2ちゃんねる スマホ用 ■掲示板に戻る■ 全部 1- 最新50    

■ このスレッドは過去ログ倉庫に格納されています

コードが汚いと開発・修正の工数が10倍になると判明

1 :仕様書無しさん:2015/01/30(金) 21:36:52.73 .net
納期に苦しんでいるSEさま必見の情報です。
今すぐコードが綺麗か汚いかを
ツールを使って調査してみましょう

317 :仕様書無しさん:2016/03/13(日) 22:39:39.12 .net
>>316
だからレビューしろって。

どこが担当しようが、レビューをしなければ
コードの品質は統一されないの当たり前じゃないか。

コードを書くっていうのは設計をやってるんだぞ。

318 :仕様書無しさん:2016/03/13(日) 23:28:07.58 .net
>>317
それじゃない

319 :仕様書無しさん:2016/03/13(日) 23:29:38.24 .net
> スゲー人が書いたスゲーコードを別のスゲー人がメンテしてスゲーコードを書いたら
> 結果スゲーメンテしづらいコードになる事象が世に溢れている

それが本当なら、そうなったものを見てみたいね(笑)


なんなら君、書いてくんない?w

320 :仕様書無しさん:2016/03/14(月) 00:34:26.77 .net
まあ、スゲーの方向性にもよるな
スゲー人力オプティマイズしたアセンブラとか

でもまあ、話の流れとか行間くらい読んでくれとは思う

321 :仕様書無しさん:2016/03/17(木) 00:49:29.74 .net
>>319
お前がメンテしたコードにもあったと思うよ
自分が当事者だと気が付かないのかやっぱり

322 :仕様書無しさん:2016/03/17(木) 01:10:41.52 .net
>>321
そのレスは間違いだね。
なんで言い訳するの?

323 :仕様書無しさん:2016/03/17(木) 03:39:40.60 .net
綺麗とか汚いとか、定量化出来ない話で盛り上がれるのは技術者じゃ無い。

324 :仕様書無しさん:2016/03/17(木) 12:20:35.01 .net
>>322
なに言ってんだお前

325 :仕様書無しさん:2016/03/17(木) 15:41:25.87 .net
俺が思う、メンテできない酷いコード(パート1)

・タブ、空白のインデントがあちこちで乱れている
とにかく汚ねえ!

326 :仕様書無しさん:2016/03/17(木) 16:29:01.26 .net
コードの見た目に関しては
わりと、どうでもいい。
フォーマッタで解決。

327 :仕様書無しさん:2016/03/18(金) 20:27:55.03 .net
余計なこといろいろ書いていたらフォーマッタでも解決できない。

328 :仕様書無しさん:2016/03/18(金) 20:42:55.79 .net
フォーマッタで整形出来ないってw
それ、コンパイルは通るのか?

329 :仕様書無しさん:2016/03/18(金) 20:58:40.22 .net
引数やローカル変数がたくさん並んでるコードは、バグもたくさん並ぶ

330 :仕様書無しさん:2016/03/18(金) 21:01:25.43 .net
>>327はどういうケースなんだろうか
具体例が知りたい
草生やしてもなんにもならんし

331 :仕様書無しさん:2016/03/18(金) 21:04:27.78 .net
>>329
個人的には、引数やローカル変数は4つ以上だと臭うなぁ
10以上とかはもう見るのも嫌

332 :仕様書無しさん:2016/03/18(金) 21:07:14.94 .net
>>331
引数は少ないに越した事は無いが、ローカル変数はむしろ多い方がいいぞ。
いっそ引数以外は全部ローカル変数にしなさい。

333 :仕様書無しさん:2016/03/18(金) 21:30:50.90 .net
ローカルがやたら多いのは機能をうまく整理・分割できてなくて、
関数内がコボル状態だろう?

334 :仕様書無しさん:2016/03/18(金) 21:36:56.26 .net
グローバル変数が多いよりよっぽどまし。

335 :仕様書無しさん:2016/03/18(金) 22:26:09.43 .net
参照可能な変数がたくさん並んでるコード(ry

メソッドの頭からお尻が巨大なtryで囲まれたコードは、人力トライ&エラーの無限ループに陥る

336 :仕様書無しさん:2016/03/18(金) 22:59:20.09 .net
local variablesにはimmutable、mutableがあるよね

mutableなのはなるべく少ない方がいいと思う
でも、なるべく少なくする為に、むやみにmethod分割するのはアンチパターンが多い気がする
関数型なlibraryや、block scopeを上手く使えば、自然と減ってくる

一方、immutableの方は、多少多くても、
それが適切なモノなら、まいっかって感じ

337 :仕様書無しさん:2016/03/18(金) 23:02:00.00 .net
>>333
コボル状態って・・・
それが何か分かってしまう俺がもう嫌

コボラーの人の書いた別の言語のコードって、それが特徴的なんだよなぁ

338 :仕様書無しさん:2016/03/18(金) 23:05:27.31 .net
>>336
youはlocal variablesよりもtoo manyなenglish wordsをsomehowした方がbetterかと

339 :仕様書無しさん:2016/03/19(土) 00:47:32.92 .net
>>338
youはsomehowという間違った英語をsomehowした方がいいと思う

340 :仕様書無しさん:2016/03/19(土) 11:07:36.28 .net
>>337
なるほどあれはコボル由来の書き方だったのか

341 :仕様書無しさん:2016/03/19(土) 11:34:35.89 .net
ZEROとかONEとかの定数が並ぶコードを書く奴は業界から追放すべき

342 :仕様書無しさん:2016/03/19(土) 15:20:40.58 .net
リファクタリングにもローカル変数が増えてきたらメソッド分ける指標って書いてあったな。

短いスコープならローカル変数はあまり気にならないけど、スクロール必要なスコープで宣言されてるとレビューしててイラっとくる。

343 :仕様書無しさん:2016/03/20(日) 02:15:30.33 .net
cssでこういう↓汚いゴミを書く奴

<div class="width-100 font-size-15">

.width-100 {
width: 100%;
}
.font-size-15 {
font-size: 15px;
}

344 :仕様書無しさん:2016/03/22(火) 16:34:15.62 .net
【偽装請負搾取盗難】

作業やめて盗難届け出せよ!

盗難被害者の例
発注者 支払 140万円 1億円の大儲け
1次受注者 報酬 120万円 20万円の盗難被害額
2次受注者 報酬 80万円 60万円の盗難被害額
3次受注者 報酬 60万円 80万円の盗難被害額

345 :仕様書無しさん:2016/03/24(木) 01:27:24.98 .net
吉野健太郎の卑怯なTwitterで検索しよう

346 :仕様書無しさん:2016/04/13(水) 21:26:36.02 .net
動きゃいいんだろ動きゃ

347 :仕様書無しさん:2016/04/24(日) 13:09:04.87 .net
工数見積りの海を彷徨う
http://hidekatsu-izuno.hatenablog.com/entry/2016/04/24/035446

この手の工数、工期という話題の時、役に立つのは次の資料だ。
IPA ソフトウェア開発データ白書
JUAS ソフトウェアメトリックス調査
素晴らしいことにどちらも PDF 版は無料で配布されているので、ダウンロードして見ることができる。
システム開発サイドだけでなく、エンドユーザ側でも有用な資料だと思う。

348 :仕様書無しさん:2016/04/24(日) 13:11:30.49 .net
問題は、工数見積りだ。工数は単価をかければそのまま金額になるわけだし、標準工期のベースともなる。
一番重要な指標と言える。しかし工数については、データを見ても分散が大きく、
IPAでも、あくまで目安として50%の信頼幅に収まっているかを見るのに使ってください、というスタンスを取っている。

JUASはもう少し踏み込み、毎年データから回帰分析を行い総工数の見積りに使える式を算出しているのだが、
各年の結果がかなり異なる。例えば、2007年度版では、総工数=1.55×画面数だと書かれている一方、
2009年度版では、総工数=1.09×画面数となっている。ここ数年の版に至っては、画面数だけの分析は削除され、
画面と帳票での分析のみが掲載されている。同じ機能数なのに工数が50%も違うのでは、見積りチェック用途だとしても使いづらい。

とはいえ以前、システム開発はもっと明朗会計にならなければいけないでも書いたように、FP法と総工数には、
分散はあるものの明らかに正の相関が認められる。FP法は「機能」をポイント化しているわけだから、
工数は機能を正しく見積ることができればある程度予測できることが想像できる。

このことから考えると、面倒ではあるが、まじめに FP を求めろという話ではあるのだが、
実際には FP 法での見積りは次のような理由があって難しい。

概算見積りを求められるのは、RFP(Request For Proposal)提示時など要件定義前の
早いタイミングでありエンティティと言った詳細な設計まではとても落とせない。

FP法は、機能ごとの見積りではないため、機能数の削減が工数に与える影響を見積るのが極めて困難である。
この機能を削るからいくら安くします、などという交渉が難しくなる。

349 :仕様書無しさん:2016/05/03(火) 15:35:39.88 .net
匿名通信(Tor、i2p等)ができるファイル共有ソフトBitComet(ビットコメット)みたいな、
BitTorrent(Covenant)が活発な情報交換・交流コミュニティでオープンソース開発されています(プログラマー募集中)

言語は何でも大丈夫だそうなので、P2P書きたい!って人居ませんか?

Covenantの作者(Lyrise氏)がそういう人と話したいそうなので、よろしければツイートお願いします<(_ _)>
https://twitter.com/Lyrise_al

ちなみにオイラはCovenantの完成が待ち遠しいプログラミングできない情報発信好きアスペルガーw


通話料が激安になるブラステル(050 Free)で、かなり遅延や音声途切れが発生する方は、以下の設定を試してください
○ Wifiと3Gのコーデックは2つ(GSM、G.711u-Law)とも有効にしておく
○ エコーキャンセルをOFF(チェックを外す)にする
○ あとの設定はデフォルトのまま
http://blog.livedoor.jp/gnunobian/archives/52013458.html
上記の設定でも音質が悪い方は、wolfsonの高音質チップを搭載した機種(Galaxy 初代S、S3、S6、 AQUOSPhone ZETA SH-06E、AQUOSPhone si SH-07E、AQUOSPhone Xx 206SH、 Galaxy Note II)に買い換えて下さい。

500円以下の格安SIMで使えて登録・月額無料、IPベース発信なら携帯へは5.5円/30秒、固定へは8円/3分(月額無料でこの価格はすごい!)
http://blog.jikoman.jp/2015/11/brastel-050-free.html

あと、050Freeの起動もしくは発着信が2週間以上ないとプッシュサーバー期限切れでプッシュ着信が出来なくなるので、Llama Location Profilesで1週間に一度050Freeを自動起動するように設定すると、2週間以上経過してもプッシュ着信できます


最後にロケットストーブの焚き口へ超省電力なDC扇風機で風を送ると、横引き煙突が12m以上あっても煙が逆流してきません。
よって、横引き煙突で超高効率な熱回収ができるので薪が少量で済みます
あと、燃焼室の大きさは『無煙竹ボイラMBG150』で検索して参考にして下さい
http://i.imgur.com/iVuglg9.jpg 
http://jp.misumi-ec.com/material/mech/KRT1/PHOTO/KRT1_221004926837.jpg
http://livedoor.blogimg.jp/zoukibayashinokai/imgs/2/a/2a3c6dc0.jpg


350 :仕様書無しさん:2017/06/08(木) 21:17:42.88 .net
素晴らしいね

351 :仕様書無しさん:2017/12/29(金) 22:14:01.49 .net
誰でも簡単にパソコン1台で稼げる方法など
参考までに、
⇒ 『宮本のゴウリエセレレ』 というブログで見ることができるらしいです。

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

ZTZH3R583H

352 :仕様書無しさん:2018/04/17(火) 07:26:56.74 .net
>>343
ごめんなさい、一カ所変えれば連動させたい部分の調整中にたまにやる

353 :仕様書無しさん:2018/05/22(火) 13:04:42.81 .net
とても簡単な自宅で稼げる方法
参考までに書いておきます
グーグルで検索するといいかも『ネットで稼ぐ方法 モニアレフヌノ』

W3427

354 :仕様書無しさん:2019/01/21(月) 18:36:08.72 .net
具体的にどのように数字を出したのか
論文にして

355 :仕様書無しさん:2019/01/21(月) 20:54:47.75 .net
スキルと時間

356 :仕様書無しさん:2019/01/21(月) 21:18:46.86 .net
汚い方ときれいな方を同時に同じ技術力のプログラマがやって試したの?

357 :仕様書無しさん:2019/01/21(月) 21:22:27.47 .net
あたりまえだのくらっかー
コード酷いと機能拡張に耐えられなくて下手すりゃ作り直しだからな

358 :仕様書無しさん:2019/01/21(月) 21:46:39.50 .net
試したの?

359 :仕様書無しさん:2019/01/22(火) 00:00:57.98 .net
競技プログラミングは害悪ってことかい?

360 :仕様書無しさん:2019/06/19(水) 12:40:56.24 .net
きれいなコードより単純なコード
必然的にコード量は少なくなり生産性が向上する。

361 :仕様書無しさん:2019/08/08(木) 20:42:47.58 .net
下手すりゃ、リファクタリングという言葉を知らないプログラマーもいそうだよな

362 :仕様書無しさん:2019/08/08(木) 20:59:03.45 .net
働かない死んでるコードを大量に残してる奴とかな

363 :仕様書無しさん:2019/08/09(金) 07:30:12.28 .net
コードの汚さって、ゴミが残されてるのも一つだけど、設計的な意味で中身が汚い。
実際リファクタリングの必要性を感じるコードというのは初心者に多く
そもそもの作り方に難があり、作成者本人でさえ仕様変更に耐えない
SEの要望を満たせないなど、主に保守性に致命的な問題があると
判断されたコードを直す場合がほとんどだね。
作成者を含み誰かしらサラサラっと難なく対応できるなら保守性の高いコード。

364 :仕様書無しさん:2019/08/22(木) 01:17:42.38 .net
同僚が、1人で1から書いたプログラムを実行しようとしたら、うまく動かなくてソース見たら、
1つの変数の中でキャメルケースとスネークケースと、ローマ字変数名と英語変数名が混ざった上に、ログの出力時刻を固定値代入して、ずっとその時刻使ってログを吐くように書いてあった

他にも色々酷い有様

基地外なのか!?

365 :仕様書無しさん:2020/01/30(木) 07:56:58 .net
【犯罪】無能時間外労働違反SEの追放【損害】
☆不利益で迷惑だから料金増やすか生産減らせ☆
【契約料金や知的財産の生涯損害促進者ばかり】
[偽装請負多重派遣の従犯SEを追放すべき]
偽装請負多重派遣SEの動機
コミュニケーション障害
コンピュータ趣味
人格障害
文系大卒
低偏差値大卒
情報処理資格非保有者
偽装請負多重派遣SEの迷惑
無償プログラム提供
事前面接
契約外期限遵守
客先指示遵守
知的財産譲渡
中間搾取促進
時間外労働違反
低予備工数見積
残業見積
無料追加
学習不足
裁判苦手
対人障害
健康障害
孤独死
偽装請負多重派遣SEの代償
低収入低技術
非婚離婚
鬱病早死

366 :仕様書無しさん:2020/02/03(月) 20:55:09 .net
競技プログラミングでシンプルなコーディングを身に着けよう!

総レス数 366
90 KB
掲示板に戻る 全部 前100 次100 最新50
read.cgi ver 2014.07.20.01.SC 2014/07/20 D ★