へたくそプログラマーが書いたコードを書きなおした話

プログラムがあまりうまくない人が書いたコードがどういうものかと言うのを紹介します。
ちなみに、書き直したら、１日処理しても終わらなかったデータ処理が１分もかからず終わるようになりました。

プログラムは、複数件のデータの削除をするというもの

問題１．１件削除するごとに、全データの書き込み、再読み込みを行っている。
改善　最初に読み込みをして、最後に書き込むようにしました。
効果　処理時間が半分になりました。

問題２．なぜか、データ処理の先頭で、毎回全データを表現するためのオブジェクトの生成処理が入っていました。
改善　使ってないのは全部削除。
効果　データの件数x3処理していたのが、１回だけの処理になり処理時間が1/500になりました。

問題３．データの削除をして切り詰める処理。処理前に全データのバックアップをとっておき、削除データがなかった場合は元に戻すという処理をしていました。
改善　バックアップリストア処理はなくして、最初にデータスキャンして、なければエラー。あった場合は処理するように変更しました。
効果　メモリ使用量が減りました。

問題４．文字列比較のために、一度、双方の文字列を大文字に変換する変数を用意しておいて、変換後＝で比較していました。
改善　string.compareで大文字小文字無視の処理に変更しました。
効果　少し無駄が減った

あんまり、プログラムが得意じゃない人は、良く分からないコードを書くという例でした ･ω･