1: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:14:43.10 ID:zDBQBoAYP
良く分からないけど動いたため暫定←こういうコメントやめろ


2: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:15:27.24 ID:I7FPWYXB0
わかりにくい変数名

4: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:15:59.87 ID:CGFbm40i0
//良く分からないけど消しちゃダメみたい

5: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:16:59.20 ID:a1bjR9mf0
こういう無意味なコメントやめろ


// var1にvar2を代入してfunc1を呼ぶ
var2 = var1;
func(var2);

6: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:17:53.19 ID:a1bjR9mf0
間違えた

// var2にvar1を代入してfunc1を呼ぶ
var2 = var1;
func1(var2);

11: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:19:19.68 ID:ymVeAfok0
>>6
まんまかw

16: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:27:22.22 ID:x3UavII7P
>>6
いやいや、別にそれはいいだろ・・・
JRのスイカーシステムは全部の行にコメントあるという話を聞いた

17: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:28:48.96 ID:TwTVH+9/P
>>16
良い訳ねーだろ
そんなもんソースみりゃわかんだよタコ

20: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:29:54.63 ID:31Sjq6Xa0
>>17
だよな
無意味すぎアホ過ぎ

22: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:31:54.74 ID:hGEeFALg0
>>20
なんかコメント入れろって言われて
困ってやったことはある

21: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:31:37.83 ID:x3UavII7P
>>17
アホ、そういうコーディング規約もあるんだよ
小規模開発からしたら無関係かもしれんがwwww

23: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:33:21.11 ID:zDBQBoAYP
>>21
1ステップごとにコメントと別途フローチャート書けって言い出した恐ろしい会社なら知ってる

24: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:34:22.49 ID:31Sjq6Xa0
俺、メーカーで大規模開発やら組み込み開発だけど
>>16みたいなコードはNGだわ

下請けが書いてもレビューで指摘する
だって>>6は無意味だから

26: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:35:27.37 ID:x3UavII7P
ソース1行コメント1行は書いたやつがバカじゃなくてそういうの強要されてんだろwww
その方が、保守的でいいと思うぞwww

無駄なら見なきゃいいだろwww
本当に大事なところで書いてないよりはまし

29: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:39:28.76 ID:31Sjq6Xa0
>>26
わかった。おまえ>>6の意図が分かってねーのな

1) これは無意味

// var2にvar1を代入してfunc1を呼ぶ
var2 = var1;
func1(var2);


2)これは正解(意味があるから)
// ラダーのニーモニック命令処理に必要な値を代入して、解析用の関数に渡す
var2 = var1;
func1(var2);

33: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:47:11.75 ID:x3UavII7P
>>29
それは分かる

無駄なコメントという定義は何?
100人したら100通りの解釈までは行かないけど
そこは人によって違うだろwwwできるやつは分かるけど、出来ないやつは分からない

それならみんな共通させて、無駄でもいいからコメントを書け
その上で>>29のパターンも検出できる

「君、このコメント意味ないよね?何で書いたの?」という話になる

47: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:57:10.52 ID:31Sjq6Xa0
>>33
んなもん後で保守する人が既存のコードみてコードのロジックを追って
メンテしやすく為だろ
機能追加でロジックを調査する際に、任された人間が>>6のコメントをみても何の役にも立たない
プログラミング初めて2日目です。ってやつにはありがたいコードかもしれないが
どうせコメント書くなら処理の全体の流れが追いやすいものにしろと

69: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:55:00.49 ID:qqOY85tBP
>>6
> // var2にvar1を代入してfunc1を呼ぶ
> var2 = var1;
> func1(var2);
いまさらだけど
コレ
func1(var1);
じゃだめなん?
いちいちvar2にするいみがわからん
データ型変換するわけでもないだろうし

>>68
上手くやれば楽だし楽しいよ
勉強しながらやってるけど自分で組めないものは外注できるし

71: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:58:24.04 ID:BDLrWFq30
>>69
func1の処理は?

74: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 14:22:01.17 ID:qqOY85tBP
>>71
わからんけど
返り値を返さないだろうと予想した

7: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:18:09.47 ID:FyYVI7Tm0
リーダブルコードを必読書にしろ

8: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:18:16.35 ID:zlmcfkVb0
コメントでおまじないとか書いてあって笑う

10: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:19:11.89 ID:zDBQBoAYP
>>8
本気出す初期パラメータ
ってのも見た

9: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:18:31.98 ID:zqCYo4Kj0
顔文字がはいってたときは絶望と高揚がまざる

12: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:20:31.39 ID:TwTVH+9/P
そんなバカいんのかよ

14: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:23:58.57 ID:zDBQBoAYP
>>12
マジで居る
しかも一年に数回ぐらい遭遇する

13: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:22:13.58 ID:T+K/sn5L0
/*この辺にバグがある、何処にあるかは不明

      ; ;
*/

18: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:29:14.23 ID:31Sjq6Xa0
まじかよ。
ダイヤ3つの電機メーカーさんでこんなコード書いたときには
馬鹿にされたり文句いわれたりだぞ

19: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:29:45.91 ID:kygAdpADi
よう分からないけど動くみたいな状況になった事がない

25: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:35:06.05 ID:zqCYo4Kj0
>>19
メーカーごとの特殊状況でなんでこいつだけうごかねぇんだよというようなものは昔はよくあった
あとはブラウザやらFlashやらが絡むとそんなバッドノウハウの山積みだったりするけどな

28: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:38:03.44 ID:kygAdpADi
>>25
なるほどね、自分は社内システムの運用保守が主だからそういうのとは無関係だは

27: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:36:17.61 ID:pSmikFDl0
昔全行80文字ヨリ右は全部コメントになっているソースがあったなぁ・・・
コード追加修正した後字数合わせなきゃいけないというバカ仕様

そんな会社も今はもう無いwwwざまぁwww

30: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:40:24.21 ID:x3UavII7P
開発体制が10人~40人位ならコメントは自由だけど、
50人以上は必須だろ

そりゃ出来るPGは無駄って言うかもしれないが、全体的に見たら
書かないより書いたほうがいい

31: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:42:38.12 ID:T+K/sn5L0
/*** 仕様変更注意 ***/
// 変数名の変更
// 型の変更
// i/fの変更、その他の変更

32: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:43:03.30 ID:kygAdpADi
コメントっていうとプログラム修正時に直す箇所は全部コメント化させて、その下に新しいコード描くってやってるから
修正しまくったプログラムはカオス化してる

64: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:36:01.35 ID:pSmikFDl0
>>32
あるわー
CVSやらSVNやら使えばそんなコメント必要なくなるのにな

34: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:48:25.04 ID:31Sjq6Xa0
ありがち

// 削除予定の関数
func(var1,var2);

↑もう使われてない関数なのに削除されてない

41: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:54:28.79 ID:kygAdpADi
>>34
実際よくあるけど、本当に使われてないか調べるのもめんどくさいし、影響がないかテストするのもめんどくさいし
関数呼びだしてる1行は消しても
関数そのものは消さない場合が多い

36: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:48:52.07 ID:N/g4/FoY0
なんかプログラマーって楽しそうだなw
結構笑ったw

37: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:51:00.28 ID:x3UavII7P
>>36
もうプログラマーじゃないけど
仕事のプログラムは全然、楽しくないwww
時間なし、仕様はクソ、それでも文句言いながら作ってるからなwww

40: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:54:13.09 ID:N/g4/FoY0
>>37
それなりに楽しいまたはやりがいのある情報系の仕事ってなんですかねー?

46: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:57:02.96 ID:x3UavII7P
>>40
大企業の社内基幹システムの保守
(要望のメールを受けて、下請けに投げるだけのお仕事)

社内システムインフラ担当
(要望を受けて、社外営業に見積もりを貰うだけのお仕事)

これ最強

48: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:58:46.13 ID:N/g4/FoY0
>>46
なるほど参考にさせてもらいます

38: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:52:23.48 ID:KeOVpHkq0
これリアルPHPのコメント


# データ取得関数
# うるう年の計算ができないという、ふざけた関数
# しかし、他に有用な関数がないので使うことにする

42: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:54:47.34 ID:x3UavII7P
>>38
ワロタwwwwwww

# データ取得関数
# うるう年の計算ができないという、ふざけた関数
# しかし、他に有用な関数がないので使うことにする

この一行追加しろよwwwwwww
# 2013/09/02 山田PM確認済

39: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:53:45.36 ID:HiN+zC1N0
前橋和弥さんだったかな、
int user_id; /* ユーザーID */ みたいな馬鹿なコメントつけるくらいなら
わかりやすいコードを書けとか言ってたな 俺もそう思う

43: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:54:57.38 ID:KeOVpHkq0
スコープを無視した変数や関数が許せないんだが

44: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:55:39.63 ID:KeOVpHkq0
# データ取得関数
# うるう年の計算ができないという、ふざけた関数
# しかし、他に有用な関数がないので使うことにする
# 2013/09/02 山田PM確認済

45: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:56:39.99 ID:vwyiraWn0
x = y;
x = y; //不安なのでもう一回

49: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 12:59:37.97 ID:KeOVpHkq0
いまこんなコメント発見した



    // この関数で最初の設定を入れるみたい
    public Form1()
    {
       // なにこれ?
      InitializeComponent();

      // 起動時の設定
      this.Icon = Properties.Resources._1377689926_45542;
      getRecord();

      // よくわからんけど、これでdataGridView1を定期的に更新できる
      t = new Timer();
      t.Enabled = true;
      t.Interval = 60000;
      t.Tick += (s, g) =>
      {
        getRecord();
      };
    }

50: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:02:17.91 ID:rZ8cUToa0
おまえら昼休み終わったぞ

仕事しろ

54: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:05:25.21 ID:KeOVpHkq0
>>50
俺は今から昼休みだ

56: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:09:07.23 ID:N+BFbvlUi
>>54
お疲れ様です

52: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:02:39.45 ID:T7wfa2+OP
変数名は日本語で定義しろと文句行ってくる会社が過去に逢った
例えば「Variable→hensuu」的な

この会社は長く続かないなと思ったら1年後あっけなく潰れた
なんであんな会社が起業出来たんだろう

53: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:05:05.37 ID:HiN+zC1N0
今は.NET時代でだいぶ楽だろうけど
C++で開発してた時代は地獄だったろうなあ
先人の方々の苦労が偲ばれる

55: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:06:49.29 ID:rZ8cUToa0
そのC++も今も標準が拡張され続けてるからなwww
11とか1yとか

コンパイラ側の対応も含めて中々対応が難しい
Windowだけしか開発しない人には無縁な話かもしれないが

57: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:11:39.66 ID:oQqvwbYQP
「よくわからんけど」っていうコメントは、これは捨てることに何のためらいもないゴミコードなんだな、という理由がわかる分まだマシじゃね?

コードに入れられない情報を入れる、という意味では

63: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:35:08.41 ID:/yxdh6mui
おまえら関数ヘッダもちゃんとメンテしろよ?以外と抜けが多いぞ

84: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 15:20:20.99 ID:hiFJYfBC0
>>63
参照渡しの可能性

67: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:46:03.59 ID:+rDGqQcQO
//僕と結婚してください

68: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:53:31.01 ID:ZiG1u8BR0
社内SEになりたい新卒

70: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 13:56:16.78 ID:xABoQ9jk0
コメントルールがない職場で働いているの?

73: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 14:00:41.20 ID:cQnUlnXWO
理想
 ソースがよく解らないからコメントを見る
現実
 なんじゃこのコメントwwwソース見よwwwwwwww

76: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 14:28:39.14 ID:bGsCAyY20
情報系学部4大卒ってだいたいプログラマになんの?

77: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 14:29:26.80 ID:KeOVpHkq0
俺はプログラミングはセンスが大事だと思ってる

78: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 14:29:57.15 ID:x3UavII7P
社内SEにも2種類いるから注意な
1.社内の要件(改善・PC入替え・ネットワーク)を自分の部門でやる場合
2.社内の要件を予算取って外注に出すタイプ

どっちが楽なのかはわかるよな?

80: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 14:37:20.88 ID:qqOY85tBP
>>78
基本①で規模がでかいと②だわ

81: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 14:42:19.75 ID:x3UavII7P
>>80
1は個人スキルないとマジ死ぬから注意wwwwwwww
2はコミュ力があれば何とかなる、出世コースに乗れる

だから、2をオススメする
1やってから中途採用で2もあり

83: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 15:14:04.28 ID:qqOY85tBP
>>81
業務系だからデータベースわかって言語はなんでもいいからプログラム組めれば何とかなるよ
一人で使うならアクセスvbaでもいいし

>>82
プログラマーではあるけど同業ではないと思う
WEB屋もphpとかあるからプログラマーだけどなんか違う気するだろー

82: 以下、名無しにかわりましてVIPがお送りします 2013/09/02(月) 14:49:21.67 ID:xABoQ9jk0
お前らどうせ電機メーカで組み込みやってるソフト屋は同業だと思ってないんだろ?