今回のテーマ
Contents
こんにちは、かっぱちゃんです。
今回はIT業界であった話をしていきます。
前回の怖い話は下記の記事です。是非こちらもみてくださいね。
-
-
参考C#のシステム改修で本当にあった怖い話 ステップ数で見積もるからこんなことになる編 IT業界志望している方は是非みてね
C#を学ぶならガチで読んで欲しい本はこちら 今回のテーマContents1 今回のテーマ2 C#のシステム改修で本当にあった怖い話3 ステップ数見積りのメリット4 C#の開発でステップ数を見積ることが ...
続きを見る
今回のテーマは下記の話をしたいと思います。
見出し(全角15文字)
C#プログラミングがよく分からない上司のコードレビュー
しょうもない話なので、適当に流しながらみてくださいね笑
それではどうぞ!!

C#のLinqをさらに学びたい方へ
C#のLinqを理解して頂くために参考書を執筆致しました!
Linqの演習問題はこちらの参考書に載せているので、問題を解いて理解を深めたい方はこちらの参考書チェックしてください!
下記の画像をクリック!!kindleunlimitedの会員であれば無料で見れます!
⇩初心者向け ⇩中級者向け
本当にあった怖い話(ソースコードレビュー1回目)
分かりやすくするために大分簡単にしています。
当時の私はストレスマックスだったと思います! 😈
かっぱちゃんが私の発言にしてみました。
******ここからが本当にあった怖い話******
某メーカー系企業に勤めていた時、私はシステム改修の開発を担当しておりました。
今回の話はC#プログラムにあまり詳しくない上司とコードレビューをした時の話です。
当時の上司はプロジェクトも上手く進んでおらず、イライラしている様子・・・。^^;

私は下記のソースコードを見せました。(勿論コードは例です。)
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 |
using System; using System.Collections.Generic; class Program { static void Main(string[] args) { List<string> robotlist = new List<string>(); robotlist.Add("Rockman"); robotlist.Add("Rollchan"); robotlist.Add("Fireman"); robotlist.Add("Iceman"); foreach(var robot in robotlist) { Console.WriteLine(robot); } robotlist.Remove("Fireman"); foreach (var robot in robotlist) { Console.WriteLine(robot); } } } |
出力結果
Rockman
Rollchan
Fireman
Iceman
Rockman
Rollchan
Iceman
上司はソースコードを見ていいました。
上司:「分かりにくい!!コメント付けろ!!」
他のブログ記事はこちらにあります。
他のブログ記事はこちら
本当にあった怖い話(ソースコードレビュー2回目)
確かにコメント書かないと分かりにくいかなと思い、
私は下記のようにコメントを付けたソースコードをみせました。

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 |
using System; using System.Collections.Generic; class Program { static void Main(string[] args) { //ロボットリストにロボットを格納する。 List<string> robotlist = new List<string>(); robotlist.Add("Rockman"); robotlist.Add("Rollchan"); robotlist.Add("Fireman"); robotlist.Add("Iceman"); //ロボットリストに格納されているロボット名を出力する。 foreach(var robot in robotlist) { Console.WriteLine(robot); } robotlist.Remove("Fireman"); foreach (var robot in robotlist) { Console.WriteLine(robot); } } } |
上司はソースコードを見ていいました。
上司:「どうみても分かりにくいだろ!Addメソッドってなんだ!?」

本当にあった怖い話(ソースコードレビュー3回目)

上司:「それを書けよ!!!」
私はコメントを付けた場所が悪いのかなと思い、
下記のようにコメントを追加をし、上司にみせました。

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 |
using System; using System.Collections.Generic; class Program { static void Main(string[] args) { List<string> robotlist = new List<string>(); //ロボットリストにロボットを追加する。 robotlist.Add("Rockman"); robotlist.Add("Rollchan"); robotlist.Add("Fireman"); robotlist.Add("Iceman"); //ロボットリストに格納されているロボット名を出力する。 foreach(var robot in robotlist) { Console.WriteLine(robot); } //ロボットリストからロボットを削除する。 robotlist.Remove("Fireman"); foreach (var robot in robotlist) { Console.WriteLine(robot); } } } |
上司はソースコードを見ていいました。
上司:「どうして1行ずつコメント書かねえんだ!!こういうことだよ!!」

他のブログ記事はこちらにあります。
他のブログ記事はこちら
本当にあった怖い話(上司の答え編)
上司:「コメントを追加するってことはこういうことなんだよ!!」
上司の修正したソースコードを見ました。
私は遂心の中で叫んでしまいました!

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 |
using System; using System.Collections.Generic; class Program { static void Main(string[] args) { //ロボットリストを生成する。 List<string> robotlist = new List<string>(); //ロボットリストにロボットを追加する。 robotlist.Add("Rockman"); //ロボットリストにロボットを追加する。 robotlist.Add("Rollchan"); //ロボットリストにロボットを追加する。 robotlist.Add("Fireman"); //ロボットリストにロボットを追加する。 robotlist.Add("Iceman"); //ロボットリストに格納されているロボットの数繰り返す foreach(var robot in robotlist) { //ロボット名を出力する。 Console.WriteLine(robot); } //ロボットリストにロボットを削除する。 robotlist.Remove("Fireman"); //ロボットリストに格納されているロボットの数繰り返す foreach (var robot in robotlist) { //ロボット名を出力する。 Console.WriteLine(robot); } } } |
上司:「どうだ!分かりやすいだろ!!」

私は心の中でこう叫びました。

上司:「他のソースコードもこのようにしとけ!そしたらコードレビュー終わりにしてやる!」
内容の指摘は何もなくコードレビューの指摘はコメントを付けるだけでした。
しかし、あまりに改修した量が多くコメントを付ける作業だけで残業を何回もしましたとさ。
めでたしめでたし。
******ここまでが本当にあった怖い話******
まとめ
今回の本当にあった怖い話はコメント付けろ編でした。
その後の私はその上司の下ではやってられないことを部長に告げ、試験が終わり次第別のプロジェクトに異動しました。

びっしりとコメントが入ったプログラムは今でも動き続けていることでしょう!^^;
このように、プログラミングのことをあまり知らない上司の下に付くと、悲惨なSE生活を仕入れられます。
このような境遇の方はすぐに転職するか異動届を出してしてくださいね。
それでは最後まで見て頂き、ありがとうございました。

ITエンジニアの転職の記事について書きました!!良ければこちらの記事をみてください!!
こちらもCHECK
-
-
システムエンジニアが人間関係などのネガティブな転職理由で、3回転職し、年収を上げることが出来た話 独立系IT企業、メーカー系IT企業、社内SEの大体の年収を公表します。
C#を学ぶならガチで読んで欲しい本はこちら 今回のテーマContents1 今回のテーマ2 転職を悩んでいる方へ3 かっぱちゃんの転職理由4 私の使用した転職サイト5 まとめ こんにちは、かっぱちゃん ...
続きを見る
年収400万以下の人は是非チェックしてください!!
こちらもCHECK
-
-
社会人4年目で年収400万以下のシステムエンジニアはすぐに転職した方がいい話!!転職がオススメの理由とオススメのIT求人専門サイトを紹介します。
C#を学ぶならガチで読んで欲しい本はこちら 今回のテーマContents1 今回のテーマ2 システムエンジニアはどこも人手不足3 転職は3回しています4 年収を上げる条件5 私の情けない転職理由6 転 ...
続きを見る
他のブログ記事はこちらにあります。
他のブログ記事はこちら
C#のLinqをさらに学びたい方へ
C#のLinqを理解して頂くために参考書を執筆致しました!
Linqの演習問題はこちらの参考書に載せているので、問題を解いて理解を深めたい方はこちらの参考書チェックしてください!
下記の画像をクリック!!kindleunlimitedの会員であれば無料で見れます!
⇩初心者向け ⇩中級者向け
新しいサイト作りました。アニメのまとめサイトです。
映画ドキュメンタリ版のブログはこちら
レインボーまとめサイト(映画ドキュメンタリー版)
ポケモンまとめブログはこちら
レインボーまとめサイト(ポケモン版)