ソフトウェア開発者の日常

こだわりなく書きたいことを書いていきます。

昔のソースコードだからといって、気づいた点は放置しないで修正

既存のシステムを流用して、文章やヘッダー画像だけ修正して、新しいシステムとして利用することがあります。
修正していると、これは直すべきだなと気づくことがあります。

f:id:AJYA:20150710054527j:plain
photo credit: HTML Code via photopin (license)

よくあるのが、tableタグの入れ子です。
左右中央配置にするために入れ子にされたようですが、tableタグのにdivタグがあって、divタグの右側のborderが隠れてしまっていました。
divタグのwidthの値を調整すれば直るのでしょうが、tableタグの入れ子自体を解消した方が後々困らないだとうと判断して、入れ子を解消しました。

他にも、プレースホルダになっていないSQLの修正や、ボタンの連打による2重登録防止を追加したりして、これなら大丈夫というレベルまで修正しています。