【雑記】 １人でシステムを製作するとどうなるか

このブログでは何度かほのめかしていますが、
私は職業プログラマとして会社勤めしています。

まぁ、会社帰りにゲーセンによって
クレーンゲームでラブプラスグッズを狙っているような
変態的な人間ですが、今回はそれは置いておきます。

３ヶ月ほど前に人事異動がありまして、今の部署に引っ越したのですが
そこで「ある精算システム」を引き継ぎました。

開発／動作環境は
・JSP
・JAVA
・Tomcat
・Postgres
です。

いわゆるひとつのブラウザを介したＷｅｂアプリケーションですね。
社名や商品名は伏せます。

私はＷｅｂアプリケーションが嫌いなのですが
そのことについて語り出すと延々と続くので
今回は割愛します。

今回の骨子はというと・・・、ソースコードがひどすぎる。
どのようにヒドイのか列挙してみます。

１．ＪＳＰファイルの中に実装を書きすぎ

ＪＳＰファイルの中に表示を担当するＨＴＭＬのコードと
処理を担当するＪＡＶＡのコードが混在しています。

まぁ、これはＪＳＰの仕様で許されているのでいいのですが
ＪＡＶＡの分量がハンパないです。

ＨＴＭＬよりも多いかもしれない。
しかも、ＪＡＶＡのクラスに切り出すことを
ほとんどしていないので関数化もされていない。

似たような処理はコピペの嵐です。
つか、同じ処理がコピペの嵐です。

そのせいでＪＳＰファイル１枚当たりの行数が膨れ上がっています。

データアクセスの部分も切り出されていません。
ＪＡＶＡの処理内でちょこちょこＳＱＬ文をダイレクトに流しています。

なおかつ、ＳＱＬ文も文字列型のローカル変数にバリバリ書いてあるので
全く分離がされていません。

あと、これは個人的な趣向ですが変数名が全角文字なんです。
こればかりはどうもジンマシンが出そうな勢いです。

２．データ構造が適当すぎ

例えば、システム設定を保存しているテーブルがあるのですが
なんとkey（項目名）とvalue（値）というカラムしかありません。
両方とも文字列です。

する／しないの２値だと "true" "false" が生の文字列で入っていますし
ＡＢＣからの３択だと "1" "2" "3" というナンバーが入っていますし
テキストのフリー入力だとそのままの値が格納されています。

ＤＢの型を無視しているので、データとして整理されていません。
タンスに本も洋服も食料も投げ入れて、ゴタ混ぜにしているようなものです。

それと、明細ＩＤみたいなのがあるのですがbigintとかで間に合うと思うのですが
１０ケタくらいのランダム文字列なんですよね。

それにより表現できる範囲が格段に上がるそうなんですが
個人的にそれはどうかと思います。

また仕事の話をしたら愚痴大会になっちゃいましたね・・・。
まぁ、ブログは自由に書ける場なのでいっか。

最後に一つだけ。

リリース間近で営業さんが価格テーブルの変更を申し出てきました。
いまからマスターデータを書き換えたら、今までのテストが無意味じゃん？！