[過去ﾛｸﾞ] Excel VBA 質問スレ Part69 (1002ﾚｽ)
上下前次1-新
抽出解除 ﾚｽ栞
このｽﾚｯﾄﾞは過去ﾛｸﾞ倉庫に格納されています｡
次ｽﾚ検索 歴削→次ｽﾚ 栞削→次ｽﾚ 過去ﾛｸﾞﾒﾆｭｰ
964(3): (JP 0H8d-IMun) 2021/03/17(水)10:52 ID:KXbC8pO1H(3/4) AAS
AA省
972: (ﾜｯﾁｮｲ 824f-pJvK) 2021/03/17(水)15:25 ID:vF0LuwhE0(1/2) AAS
>>964
うーん
頭ごなしに糞というつもりはないけど
そうだね、もし改善点を挙げるとすれば・・・
まぁ聞き流して良いけどね。
・変数名は一貫性を持たせて、一見して何を意味しているのか分かるようにすること。
例えば頭の1文字が大文字だったり小文字しているのをキャメル形式に統一するとか。
・ActiveSheetは極力使わない。
キチンとした意味を持っているならそれでいいけど、フォームやツールバーから呼ばれることやそのように改修される可能性も考慮してどのシートを使うか明示的に指定する方が良い。
もちろんその際はシート名が変更されてしまうことを考慮して
省16
977(2): (ﾜｯﾁｮｲ 824f-pJvK) 2021/03/17(水)19:38 ID:vF0LuwhE0(2/2) AAS
>>964
ついでにもう一つ言っておくと、
ディクショナリやコレクションの要素数は1バイトで扱える数、つまり1から数えて256まで。
着眼点は良かったけど、3000行を考慮すると
オーバーフローで落ちる可能性があるよ。
991(3): (ﾜｯﾁｮｲ 02bd-I38D) 2021/03/18(木)13:36 ID:pzBF2kYz0(1/2) AAS
>>964
64bit版の時はlonglongで宣言するようにしないとエラーになる可能性があるよ
上下前次1-新書関写板覧索設栞歴
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ AAｻﾑﾈｲﾙ
ぬこの手 ぬこTOP 0.034s