ソースコードが汚いことで発生する問題点 [無断転載禁止]©2ch.net

**仕様書無しさん** · 2016/06/01(水) 00:51:12.80

修正するたびにバグが増える。
修正にかかる見積が大幅にずれる

**仕様書無しさん** · 2017/01/22(日) 15:25:07.20

インデントは直しゃいいし
実際に条件が複雑ならしょうがなくね？

**仕様書無しさん** · 2017/02/13(月) 03:35:53.28

ちゃんとオブジェクト指向なり守ってたらどんなに複雑でもそんな多重ネストになるわけない
ネストなんてサボって二重、それもスクロールせずに表示できる行数内で済むレベルのみ
三重超えるようなら設計がおかしい

**仕様書無しさん** · 2017/02/13(月) 22:55:14.20

全角スペースを使う奴
行末に余分な半角スペースを残す奴
こいつらのソースは汚い

俺はエディタでそういう文字を強調表示しているから
そういう奴らのソースはチカチカして目が痛いｗ

**仕様書無しさん** · 2017/02/14(火) 00:07:06.05

>>158
> 行末に余分な半角スペースを残す奴
markdownかけねぇw

冗談はさておき、今時のエディタっていらねぇデフォルトでスペース勝手に消してくれない？
Atomでmarkdown書いてて勝手に消されて設定変えるのめんどかった

**仕様書無しさん** · 2017/02/14(火) 08:07:12.71

>>159
markdownのそれは余分じゃないだろ？
例外見つけてはウキウキ

**仕様書無しさん** · 2017/02/14(火) 08:29:45.59

>>160
余分か余分じゃないかってどう判断するの？

**仕様書無しさん** · 2017/02/14(火) 08:38:30.85

たいしたことないシステムほどデグレがどーのこーのうるさい

**仕様書無しさん** · 2017/02/14(火) 08:50:19.24

デグレ？

**仕様書無しさん** · 2017/02/14(火) 12:47:42.57

デフレじゃないぞ。

**仕様書無しさん** · 2017/02/14(火) 20:45:11.85

おまえらデグレって言う？
本場の英語ではリグレッションのはず
でもリグレって言うとかっこわるいのだ

**仕様書無しさん** · 2017/02/14(火) 20:53:36.27

デグレはデグレードのことだろ。レベルダウンともいう。
リグレはリグレッション。元の木阿弥とか前のほうがいいんじゃね？的な。

でもリグレッションという言葉をよく聞くのはリグレッションテストじゃ
ないかと思う。

**仕様書無しさん** · 2017/02/14(火) 20:53:56.97

どっから発生した用語か不明だが、
レベルダウンて呼んでたので、
当初はデグレが？？だった。

**仕様書無しさん** · 2017/02/15(水) 07:28:45.50

>>167
レベルダウンはSIでよく聞くな
５０代、６０代とかの人間がよく使う傾向
異常終了のことをアベンドとか言うし

**仕様書無しさん** · 2017/02/15(水) 07:29:55.38

ｽﾚﾁｽﾏｿ

**仕様書無しさん** · 2017/02/15(水) 08:28:27.81

デグレ出してないか、リグレッションテストするんじゃない？
アベンドは汎用機屋さんが居たような所だと逆に若手にも通じるし便利。

**仕様書無しさん** · 2017/02/15(水) 20:54:23.14

>>170
ここはソースコードの汚いスレだ !

**仕様書無しさん** · 2017/02/15(水) 21:44:03.31

>>171
ここはソースコードが汚いスレだ

**仕様書無しさん** · 2017/02/16(木) 05:35:33.02

ここは酷いインターネッツですね

**仕様書無しさん** · 2017/02/17(金) 12:29:15.15

すまん、1ヶ月前に自分が書いた200行程度のコードが読めん猛者おる？

**仕様書無しさん** · 2017/02/17(金) 12:46:51.51

一週間も経てば初見の気分だし

**仕様書無しさん** · 2017/02/18(土) 01:08:47.93

>>174
ソースは汚いですか？

**仕様書無しさん** · 2017/02/20(月) 23:15:24.51

誰も引き継げないまま次々とプログラマーが辞める
担当者自身も強硬離脱

引き取るハメになった別会社の担当者が発狂

**仕様書無しさん** · 2017/02/22(水) 13:25:28.07

ロマンの無い考古学

**仕様書無しさん** · 2017/02/22(水) 23:41:46.76

メランコリーな博物学

**仕様書無しさん** · 2017/02/27(月) 14:39:37.80

1.技術ブログで有名な会社からコードベース引き継ぐ
2.どんなコードかワクワク
3.設計どころかコピペまみれで規約やフォーマットすら統一されておらず英語のスペルミスもそこら中にあるクソコードだった(署名から判断して書いたのは社員)

リアルにあった話

**仕様書無しさん** · 2017/02/27(月) 15:48:25.80

そりゃ全員が優秀なわけじゃないからな
あと、自分を見詰めてる技術者は少ない

**仕様書無しさん** · 2017/03/02(木) 20:35:13.86

>>177
デスコードと命名しよう

スパゲティの上を行く

**仕様書無しさん** · 2017/03/03(金) 06:11:21.64

>>177
フィクションのようだが実在するから困る

**仕様書無しさん** · 2017/03/03(金) 06:17:18.78

しかも保守(引き取り)なので安い

**仕様書無しさん** · 2017/03/03(金) 20:54:45.83

ソースの汚れを落とすにはどの洗剤がいいですか？
・教育
・リファクタリング
・書き直し
・老害or無能追放
・転職

**仕様書無しさん** · 2017/03/03(金) 21:48:26.44

転職でキレイになるん？

**仕様書無しさん** · 2017/03/04(土) 06:08:55.88

>>186
別の汚れた服を着るだけ

**仕様書無しさん** · 2017/03/04(土) 09:34:14.51

製品の品質を保証するのはテストでコードの読みやすさじゃない

テストしやすいよう機能がきちんと整理されてれば
中身がぐちゃぐちゃで見るに堪えないものでも普通に動く

悲しい現実

**仕様書無しさん** · 2017/03/04(土) 12:19:50.54

>>188
いや、それ当たり前のことなんだが。

だからテスト駆動開発という
テスト→テストを通す最小限のコード→リファクタリング
という流れの開発手法ができたでしょ？

リファクタリングする前のコードだってテスト通すしちゃんと動く。

それから、汚いコードっていうのは、大抵が設計レベルで汚いって意味なんだよ
構造がめちゃくちゃモジュールの構成も意味不明で依存関係もおかしい。
テストしやすいコードにするってことは、設計レベルではきれいになるということ
設計レベルでぐちゃぐちゃだとテストしやすいようはならない。

関数の中身レベルでの汚さってのは大きな問題じゃない。
もちろん問題ないと言っても、関数の中身レベルでちゃんとしたコードを書くのは
プログラマにとってはマナーみたいなものだから、それが出来てないと恥ずかしいけどね。

**仕様書無しさん** · 2017/03/04(土) 14:43:50.74

コードは簡潔で綺麗だけど、
説教モードに入ると長いのは何故？

**仕様書無しさん** · 2017/03/04(土) 15:54:27.16

自分に自信がないんだろう
説明するときに饒舌になる奴はね

**仕様書無しさん** · 2017/03/04(土) 16:22:56.92

>>198
コードはわかっている人（一人前のプログラマ）に向けて書くものだから。
初心者プログラマのために仕事しているわけじゃない。

説教モードが長いのは、分かってない人に向けて書くものだから
初心者プログラマは分かってないことが多いから
それだけ説明の文章も長くなる。

**仕様書無しさん** · 2017/03/04(土) 16:27:19.74

>>192
それはそのとおりだなｗ

プロだと一言、DRY原則に反してるとか
SOLIDとかYAGNIとかデメテルの法則とか言えばそれだけで通じる。

初心者相手だと、その用語がどういう意味かを説明しなきゃいけないし、
もっとひどいと、説明しても利点を理解できないからもっと説明も長くなる

**仕様書無しさん** · 2017/03/05(日) 16:41:15.93

DRYに関しては初期から闇雲に共通化する必要はないと思ってる。
コード上の文脈は同じだけど業務上の文脈は異なる場合も意外に多いので。
そういう場合はあそこの箇所と同じだけどこういう理由から現段階では共通化はしてないとコメントに書くようにしている。(そうしないと他人や忘れた頃にコードを眺めた自分がやらかすかもしれないので)
で、そういうのが増えて来たら改めて業務分析(ってほど大したものでは無くて詳しい人に聞く程度)と何らかの設計パターン適用を検討、実施して初めてDRYにする。

(継続的に成長させたいシステムやサービス的な視点で)綺麗なコードってそういうのの繰り返しで生まれるんじゃないかなと最近は思ってる。
最初から共通化すべき箇所を見極めるのはマジで難しい。

ライブラリやさらに低レイヤーな部分書く人にとってはまた別の綺麗さ(あるいはそれを捨ててでもパフォーマンス優先するとか)の基準があるのだろうけどそっちはあまり分からない。

こういうのは納品したら終わりなやつには絶対に向かない手法だと思ってる。

**仕様書無しさん** · 2017/03/05(日) 16:49:51.96

納品した後で業務分析してリファクタするのか

結合試験もやり直しだろうに
よくOK出るな

**仕様書無しさん** · 2017/03/05(日) 18:01:13.05

共通化とかライブラリとか言ってる時点で…

もっと今風に組もうぜ

**仕様書無しさん** · 2017/03/05(日) 19:06:34.01

>>195
> 結合試験もやり直しだろうに
> よくOK出るな

結合試験でバグが出たらやり直しだろ？
それと何が違うんだ？

**仕様書無しさん** · 2017/03/05(日) 19:40:57.53

今風にDLLだよな

**仕様書無しさん** · 2017/03/05(日) 23:41:14.73

comかもしれん

**仕様書無しさん** · 2017/03/24(金) 03:41:38.71

>>174
半年前くらいなら10分くらいで思い出して余裕で読めるし直せるかなーくらい
1年まえだと少しキョドる

**仕様書無しさん** · 2017/03/24(金) 23:48:46.16

>>175
('A`) ﾋｬｱｱｱｱｱｱｱｱｱｱｱｱｱｱ

**仕様書無しさん** · 2017/06/24(土) 06:52:51.84

ファイルの予備部に値が格納されている。

**仕様書無しさん** · 2017/12/29(金) 20:07:51.27

誰でも簡単にパソコン1台で稼げる方法など
参考までに、
⇒　『宮本のゴウリエセレレ』というブログで見ることができるらしいです。

グーグル検索⇒『宮本のゴウリエセレレ』

AQ6FA9SZ0Y

**仕様書無しさん** · 2018/02/16(金) 01:02:57.15

☆ 私たち日本の、改憲を行いましょう。現在、衆議員と
参議院の両院で、改憲議員が３分の２を超えております。
『憲法改正国民投票法』、でググってみてください。国会の発議は
すでに可能です。平和は勝ち取るものです。お願い致します。☆☆

**仕様書無しさん** · 2018/05/22(火) 14:50:19.49

とても簡単な自宅で稼げる方法
参考までに書いておきます
グーグルで検索するといいかも『ネットで稼ぐ方法　モニアレフヌノ』

WBKFU

**仕様書無しさん** · 2020/03/28(土) 00:25:10.88

いや