アカウント名:
パスワード:
今時内部ツールとはいえコードレビュー無しとか、平成も終わるというのにどこの昭和ですか。
コードレビューで見つけられるのは、処理方式や書き方が難解(属人化してるとか、処理の流れが読みづらくバクを埋め込みやすくなっているとか)となっていないかであって、
計算間違い(プログラムのバグなのか、実装しようとした処理自体が目的と合っていなかったか)は、テスト(前者なら単体テストで、後者ならシステムテスト)で見つけるものではないでしょうか?
ソースコードが読めない人ではテストを実施できないというわけではないでしょうし、テスト仕様書がコードを読める人でないと書けなかったとしても、さすがにそれは日本語で書かれているだろうし、何を修正するかは実装者以外も把握しているだろうから、COBOLが読めない人でも、内容が妥当かのチェックはできると思うのですが...
COBOLが読める人が1~2人しかいないというより、仕様を把握している人が少ないのが原因だと思うのですが、どうでしょう?
# 昭和というより、テストしないでリリースする某ゲームry
実際の所は、情報が不足しているのでどうなのかは全くわかりませんですよ。というのを前提として。
私がコードレビューの話を持ち出したのは、アゴラの記事にある
データに抽出率逆数をかける復元をシステム担当者が忘れたバグに尽きるのだ。これはCOBOLで書かれた特殊なプログラムなので高齢者しか読めず、そのミスがチェックできないので
という部分から。内容のチェックすら誰もしてないのが見て取れるからです。
でまぁ、一般的な開発現場における話は、お書きになったとおりです。COBOLは単体テストが極めてやりにくそうでありますが…。原因は、正直これだけでは判断つかないですが、根本は人材不足、労働管理能力不足かなぁとはにらんでおります。おそらく、設計能力のある人がいたら、さっさと書き直せるレベルじゃないのかなぁと。
もっと言えば、妥当な内容を保つインセンティブが何もないんだろ
役人にそんなもの無いよな
コードレビューをしてない事にしないと、プログラムのせいに出来ないじゃないですか!!
お役所の体制が空洞化してるってことなんじゃないかなあ。「役人がCOBOLやRを学ぶべき」という結論にはならんだろ。何の決裁機構も文書化もないまま本番系を漫然と改変し続け、作業者から大臣に至るまで誰も指摘せずできずできることを求められもしなかった、そういうことだと思う。厚生労働省ですらアマチュアが椅子に座って数字を送り出していれば問題とされないというレベルまで中央政府が崩壊してるんだよ。
今より国家のやる事が少なかった昔からずっと人が減り続けてるからねそりゃあまともに動かないよ
中堅企業でさえ3割がひとり情シスなのに。コードレビューだけ外注に出すか。
コードレビューねぇ、自分の経験から言うとチームの人間が 10 人居たとして、コードを理解してレビューできるのは、そのうちの 1 か 2 人だな。一応今の生業は linux kernel driver 屋さんチームだ。
パーキンソンの凡俗法則やな。https://ja.wikipedia.org/wiki/%E3%83%91%E3%83%BC%E3%82%AD%E3%83%B3%E3%... [wikipedia.org]
なにも言えなきゃ黙ってりゃいいのにな。命名とかタブ幅とかなんかどうでもいいんやから好きにさせろっつーのに、そういうことをわざわざ言うヤツは何を考えてんのかな?ヒマなんかな?
> カンマやスペースの入れ方と言ったコーディングスタイルの違いばかり指摘している。
git commit 時のフックでフォーマッター噛ませないの?
Cの皮を被ったアセンブラなんて、すぐその場で読めというほうが無茶。
ごめんなさいポインター使ったたらレビューで読みにくいと言われました。
これって日本国内での経験?
自分は日本とベトナムの2チームの委託先と仕事してるんだけど、日本のチームは同様の状況。コード理解してる人もコピペマンも一緒にチーム組んで、同じ仕事をしてる。
一方ベトナムのチームは、打ち合わせに出てくるのはコード理解してる人だけ。テスト専門のコード理解していない人もたくさんいるんだけど、職種が違うらしく一緒に仕事しないし給料も3倍ぐらい違うらしい。大学で専門教育を受けた人しか設計の職には就けないので、プライド高くて学習意欲もすごく高い。
あなたこそどこの時代の人ですか最近のトレンドは、人不足、予算不足、知識不足でコードレビューなしが主流ですよ!
おぉぅ、私は時代に取り残されてしまったようデスね…。(追いつきたくもないが)
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
Stableって古いって意味だっけ? -- Debian初級
コードレビューなしとは (スコア:2)
今時内部ツールとはいえコードレビュー無しとか、平成も終わるというのにどこの昭和ですか。
ほえほえ
Re:コードレビューなしとは (スコア:2)
コードレビューで見つけられるのは、
処理方式や書き方が難解(属人化してるとか、
処理の流れが読みづらくバクを埋め込みやすくなっているとか)
となっていないかであって、
計算間違い(プログラムのバグなのか、実装しようとした処理自体が
目的と合っていなかったか)は、テスト(前者なら単体テストで、
後者ならシステムテスト)で見つけるものではないでしょうか?
ソースコードが読めない人ではテストを実施できないというわけではないでしょうし、
テスト仕様書がコードを読める人でないと書けなかったとしても、
さすがにそれは日本語で書かれているだろうし、
何を修正するかは実装者以外も把握しているだろうから、
COBOLが読めない人でも、内容が妥当かのチェックはできると思うのですが...
COBOLが読める人が1~2人しかいないというより、仕様を把握している人が少ないのが原因だと
思うのですが、どうでしょう?
# 昭和というより、テストしないでリリースする某ゲームry
Re:コードレビューなしとは (スコア:2)
実際の所は、情報が不足しているのでどうなのかは全くわかりませんですよ。というのを前提として。
私がコードレビューの話を持ち出したのは、アゴラの記事にある
という部分から。内容のチェックすら誰もしてないのが見て取れるからです。
でまぁ、一般的な開発現場における話は、お書きになったとおりです。COBOLは単体テストが極めてやりにくそうでありますが…。原因は、正直これだけでは判断つかないですが、根本は人材不足、労働管理能力不足かなぁとはにらんでおります。おそらく、設計能力のある人がいたら、さっさと書き直せるレベルじゃないのかなぁと。
ほえほえ
Re: (スコア:0)
もっと言えば、妥当な内容を保つインセンティブが何もないんだろ
Re: (スコア:0)
役人にそんなもの無いよな
Re:コードレビューなしとは (スコア:2, 荒らし)
コードレビューをしてない事にしないと、プログラムのせいに出来ないじゃないですか!!
Re: (スコア:0)
お役所の体制が空洞化してるってことなんじゃないかなあ。「役人がCOBOLやRを学ぶべき」という結論にはならんだろ。
何の決裁機構も文書化もないまま本番系を漫然と改変し続け、作業者から大臣に至るまで誰も指摘せずできずできることを求められもしなかった、そういうことだと思う。
厚生労働省ですらアマチュアが椅子に座って数字を送り出していれば問題とされないというレベルまで中央政府が崩壊してるんだよ。
Re: (スコア:0)
今より国家のやる事が少なかった昔からずっと人が減り続けてるからね
そりゃあまともに動かないよ
Re: (スコア:0)
中堅企業でさえ3割がひとり情シスなのに。コードレビューだけ外注に出すか。
Re: (スコア:0)
コードレビューねぇ、自分の経験から言うとチームの人間が 10 人居たとして、コードを理解してレビューできるのは、そのうちの 1 か 2 人だな。一応今の生業は linux kernel driver 屋さんチームだ。
Re:コードレビューなしとは (スコア:2)
パーキンソンの凡俗法則やな。
https://ja.wikipedia.org/wiki/%E3%83%91%E3%83%BC%E3%82%AD%E3%83%B3%E3%... [wikipedia.org]
なにも言えなきゃ黙ってりゃいいのにな。
命名とかタブ幅とかなんかどうでもいいんやから好きにさせろっつーのに、そういうことをわざわざ言うヤツは何を考えてんのかな?ヒマなんかな?
Re:コードレビューなしとは (スコア:1)
> カンマやスペースの入れ方と言ったコーディングスタイルの違いばかり指摘している。
git commit 時のフックでフォーマッター噛ませないの?
Re: (スコア:0)
Cの皮を被ったアセンブラなんて、すぐその場で読めというほうが無茶。
Re: (スコア:0)
ごめんなさい
ポインター使ったたらレビューで読みにくいと言われました。
Re: (スコア:0)
これって日本国内での経験?
自分は日本とベトナムの2チームの委託先と仕事してるんだけど、日本のチームは同様の状況。
コード理解してる人もコピペマンも一緒にチーム組んで、同じ仕事をしてる。
一方ベトナムのチームは、打ち合わせに出てくるのはコード理解してる人だけ。
テスト専門のコード理解していない人もたくさんいるんだけど、職種が違うらしく一緒に仕事しないし給料も3倍ぐらい違うらしい。
大学で専門教育を受けた人しか設計の職には就けないので、プライド高くて学習意欲もすごく高い。
Re: (スコア:0)
Re: (スコア:0)
あなたこそどこの時代の人ですか
最近のトレンドは、人不足、予算不足、知識不足でコードレビューなしが主流ですよ!
Re:コードレビューなしとは (スコア:2)
おぉぅ、私は時代に取り残されてしまったようデスね…。(追いつきたくもないが)
ほえほえ