1read 100read
2012年3月プログラマー79: イラッつとするコーディングスタイル (194) TOP カテ一覧 スレ一覧 2ch元 削除依頼
\(^o^)/●ボーナス支給額 報告スレ●\(^o^)/ (323)
35歳までに上流工程に食い込めなかった馬鹿集まれ (121)
N88 BASIC (694)
A型のプログラマーは神 (288)
デバッグにかかる時間は、開発にかかる時間以上 (125)
ビルゲイツ「」←何言わせても勝ち組 (684)

イラッつとするコーディングスタイル


1 :12/03/10
文法的には何ひとつ間違ってはいないし、本人なりに見やすくしようとする意図は汲み取れるのだが、
どうにも気持ち悪くて、「修正してやる!」と叫びながらキーボードを激しく連打したくなる
そういう薄気味悪いコーディングスタイルを発見したら書き込むスレッド

2 :12/03/10
for(i = 0;i < IMAX;i++){・・・}

3 :12/03/10
#define { begin
#define } end

4 :12/03/10
pBase = (条件) ? new Child1() : new Child2();

5 :12/03/11
スレタイがイラッつとする

6 :12/03/11
             ☆
     ×         ' .                    ×
    x            ` .        x            ヽ . ☆
                         X   ,. -. ‐'´ ̄``丶、 ノ}          X
                     /} /: : : : : : : : : ーヘ
                 ×   / //: : : :l: .:. .} | . : : : : } . . ゜
                     _ノ_,ム′: : : |:::::::/! l.::. : !  /: :\        , ☆
               _/ /,. -‐〉 : : :_ !:;イ¬.|:i::: |i.|. :.i::: : : ヽ     ;
        ☆ . .     ,. '´!{  ゝ-‐''^¨二.ノ:〔__− V!::!LTV{::: : : :.',
            ×x .   ぃ     .イ::. : : |⌒`  }:リ'示Y1:: : : i }         x  ×
       . '´     ,. 介iー-、 {:::::::::. : : !     ^' ヒ'リ ',.|:::. :..  :.!リ          X
  X    /       /ヽ' L!  ヽ. Y::::i::::. .::. |r:ゥ- 、' `^ /:::l::::.::::.x:リ′ ; ☆ イラッ☆
   x /      / /⌒ヽ  込Jヽ:ト:{>、:ィ八_ ,.‐く  イ:ィ:::!x::X::/ ゛
   i'´       /-r‘ー、  ヘ-┴‐〉 `'i¬  ヘ.__{:::::::}.彳〔__レ1::ル'゜   ゜ .
    ー 、 --−'´   コ:..:.}:    \ 丶  .l | -、匸⌒´:_;-、ノ }_ ´        ゜ ×.
     `ヽ、    └;.:.    ..  }  〉 ヽ.| 〈__:,.イv/´〕、冫`i  x          ☆
        ` ¬ゥ´:..:....:..:..:..: ,ノ、 \ { !    {.{j_/,ィう′ !
      x '   Y:..:..:..X:..:..:.∠.._    ヽ.} |.     } `マ^V   |      X
          , ゛ヽ:..:..:..:../  ,.⊥_  /小\¬-{   ∨ヘ._,. -‐¬、
        ☆     ` ー′  j:..:..:..:Y´:´/ハ卜':..! :ヽ  ∧::ヘ .:..:... }
                      /:..:..:..:..j/:..:.`:..:´:..:..i  :..:ト-_ノ マ'’:..:..:..:..ヘ 

7 :12/03/11
#define private public

8 :12/03/11
String hoge
 =
 null
 ;
hoge
 =
 getSQL(
  flg1
  flg2
 )
 ;

9 :12/03/11
>>8の9行目訂正
  flg1,

10 :12/03/11
GNUの字下げスタイルはなんか嫌だ

11 :12/03/11
コードの書き方にこだわるようじゃ3流だな。
こだわるべきはアルゴリズムだろ。
そこがきちっとしていればコードが短くなる。コードが短くなれば自然と読みやすくなる。
早くその領域に達するといいな。

12 :12/03/11
他人のコードを読んだことないの?

13 :12/03/11
>>11
既知のパターンで書かれているかも重要
パターンとパターンをつないでいくような作りであると、読みながら何をしたいのかが分かる。

14 :12/03/11
>11
コードを短くするにはアルゴリズムではなく
関数にすることが重要。
どんなに長いコードでもたった一行にできる。
しかもすごく読みやすくなる。

15 :12/03/11
というか設計だな

16 :12/03/11
>>1の条件に該当するヤツは
一行が長くならないようにコメントを適当に改行してるんだけど
単語の途中を改行でぶった切られてのぐらいしか思いつかない
あとはイラッっとするので
> 本人なりに見やすくしようとする意図は汲み取れるのだが
で該当したものはない

17 :12/03/11
古来から言われていることだが、ポインタ変数と普通の変数をいっぺんに宣言するとまぎらわしい。

18 :12/03/11
>>17
int* ptr_A, B;
こんな感じ?

19 :12/03/11
K&R流では
int *pa, b;
だね
*(アスタリスク)の前に空白を入れるのか後に入れるのかという違いだけど、
>>18のスタイルの発祥はどこなんだろ?

20 :12/03/11
>>18
>int* ptr_A, B;
これで両方ポインタになるべきだと
俺は思うのだが

21 :12/03/11
クラスのプロトタイプ宣言ファイルにそのクラスが使う複数の構造体の宣言を一緒に入れる
クラスのプロトタイプ宣言のなかにそのクラスが使う構造体の宣言をごっそり入れる
どう思う?

22 :12/03/11
>>20
ていう思い違いをしやすいから*は識別名にくっつけろ、という話じゃね?

23 :12/03/11
いやそうじゃなく文法自体が不適切なんじゃないのかって話

24 :12/03/11
>>23
そりゃあ「コンパイラに優しい言語仕様」を目指したわけだし
てかそもそもそういうスレじゃないし

25 :12/03/12
if(hoge != null) {
if(hoge.length() > 0) {
// 糞処理
}
}
ネスト深くせずreturnしてほしいわ
if(hoge != null && hoge.lenght() > 0) { return; }
// 糞処理

26 :12/03/12
>>25
バカっ☆

27 :12/03/12
定数宣言で=の位置を縦に揃えるのがあまり好きじゃない

28 :12/03/12
Linux環境でC++ソースの整形コマンドってなかったっけ?

29 :12/03/12
>27
俺は逆に、揃えてないほうが不快。

30 :12/03/12
揃ってると見やすいけど修正しづらい

31 :12/03/12
後で長い名前の定義を足した時に揃え直すか迷う

32 :12/03/12
>>27 and >>29,30
縦揃えにTabを使わないのが問題なんじゃね?
自分はタブ幅4(:se ts=4 sw=4 ai)だけど、全く苦にならない

33 :12/03/12
俺はインデント以外でタブ文字使われるのが大嫌い

34 :12/03/12
キナ臭くなってまいりました

35 :12/03/12
タブサイズを8以外にしてるのがイラっとするわ。

36 :12/03/12
indentのデフォルトは2じゃね?

37 :12/03/12
=の位置で簡単に揃えられない
エディタを使う奴が無能なだけ。

38 :12/03/12
そもそも変数名の長さがまちまちなところに問題の根本が潜んでいる気がする

39 :12/03/12
だからコボラーはCOL001,COL002,COL003…みたいな変数にしてたんだろ

40 :12/03/13
エディタ表示部が勝手に検知して勝手に揃えて表示するというのはありなような気はする
画面表示がディスク上のファイルのバイナリ列と完全に一致しなければならないわけじゃないしな

41 :12/03/13
2行以上のコメントを//で書いてあるとイラッとくる

42 :12/03/13
>>39
変数の長さを規約で決めてしまうのか
ソースを見やすくするっていう規約の存在意義を考えるとアリだな

43 :12/03/13
>>36
インデントは2でも4でも8でもいいけど、タブのサイズを4とかにしてタブでインデントしてるヤツは
イラっとするわ。

44 :12/03/13
>41
それは//のほうが正しいだろ。
/*〜*/で普段のコメントが書いてあると、一部の処理をして試したいときに、普段のコメントがそれを邪魔する。
関数の中身に書くコメントは//じゃないと邪魔だ。
JavaDoc形式については別だぞ。

45 :12/03/13
for(i = 0; i < max; i++);  ←※
{
  // 云々
}
こーゆーミスするなら中括弧の前に改行入れんじゃねえよハゲ

46 :12/03/13
てへぺろ;-P

47 :12/03/13
>>43
MS「(;゚Д゚)エッ…」

48 :12/03/13
>>43
なんで?

49 :12/03/13
>>43
なんで?

50 :12/03/13
>>43はRuby使いなんじゃね?
Rubyには標準ライブラリを含めてインデント2で書かれたコードが多い

51 :12/03/14
インデント2は明らかに少なすぎ。
デザイン的に空間が分かれてるように見えない。

52 :12/03/14
>>43
チカンすればいいじゃん

53 :12/03/14
>>52
おまわりさんこっちです

54 :12/03/14
生ポインタとusingを禁止で全て書き直せとのお達し
配列長が必要なので、shared_arrayは使えない
vector<Hoge*>* hoge;

boost::shared_ptr<std::vector<boost::shared_ptr<Hoge>>> hoge;
マジキチ
下手すりゃdelete漏れを探すよりもカオスなことになりそうだぜ

55 :12/03/14
>>47-49
タブサイズは8に決まってるだろ。
タブサイズを8以外にしてるやつは迷惑だわ。
インデントを8以外にしたいときにはスペース使え。
>>52
どっちにしても面倒だし、チカンするくらいならエディタの設定を変えたほうが速いだろ。

56 :12/03/14
インデントはタブサイズの設定関係ないだろ
インデント以外のレイアウトにタブ使うヤツがクソ野郎

57 :12/03/14
みんなそれぞれ意見がバラバラなのが面白いぜw

58 :12/03/14
コーディングスタイルはしばしば宗教に例えられる

59 :12/03/14
コーディングルール・コーディングスタイル議論は山のようにあって正直ウンザリなので、
このスレでは「イラッつとする」かどうかのみで判断した感情的なレスをお願いします

60 :12/03/14
>>56
関係あるだろ。
タブサイズ4前提でインデントしてて、8に設定してあるエディタで見ると崩れるやつとかいる。

61 :12/03/14
いまやってるphpのシステムで関数の引数を
function hoge($arg1, $arg2, $arg3)
{
$weight = arg1;
$height = arg2;
$age = arg3;

}
と必ず$arg1, $arg2…みたいな意味の無い名前の変数でうけて、関数の中で意味のある
名前の変数に移してるんだけど、
普通に
function hoge($weight, $height, $age)
でいいじゃないか。
なんか意味あるのか。

62 :12/03/14
コードを追うと突如現る謎の空白行
ふと右を見ると変数の頭文字らしきものがニョキっと生えてる
タブ8とか3階層ネストするだけで宇宙に行ってまうわ
気持ち悪いったらありゃしない

63 :12/03/14
Full HDなモニタ買ってもらえ。

64 :12/03/14
>>61
Perl厨のせいなんじゃね?
Perlには仮引数がないから
sub hoge {
 $weight = shift;
 $height = shift;
 $age = shift;
}
ってやるよ。

65 :12/03/14
>>60
すまん、インデントの意味を間違えてた
段落を意図したインデントのみタブを使えって言いたかった
こういうこと言いたかったんです
ttp://ameblo.jp/argv/entry-10000648280.html

66 :12/03/15
>>65
正しい日本語使おうな。
たしかに、タブとスペース混ぜられたり、後ろに不要な空白残したりされると殴りたくなる。
てめーのことだぜ先輩!

67 :12/03/15
いえ、わたくしは、イライラしながらフォーマッタでポチポチ揃えてる側の人間ですが。

68 :12/03/15
昔はタブサイズは8にするべきだって思ってたんだけどね
今は1か2がちょうどよく思えてきたよ

69 :12/03/15
タブのサイズを1に設定してタブを使うなら、ふつーにスペース使ったほうがよくね?

70 :12/03/15
bool hoge(){
 if( fuga() != false ){ return false; }
 else{ return true; }
}
なぜreturn !fuga();としないのだ……。

71 :12/03/15
/* 2011.3.11 なんかエラーになるのでとりあえず外す
 return false;
*/
}
return true;

72 :12/03/15
>>70
A:論理値をリテラルと比較するような阿呆だから

73 :12/03/15
>>71
3.11・・・

74 :12/03/15
>>73
rev.666 2011-03-11 15:47
ほぼ100%職場おわるので中間コミット
SyntaxError出るけどこれ以上はやばいのでかんべんしてください
去年下請けと組んでやった案件の作業ログにこんなのあったの思い出したわ

75 :12/03/15
修正したり追加した行に日付と名前が書いてあるのは
お前の名前分かっても意味無いんじゃ、って思うな。

76 :12/03/15
日付は何故そう修正したか雰囲気がわかったりするから無いよりはマシ
それよりはまともなコメントを書けよハゲって話なんだが
担当者を入れるのは責任問題の押し付け合いをするためのものと理解している

77 :12/03/15
日付と名前は必須だろ
どこにバグがあるか特定する時に一番役に立つ

78 :12/03/15
修正履歴なんて入れてないでソース管理ツール使えよって感じだけど、ドカタの現場だとただのファイル共有ツールって認識だし使っても同じか。

79 :12/03/15
わざわざソース管理ツールをインストールするの面倒じゃん
おまえらすぐ管理ツールを変更しちゃうから古いソースを見る時に大変なんだよ

80 :12/03/15
やっぱり新しいツールや技術についていけない無能に合わせるしかないよな

81 :12/03/15
名前がファミリーネームどころかファーストネームですらなく、
親しい間でなければ使わないような愛称
鼻穴に5センチほど割り箸突っ込んでグググと水平に近づけて
後遺症が残らない程度に苦痛を与えることで反省を促したい

82 :12/03/16
俺もちょっと前にPHPで
return hoge ? false : true ;
って書いてた。恥ずかしい

83 :12/03/16
return func();
とか、気持ち悪くないか?

84 :12/03/16
返却値の柔軟性を奪っておいた方が後々不具合が少ない気はしないでもない。

85 :12/03/16
/*2008.01.01 障害対応 start */
/*2009.09.15 障害対応 start */
/*2010.12.11 障害対応 start */
/*2011.02.13 障害対応 start */
return true;
/*2011.02.13 障害対応 end */
/*2010.12.11 障害対応 end */
/*2009.09.15 障害対応 end */
/*2008.01.01 障害対応 end */
こんなのを見ると腹立つ
消すなって言われると帰りたくなる

86 :12/03/16
>>85
え?なんで修正した箇所のソースが残ってないの?
普通はコメントアウトして残すだろ?
コメントアウトした部分を削除する時は日付も削除するし
そんな状態にはならない

87 :12/03/16
>>86
>普通はコメントアウトして残すだろ?
普通は…な…。

88 :12/03/16
正月から大変なんだなw

89 :12/03/17
>85
そもそもSubversionとか使ってないの?

90 :12/03/17
そこかよw

91 :12/03/17
全体の設計があきらかにアレなコードで
いちいち修正をコメントで残されてもなー

92 :12/03/17
ありきたりだけど、コメントが疑問系のやつ
一回それに対する回答コメントがあってワラタ

93 :12/03/18
LINQの使い方を知って以来、foreachまみれのソースは基本的にイラつく

94 :12/03/18
foreach?gotoでループを表現しているコードをいじらされるよりだいぶマシだな

95 :12/03/19
>>14
ちゃんと意味のある関数ならいいけどな。

96 :12/03/19
>>20
C#はなるんじゃなかった?

97 :12/03/19
C#にはポインタはありません(すくなくとも表面的には)
それよりもここのタイトルの「イラッつと」って書き方にいらっと来た。

98 :12/03/19
>>97
あからさまに仕様上あるわけだが>C#のポインタ
そんなことより、スレタイに関しては >>5 で既出なんだが
ホントに直近のレスだけしか見てないんだなあ(´・ω・`)

99 :12/03/19
何のひねりもないマジレスにイラッつとした

100read 1read
1read 100read
TOP カテ一覧 スレ一覧 2ch元 削除依頼
ハッカー&クラッカー専用 (437)
例外を正しく使えないプログラマ多いね。 その7 (515)
長門有希が質問に答えるスレ (325)
(´・ω・`)知らんがな5 (541)
\(^o^)/●ボーナス支給額 報告スレ●\(^o^)/ (323)
【うゆ】uyスレ6【クソコテホイホイ】 (841)
--log9.info------------------
【つま先】フォアフット総合スレ【着地】 (144)
【安井・光延】2012高2世代 part11【坂本・高森】 (235)
陸上板自治スレ (442)
【100m】Sogelau TUVALU【15.66】 (181)
【9秒58】ウサイン・ボルトPart26【19秒19】 (776)
兵庫の陸上競技を語るスレ (183)
石川県の陸上競技を語る (104)
カロキ応援スレ (138)
【中谷・石川】2012年新高校3年世代7【大塚・一色】 (824)
☆☆大東文化大学女子陸上部☆☆ (794)
高校男子短距離スレ part3 (794)
【佐倉アスリート倶楽部】新谷仁美Part.7【興譲館】 (341)
広島県高校長距離応援スレッド (906)
駒澤大学PART155 (1001)
山梨学院大附属高校陸上部Part.1 (771)
市民マラソン in 東京・荒川河川敷4 (547)
--log55.com------------------
アセロラちゃんに萌えるスレ
リーリエちゃんに萌えるスレ★2
新ポケモンレーティング総合 part1
(・大・)<ぶっし… ナマコブシ2投目
セレナ嫌われすぎじゃね?
【フクスロー】モクロー族スレ2【ジュナイパー】
真夏の夜のポケモン交流所 第三章
ミミッキュ対策本部【シングル】