ほんとうにあった開発生産性が爆下がりする話 - Qiita

昨今、継続的にプロダクト開発していくことが主流となり、Four Keysなどの開発パフォーマンスを測る指標なども出てきており開発生産性を向上させることが注目されています。 しかし、かつての開発現場では今では信じられないような開発生産性を爆下げするようなことをやっていました。 この記事では10年以上前に私が経験した開発生産性を爆下げする事例を書いていこうと思います。 (私が体験したことをベースに書いているので10年前は全てがこうだったということではないのでご留意ください ) 修正前のコードはコメントアウトで残す 当時、ウォーターフォールで開発していました。 ウォーターフォールでは開発工程とテスト工程が分かれています。 開発工程で一通りコーディングして、テスト工程で動作確認を行いバグを潰します。 問題はここからです。 とある現場では、テスト工程でバグを直すときにコードを破壊的に直すのではなく、

[denimn]

全部経験ある…。ユニットテストコードを導入しようとしたら開発量が倍になるから客からNGと言われた思い出。（そのくせ、ブレークポイントでキャプチャ）

[chintaro3]

高額の金銭が絡む話になったり裁判になったりする恐れに対抗しようとするとこうなるんだろな。人命にかかわる場合もこんなことになりがちかもしれん。生産性より重要なことがある場合もある。ケースバイケース

[teruspr]

印刷して納品、で死んだ。受注した奴はちゃんと読めよ。

[dotneet]

実は今でもやってるとこありそう。

[Goldenduck]

爆上がりすることはまずないけど、爆下がりすることは稀によくあるので潰して回るだけでもかなり改善できるはずなんだ

[lenore]

IDが付いたクラス名病、ホントにやめてほしい。これPGだけじゃないからね。こないだAzure見たらいろんなリソースが全部番号で振られてて参った。

[oktnzm]

こういう苦行を体験すると純粋にプログラミングに集中できる環境のありがたみが身に染みるから一度は体験してみることをおすすめ（しません）。

[aipacommander]

“過去最高にツラかったのは単体テストのホワイトボックステストで分岐前後のキャプチャを残すというものでした。” ????????????????????? 辛い

[sds-page]

ケースバイケースで判断できればいいけど「これ全部要らない！」ってアホがプロジェクトを滅茶苦茶にしていくから規約として盛り込まれる事になるんだよな

[goronehakaba]

/*** Add Start YYYY/MM/DD ***/と/*** Add ENDYYYY/MM/DD ***/ってなので挟んで書くというルールがあって修正していくうちに入れ子になってドッタンバッタン

[shingo-sasaki-0529]

システム開発老人会の話なのではと思いつつ、エビデンスを重視する開発プロセスだとまだまだありえるのかな。

[t-tanaka]

これが運用の現場ではいまも当たり前に行われてる悪夢。なんちゃら.confの中には，コメントアウトされた大量の変更履歴。ID名のスクリプト。Excelによる業務手順書。なんでもかんでも画面キャプチャ。

[syu-m-5151]

ほんとうに夏に聞いたら体温が下がりそう。。。

[diveintounlimit]

あるあるすぎてつらい

[sigwyg]

コードの行数で進捗管理はクソだが、それはそれとして、ワンライナーも良くない。改行は適切に行いましょう

[nisisinjuku]

納品に関してのエビデンスは無いと係争になった場合逃げ場がないからな。「つかったことない」ってのは生存バイアスでしかない。

[wkoichi]

さすがに「デバッガで止めた状態をキャプチャ」は経験ないな…

[send]

SIerまだそんなことしてるのか……

[masayan523]

1年前にいた前職がまさにこれだった。後から入ってきた人がこのやり方を推進しまくってたので今もそうなのかな..。合掌。

[otation]

民間企業なら競争力失うだけだからいいけど、官公庁は税金を垂れ流してるというストレスも…

[vegnpomn]

画面については強引な英語をつけられるよりカテゴリ分けされたIDと日本語名が書かれた対応表があるほうが数段把握しやすいと思ったりもするので、絶対悪とは言いにくいかな。汎用的なのはIDやめてほしいが

[healthyprog]

日本語コーディングは複雑なロジックの場合は役に立つかも。自明な場合は要らないけど

[hdampty7]

何言うてるの？これ、自社開発ならそうだけど、請負ならその分の工数貰えるから逆にお得。そういう体質じゃん、今のこの業界。

[stabucky]

エビデンスを残すなんて本当は要らないから。

[komkom0428]

俺は幸いそんな所からは逃げられたが、残念ながら2年くらい前にSIer系の現場でほぼ全部経験したよ。あれは早く滅ぼした方が良い

[naggg]

“この記事では10年以上前に私が経験した開発生産性を爆下げする事例を書いていこうと思います。”

[ducktoon]

デバッガで止めてスクショって狂ってる

[Phenomenon]

なにこの怖い話

[koroharo]

残念ながらSI屋だと今でも普通にあるよ。マネージャになると、これをやらせたくなる病気を発症するよ。自分はやりたくないけど、人にはやらせたいのよ。

[sirobu]

“デバッガで止めた状態をキャプチャ”Eclipseのカバレッジツールで全行が緑になっていることを確認（スクショ取得してExcel貼り付け）はあったな

[proverb]

かわいそう

[puruhime]

最初の方はよく聞いた話だったけど、ブレークポイントのキャプチャは初めて聞いたよ。創作だと思いたい。そうだと言って…(絶望)

[mr_ozin]

本当に悲しいのは、若手の人がこういうのを当たり前だと思って、転職しようとした時にロースキルを自覚させられるやつだよ

[fukken]

「普通はこう書くんだけど、精度の問題や最適化の都合でこう書いてる」みたいな場合、意図の説明を兼ねて普通のコード（つまり読みやすい）の方をコメントで残したりすることはある。

[uturi]

いくつかは自分も経験したことがある。ただ、商習慣に結びついてたりするのでなかなか改善は難しかった記憶。なぜ生産性が下がるルールができたのかの説明も欲しかった。

[remonoil]

コードを紙に印刷して赤鉛筆でなぞりながら机上テストしていくってのがあった / 意外とバグとかtypoとか見つかる見つかる

[kyoruni]

コメントアウトされたものを戻すときも、コメントを書きます

[sota2502]

税務系ならIDクラスは普通にありそう。帳票のXMLで氏名が<BLA12345>山田太郎</BLA12345>で表現されたりする。

[diffie]

IDは役に立つときもあるんだよな。10年以上前にかかわったシステムのバッチ、005がメイン処理で、018で補正、068が大口集計、070が後処理。メンバ間で認識共有しやすかった。いまだに覚えてるし