たかちゃんさん:

おはようございます。

> 先生の講座は本当に楽しいです。
> 最初は伝票が作れたら満足って思っていましたが、今では密かに講座を全制覇目指すことも可能じゃないか？と
> 思いはじめました。（^^;
> 本当に良い講座に出会えました。

それは何よりです (^^*


> 伝票作成は、０から作成しようとすると一瞬不安になりましたが、落ち着いてゆっくりでしたが書けました！
> 恐らく仕事で、「初めて作成する物」だったり「もし作成出来なかったらどうしよう？」みたいな環境だと
> パニックになってしまいそうですが・・・。

型に慣れれば、ささっとできるようになりますよ。
料理と同じです (^^

> ３つ疑問が出来たので、質問させて下さい。
>
> 【１つ目】Setで設定する位置について。

オブジェクト型変数に限らず、「ローカル変数を宣言するタイミング」ということで広く一般的に言えることを書きます。
以下のとおりに整理して理解しておいてください。
[1] 変数は、その変数に値なり参照なりを設定するより前に宣言するのであれば、どこで宣言してもよい。
[2] ループの中で宣言するのは好ましくない

[1]について。
以下のそれぞれについて、どちらが良いか？はケースバイケースです。
それぞれにメリット/デメリットがあります。
[a] 変数宣言をプロシージャ上部にまとめる
[b] 変数を利用する箇所のなるべく近くにするか、

[a] のメリット:
・整然としている
・プロシージャ上部の変数群(と、必要に応じてコメント)を見れば何をしたいのかだいたい見当がつく。(もちろん、変数名が適切で、コメントが分かりやすい場合)

[b] のメリット:
・変数を実際に活用するタイミングで、その変数の宣言箇所がすぐ目につく。なので、その変数のデータ型やその変数の宣言趣旨が(コメントが付与されていれば)すぐ分かる。
・その変数を活用する直前まで変数を宣言しなければ、予め頭にその変数を入れずにコードを読めるので、読む時に負担が少ない。特に、プロシージャ内で何度も構造が出てくる等の理由で全体の見通しが悪い場合など。

僕の好みは[a]です。なるべく[a]になるようにコーディングします。
[b]の状況になることは、以下の方法でなるべく避けます。

[b-1] そもそも、ひとつのプロシージャをあまり大きなボリュームのものにしない。
[b-2] そもそも、ひとつのプロシージャ内で For ... Next 文を何度も入れない。

発展編2に進むと、上に示したような手直しをさらに容易にできるようになります。

[2]について。
ループの中で変数宣言してしまうと、ループの回数だけその変数を宣言することになります。これは非効率。ループの手前で宣言して、それをループ内で使い回すほうが良い。


> 【２つ目】dt = shFm.Range("C" & lnFm).Valueの設定する位置について。
> この↓の記述の直前に設置した方が良いのか？、For lnFm = 2 To lnFmMxの直後に書いた方が良いのか？
> 迷いました。結局、動くので同じなんですが、設定するのは動作に影響が出ない範囲でなるべく上の方がいいのかなぁと
> 一瞬思ったんですが、推奨する位置とかありますか。
>
> shTo.Range("B" & lnTo).Value = Right(Year(dt), 2)
> shTo.Range("C" & lnTo).Value = Month(dt)
> shTo.Range("D" & lnTo).Value = Day(dt)

僕は直前が良いと思います。
【１つ目】への回答[b]と趣旨は同じです。B, C, D列に値を入れようとするこの3行にしか影響しない話題なので。


> 【３つ目】書いているとついつい、見やすいように空白行を入れてしまうことがあります。
> 見本を見ると、タブで横にはずれているものの、殆ど詰めて書いてありました。
> 空白行は、こんな時以外は入れない方が良いなどありますか？

その空白行にどのような意味をもたせるのか？という目的次第です。
メールの文章でも、メリハリのために空白行を入れることがありますね。どのタイミングで空白行をどの程度の入れるか？は、読者に期待する読解力次第でもあります。

2020/09/26 16:30