アカウント名:
パスワード:
> 同僚の書く酷いコード、どうやって気づかせる?気づいても直らないかも。
気づかせるだけなら、スマートなコードを書いて比べさせればいいと思う。メンテナンスのしやすさ、修正が来たときの直しやすさ、良く使うならクラス化とか、そういうのは比べさせれば、とりあえず気づくと思う。一番いいのはダメージが大きくてもハッキリ言うこと。
でも直らないんじゃなかろうか。私は人並みにしかコードが書けませんが、人並み以下のコードを指摘しても直った試しがないです。
自分もそう思います。この人は1にしか対処できない人で、複数が見えないのだと思います。
1にしか対処できないので、世界をそれで対処可能な様に再定義しないと生きて行けないので、自然と再定義能力に富み、頭がいいと思われているだけかも、知れません。
あきらめて、(本家の)私さんがそのソースの全面的権限を得た時に、リファクタすればよいだけでは?
#人間の認識形態の問題になったら、なんであれ、直らないと思います。
> あきらめて、(本家の)私さんがそのソースの全面的権限を得た時に、リファクタ> すればよいだけでは?
分野にもよるんだろうけど、リファクタリングの機会って多い?「動いているコードは修正するな」なことが多いので、ベースがクソコードだと、それに対する機能追加・bug fixもadhocになりがち。
テストコードの資産が十分にあればリファクタリングしても比較的安全なんだろうけど、クソコードに対するテストコードが十分とは考えにくい...
「ベースがクソコードだと」という条件は、とうの昔にリファクタリングが必要になったにもかかわらず、それは長期間してこなかった結果だから。
そこまで行ったら既に手遅れ。#まるで「末期癌で、もはや手が付けられません。」状態。
もしやるならば、それは「リファクタリング」ではなく「作り直し」。
「リファクタリングの必要があるか?」なら、この場合は「必要があった」だろう。「リファクタリングする機会があるか?」なら「無いことも多い」。
なぜならば、リファクタリングする必要がある時に、それに気付かず手をこまねいているだけで、手遅れになるまで放置する無能なマネージャーがゴロゴロしてるから。
その昔「プレイングマネージャーって"Praying Manager"じゃね?」というのを見て、なるほどと思った。
なぜマネージャーが気付かないんですか?マネージャーがコードレビューしないから?コードの質は誰が把握してるんです?警告は誰が行う事になってるんですか?
#なんだろう、自分で書いていてなぜか胃の辺りが痛い…
ある中小企業での話。
>なぜマネージャーが気付かないんですか?マネージャーがコードレビューしないから?コードレビューどころかコードが読めません。上がってる報告も理解できないと豪語してました。
#というか、コードが読めるマネージャーって、日本に実在するんですか?#せいぜい「プログラミング言語の文法の基礎が分かる」程度では。#ちょっと難しいコードになるとお手上げで、コードの善し悪しを判断するなんて夢のまた夢。
>コードの質は誰が把握してるんです?おそらく私だけでしょう。私以外は誰一人読めなかったので、質が悪いことは他のメンバーも薄々気がついていたでしょうけど。
静的解析ツールとか使おうよ。利用費用は高いけど。潜在的なバグも減るよ。
静的「解析」ツールを使った所で、「バグが減る」わけないじゃん.減らすのは結局プログラマーの仕事なんだよ。解析ツールは何もやってくれない。せいぜいバグ候補をリストアップするだけ。
しかもこれはいたる所に不具合のあるスパゲッティプログラムの場合だから、バグの十や二十を見つけるのに手間はかからん。難しいのは「見つける」ことではなく「治す」こと。
それに静的解析ツールは、バグではない警告もわんさか出してくれるから、そのレベルのコードだと、意味の無い警告の山で役に立たんわ。
下手を打っても無駄な仕事が増えるだけ。#参考までに http://srad.jp/~YoR/journal/561405 [srad.jp]
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
「科学者は100%安全だと保証できないものは動かしてはならない」、科学者「えっ」、プログラマ「えっ」
気づいても直らないかも。 (スコア:5, すばらしい洞察)
> 同僚の書く酷いコード、どうやって気づかせる?
気づいても直らないかも。
気づかせるだけなら、スマートなコードを書いて比べさせればいいと思う。
メンテナンスのしやすさ、修正が来たときの直しやすさ、良く使うならクラス化とか、
そういうのは比べさせれば、とりあえず気づくと思う。
一番いいのはダメージが大きくてもハッキリ言うこと。
でも直らないんじゃなかろうか。
私は人並みにしかコードが書けませんが、人並み以下のコードを指摘しても直った試しがないです。
Re: (スコア:0)
自分もそう思います。この人は1にしか対処できない人で、複数が見えないのだと
思います。
1にしか対処できないので、世界をそれで対処可能な様に再定義しないと生きて
行けないので、自然と再定義能力に富み、頭がいいと思われているだけかも、
知れません。
あきらめて、(本家の)私さんがそのソースの全面的権限を得た時に、リファクタ
すればよいだけでは?
#人間の認識形態の問題になったら、なんであれ、直らないと思います。
Re: (スコア:0)
> あきらめて、(本家の)私さんがそのソースの全面的権限を得た時に、リファクタ
> すればよいだけでは?
分野にもよるんだろうけど、リファクタリングの機会って多い?
「動いているコードは修正するな」なことが多いので、ベースがクソコードだと、
それに対する機能追加・bug fixもadhocになりがち。
テストコードの資産が十分にあればリファクタリングしても比較的安全なんだろうけど、
クソコードに対するテストコードが十分とは考えにくい...
Re: (スコア:2)
「ベースがクソコードだと」という条件は、
とうの昔にリファクタリングが必要になったにもかかわらず、
それは長期間してこなかった結果だから。
そこまで行ったら既に手遅れ。
#まるで「末期癌で、もはや手が付けられません。」状態。
もしやるならば、それは「リファクタリング」ではなく「作り直し」。
「リファクタリングの必要があるか?」なら、この場合は「必要があった」だろう。
「リファクタリングする機会があるか?」なら「無いことも多い」。
なぜならば、リファクタリングする必要がある時に、それに気付かず手をこまねいているだけで、
手遅れになるまで放置する無能なマネージャーがゴロゴロしてるから。
その昔「プレイングマネージャーって"Praying Manager"じゃね?」というのを見て、なるほどと思った。
Re: (スコア:1)
なぜマネージャーが気付かないんですか?マネージャーがコードレビューしないから?
コードの質は誰が把握してるんです?
警告は誰が行う事になってるんですか?
#なんだろう、自分で書いていてなぜか胃の辺りが痛い…
Re: (スコア:1)
ある中小企業での話。
>なぜマネージャーが気付かないんですか?マネージャーがコードレビューしないから?
コードレビューどころかコードが読めません。
上がってる報告も理解できないと豪語してました。
#というか、コードが読めるマネージャーって、日本に実在するんですか?
#せいぜい「プログラミング言語の文法の基礎が分かる」程度では。
#ちょっと難しいコードになるとお手上げで、コードの善し悪しを判断するなんて夢のまた夢。
>コードの質は誰が把握してるんです?
おそらく私だけでしょう。
私以外は誰一人読めなかったので、質が悪いことは他のメンバーも薄々気がついていたでしょうけど。
Re:気づいても直らないかも。 (スコア:0)
静的解析ツールとか使おうよ。利用費用は高いけど。潜在的なバグも減るよ。
Re: (スコア:0)
Re: (スコア:0)
静的「解析」ツールを使った所で、「バグが減る」わけないじゃん.
減らすのは結局プログラマーの仕事なんだよ。
解析ツールは何もやってくれない。せいぜいバグ候補をリストアップするだけ。
しかもこれはいたる所に不具合のあるスパゲッティプログラムの場合だから、
バグの十や二十を見つけるのに手間はかからん。
難しいのは「見つける」ことではなく「治す」こと。
それに静的解析ツールは、バグではない警告もわんさか出してくれるから、
そのレベルのコードだと、意味の無い警告の山で役に立たんわ。
下手を打っても無駄な仕事が増えるだけ。
#参考までに http://srad.jp/~YoR/journal/561405 [srad.jp]