アカウント名:
パスワード:
提案するからだめなのだと。するなら依頼か命令で。
「読みにくいから直してくれ」と具体的にどこがダメなのかを明確に
タレコミのようなコード書く人が居るから・ループの入れ子はnこまで・1行n文字以内・1メソッドn行居ない
等などのアホなコーディング規約出てくるのでしょうね。でもそういうので縛るのが単純で効果ありそう。
業務系の画面なんかは美しくないかもしれないけどだらだら書いたほうが読みやすいです。勝手な解釈でやたらめったら分割されたら読み難い。適切で誰でもわかって短くて見分けのつきやすいわかりやす~いメソッド名、引数の変数名がするする出てくる人なんてそうそう居ない。ってか無理でしょう?
たぶん、ワザとだと思うけど「読みにくいから」というのはかなりあいまいな指摘な気がする。
普通にこう書くと、こういう利点があるってのを示していけば取り入れてくれるけどなあ。
「読みにくいから」と言っても「お前のスキルが低いからだ」と返されるパターンかと
静的解析ツールにかけて「ツールがこれだけ文句を言ってきている」と突きつければ良いのでは?
コーディング規約でツールに文句を言われない状態にすることって入れておくといいと思うよ。
底辺で仕事してるとなぁ…。
そういうコードを書く人は、静的解析ツールの指摘内容が理解できないんですよ。もしくは指摘を黙らせるためだけに、更に悪化させるコード修正をしてきます。それが現実。
こういう人はツールに文句を言いますよ。結論はツールを使うな。
実話なのでAC orz
静的解析ツールは参考にはなるが、盲信は非常に危険。静的解析ツールはそこまで賢くない。
その静的解析ツールにもよるけれど、たとえば「一メソッドの行数が100行を超えた。」みたいな警告を出してくるのもある。
行数が長すぎるのは良くない臭いではあるが、例えばやってる条件判定の数が多くて、単純なswitch~caseがずらずら並んだ場合でさえも警告を出してくるので、それに従うのは無意味だ。
これなんかは比較的単純だから無意味であることを説明するのも簡単だけど、判定基準が複雑になればなるほど、無意味であることを(おそらくはプログラムのことを全く分かってない上司に)説明するのに無駄な時間を取られることになる.
>コーディング規約でツールに文句を言われない状態にすることって入れておくといいと思うよ。こういうことをするのはプログラミング経験0の糞マネージャの証。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
192.168.0.1は、私が使っている IPアドレスですので勝手に使わないでください --- ある通りすがり
単刀直入に (スコア:1)
提案するからだめなのだと。
するなら依頼か命令で。
「読みにくいから直してくれ」と
具体的にどこがダメなのかを明確に
Re:単刀直入に (スコア:2)
タレコミのようなコード書く人が居るから
・ループの入れ子はnこまで
・1行n文字以内
・1メソッドn行居ない
等などのアホなコーディング規約出てくるのでしょうね。
でもそういうので縛るのが単純で効果ありそう。
業務系の画面なんかは美しくないかもしれないけど
だらだら書いたほうが読みやすいです。
勝手な解釈でやたらめったら分割されたら読み難い。適切で誰でもわかって短くて見分けのつきやすいわかりやす~いメソッド名、引数の変数名がするする出てくる人なんてそうそう居ない。ってか無理でしょう?
Re: (スコア:0)
たぶん、ワザとだと思うけど「読みにくいから」というのはかなりあいまいな指摘な気がする。
普通にこう書くと、こういう利点があるってのを示していけば取り入れてくれるけどなあ。
Re: (スコア:0)
「読みにくいから」と言っても
「お前のスキルが低いからだ」と返されるパターンかと
Re: (スコア:0)
静的解析ツールにかけて
「ツールがこれだけ文句を言ってきている」
と突きつければ良いのでは?
コーディング規約でツールに文句を言われない状態にすることって入れておくといいと思うよ。
Re:単刀直入に (スコア:1)
底辺で仕事してるとなぁ…。
そういうコードを書く人は、静的解析ツールの指摘内容が理解できないんですよ。
もしくは指摘を黙らせるためだけに、更に悪化させるコード修正をしてきます。
それが現実。
Re: (スコア:0)
こういう人はツールに文句を言いますよ。
結論はツールを使うな。
実話なのでAC orz
Re: (スコア:0)
Re: (スコア:0)
静的解析ツールは参考にはなるが、盲信は非常に危険。
静的解析ツールはそこまで賢くない。
その静的解析ツールにもよるけれど、
たとえば「一メソッドの行数が100行を超えた。」みたいな警告を出してくるのもある。
行数が長すぎるのは良くない臭いではあるが、例えばやってる条件判定の数が多くて、
単純なswitch~caseがずらずら並んだ場合でさえも警告を出してくるので、
それに従うのは無意味だ。
これなんかは比較的単純だから無意味であることを説明するのも簡単だけど、
判定基準が複雑になればなるほど、無意味であることを(おそらくはプログラムのことを
全く分かってない上司に)説明するのに無駄な時間を取られることになる.
>コーディング規約でツールに文句を言われない状態にすることって入れておくといいと思うよ。
こういうことをするのはプログラミング経験0の糞マネージャの証。