アカウント名:
パスワード:
提案するからだめなのだと。するなら依頼か命令で。
「読みにくいから直してくれ」と具体的にどこがダメなのかを明確に
「読みにくいから」と言っても「お前のスキルが低いからだ」と返されるパターンかと
静的解析ツールにかけて「ツールがこれだけ文句を言ってきている」と突きつければ良いのでは?
コーディング規約でツールに文句を言われない状態にすることって入れておくといいと思うよ。
静的解析ツールは参考にはなるが、盲信は非常に危険。静的解析ツールはそこまで賢くない。
その静的解析ツールにもよるけれど、たとえば「一メソッドの行数が100行を超えた。」みたいな警告を出してくるのもある。
行数が長すぎるのは良くない臭いではあるが、例えばやってる条件判定の数が多くて、単純なswitch~caseがずらずら並んだ場合でさえも警告を出してくるので、それに従うのは無意味だ。
これなんかは比較的単純だから無意味であることを説明するのも簡単だけど、判定基準が複雑になればなるほど、無意味であることを(おそらくはプログラムのことを全く分かってない上司に)説明するのに無駄な時間を取られることになる.
>コーディング規約でツールに文句を言われない状態にすることって入れておくといいと思うよ。こういうことをするのはプログラミング経験0の糞マネージャの証。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
一つのことを行い、またそれをうまくやるプログラムを書け -- Malcolm Douglas McIlroy
単刀直入に (スコア:1)
提案するからだめなのだと。
するなら依頼か命令で。
「読みにくいから直してくれ」と
具体的にどこがダメなのかを明確に
Re: (スコア:0)
「読みにくいから」と言っても
「お前のスキルが低いからだ」と返されるパターンかと
Re: (スコア:0)
静的解析ツールにかけて
「ツールがこれだけ文句を言ってきている」
と突きつければ良いのでは?
コーディング規約でツールに文句を言われない状態にすることって入れておくといいと思うよ。
Re:単刀直入に (スコア:0)
静的解析ツールは参考にはなるが、盲信は非常に危険。
静的解析ツールはそこまで賢くない。
その静的解析ツールにもよるけれど、
たとえば「一メソッドの行数が100行を超えた。」みたいな警告を出してくるのもある。
行数が長すぎるのは良くない臭いではあるが、例えばやってる条件判定の数が多くて、
単純なswitch~caseがずらずら並んだ場合でさえも警告を出してくるので、
それに従うのは無意味だ。
これなんかは比較的単純だから無意味であることを説明するのも簡単だけど、
判定基準が複雑になればなるほど、無意味であることを(おそらくはプログラムのことを
全く分かってない上司に)説明するのに無駄な時間を取られることになる.
>コーディング規約でツールに文句を言われない状態にすることって入れておくといいと思うよ。
こういうことをするのはプログラミング経験0の糞マネージャの証。