すばらしきソースコード

現在sugarballは関わっている案件で、不具合修正を依頼されております。

言語はjavaなんですが、ソースコードを見て愕然としました。

ファイル全体の行数は3000行ほど。その中で、700行を超えるメソッドが2つ

こんなのメンテできません。てか、ファイルのヘッダに改定履歴とか製作者が書かれてるのですが、どうやらこのときの彼が作ったソースコードでした。「またお前か!」って思ってしまいました…。

ちなみに、その彼、この時記事の人と同一人物です。

頼むからリファクタリングを読んでくれ。こんな不吉な匂いがプンプン漂うコードを書かれたらメンテする人は大変なんだー、と。

しかも、どう見たってコピペしたとしか思えないような重複コードがあったので、彼に関数分けを勧めても「この方がシンプルでいいんです」とか言い出す始末。シンプルの意味が分からないです。

ならば「コードが長くなるから可読性が低くなって、バグが入りやすくなりますよ」と諭せば「関数分けなんかしたらバグが入り込むじゃないですか。信用できないですよ」とか言い出しやがります。

一瞬、ものすごく殴りたい気分になりました。そんな彼はjava歴3年over。3年間何をしていたのか小一時間問い詰めたくなりました。

PL殿、お願いですから彼を再教育してあげて下さい。もしくは、プロジェクトから外してください。

※すべて実話です。紛れも無く(泣

スポンサーリンク
本文中広告




本文中広告




シェアする

  • このエントリーをはてなブックマークに追加

フォローする

コメント

  1. SHIN-ICHI より:

    もしかして、僕の前の前にいた会社の先輩ですか?(笑
    Java歴3年・・・言い訳もそっくりです。

    その彼は、JSPのページでしたが、指示されて自分で作ったライブラリを使わず、ページに直接コードを埋め込んでバグを量産していました。

    自分で考えてプログラムを組んでない人間はこんなもんですね。殴って良いですよ?(苦笑

  2. sugarball より:

    >SHIN-ICHIさん
    >もしかして、僕の前の前にいた会社の先輩ですか?(笑
    そうかもしれませんw

    いや、というより、そういうコードを書くPGがどこかで大量生産されているのかも。デスマ誘発PGの生産拠点が世界のどこかに…

    >その彼は、JSPのページでしたが、指示されて自分で作ったライブラリを使わず、
    >ページに直接コードを埋め込んでバグを量産していました。
    うーん。せっかく自分で作ったライブラリなのに。まぁ、ライブラリを作る前から、使うときのことを全く考えずにいたわけですね。

    >殴って良いですよ?(苦笑
    あぁー、殴りたいっ。けどsugarは外勤、その彼は社員さん。あんまり目をつけられることもできないのです。

    まぁ、同じ会社の社員でも殴ったりはしませんがw

  3. 紙コップ より:

    プロの現場の話が聞けて楽しいです^^

  4. sugarball より:

    >紙コップさん
    単価の安いプロの話でよければいくらでもw

    てか、ひそかにリンクをはっていただいてたので、はり返しました。。

    こんな弱小ブログにリンクをはっていただいて大変驚いております。。。
    —–