[過去ﾛｸﾞ] SES(客先常駐)エンジニアの集会所 32 (1002ﾚｽ)
上下前次1-新
抽出解除 ﾚｽ栞
このｽﾚｯﾄﾞは過去ﾛｸﾞ倉庫に格納されています｡
次ｽﾚ検索 歴削→次ｽﾚ 栞削→次ｽﾚ 過去ﾛｸﾞﾒﾆｭｰ
763(7): 05/20(火)04:36 AAS
いやいや、だから待てって
なんで通らないコードを残しておくんだ？
そこにまずはなんらか理由が必要だろう
それを聞かない限り、現状与えられた情報からだとどちらもズレてる気がするが
764: 05/20(火)05:03 AAS
>>763
日本だとソース管理使っててもコードコメントアウトする馬鹿な文化があるとか
さっき言った様に実は悪意のあるコード入れる為とか
765(54): 05/20(火)08:33 AAS
>>763
デブが正しいよ
確かにmisra-cでも実行されないコードを作るなというルールがあるが
それはコードレビューのフェーズで検討すべきことだろうね
758は見るからにテストフェーズなので存在意義より実行されるかどうか検証するのは意味あることだろ
単に君の経験が浅いだけかもね
789: 05/20(火)21:20 AAS
>>782
>(検証に)必要ないと判断しているだけで最終的な要否は別問題。
要否の判断、またその経緯に検証の要否が依拠してくるため、そこを論点に挙げてるわけだが
>>781 で述べたよう、既存の修正であり、更に開発現場のスタイル、事情によってコメントアウトが是とされるなら、もはや議論の余地はない
嗜好性とは別議論なため、>>763 の主張は無意味となるが、しかし現時点では上記のことが判然としないと言っている
また、気になるのは「最終的な（コメントアウトの）要否は別問題」とされてるが、上述のとおり、検証の要否はそこに依拠する
前者を棚上げした上でなにゆえ後者を正しく判断できようか
省4
793(1): 05/20(火)21:30 AAS
>>784
>テストフェーズであることは認めたようだね
論点そこじゃなくて、主語がデカい、だから粒度を聞いてるわけよ
とりあえずこちらの認識ね
実装 -> 単体 -> コードレビュー
やり取りからコードレベルの議論してるから、コードレビュー前だとこちらは認識している
だからそもそも「どちらもズレてる(>>763」としてるわけ
省2
817: 05/20(火)22:49 AAS
>>802
>あとからコードレビューするライフサイクルするケースも少なくないよ
はい
で、本件はいずれか判然としてますか？
>>800 や >>803 にて述べたよう、コードレビュー、もしくは規約など何かしら当該行為（コメントアウト）を正当化するものがなければ、本件における二人のやり取り自体の要否にそれこそ疑問が出てくる
今どきのコーディングスタイル一般に照らせば、コメントアウトは否であるまいか
さすれば、そもそも正当性の担保されていない、今どきのコーディングスタイル一般ではない、それ（コメントアウトされたコード）についてテストケースの要否を議論すること自体がズレているのでは？がこちらの当初からの立場であって(>>763
省4
845(3): 05/21(水)00:19 ID:PeHvRXYZ(3/18) AAS
>>840
>新規コードであろうと
>既存コードの改定であろうと対応は変わらない。
>コード修正が求められる。
具体的には？
つまりコード不正対策の観点からもコメントアウトを否とするという理解で合っているかな？
だから >>761（これも自分のレス） や >>763 の認識で合ってるべ？
省3
942(1): 05/22(木)20:34 ID:qLJdXdia(10/17) AAS
>>926
>なので、この後の議題については逸脱をゆるす運用の場合は除外した議論としなければならないよ。
つまり「各人の判断で不正を許す」ような例外を除けば、ケースバイケースで許容もありえると
>>927
>名古屋系と広島系と大阪系とカルロスーン系も
これ、なんのことです？
>いずれもコメントアウトはダメで不正 削除が必要
省18
上下前次1-新書関写板覧索設栞歴
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ AAｻﾑﾈｲﾙ
ぬこの手 ぬこTOP 0.041s