コードの行数を減らすと生産性があがりバグも減る [転載禁止]©2ch.net

**仕様書無しさん** · 2015/09/12(土) 10:34:48.38

この事実を発見してから我が社では既存のコードを修正し、
行数（正確にはステップ数）を極限までに減らす修正をすることで
バグを大幅に減らして生産性も大きく上げる事に成功した。

**仕様書無しさん** · 2015/09/12(土) 11:41:44.60

>>1
可読性まで犠牲にして逆にバグが発見でき無いコードにならんようにな。

**仕様書無しさん** · 2015/09/12(土) 14:50:35.49

縦スパゲッティか横スパゲッティの違いじゃないの？

**仕様書無しさん** · 2015/09/12(土) 14:56:15.81

できるだけ改行しなければいいのか。

そんなソースコードとは関わりたくないが。

**仕様書無しさん** · 2015/09/12(土) 15:09:44.46

全部1行で書けば良いんだな。

**仕様書無しさん** · 2015/09/12(土) 16:24:51.96

アホやw

行数よりも
コードの見やすさとコメント量の方が大事

誰が見てもわかりやすいコーディング

メンテで差がつく

**仕様書無しさん** · 2015/09/12(土) 16:30:30.70

でもコードの行数がすくなければ
メンテする量も減るんですよ。

**仕様書無しさん** · 2015/09/12(土) 17:00:07.26

>>7
そりゃ、処理、機能がないんだからそうなるなw

**仕様書無しさん** · 2015/09/12(土) 18:01:10.45

コメントも多すぎると、そのコメントのメンテナンスも
必要になるから、コメントも少ないほうがいい。

**仕様書無しさん** · 2015/09/12(土) 18:06:43.59

>>9
おまえが文章書くのが苦手なのは分かった。

**仕様書無しさん** · 2015/09/12(土) 18:10:05.58

>>10
どこかまずい所があったら直しますんで、
具体的に指摘してみてください。

**仕様書無しさん** · 2015/09/12(土) 18:11:48.38

コメント三回は多すぎるわなぁ

**仕様書無しさん** · 2015/09/12(土) 18:20:39.62

>>12
え？そんだけ？

**仕様書無しさん** · 2015/09/12(土) 18:54:26.57

こんなん当たり前の話とちがうん？

昔のエライ人が言ってだろう？

「シンプル・イズ・ベスト」

簡単なモノは壊れにくい。複雑なモノほど壊れやすい。
これはプログラムコードにも当てはまるんよ。

俺はもう設計屋だから、コード書かせるときに必ず若いもんにこれ言うわ。

**仕様書無しさん** · 2015/09/12(土) 19:50:30.60

>>14
その当たり前のことが出来ないんだよね。
ソフトウェアの難しいのは、
バージョンアップしていく所。

大きい物と小さい物は作り方が違うんだけど、
小さく作って大きくしていく時に、
既存の部分を大きい作り方に直さないといけないんだけど、
「既存の部分に手を付けるな」とかいう馬鹿な奴が
指揮をとると、どんどん複雑になって手がつけられなくなる。

既存の部分を修正すれば、1.1の量で済むことが、
手を付けれないからコピーして、2になる。

**仕様書無しさん** · 2015/09/12(土) 21:44:56.56

本当に手を付けられないコードは、ある。

コードに手を入れればコード増加は少ないだろうが、リグレッションテストの工数が増えるだろ？

どっちの工数が少ないか秤にかけ、コードに手を入れない事は、フツウにあるだろ？

どっちがいいかは、プログラマには関係ないことだ。

**仕様書無しさん** · 2015/09/12(土) 22:05:13.11

> リグレッションテストの工数が増えるだろ？

そうならないように自動化すればいい。

結局オレが言ってることと一緒で、
手を入れるから増えるのではなく、
手遅れになってるから増えるんだよ。

**仕様書無しさん** · 2015/09/12(土) 23:42:51.35

>>16
> どっちがいいかは、プログラマには関係ないことだ。
いや、プログラマが判断することだろ。

**仕様書無しさん** · 2015/09/13(日) 00:11:27.61

>>1
生産性の定義すら提示してないから、もともと議論する気はなさそうだな。

**仕様書無しさん** · 2015/09/13(日) 00:22:41.16

生産性の定義なんか必要あるのか？

**仕様書無しさん** · 2015/09/13(日) 01:01:54.35

自動化なんかしたら工数減るだろ！

**仕様書無しさん** · 2015/09/13(日) 04:20:58.66

シンプルなソースと行数は相関関係がないと思うんだが
一行に纏めて読みづらいif文よか5行でも読みやすい方が
保守工数は減るんでね

**仕様書無しさん** · 2015/09/13(日) 10:14:22.51

>>19
単位時間当たりの生産量だろ？プログラマなら、単位時間当たりの
コーディング量＋テスト量って事になる。

**仕様書無しさん** · 2015/09/13(日) 18:00:38.85

>>22
共通関数用意してるのに使えないから誰も使わなくて至る所で同じような処理が行われてるとか、
しかもそれが微妙に違うから各モジュールでデータをやり取りするときに変換かけないといけないとか
そういうのは？

**仕様書無しさん** · 2015/09/13(日) 18:20:29.09

つまりなんだ、このスレはソースコードの負の遺産の話でしょ
ゴミや糞は片付けよう！

**仕様書無しさん** · 2015/09/13(日) 18:20:46.50

少ない行で書けるシンプルなコードを書けということで行数を減らすのが目的ではないよな

**仕様書無しさん** · 2015/09/13(日) 19:26:52.62

結果的に行数が減るだろ？

**仕様書無しさん** · 2015/09/13(日) 21:06:13.85

行数が問題なんじゃねえよ

instructionが少なくなるように書けっつってっだろ？

**仕様書無しさん** · 2015/09/13(日) 23:10:18.70

>>28

>>1の
> 行数（正確にはステップ数）を
を読んでますか？

**仕様書無しさん** · 2015/09/14(月) 01:53:05.12

ワンライナーってやつか

**仕様書無しさん** · 2015/09/14(月) 07:09:08.63

>>29
はあ？
instructionだよアホ

頭ん中で簡単なアセンブリしながら書けっての

**仕様書無しさん** · 2015/09/15(火) 02:02:28.77

if文に関数コール埋め込んだりすると行数は減るがデバッガで追いにくくなるんだが

**仕様書無しさん** · 2015/09/15(火) 08:42:35.42

三項演算子とかも二重三重にまとめ書きして行数減らすのが推奨？
解読困難になるけど

**仕様書無しさん** · 2015/09/15(火) 12:33:59.71

>>32
最近はデバッガで追うようなことはしなくなってるよ。
まずはテストコードが書けるようなシンプルな関数を作る。
テストコードがあればデバッガはほとんど必要ない。

**仕様書無しさん** · 2015/09/15(火) 20:55:41.82

コーディングスタイルで行数を減らすって話じゃないだろ
その機能を実現するのに最適なロジックを選択したら行数も減るってことの裏返しを言ったんだよな？>>1

**仕様書無しさん** · 2015/09/15(火) 22:26:56.30

>>34
いいこと言うなあ
その通りだよな

**仕様書無しさん** · 2015/09/16(水) 00:26:51.04

>>20
お前だけ、経営者が決めた生産性でやってれば良いんじゃね？

**仕様書無しさん** · 2015/09/16(水) 01:38:23.18

>>34
テストが通らないときはどうするの？

**仕様書無しさん** · 2015/09/16(水) 04:29:24.38

>>38
ソースコードを眺めて（セルフレビュー）して
間違いを見つける。

これができない＝コードが複雑ということ

**仕様書無しさん** · 2015/09/16(水) 06:56:26.57

>>39
のんびりとした開発してるんだね

**仕様書無しさん** · 2015/09/16(水) 07:34:06.22

ただの裏方データ処理ならいくらでもシンプルにできるだろうけど
GUIが入ると人間の曖昧性を処理しなきゃいけないから
コードをシンプルにすればするほどゴミUIに近付いていく

**仕様書無しさん** · 2015/09/16(水) 08:08:20.95

>>40
デスマ乙。

**仕様書無しさん** · 2015/09/16(水) 10:38:38.57

>>40
コード読むとのんびり？
あんた、他人がコードレビューしてないの？
読むのに時間がかかるコードだから？
終わってるねｗ

**仕様書無しさん** · 2015/09/16(水) 10:56:39.34

眺めるならテストコードとパターンだろ。
テスト結果が合えば終わり。
合わなきゃテストパターンから原因箇所は特定できる。
お前のセルフレビューが正しいって保証はどこにあんだよ。

**仕様書無しさん** · 2015/09/16(水) 13:12:44.83

>>43
レビューはデバッグの時間じゃないぜ

**仕様書無しさん** · 2015/09/16(水) 13:39:49.39

>>45
そうだよ？

レビューとデバッグは両方やるし、その両方でコードを読む。
だからコードはスムーズに読めるに、なっていなければならない。
コードは解析するものじゃない。

デバッグで解析は楽ですーっていっても、
レビューでコードを読むわけで、コードを読む＝のんびり。であるなら
まずその問題を解決しなければならないよね？

で、その問題が解決できれば、コードを読んでバグの修正もできる。

**仕様書無しさん** · 2015/09/16(水) 15:00:27.70

>>46
問題にしているのは、コードを読む時間ではない。
バグの修正にデバッガを使わないことで無駄になる時間を問題にしている。
あるバグを修正するのに、デバッガを使えば1分で原因が特定できたのに
使わなかったために原因の特定に１０分かかった場合は、９分間給料泥棒
やってたことになる。
３０分コード見ても原因が特定できずに、結局デバッガ使うことになったら
３０分まるまる給料泥棒だ。

**仕様書無しさん** · 2015/09/16(水) 15:13:17.77

>>47
そもそも、デバッガを使っても
時間は短くならない。

**仕様書無しさん** · 2015/09/16(水) 15:17:56.63

>>48
そりゃ酷いコードだ

**仕様書無しさん** · 2015/09/16(水) 15:31:48.72

>>49
逆w

酷いコードだと
中で何やってるかわからないから
デバッガ使って動かしてみないとわからない。

ちなみに基本的にコードレビューというのは
実行しないで行う。

**仕様書無しさん** · 2015/09/17(木) 07:28:06.59

主な使い捨て実態派遣作業
・データ > ロジック
・簡単ロジック
・大量データ
・フレームワーク
・Web
・DB
・SE適性不要
・IT資格不要
・大卒資格不要
・文科系対象
・体育系対象
・業務系処理
・商業系業種

**仕様書無しさん** · 2015/09/18(金) 21:01:01.79

同じコードを2度書くな
大切なことだからもう一度書いとくよ
同じコードを2度書くな

コードの行数は減ることが多い
単体レベルのテストは減る
同等のコードなので生産性は上がらない
コードが分散しないのでバグは減る

同じコードを2度書くな(大切なのでry
違和感があるときはリファクタリングのタイミング

**仕様書無しさん** · 2015/09/19(土) 10:38:44.77

>>52
二度以上書かないとならない数行のコードなんて普通にあるから。
おまいの書いた内容程度の行数なら何回でも同じコードを書いてもいいよw

**仕様書無しさん** · 2015/09/19(土) 11:09:54.23

類似性のある複数のコードは、相違点だけを引数とかで吸収するようにして纏めてしまえ、てことなんでしょ

纏め方も、サイズ重視か速度重視かで関数なのかマクロなのか変わるだろうけど