misc.log

日常茶飯事とお仕事と

ちょっとまとめよう: ひどい設計、実装

  • For i=0 to 10で始まるFor文の中で、iを使わずに別のカウンタが宣言されて自力インクリメントしている。
  • Functionとして定義された処理で、引数で渡された値が全く使われていない。
  • フォーム上のコントロールインスタンスの名称がデータベースに記載されており、そこに権限ごとの表示可否情報が記されている。
    • 設定ファイルでしょそれやるなら。フォームのコントロールインスタンスである以上、DBだけ変更するとかあり得ないのでどのみちアプリ配布になり、設定ファイル再配布。DBだけ修正で配布を端折れる機会は無い上、画面の変更=DBデータパッチ適用という余計な手間が出る。
    • おまけにコントロールインスタンス名の文字数がDB側のカラムサイズで制限されているため、プログラム側で自由な名前をつけられない。