ソースコードが汚いことで発生する問題点

1 ：仕様書無しさん：2016/06/01(水) 00:51:12.80 .net

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

2 ：仕様書無しさん：2016/06/01(水) 03:42:41.54 .net

ソース汚いのは初心者にありがち。

3 ：仕様書無しさん：2016/06/01(水) 05:15:53.72 .net

従業員が辞めていく

4 ：仕様書無しさん：2016/06/01(水) 07:14:50.79 .net

Fワードが職場に蔓延する

5 ：仕様書無しさん：2016/06/01(水) 10:26:08.61 .net

>>1
ちょっとした修正でも、ソース解析やテストに膨大な時間がかかるから、ユーザにとっても保守費用がかさむ。

おそらく、きれいに作っていれば、DBのデータとか設定ファイルの変更で対応できそうなことを、
数日かけてソース解析して改修してるシステムを実際に見たことがある。
正社員で入ってきたプログラマが入社1週間くらいで辞めてた。

6 ：仕様書無しさん：2016/06/01(水) 13:50:27.02 .net

やる気がでない

7 ：仕様書無しさん：2016/06/01(水) 14:10:28.90 .net

まさに今の俺
資料はおろかコメントすらないVB6.0のプロジェクトを引き継いじまった

8 ：仕様書無しさん：2016/06/01(水) 14:50:45.69 .net

ソースが仕様です！ (｀・ω・´)ｷﾘｯ

9 ：仕様書無しさん：2016/06/01(水) 15:19:46.40 .net

きれいに作ってたら納期に間に合わん。
短期で過剰の成果を要求するな。

10 ：仕様書無しさん：2016/06/01(水) 19:17:15.73 .net

>>9
開発と保守の違いがわかってないおばかさん

11 ：仕様書無しさん：2016/06/01(水) 20:51:12.62 .net

if() {
if() {
if() {
if() {
if() {
if () {
}
}
}
}
}
}

12 ：仕様書無しさん：2016/06/01(水) 21:14:13.31 .net

汚い奴ソースを書く人は概して頭が悪い
コードスメルを認識できないほと頭が悪い

13 ：仕様書無しさん：2016/06/01(水) 22:26:06.73 .net

>>9
今はソースコードが汚いことで発生する問題点の話をしている。
綺麗にするかどうかは別問題

14 ：仕様書無しさん：2016/06/01(水) 22:26:57.79 .net

そもそも最初から綺麗に作ればいいわけで

15 ：仕様書無しさん：2016/06/01(水) 22:28:44.04 .net

汚く作れるのってある意味才能だよな

16 ：仕様書無しさん：2016/06/01(水) 22:40:12.92 .net

>>10
大抵の開発は、保守と同じようなことしてるけどな。
プロトタイプの開発でもない限り。

バグがあったら修正するわけだし、
ソースコードが汚いと自分で書いたコードに苦しめられるｗ

17 ：仕様書無しさん：2016/06/02(木) 08:26:59.19 .net

【主な偽装請負従犯SEの作業】
[技術不要の使い捨てスキル]
コマンド
データ > ロジック
簡単ロジック
大量データ
SE適性不要
IT資格不要
大卒資格不要
文科系対象
体育系対象
商業系業種
業務系処理

[業務ソフト作り捨てソフト]
ノンプログラミングツール
フレームワーク
COBOL
VＢ
.net
Java
Web
DB
ERP
SAP7

18 ：仕様書無しさん：2016/06/02(木) 18:29:38.03 .net

仕様が曖昧で二転三転する中、なんとかそれっぽいものをでっち上げたら、
それぞれのメソッドからの返り値が迷路のように複雑に絡み合う、
超難解ソースが出来上がりましたとさ。
メンテ不能だから、これ。

19 ：仕様書無しさん：2016/06/02(木) 20:50:49.25 .net

それは仕様変更の時に、機能を満たす事だけ考えて、
「まいっか、後で直せばいいや」と、場当たり的な対応してるからです

場当たり的な対応をせざるを得ない場合であっても、
メモるなり、TODO書くなりすればよいのです

後に直す事を覚えて、徐々に直していけば、どんなに仕様が二転三転しても、
そこまで酷くはならないです

もう一度繰り返します

殆どのケースでは「まいっか、後で直せばいいや => メモを取らない」が元凶です

20 ：仕様書無しさん：2016/06/03(金) 00:40:07.68 .net

そもそも、頭が悪いと何が綺麗なコードで何が汚いコードなのか分からん

21 ：仕様書無しさん：2016/06/03(金) 01:00:09.32 .net

>>19
動いたからOK。
あとで直す？何を？動いているのに？

22 ：仕様書無しさん：2016/06/03(金) 02:08:30.83 .net

>>19
TODO: だらけのコードになるわけですね

23 ：仕様書無しさん：2016/06/03(金) 03:30:35.82 .net

クソコード書く奴等ってなんでまともな形にせずに逃げるの？

24 ：仕様書無しさん：2016/06/03(金) 04:57:37.11 .net

契約終了と言ったのはそちらですよ

25 ：仕様書無しさん：2016/06/03(金) 05:23:26.56 .net

>>23
俺も長年疑問に思っていたが、だんだんわかってきたよ
彼らは"まともな形"が分からない人たちなんだよ
「動いたこれでよし」と思うからそれで終わってしまう
>>21みたいに、何を直すのか分からない
あなたは知ってそうだから、それでいい

26 ：仕様書無しさん：2016/06/03(金) 20:09:19.88 .net

>>25
それは違うよ
派遣が多いから「どうせずっとかかわるシステムじゃないし」と思って雑になるんだよ
ソースが汚くても苦労するのは自分じゃないからね
酷い現場だとわざと汚くしたりするのさ

27 ：仕様書無しさん：2016/06/03(金) 20:58:14.48 .net

たぶん>>25と>>26の両方だな
俺がいいとこ取り

28 ：仕様書無しさん：2016/06/03(金) 21:31:21.97 .net

>>26
> 派遣が多いから「どうせずっとかかわるシステムじゃないし」と思って雑になるんだよ

その理屈だと、なんで正社員にさせないの？って
聞かれるだけだよ。

そうすれば本当の理由がわかるだろ？
金がないから、派遣に頼らざるをえない。
金がないから、雑になる。

29 ：仕様書無しさん：2016/06/03(金) 21:39:13.77 .net

自宅で書くコードは美しいんだけど職場だとどうしても汚くなる
仕事だとしがらみが多すぎてな

30 ：仕様書無しさん：2016/06/03(金) 21:40:29.04 .net

インデントすらできない奴とか死んだ方がいいよ

31 ：仕様書無しさん：2016/06/03(金) 21:44:10.27 .net

インデントはVisualStudioが全部やってくれるだろ

32 ：仕様書無しさん：2016/06/03(金) 23:51:12.65 .net

「ソースコードが汚い」と大声で騒ぎ立てる馬鹿が湧く

33 ：仕様書無しさん：2016/06/04(土) 00:57:25.14 .net

>>32
店が汚いと「店が汚い」と大声で叫ぶ客が出てくると思うが、
誰が悪いんだい？

34 ：仕様書無しさん：2016/06/04(土) 09:08:41.02 .net

>>33
汚いくて悪いのは店だが客はキチガイだな

35 ：仕様書無しさん：2016/06/04(土) 09:14:31.91 .net

「誰が悪いんだい」だってｗｗｗ
こいつもキチガイクレーマーなんかなｗ

36 ：仕様書無しさん：2016/06/04(土) 09:23:21.74 .net

>>1
解析に時間がかかり、改修方法が複雑。

37 ：仕様書無しさん：2016/06/04(土) 10:01:31.31 .net

SEの知的財産と契約料金の搾取対策

早死に貧困の助長だから偽装請負の従犯は辞めろ！
相場下がって迷惑だから報酬増やすか作業減らせ！

・IT社長に贅沢資金を搾取させるな
・平均年齢40歳未満の会社は辞めろ
・1,000万円/年以下の会社は辞めろ
・80万円/月以下の契約は辞めろ
・5,000円/時間以下の契約は辞めろ
・6時間/日以上のPC使用は辞めろ
・100万円/月以下のプログラムは作るな
・多重契約は辞めろ
・残業見積りは辞めろ
・時間外労働違反は辞めろ
・契約外作業期日は守るな
・客先指示に従うな
・知的財産を渡するな
・不利益な依頼は断れ
・残業しないで学習しろ
・残業しないで副業しろ
・損害は訴えろ

【非婚】SI受注SEは3億円以下の低生涯収入【離婚】
http://hanabi.2ch.net/test/read.cgi/infosys/1451213054/
2

38 ：仕様書無しさん：2016/06/04(土) 19:22:38.02 .net

理論は完璧そうなあいつほど
ソースがクソ汚いはありがち

39 ：仕様書無しさん：2016/06/04(土) 19:34:57.67 .net

>>38
そういうやつは文章を書かせると難解な文章を書くから見抜けるよ。

40 ：仕様書無しさん：2016/06/04(土) 20:08:03.93 .net

開発コストを減らせばコードは汚くなり保守コストが上がる
つまり開発と保守を別の会社にやらせると資本主義の原則に則ってコードが汚くなる

41 ：仕様書無しさん：2016/06/04(土) 20:27:23.24 .net

コーダー飼うコストとか全体から見れば誤差の範囲なのですがw

42 ：仕様書無しさん：2016/06/10(金) 03:37:11.48 .net

綺麗に書いたコードでも3ヶ月後には汚くみえる
ただ元々汚いコードは3ヶ月後には解読不可能になる

43 ：仕様書無しさん：2016/06/10(金) 12:36:52.41 .net

但しバカに限る

44 ：仕様書無しさん：2016/06/10(金) 20:46:17.90 .net

>>39
頭混乱してるから文章回りくどいよな

45 ：仕様書無しさん：2016/06/10(金) 21:21:27.38 .net

汚いと解析できないから、改修時にビクビク怯えて、
コピペで追加する汚いコードが増える
綺麗なコードは簡単に見てわかるから、手を入れられる

46 ：仕様書無しさん：2016/06/10(金) 23:04:56.06 .net

改修のしやすさは綺麗汚いじゃなくて設計と規約が良くできてるかどうかだろ

47 ：仕様書無しさん：2016/06/10(金) 23:16:28.41 .net

設計と規約が良くできていることを
綺麗だって表現するんだよ

水が飲めるかどうかは、綺麗汚いじゃなくて、不純物が少なく透き通っているかどうかだろ
って言ってるようなもの。

48 ：仕様書無しさん：2016/06/10(金) 23:53:22.95 .net

つまり美少女はウンコしないのに美少女のウンコなら食えるって言うようなものだな、了解した

49 ：仕様書無しさん：2016/06/11(土) 02:18:44.63 .net

俺がウンコードの解析させられてハゲる

50 ：仕様書無しさん：2016/06/11(土) 08:47:14.85 .net

SEの知的財産と契約料金の搾取対策

早死に貧困の助長だから偽装請負の従犯は辞めろ！
相場下がって迷惑だから報酬増やすか作業減らせ！

・IT社長に贅沢資金を搾取させるな
・平均年齢40歳未満の会社は辞めろ
・1,000万円/年以下の会社は辞めろ
・80万円/月以下の契約は辞めろ
・5,000円/時間以下の契約は辞めろ
・6時間/日以上のPC使用は辞めろ
・100万円/人月以下のプログラムは作るな
・多重契約は辞めろ
・残業見積りは辞めろ
・時間外労働違反は辞めろ
・契約外作業期日は守るな
・客先指示に従うな
・知的財産を渡するな
・不利益な依頼は断れ
・残業しないで学習しろ
・残業しないで副業しろ
・損害は訴えろ

【非婚】SI受注SEは3億円以下の低生涯収入【離婚】
http://hanabi.2ch.net/test/read.cgi/infosys/1451213054/
3

51 ：仕様書無しさん：2016/06/11(土) 10:16:03.48 .net

下書きをしてからペン入れや、習作をいくつか作ってから本番のを書くとか
試行錯誤をしてからアウトプットを作るとかいう考えに思い至らない人が
多分多いんだろうね。試行錯誤をしたものをそのまま出してもある程度
動くなら問題が出ないことの方が多いし。 手法としてはプロトタイピングが
近いのだが、あれでも捨ててもいいよとか捨てろとか言っても頑なに拒否
する人は少ないからなあ。

52 ：仕様書無しさん：2016/06/19(日) 17:11:53.62 .net

保守性下がるんだよね

53 ：仕様書無しさん：2016/07/04(月) 23:09:05.12 .net

>>51
初心者は作り始めてから、あーでも、こーでもないとやってぐちゃぐちゃになる。

普通は試行錯誤のこの過程を経て、成長していくんだか、なぜか成長せず、クソコードを作りつづける人間がいる。

54 ：仕様書無しさん：2016/07/05(火) 08:23:42.76 .net

年収1,000万円以下の低レベルPGへ

SEの低生涯収入と短勤続年数の対策を考えろよ！
相場下がって迷惑だから交渉するか作業減らせ！
生産下がって迷惑だから技術は報酬で評価しろ！

[推定平均生涯収入]
100万/月 3億5,000万円以上(大卒サラリーマン上位レベル)
90万/月 3億円以上(大卒サラリーマン平均レベル)
80万/月 2億5,000万円以上(大卒サラリーマン下位レベル・高卒サラリーマン上位レベル)
70万/月 2億円以上(高卒サラリーマン平均レベル)
60万/月 1億5,000万円以上(高卒サラリーマン下位レベル)
50万/月 1億円以上(フリーターレベル)
40万/月 5,000万円以上(パートレベル)4

55 ：仕様書無しさん：2016/07/05(火) 16:00:31.84 .net

コード書きながらこれ汚いなって思うけど、ローンチしたらどうせプロジェクトを外れるわけで、
わざわざ時間をかけて綺麗にしようというインセンティブがない

56 ：仕様書無しさん：2016/07/05(火) 16:07:03.85 .net

>>55
開発プロジェクトを開発が終わったらはずれる立場を繰り返していると運用・保守の観点がなかなか身につかない。

57 ：仕様書無しさん：2016/07/05(火) 18:24:36.38 .net

>>56
運用保守を含む契約での仕事もしてるから大丈夫
>>55は開発で契約したときの話

58 ：仕様書無しさん：2016/07/06(水) 07:53:05.09 .net

【主な偽装請負従犯SEの作業】
[技術不要の使い捨てスキル]
コマンド
データ > ロジック
簡単ロジック
大量データ
SE適性不要
IT資格不要
大卒資格不要
文科系対象
体育系対象
商業系業種
業務系処理

[業務ソフト作り捨てソフト]
ノンプログラミングツール
フレームワーク
COBOL
VＢ
.net
Java
Web
DB
ERP
SAP8

59 ：仕様書無しさん：2016/07/06(水) 10:54:49.28 .net

まぁ2度と関わらない会社の案件ならそうなるわな

60 ：仕様書無しさん：2016/07/06(水) 19:30:33.13 .net

ソースが汚くて保守せいが悪い？
工数が増えていいことじゃないか。

61 ：仕様書無しさん：2016/07/06(水) 19:39:30.37 .net

ソースが汚ないぐらいで工数が増えるってどんだけ低スキルなんだよw

62 ：仕様書無しさん：2016/07/06(水) 20:07:06.66 .net

ＦＪネクスト不買運動で検索すると分かります

63 ：仕様書無しさん：2016/07/06(水) 20:24:19.84 .net

汚いだけなら整形アプリで整えちまえば済む。

64 ：仕様書無しさん：2016/07/06(水) 23:31:05.95 .net

>>63
「汚いコードとは、整形されていないコードのことなのだ！」

65 ：仕様書無しさん：2016/07/07(木) 08:21:09.86 .net

無能ITドカタへ

無能残業・低価格化・健康障害・対人障害のせいだろ！
相場下がって大迷惑だから技術評価は報酬金額で表せ！

ＳＥの異常レベルを立証
正社員の人手不足業界ランキング
1位：情報サービス　59.3%
2位：建設 54.6%
3位：医薬品・日用雑貨品小売 53.6%
4位：放送　53.3%
5位：旅館・ホテル　52.8%
6位：人材派遣 52.6%
7位：運輸・倉庫　50.0%
8位：金融　49.1%
9位：専門サービス　48.3%
10位：メンテナンス・警備　48.1%

人手不足業界は独身率も高い
http://raorsh.com/hitode

66 ：仕様書無しさん：2016/07/09(土) 15:26:25.22 .net

C言語での開発なのにアセンブラとgotoの荒らし

かっこいい

67 ：名無しさん＠そうだ選挙に行こう！ Go to vote!：2016/07/10(日) 13:39:17.75 .net

>>60
正確に見積もれる程度の汚さで収まってるならな。
実際は地雷踏まされたり、延焼したり、二次遭難したりするのが普通だからなあ。

68 ：仕様書無しさん：2016/07/11(月) 18:04:56.13 .net

インデントがめちゃくちゃで｝が余計にあるのか足りないのかわからなくなる

69 ：仕様書無しさん：2016/07/11(月) 18:21:44.82 .net

名前と意味と役割が合ってないと困るな
isXxxとisNotXxxが逆の意味とか
未使用だったDBカラムに全く別の意味持たせたりとか
そいつの書いた物が全部信用出来なくなる

70 ：仕様書無しさん：2016/07/13(水) 20:12:56.89 .net

if文のなかで目茶苦茶に5個くらいandとorが入り交じった
わけわからん判定文がある

それが1つのファイル内に30個以上ある

71 ：仕様書無しさん：2016/07/13(水) 21:02:20.21 .net

ふっ
そんな条件式の途中がコメントアウトされてて
似た様な式が幾つもあるみたいな奴なんかザラだぜw

72 ：仕様書無しさん：2016/07/13(水) 21:25:06.75 .net

>>70
普通はネストでわかりやすく分割するんだけどな。

73 ：仕様書無しさん：2016/07/13(水) 21:50:55.33 .net

>>68
あるあるw