メンテ保守性の向上

Question

中小企業の管理部付社内SEをやっております。
サーバー入替に伴い社内LAPP業務システムを多少ブラッシュアップする予定です。

なぜLAPP、、という類のご意見はご遠慮ください

引継同時は完全一致するCGIスクリプト(約40scripts)とテンプレートHTMLが利用拠点(10箇所近く)のディレクトリ毎に同セットがおいてある状態だったので
これは修正の度にキレる自分が容易に想像でき
段階的に関数をライブラリ化してコード整理(コード行1/3に!)して
テンプレートHTMLも非推奨タグを一掃して、1セットを全利用拠点で参照するように変更済(ファイル数は1/10に!)です。

ならばこの機会に、とCGIスクリプトも1セットにと思い
起動パラメータで初期値・設定ファイルを分岐すればいい、と考えました。
すでに引数が一つで済むような単純な機能はこの方法で実装しています。

ふと こういう考え方？というかやり方は
プログラムの書き方というか、保守性やメンテ効率の為になら「アリ」なんだろうか？

と疑問に思い、同業の方のご意見頂戴したく投稿いたしました。

Answer 1

前任者様がコピペでコードを増殖させていたのは未熟さのせいかもしれませんがもしかしたらバックオフィスという職種の性質のせいかもしれません。
成果は評価されないがミスは全社から責められるというバックオフィス業務の性質上、新規ページを追加する際に既存ページに絶対にデグレが出ないように、つまり手を触れなくていいようにコピペを繰り返していたのではないかということです。

同レベルの安全性を、いや保身をどのように実現するかというと、モダンプログラマならやはりテスト駆動でしょう。
コードの正しさをがちがちのテストコードで保証し、デグレを作りそうになってもデプロイ前に未然に潰せる環境を作ることがセットで必要になるかと存じます。

と言っても、これだけ大規模なリファクタリングをしたのならすでにテストコードはほぼ整備済みということでしょうかね。だとしたら釈迦に説法すみません。

Comments

[退会済みユーザー]

yuba様、回答ありがとうございます。

すみません、質問がわかりづらかったようで
前任者のコードが「なぜそうなったのか？」を推測していただいたようなのですが
今回皆様からご意見頂戴したいのは

こんなふうに
http://pekepeke.jp/shop111/sample.cgi
http://pekepeke.jp/shop112/sample.cgi
http://pekepeke.jp/shop113/sample.cgi
:
店舗の数だけ同じスクリプトを設置するのは無駄だしメンテが面倒だから
こうすればいいんじゃないか？

http://pekepeke.jp/scripts/sample.cgi?パラメータ1=aaa&パラメータ2='xyx'

という私の考えは「あり」なのかな？？です。

宜しければ引き続きお付き合いください

[yuba]

こちらこそ、回答の一番大事な部分を省略していました。
ありだと思いますし、ありを通り越して当然だと思います。
前任者様のソース構成はあまりにもひどいかと思いますが、そのひどさにも理由があるのかなと考察を述べました。

[退会済みユーザー]

yuba様

続投に気づかず失礼しました。

< ありだと思いますし、ありを通り越して当然だと思います。

後押ししていただいた気分。俄然心強いです。
ちょうど新サーバができあがってきたので迷わずにやってみようと思います。

< 前任者様のソース構成はあまりにもひどいかと思いますが、そのひどさにも理由があるのかな

はい、「すでに退職を決めてたからやっつけ仕事」とも聞いてますし
ローカル環境でコード書いてftpアップロードするタイプだったので
同じものをディレクトリ毎に放り込む方が楽だったんだと、予想しております。

サーバ移行は初めてなのでまた質問させていただきます。
助言いただき、本当に有難うございました。