コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

RSS
  • 受付中

    回答
    16

    皆さんにお聞きしたいです。「書くコードが酷い会社」で働いて得る経験は大きいと思いますか?

    teratailには素晴らしいエンジニア経験をお持ちの方がいらっしゃると思うので質問させてもらいます。 タイトルの通りになりますが、 「書くコードが酷い開発会社(ベンチャー)」で働いた経験がある方(なくても似たような経験をした)に特に回答頂けたらとても嬉しいです。 プログラマーの皆様は「コードの品質」というものに、どれだけこだわりを持っていますか?

    • -8評価
    • 1294PV
    Argent Argent 3週間前に コメント
  • 解決済

    回答
    2

    変数名についてご相談

    where id < ? の?の部分の部分の値に何という変数名が適当が思いつきません。。max_idを検討したのですがこれだと「id <= ?」という意味になってしまうので適当ではないいと思います。 また、メソッド名についてもいつも適当なものが思いつかずにかなり困っています。 例えばDBから値を取得するメソッドを定義する場合、必ずgetHogeみたいな

    • 0評価
    • 146PV
    yuba yuba 4ヶ月前に ベストアンサー
  • 解決済

    回答
    1

    python 画像を保存する複数の関数を任意で実行したい

    悩んでいること、分からないこと 以下のコードは、画像処理を行って処理後の画像を確認するために、保存しています。しかし、画像を保存するのには、僅かでも時間がかかります。それに、gray.jpg画像なんってめったに確認しません。出来れば、一か所にまとめて、「保存する/しない」を切り替えられるようにしたいのですが、自分の堅い頭ではどうコードを書くか分かりません。

    • 0評価
    • 356PV
    ehu ehu 7ヶ月前に ベストアンサー
  • 解決済

    回答
    4

    コードレビューで生じる開発遅延について

    いつもお世話になっております。 コードレビューについて書かせていただきます。 私はプログラミング能力が低いので、結構指摘を受けます。 でも、コードレビューって良い文化ですよね。 コードレビューのメリットって皆様もわかっているだろうし、 ネットにも色々あがっているのでここでは割愛します。 ただ、コードレビューって開発スケジュールの遅延を発生させ

    • -1評価
    • 359PV
    pgm_bakabon pgm_bakabon 8ヶ月前に コメント
  • 受付中

    回答
    9

    適切な分岐処理について

    前提・実現したいこと 以下のような簡単な図の分岐処理のスマートな書き方について A B 重要度 × × 低い × ○ 普通 ○ × やや高い ○ ○ 高い 該当のソースコード if(A == "×" && B == "×"){ Console.WriteLine("低い"); }else if(A == "×"

    • 3評価
    • 1063PV
    swordone swordone 9ヶ月前に コメント
  • 解決済

    回答
    5

    他の人が書いたコードに触れる機会をどうやって設けてますか?

    現状と感じている課題 同じプロジェクトで5年程コードを書いてます ずっとPHPです 私はこのプロジェクトの書き方しか知りません 最近他のプロジェクトのコードにも触れておかないと、ガラパゴス状態となる危険性があるのでは?という危機感を感じてます お聞きしたい事 皆さんは仕事で関わっているプロジェクト以外のコードを触る・見る

    • 3評価
    • 532PV
    coffee coffee 1年以上前に ベストアンサー
  • 解決済

    回答
    2

    PHPの標準入力を汎用的に使えるコードを教えてほしい

    <?php $av = array(); while(true) { //$a = trim(fgets(STDIN)); $a=fgets(STDIN); if ($a === FALSE ) { break; }

    • 0評価
    • 301PV
    Touhoku Touhoku 11ヶ月前に ベストアンサー
  • 受付中

    回答
    5

    レビューに関して(設計レビュー~ソースレビュー)

    設計レビューやソースレビューの意義や効率的な進め方に関して 背景として、属人的なプログラム実装や設計をフラットにしてレガシーな環境をレビューを行って何とか減らしていきたいという思いがあります。gitやチャットを導入して、WEBを介してある程度の情報量は共有出来ていると思っているのですが、チームで共有出来ているかというと感覚としては浸透していない印象です。

    • 3評価
    • 688PV
    ShintaroIshida ShintaroIshida 1年以上前に コメント
  • 解決済

    回答
    1

    MNISTに誤差逆伝播法を実装したが精度が下がる(C#)

    前提・実現したいこと C#で機械学習の勉強を行っており、手始めにMNISTの文字認識を勉強しています。 ただ誤差逆伝播法を入れた時に学習率が向上せず、どこがおかしいのかもわからず困っております。 プログラムの概要 28*28の元画像に対して畳み込みとプーリングを繰り返して4*4の画像×16フィルタまで落とし込み、 4*4*16=256個の値を入

    • 0評価
    • 713PV
    kku6 kku6 1年以上前に 回答
  • 解決済

    回答
    1

    有料でコードレビューを依頼できるサイトなどはありませんか?

    自分は趣味でプログラミングをしていて、基本一人で開発をしております。 なのでコードレビューしてもらう人材もいないので、いままでコードレビューをしたことがありませんでした。 自分の技術力を上げるためにも、コードレビューはやった方が良いと思うので、 コードレビューを依頼できるサイトなどご存知でないでしょうか? なお、自分がよく使っている言語はpyth

    • 1評価
    • 500PV
    tell_k tell_k 1年以上前に ベストアンサー
  • 解決済

    回答
    3

    Cppcheck結果をID順にソートして確認したい

    前提・実現したいこと GUI版Cppcheckの一覧画面にはID列を表示することができます。 このID値は、解析内容をグルーピングして確認するのに有用です。 一覧画面のID列ヘッダクリックにてID順に並び替えられますが、ファイルをまたいで並び替えができないようです。 複数ファイルにまたがってID順にソートして内容を確認したいです。 実行環境

    • 0評価
    • 588PV
    can110 can110 1年以上前に コメント
  • 解決済

    回答
    1

    Cppcheckエラー行位置を指定してサクラエディタで開きたい

    実現したいこと Cppcheckでは、解析結果の一覧画面の行ダブルクリックで、エラー元のファイルをメモ帳で開くことができます。 これを普段使い慣れているサクラエディタで、行位置までスクロールされた状態で開きたいです。 試したこと 以下手順にて、メモ帳ではなくサクラエディタで開くことまではできました。 Cppcheckの編集メニュー→設定→ア

    • 0評価
    • 667PV
    can110 can110 1年以上前に ベストアンサー
  • 解決済

    回答
    6

    アサインされたプロジェクトのソースを理解するときどこから読み始めますか?

    プロジェクトにアサインされた際(もしくは新しい会社に入社した際)プロジェクトのソースを読み込むのに何日か時間を与えられたとき、どこからソースを読み始めるものでしょうか? 自分の場合、どこから読み始めて良いか分からず途方にくれてしまい、とりあえず気になった機能をデバッグしてみるというようなあまり効率的でない時間の使い方をしてしまいます。 プロジェク

    • 1評価
    • 721PV
    tmizuma tmizuma 1年以上前に ベストアンサー
  • 解決済

    回答
    1

    MNISTのコードについて

    ディープラーニングのMNISTで以下のコードがありました。 下記のコードの def img_show(img): pil_img = Image.fromarray(np.uint8(img)) pil_img.show() についてお聞きしたいのですが、 【質問1】img_show関数の引数としてimgを指定する意味を教えてください。 【質問

    • 0評価
    • 496PV
    takeshi takeshi 1年以上前に ベストアンサー
  • 受付中

    回答
    2

    コード内コメントの記述方式

    コード内のコメント(コメントアウト)で可読性を上げたいのですが、便利なコメントの書き方とか、こういうの書き方があるという情報がまとまったWebページ・書籍等はありませんでしょうか? また、皆さんはどういう風に書いてるとか決めておられるのでしょうか? 人によって、会社・プロジェクト、言語によって、慣例のようなものはあると思いますが・・・ ちなみに

    • 0評価
    • 701PV
    mit0223 mit0223 1年以上前に コメント
  • 解決済

    回答
    5

    「他のクラスに書かれている処理に遷移するためのメソッド」について

    閲覧ありがとうございます。 プログラムを書いていると、以下の例の★のように 1クラスの1メソッドを極力少なくするために「他のクラスに書かれている処理に遷移するためのメソッド」を作ることがあります。 (以下の例はロジックを書くクラスです。コントローラーにtrue,falseを渡して画面遷移の判断材料にしています) 例 public boole

    • 0評価
    • 558PV
    otabeb otabeb 1年以上前に ベストアンサー
  • 解決済

    回答
    8

    きれいなコードとはどんなものでしょうか?

    ソースが汚いという質問をみて、そういえはいろいろ詳しい人に聞いてみたかったことを思い出しました。 それほど困ってはいないのですが、”きれいなコード”という表現を聞くことあります。抽象的で何を指しているのかわからずこの言葉を聞くたびに軽く混乱してしまいます。 その時々で尋ねればいいのですが、ついでで訊くにはややこしい問題かもしれないと思い控えていま

    • 5評価
    • 1098PV
    iwamoto_takaaki iwamoto_takaaki 1年以上前に ベストアンサー
  • 受付中

    回答
    8

    ソースが汚い

    自分の書いたコードが汚くて悩んでいます DRY原則は意識してるつもりなんですが、、、 良い方法ありませんか

    • 0評価
    • 877PV
    LLman LLman 1年以上前に 回答
  • 解決済

    回答
    6

    グローバル変数を使用しない設計方法

    前提・実現したいこと 今、下記のようなグローバル変数が定義されているとします。 このグローバル変数はプログラム中のいたるところで使用されます。 (例えばボタンがクリックされたらクリックされたボタンに応じてグローバル変数の値を書き換えたり、これらの値を使って計算したり、色々です。) 実現したいこととしましては、これらのグローバル変数を使わないような設計

    • 0評価
    • 1125PV
    kerokero335 kerokero335 1年以上前に ベストアンサー
  • 解決済

    回答
    7

    読みやすいコード(これは必須ではないですよね?)

    読みやすいコードを書くための作法はいろいろとあるとは思うのですが、以下のように関数やメソッドの引数に渡す値の左右にスペースをあける書き方は、一般的にチームでコーディングする時に期待されるでしょうか? //こうじゃなくて var name = getNameById(1); //こういうの var name = getNameById( 1 ); //

    • 0評価
    • 1088PV
    harashow1701 harashow1701 1年以上前に ベストアンサー
1

タグ情報

  • あなたの順位

    -

  • タグ登録者数

    29

  • 総質問数

    32

  • 今週の質問数

    0

関連するタグ

コードレビュータグのよく見られている質問

思考するエンジニアのためのQ&Aサイト「teratail」について詳しく知る