久々に思い出したように書いてみる。
完全に社外のExcel-VBAを解析・調整する仕事が飛んできて、
見てみたら恐ろしいくらいひどいソースコードだった・・・orz
・変数が日本語
・関数やプロシージャまで日本語
・段組に統一性が全く無し(if もループもどこまでだかわかりにくい)
・メインコードが700行（すぱげってぃ！）
・ほぼPublic変数で処理（Local変数と同じ名前のもあったり）
なんで、こんなものが回ってきたのかというと、
「できる」プログラマさんがいなくなってしまって
書き換えなきゃいけないとかで・・・
よくあるお話だけど、できるレベルの感覚がだいぶおかしい気しかしない。
解析を進めていると、
複数のワークブック・ワークシートを見に行っていて、
頻繁にいろんなBook/SheetをActivateをして
RangeやらCellやらWorkSheetを省略して書いているので
データ自体はどこを参照しているのかぱっと見で理解しにくいという。
たくさんWorkSheetを扱っていても
こんな感じでちゃんとWorkSheetを変数で扱って
明示してかけばわかりやすいですよっと。
PR
