none
制作したコードのレビューについて RRS feed

  • 質問

  • 初めての質問になります。

    今月よりVisual C#でWindowsフォームアプリの制作を学んでおり、いくつか入門書レベルの小さなアプリを作ってみています。
    一応エラーも出ず意図したとおりの動作はするのですが、コーディングとして良いかかどうか(改修のしやすさ、機能追加のしやすさなど)や、問題がある記述となっていないかどうかを自分で確認することができない状態です。

    環境としては書籍を読みながらの独習で、ほかにプログラミングのわかる人間がやや忙しい別部署にしかおらず、気軽にレビューを頼める状況でもありません(それしか方法がなければやむを得ないとは考えています)。

    自らでコード記述のチェックを行う方法で何か良いものはありますでしょうか?

    2021年3月19日 6:39

回答

  • Hoshina です
    こんにちは

    あくまでも個人の感想ですが、
    初心者の方では自分で自分を検証するのは難しいと思います。

    実は、経験者であってもたやすいこととは思いません。
    身近でレビューをしてもらえそうな人を地道に探すのがよいと思います。

    お住いの地域やお勤め先などの条件によりますけれど。手広く探してみてはいかがでしょうか?
    お仕事の学習の一環というように読めますので、職場の上司の方に相談してみるのも一案と思います。
    部下が、そうした案件で上司に相談を持ち掛けてきた場合、
    上司としては、それなりにうれしいもので、協力を得られるかもしれません。

    参考にならない投稿でごめんなさい。

    それでは

     
    • 回答としてマーク ながぽん 2021年3月20日 1:26
    2021年3月19日 7:01
  • 基本的にHoshinaさんに賛成です。上司がレビュー不要と判断するならそれはそれで一つの解です。

    そのうえで補足情報ですが、Visual Studioにはソース コード分析機能があります。これはコンパイラーの警告よりももう一段深い指摘が行われます。コンパイラーの警告やコード分析による指摘があれば、それを無理やり抑止するのではなく警告内容に真摯に向き合い修正するのも一つの手です。
    # 例えば「変数が初期化されていない」という警告に対し、「とりあえず0やnullを代入する」という対応は真摯に向き合っているとはいいがたいです。

    • 回答としてマーク ながぽん 2021年3月20日 1:26
    2021年3月19日 8:27

すべての返信

  • Hoshina です
    こんにちは

    あくまでも個人の感想ですが、
    初心者の方では自分で自分を検証するのは難しいと思います。

    実は、経験者であってもたやすいこととは思いません。
    身近でレビューをしてもらえそうな人を地道に探すのがよいと思います。

    お住いの地域やお勤め先などの条件によりますけれど。手広く探してみてはいかがでしょうか?
    お仕事の学習の一環というように読めますので、職場の上司の方に相談してみるのも一案と思います。
    部下が、そうした案件で上司に相談を持ち掛けてきた場合、
    上司としては、それなりにうれしいもので、協力を得られるかもしれません。

    参考にならない投稿でごめんなさい。

    それでは

     
    • 回答としてマーク ながぽん 2021年3月20日 1:26
    2021年3月19日 7:01
  • 基本的にHoshinaさんに賛成です。上司がレビュー不要と判断するならそれはそれで一つの解です。

    そのうえで補足情報ですが、Visual Studioにはソース コード分析機能があります。これはコンパイラーの警告よりももう一段深い指摘が行われます。コンパイラーの警告やコード分析による指摘があれば、それを無理やり抑止するのではなく警告内容に真摯に向き合い修正するのも一つの手です。
    # 例えば「変数が初期化されていない」という警告に対し、「とりあえず0やnullを代入する」という対応は真摯に向き合っているとはいいがたいです。

    • 回答としてマーク ながぽん 2021年3月20日 1:26
    2021年3月19日 8:27