SE道場 - 似て非なるロジック(後編) |     ✤ We Love Softbrain ✤ 

    ✤ We Love Softbrain ✤ 

        日々の仕事や、プライベートな趣味まで掲載しています。

みなさん、こんばんは~
シニアAですビックリマーク

いつもSE道場をお読みいただきありがとうございますクローバー

前回は新人を残して出張に行くところまででしたひらめき電球


(続きをどうぞ)
『ええ~、出張ですかはてなマーク ちょっと無理ですよあせる
『そこを何とかビックリマーク

『だけど5月29日納期の仕事があって・・・』
『先方はどうしてもSEに来てほしいと言っているんだビックリマーク

『しかも新人がこれからコーディングなのに・・・』
『お願いビックリマーク この通りだ!!

『そんなに言われても困りますビックリマーク
『君しかいないんだ! 頼むよビックリマーク

出たビックリマークこの殺し文句。前編でも言ってた。

この会社の上の人はみんな同じ殺し文句をつかうなガーン



こうして出張にいくことになり、翌週になって、わたしは帰ってきた新幹線

出張中に考えていたのは、ハードウェア制御部分を共通関数化したから、共通関数の呼び出し部分のコーディングはさぞかし分かりやすいだろうということだニヤリ

きっと問題なく出来ていると期待して、新人とコーディングレビューを始めた音譜



画面メニューの項目は全部で17あるひらめき電球

そのうち13項目は共通関数を似たような形で呼び出しているビックリマーク

ここまでは良かったが、その先はとても分かりやすいとは思えないショボーン


『これは、わかりにくいな。こんなに複雑怪奇になる訳はないんだが・・・』
『複雑複雑って、面白い言い回しですねひらめき電球

『君は自分で分かりにくいとは思わないのかはてなマーク
『そんなに複雑ではないと思いますよ。もっとよく見て下さいビックリマーク

『こことここは共通関数の呼び出し順がかなり入れ替わっちゃってるよ。なんでだはてなマーク
『同じ順番でなくてもいいと思ったんで入れ替えちゃいましたウインク

『そもそも、呼び出しパターンが17通りもあったら、何でこれを関数化していないんだはてなマーク
『やだな~、同じロジックの時だけ関数化するって習いましたよニヤリ

『それしか教えなかったかはてなマーク
『はい、だからこことことは同じロジックだったので、関数化してありますビックリマーク

『同じロジックだけ関数化してもたかが知れてる。これからは似て非なるロジックも関数化するんだビックリマーク



言葉のあやというか、わかりやすくするために言った「同じロジック」「似て非なるロジック」に置き換えておくべきだったあせるあせる

後悔してもあとのまつり、いまから修正しても到底間に合わない汗

この仕事は、似て非なるロジックを関数化できずに終わってしまったガーン

苦い経験だった汗

プロジェクトが終わったとき祝杯を挙げたが、反省の多いプロジェクトだった。
(終わり)


次回のSE道場をお楽しみに・・・

 


SE道場では、プログラミングの極意や開発の苦労話等、様々な記事を投稿していますビックリマーク是非ご覧くださいクローバー

 

1つ前の開発秘話は...