- VBAなんでも質問スレ
869 ：デフォルトの名無しさん[sage]：2014/09/13(土) 02:10:36.52 ID:aRKTTaf3 - 俺は、読みにくいけどわかりにくいとは思わんな
わかりにくいと感じるのは、実際のワークシートや印刷ページを見てないからだと思う
単にデータを並べ替えてるだけみたいだし、たぶん実際のシートを見れば何やってるか一目瞭然だと思う
まあ変数名はもうちょっと考えてもいいかもしれんが
>>868
変数名は、悩むぐらいなら無理に英語にせずに日本語で書けばいい
こう書くと日本語の変数名には絶対反対と言うやつも出てくるけど、コーディング規則なんて宗教みたいなもんだからなあ
何を信じるかは個人の自由
あと、コメントは例えば
「行のループ番号を格納する変数」
じゃなくて
「行番号を格納するループ変数」
みたいに書くかな、俺なら
|
- VBAなんでも質問スレ
872 ：デフォルトの名無しさん[sage]：2014/09/13(土) 03:55:30.91 ID:aRKTTaf3 - とりあえずインデントつけてみた
https://friendpaste.com/7iW6FESeuW3vqGQb1Y08Vz
|
- VBAなんでも質問スレ
873 ：デフォルトの名無しさん[sage]：2014/09/13(土) 04:04:20.35 ID:aRKTTaf3 - >>871
俺なら「印刷用のシートをActiveSheetにする」と決めてコーディングする
先頭の方でActiveSheetをコメント付きで明示して
Worksheets(Invoice_WS).Activate ' 印刷用シートをアクティブにする
データのコピー元シートは2枚あるので、Withは使わずにオブジェクト変数に入れる
まあ全シートを省略せずに書いた方が間違いは減りそうだけど
|
- Excel VBA 質問スレ Part34
572 ：デフォルトの名無しさん[sage]：2014/09/13(土) 04:08:15.45 ID:aRKTTaf3 - >>563
単なるデータベース管理だろ？Access使え
|
- VBAなんでも質問スレ
875 ：デフォルトの名無しさん[sage]：2014/09/13(土) 04:22:56.22 ID:aRKTTaf3 - >>874
別にどれをActiveにしてもいいんだけど、ExcelではActiveシートを無視することはできてもなくすことはできないから、
どれにするか決めて活用した方がコーディングが楽になるよねっていう単なる経験談
今回のケースではコードが短かすぎて恩恵が少ないけど
|
- VBAなんでも質問スレ
876 ：デフォルトの名無しさん[sage]：2014/09/13(土) 04:28:55.43 ID:aRKTTaf3 - ここで言ってる「Activeシートを無視」ってのは、プログラムを作る時にどこがActiveかまったく考えずにシート名をいちいち書くって意味ね
|
- VBAなんでも質問スレ
879 ：デフォルトの名無しさん[sage]：2014/09/13(土) 10:17:51.44 ID:aRKTTaf3 - >>878
セルの指定方法については考え方が逆だな
A-Zは脳内で数字に変換できるけど2文字以上は暗算が困難だから、既存の表からデータを拾う場合は文字で指定した方が確実
|
- Excel VBA 質問スレ Part34
576 ：デフォルトの名無しさん[sage]：2014/09/13(土) 23:11:38.58 ID:aRKTTaf3 - >>574
ネットで検索
ただし一致する定数があるとは限らない
|
- VBAなんでも質問スレ
886 ：デフォルトの名無しさん[sage]：2014/09/13(土) 23:16:27.49 ID:aRKTTaf3 - >>884
変数名の大文字小文字と単語間の区切り方はプロジェクト単位で統一されてればどうでもいい派
それぞれの流儀にちゃんと名前もある
http://blog.nabe-ch.net/?p=1300
個人的にはN-BASICから入った人間なので1文字で済まそうとする悪い癖がいまだに抜けない
|