トップページ > プログラム > 2016年03月19日 > M34OaI99

書き込み順位&時間帯一覧

57 位/197 ID中時間01234567891011121314151617181920212223Total
書き込み数0000000000000000100001002



使用した名前一覧書き込んだスレッド一覧
デフォルトの名無しさん
Excel VBA 質問スレ Part40 [無断転載禁止]©2ch.net

書き込みレス一覧

Excel VBA 質問スレ Part40 [無断転載禁止]©2ch.net
653 :デフォルトの名無しさん[sage]:2016/03/19(土) 16:57:05.21 ID:M34OaI99
cadテキストファイルを読み込んでexcel上で見やすい形にして、エラー確認が出来たらaccessにぶっ込んで集計処理を行うっていうの作ってたのですが
あまりにも条件が多くなってきたことと2年ほど前から書いたコードだから統一がされてないのとで
超スパゲッティコードになってしまいました。

合計1万行、モジュール20、クラス4、ボタン10個、ユーザーフォーム2ぐらいあるんですが
こういうのはやっぱり新しく書き直した方がいいんでしょうか…?
またこのように自分で作ったプログラムを1から書き直したことがある方、気をつけることがあれば教えていただけますでしょうか?
Excel VBA 質問スレ Part40 [無断転載禁止]©2ch.net
686 :デフォルトの名無しさん[sage]:2016/03/19(土) 21:42:27.21 ID:M34OaI99
>>668
ありがとうございます。リファクタリングですね、
・外部的な振る舞いは変えない
・内部構造を洗練していく
・作業は一歩ずつ進める
・安全に変更を行う
非常にためになる言葉ですね。開発当時の自分をぶん殴りたい気分です。
内部構造の洗練はとても難しいですね。
でも調べてくうちに
excel標準機能にあるワークシート関数やら書式設定やらはコードに実装しない
if文やselect caseがあるとその分だけテストケース増やさなければならない=複雑になる
行数が多い=それだけ読まなければいけない=他の関数との凝集度が悪くなる
この辺は痛感しました…書き直してきます


※このページは、『2ちゃんねる』の書き込みを基に自動生成したものです。オリジナルはリンク先の2ちゃんねるの書き込みです。
※このサイトでオリジナルの書き込みについては対応できません。
※何か問題のある場合はメールをしてください。対応します。