1read 100read
2012年3月プログラマー79: イラッつとするコーディングスタイル (194) TOP カテ一覧 スレ一覧 2ch元 削除依頼
UMLなんていらない (344)
偽装請負の温床、JIET(日本情報技術取引所) (216)
【うゆ】uyスレ6【クソコテホイホイ】 (841)
サイボウズラボ 3.0 (609)
富士通の残業だとかいろいろ教えて>< (742)
【孤立】偽装派遣で一人で常駐 2社目【根性】 (296)

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


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元 削除依頼
プログラマ業界で生きていくための最低条件 (720)
★☆☆ アスペルガーだけどプログラマ (646)
     (858)
プログラマ業界で生きていくための最低条件 (720)
ユニシステム株式会社 (176)
■自宅待機プログラマが自宅から集うスレ 6ヶ月目■ (494)
--log9.info------------------
【ルール無視】ゾンビ野郎手配書 4冊目【モラル欠落】 (284)
タナカ.380ガバメントってどうよ (394)
【ゾンビ】エバンってどうなの?1【珍砲団】 (612)
【特定】無線関係スレ【省電力】 (867)
FN銃総合スレッドPart3 (207)
三八式歩兵銃とM1ガーランド (276)
【SIIS】エスツーエス総合スレver.5【S2S】 (769)
彡⌒ミして一時間まったく!されなかったら髪 (509)
ミスチルのHANABIって名曲だな。 (191)
【天地人】長岡まつり大花火大会5【フェニックス】 (298)
花火の火薬から (677)
( ´_ゝ`)流石だよな俺ら@花火板 (121)
日大・専修・南山・中京・京都産業・西南と花火 (141)
撮影はカメラ それとも ビデオ どっちがいいか (117)
土浦全国花火競技大会が日本一だ!10発目 (262)
※※ これから行われる花火大会 4 ※※ (410)
--log55.com------------------
【バーチャルYoutuber】にじさんじ有ンチスレ14233【裸の舞元とオレ】
【うんこちゃん】加藤純一総合スレpart864
【かっさん】加藤純一ID無しスレpart3119【千倉交番】
【バーチャル】hololiveアンチスレ#2478【youtuber】
【バーチャルYouTuber】.LIVEアイドル部アンチスレ#9289【アップランド】
【バーチャル】hololiveアンチスレ#2479【youtuber】
【バーチャルYoutuber】にじさんじ有ンチスレ14234【次スレどこやー😭】
【かっさん】加藤純一ID無しスレpart3120【千倉交番】