- 1 名前：仕様書無しさん [2012/03/10(土) 22:20:29.12 ]
- 文法的には何ひとつ間違ってはいないし、本人なりに見やすくしようとする意図は汲み取れるのだが、
どうにも気持ち悪くて、「修正してやる！」と叫びながらキーボードを激しく連打したくなる
そういう薄気味悪いコーディングスタイルを発見したら書き込むスレッド
- 2 名前：仕様書無しさん mailto:sage [2012/03/10(土) 22:24:49.50 ]
- for(i = 0;i < IMAX;i++){・・・}
- 3 名前：仕様書無しさん mailto:sage [2012/03/10(土) 22:26:16.62 ]
- #define { begin
#define } end
- 4 名前：仕様書無しさん mailto:sage [2012/03/10(土) 22:27:07.72 ]
- pBase = (条件) ? new Child1() : new Child2();
- 5 名前：仕様書無しさん mailto:sage [2012/03/11(日) 01:07:15.09 ]
- スレタイがイラッつとする
- 6 名前：仕様書無しさん mailto:sage [2012/03/11(日) 01:10:07.48 ]
- ☆
× ' . ×
x ｀ . ｘ ヽ . ☆
Ｘ ,. -. ‐'´￣｀`丶､ ﾉ} Ｘ
／} ／: : : : : : : : : ｰﾍ
× / //: : : :l: .:. .} | . : : : : } . ． ゜
_ノ_,ﾑ′: : : |:::::::/! l.::. : ! /: :＼ , ☆
_／ /,. -‐〉 : : :_ !:;ｲ¬.|:i::: |i.|. :.i::: : : ヽ ;
☆ . . ,. '´!{ ゝ-‐''＾¨二.ノ:〔__− V!::!LTV{::: : : :.',
×ｘ . ぃ .イ::. : : |⌒` }:ﾘ'示Y１:: : : ｉ } x ×
. '´ ,. 介iｰ-､ ｛:::::::::. : : ! ＾' ﾋ'ﾘ ',.|:::. :.. :.!ﾘ Ｘ
Ｘ ／ ／ヽ' Ｌ! ヽ. Y::::ｉ::::. .::. |ｒ:ｩ- ､' ｀＾ /:::l::::.::::.x:ﾘ′ ; ☆ イラッ☆
ｘ ／ / ／⌒ヽ 込Jヽ:ﾄ:{>､:ィ八_ ,.‐く ｲ:ィ:::!x::X::/ ゛
i'´ ／-r‘ｰ､ ﾍ-┴‐〉 ｀'i¬ ヘ.__{:::::::}.彳〔__レ1::ﾙ'゜ ゜ ．
ｰ ､ --−'´ ｺ:..:.}: ＼ 丶 .ｌ | -､匸⌒´:_;-､ノ ｝_ ´ ゜ ×.
｀ヽ､ └;.:. .. } 〉 ヽ.｜ 〈__:,.イv/´〕、冫｀i x ☆
｀ ¬ゥ´:..:....:..:..:..: ,ﾉ､ ＼ { ! {.{j_/,ィう′ !
ｘ ' Y:..:..:..Ｘ:..:..:.∠.._ ヽ.} |. } ｀ﾏ^V | X
, ゛ヽ:..:..:..:..／ ,.⊥_ ／小＼¬-{ ∨ﾍ._,. -‐¬､
☆ ｀ ー′ j:..:..:..:Y´:´/ハ卜':..! :ヽ ∧::ﾍ .:..:... }
/:..:..:..:..j/:..:.｀:..:´:..:..i :..:ﾄ-_ノ ﾏ'’:..:..:..:..ﾍ
- 7 名前：仕様書無しさん mailto:sage [2012/03/11(日) 01:55:03.52 ]
- #define private public
- 8 名前：仕様書無しさん mailto:sage [2012/03/11(日) 02:43:59.61 ]
- String hoge
=
null
;
hoge
=
getSQL(
flg1
flg2
)
;
- 9 名前：仕様書無しさん mailto:sage [2012/03/11(日) 02:44:47.10 ]
- >>8の9行目訂正
flg1,
- 10 名前：仕様書無しさん mailto:sage [2012/03/11(日) 03:05:02.57 ]
- GNUの字下げスタイルはなんか嫌だ
- 11 名前：仕様書無しさん mailto:sage [2012/03/11(日) 06:31:52.11 ]
- コードの書き方にこだわるようじゃ3流だな。
こだわるべきはアルゴリズムだろ。
そこがきちっとしていればコードが短くなる。コードが短くなれば自然と読みやすくなる。
早くその領域に達するといいな。
- 12 名前：仕様書無しさん mailto:sage [2012/03/11(日) 07:15:38.84 ]
- 他人のコードを読んだことないの？
- 13 名前：仕様書無しさん mailto:sage [2012/03/11(日) 09:14:43.77 ]
- >>11
既知のパターンで書かれているかも重要
パターンとパターンをつないでいくような作りであると、読みながら何をしたいのかが分かる。
- 14 名前：仕様書無しさん mailto:sage [2012/03/11(日) 12:56:09.17 ]
- >11
コードを短くするにはアルゴリズムではなく
関数にすることが重要。
どんなに長いコードでもたった一行にできる。
しかもすごく読みやすくなる。
- 15 名前：仕様書無しさん mailto:sage [2012/03/11(日) 13:09:35.46 ]
- というか設計だな
- 16 名前：仕様書無しさん mailto:sage [2012/03/11(日) 16:11:30.97 ]
- >>1の条件に該当するヤツは
一行が長くならないようにコメントを適当に改行してるんだけど
単語の途中を改行でぶった切られてのぐらいしか思いつかない
あとはイラッっとするので
> 本人なりに見やすくしようとする意図は汲み取れるのだが
で該当したものはない
- 17 名前：仕様書無しさん mailto:sage [2012/03/11(日) 17:29:28.29 ]
- 古来から言われていることだが、ポインタ変数と普通の変数をいっぺんに宣言するとまぎらわしい。
- 18 名前：仕様書無しさん mailto:sage [2012/03/11(日) 18:26:18.08 ]
- >>17
int* ptr_A, B;
こんな感じ？
- 19 名前：仕様書無しさん mailto:sage [2012/03/11(日) 18:34:53.32 ]
- K&R流では
int *pa, b;
だね
＊(アスタリスク)の前に空白を入れるのか後に入れるのかという違いだけど、
>>18のスタイルの発祥はどこなんだろ？
- 20 名前：仕様書無しさん mailto:sage [2012/03/11(日) 19:02:06.53 ]
- >>18
＞int* ptr_A, B;
これで両方ポインタになるべきだと
俺は思うのだが
- 21 名前：仕様書無しさん mailto:sage [2012/03/11(日) 19:45:40.12 ]
- クラスのプロトタイプ宣言ファイルにそのクラスが使う複数の構造体の宣言を一緒に入れる
クラスのプロトタイプ宣言のなかにそのクラスが使う構造体の宣言をごっそり入れる
どう思う？
- 22 名前：仕様書無しさん mailto:sage [2012/03/11(日) 22:13:55.60 ]
- >>20
ていう思い違いをしやすいから*は識別名にくっつけろ、という話じゃね？
- 23 名前：仕様書無しさん mailto:sage [2012/03/11(日) 22:39:10.87 ]
- いやそうじゃなく文法自体が不適切なんじゃないのかって話
- 24 名前：仕様書無しさん mailto:sage [2012/03/11(日) 23:25:48.90 ]
- >>23
そりゃあ「コンパイラに優しい言語仕様」を目指したわけだし
てかそもそもそういうスレじゃないし
- 25 名前：仕様書無しさん [2012/03/12(月) 02:04:41.58 ]
- if(hoge != null) {
if(hoge.length() > 0) {
// 糞処理
}
}
ネスト深くせずreturnしてほしいわ
if(hoge != null && hoge.lenght() > 0) { return; }
// 糞処理
- 26 名前：仕様書無しさん mailto:sage [2012/03/12(月) 02:11:39.25 ]
- >>25
バカっ☆
- 27 名前：仕様書無しさん mailto:sage [2012/03/12(月) 02:39:19.67 ]
- 定数宣言で=の位置を縦に揃えるのがあまり好きじゃない
- 28 名前：仕様書無しさん mailto:sage [2012/03/12(月) 06:45:08.76 ]
- Linux環境でC++ソースの整形コマンドってなかったっけ？
- 29 名前：仕様書無しさん mailto:sage [2012/03/12(月) 13:02:39.61 ]
- >27
俺は逆に、揃えてないほうが不快。
- 30 名前：仕様書無しさん mailto:sage [2012/03/12(月) 14:48:57.42 ]
- 揃ってると見やすいけど修正しづらい
- 31 名前：仕様書無しさん mailto:sage [2012/03/12(月) 15:04:21.10 ]
- 後で長い名前の定義を足した時に揃え直すか迷う
- 32 名前：仕様書無しさん mailto:sage [2012/03/12(月) 15:10:16.44 ]
- >>27 and >>29,30
縦揃えにTabを使わないのが問題なんじゃね？
自分はタブ幅4(:se ts=4 sw=4 ai)だけど、全く苦にならない
- 33 名前：仕様書無しさん mailto:sage [2012/03/12(月) 16:30:15.35 ]
- 俺はインデント以外でタブ文字使われるのが大嫌い
- 34 名前：仕様書無しさん mailto:sage [2012/03/12(月) 19:58:47.49 ]
- キナ臭くなってまいりました
- 35 名前：仕様書無しさん mailto:sage [2012/03/12(月) 20:46:52.41 ]
- タブサイズを8以外にしてるのがイラっとするわ。
- 36 名前：仕様書無しさん mailto:sage [2012/03/12(月) 21:11:14.76 ]
- indentのデフォルトは2じゃね？
- 37 名前：仕様書無しさん mailto:sage [2012/03/12(月) 21:35:49.34 ]
- =の位置で簡単に揃えられない
エディタを使う奴が無能なだけ。
- 38 名前：仕様書無しさん mailto:sage [2012/03/12(月) 21:38:11.18 ]
- そもそも変数名の長さがまちまちなところに問題の根本が潜んでいる気がする
- 39 名前：仕様書無しさん mailto:sage [2012/03/12(月) 21:53:50.70 ]
- だからコボラーはCOL001,COL002,COL003…みたいな変数にしてたんだろ
- 40 名前：仕様書無しさん mailto:sage [2012/03/13(火) 00:48:50.68 ]
- エディタ表示部が勝手に検知して勝手に揃えて表示するというのはありなような気はする
画面表示がディスク上のファイルのバイナリ列と完全に一致しなければならないわけじゃないしな
- 41 名前：仕様書無しさん mailto:sage [2012/03/13(火) 06:49:57.92 ]
- 2行以上のコメントを//で書いてあるとイラッとくる
- 42 名前：仕様書無しさん mailto:sage [2012/03/13(火) 07:26:43.39 ]
- >>39
変数の長さを規約で決めてしまうのか
ソースを見やすくするっていう規約の存在意義を考えるとアリだな
- 43 名前：仕様書無しさん mailto:sage [2012/03/13(火) 09:01:52.99 ]
- >>36
インデントは2でも4でも8でもいいけど、タブのサイズを4とかにしてタブでインデントしてるヤツは
イラっとするわ。
- 44 名前：仕様書無しさん mailto:sage [2012/03/13(火) 13:39:53.08 ]
- >41
それは//のほうが正しいだろ。
/*〜*/で普段のコメントが書いてあると、一部の処理を殺して試したいときに、普段のコメントがそれを邪魔する。
関数の中身に書くコメントは//じゃないと邪魔だ。
JavaDoc形式については別だぞ。
- 45 名前：仕様書無しさん mailto:sage [2012/03/13(火) 19:40:55.20 ]
- ｆｏｒ（i = 0; i < max; i++）; ←※
{
// 云々
}
こーゆーミスするなら中括弧の前に改行入れんじゃねえよハゲ
- 46 名前：仕様書無しさん mailto:sage [2012/03/13(火) 20:55:25.95 ]
- てへぺろ;-P
- 47 名前：仕様書無しさん mailto:sage [2012/03/13(火) 22:54:43.60 ]
- >>43
MS「(；ﾟДﾟ)ｴｯ…」
- 48 名前：仕様書無しさん mailto:sage [2012/03/13(火) 23:50:12.33 ]
- >>43
なんで？
- 49 名前：仕様書無しさん mailto:sage [2012/03/13(火) 23:52:59.28 ]
- >>43
なんで？
- 50 名前：仕様書無しさん mailto:sage [2012/03/13(火) 23:57:17.22 ]
- >>43はRuby使いなんじゃね？
Rubyには標準ライブラリを含めてインデント2で書かれたコードが多い
- 51 名前：仕様書無しさん mailto:sage [2012/03/14(水) 00:10:57.34 ]
- インデント2は明らかに少なすぎ。
デザイン的に空間が分かれてるように見えない。
- 52 名前：仕様書無しさん mailto:sage [2012/03/14(水) 00:35:03.78 ]
- >>43
チカンすればいいじゃん
- 53 名前：仕様書無しさん mailto:sage [2012/03/14(水) 02:52:09.84 ]
- >>52
おまわりさんこっちです
- 54 名前：仕様書無しさん mailto:sage [2012/03/14(水) 07:34:18.44 ]
- 生ポインタとusingを禁止で全て書き直せとのお達し
配列長が必要なので、shared_arrayは使えない
vector<Hoge*>* hoge;
↓
boost::shared_ptr<std::vector<boost::shared_ptr<Hoge>>> hoge;
マジキチ
下手すりゃdelete漏れを探すよりもカオスなことになりそうだぜ
- 55 名前：仕様書無しさん mailto:sage [2012/03/14(水) 13:02:41.26 ]
- >>47-49
タブサイズは8に決まってるだろ。
タブサイズを8以外にしてるやつは迷惑だわ。
インデントを8以外にしたいときにはスペース使え。
>>52
どっちにしても面倒だし、チカンするくらいならエディタの設定を変えたほうが速いだろ。
- 56 名前：仕様書無しさん mailto:sage [2012/03/14(水) 16:12:41.42 ]
- インデントはタブサイズの設定関係ないだろ
インデント以外のレイアウトにタブ使うヤツがクソ野郎
- 57 名前：仕様書無しさん mailto:sage [2012/03/14(水) 16:20:32.15 ]
- みんなそれぞれ意見がバラバラなのが面白いぜw
- 58 名前：仕様書無しさん mailto:sage [2012/03/14(水) 16:49:24.33 ]
- コーディングスタイルはしばしば宗教に例えられる
- 59 名前：仕様書無しさん mailto:sage [2012/03/14(水) 17:57:48.38 ]
- コーディングルール・コーディングスタイル議論は山のようにあって正直ウンザリなので、
このスレでは「イラッつとする」かどうかのみで判断した感情的なレスをお願いします
- 60 名前：仕様書無しさん mailto:sage [2012/03/14(水) 19:30:13.99 ]
- >>56
関係あるだろ。
タブサイズ4前提でインデントしてて、8に設定してあるエディタで見ると崩れるやつとかいる。
- 61 名前：仕様書無しさん mailto:sage [2012/03/14(水) 19:41:56.11 ]
- いまやってるphpのシステムで関数の引数を
function hoge($arg1, $arg2, $arg3)
{
$weight = arg1;
$height = arg2;
$age = arg3;
}
と必ず$arg1, $arg2…みたいな意味の無い名前の変数でうけて、関数の中で意味のある
名前の変数に移してるんだけど、
普通に
function hoge($weight, $height, $age)
でいいじゃないか。
なんか意味あるのか。
- 62 名前：仕様書無しさん mailto:sage [2012/03/14(水) 19:47:25.12 ]
- コードを追うと突如現る謎の空白行
ふと右を見ると変数の頭文字らしきものがニョキっと生えてる
タブ8とか3階層ネストするだけで宇宙に行ってまうわ
気持ち悪いったらありゃしない
- 63 名前：仕様書無しさん mailto:sage [2012/03/14(水) 20:32:57.87 ]
- Full HDなモニタ買ってもらえ。
- 64 名前：仕様書無しさん mailto:sage [2012/03/14(水) 20:42:50.52 ]
- >>61
Perl厨のせいなんじゃね？
Perlには仮引数がないから
sub hoge {
$weight = shift;
$height = shift;
$age = shift;
}
ってやるよ。
- 65 名前：仕様書無しさん mailto:sage [2012/03/14(水) 23:14:52.71 ]
- >>60
すまん、インデントの意味を間違えてた
段落を意図したインデントのみタブを使えって言いたかった
こういうこと言いたかったんです
ttp://ameblo.jp/argv/entry-10000648280.html
- 66 名前：仕様書無しさん mailto:sage [2012/03/15(木) 01:23:10.20 ]
- >>65
正しい日本語使おうな。
たしかに、タブとスペース混ぜられたり、後ろに不要な空白残したりされると殴りたくなる。
てめーのことだぜ先輩！
- 67 名前：仕様書無しさん mailto:sage [2012/03/15(木) 02:07:12.65 ]
- いえ、わたくしは、イライラしながらフォーマッタでポチポチ揃えてる側の人間ですが。
- 68 名前：仕様書無しさん mailto:sage [2012/03/15(木) 09:47:40.20 ]
- 昔はタブサイズは8にするべきだって思ってたんだけどね
今は1か2がちょうどよく思えてきたよ
- 69 名前：仕様書無しさん mailto:sage [2012/03/15(木) 10:36:11.89 ]
- タブのサイズを1に設定してタブを使うなら、ふつーにスペース使ったほうがよくね?
- 70 名前：仕様書無しさん [2012/03/15(木) 11:36:52.65 ]
- bool hoge(){
if( fuga() != false ){ return false; }
else{ return true; }
}
なぜreturn !fuga();としないのだ……。
- 71 名前：仕様書無しさん mailto:sage [2012/03/15(木) 12:14:05.59 ]
- /* 2011.3.11 なんかエラーになるのでとりあえず外す
return false;
*/
}
return true;
- 72 名前：仕様書無しさん mailto:sage [2012/03/15(木) 12:26:17.66 ]
- >>70
Ａ：論理値をリテラルと比較するような阿呆だから
- 73 名前：仕様書無しさん mailto:sage [2012/03/15(木) 13:45:30.08 ]
- >>71
3.11･･･
- 74 名前：仕様書無しさん mailto:sage [2012/03/15(木) 14:00:09.04 ]
- >>73
rev.666 2011-03-11 15:47
ほぼ100％職場おわるので中間コミット
SyntaxError出るけどこれ以上はやばいのでかんべんしてください
去年下請けと組んでやった案件の作業ログにこんなのあったの思い出したわ
- 75 名前：仕様書無しさん mailto:sage [2012/03/15(木) 14:05:41.36 ]
- 修正したり追加した行に日付と名前が書いてあるのは
お前の名前分かっても意味無いんじゃ、って思うな。
- 76 名前：仕様書無しさん mailto:sage [2012/03/15(木) 18:26:58.01 ]
- 日付は何故そう修正したか雰囲気がわかったりするから無いよりはマシ
それよりはまともなコメントを書けよハゲって話なんだが
担当者を入れるのは責任問題の押し付け合いをするためのものと理解している
- 77 名前：仕様書無しさん mailto:sage [2012/03/15(木) 19:03:15.35 ]
- 日付と名前は必須だろ
どこにバグがあるか特定する時に一番役に立つ
- 78 名前：仕様書無しさん mailto:sage [2012/03/15(木) 20:25:45.12 ]
- 修正履歴なんて入れてないでソース管理ツール使えよって感じだけど、ドカタの現場だとただのファイル共有ツールって認識だし使っても同じか。
- 79 名前：仕様書無しさん mailto:sage [2012/03/15(木) 21:04:29.64 ]
- わざわざソース管理ツールをインストールするの面倒じゃん
おまえらすぐ管理ツールを変更しちゃうから古いソースを見る時に大変なんだよ
- 80 名前：仕様書無しさん mailto:sage [2012/03/15(木) 21:12:59.59 ]
- やっぱり新しいツールや技術についていけない無能に合わせるしかないよな
- 81 名前：仕様書無しさん mailto:sage [2012/03/15(木) 21:13:26.95 ]
- 名前がファミリーネームどころかファーストネームですらなく、
親しい間でなければ使わないような愛称
鼻穴に５センチほど割り箸突っ込んでグググと水平に近づけて
後遺症が残らない程度に苦痛を与えることで反省を促したい
- 82 名前：仕様書無しさん mailto:sage [2012/03/16(金) 00:11:44.00 ]
- 俺もちょっと前にPHPで
return hoge ? false : true ;
って書いてた。恥ずかしい
- 83 名前：仕様書無しさん mailto:sage [2012/03/16(金) 06:55:35.96 ]
- return func();
とか、気持ち悪くないか？
- 84 名前：仕様書無しさん mailto:sage [2012/03/16(金) 11:17:23.41 ]
- 返却値の柔軟性を奪っておいた方が後々不具合が少ない気はしないでもない。
- 85 名前：仕様書無しさん mailto:sage [2012/03/16(金) 19:59:37.11 ]
- /*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 名前：仕様書無しさん [2012/03/16(金) 20:10:41.24 ]
- >>85
え？なんで修正した箇所のソースが残ってないの？
普通はコメントアウトして残すだろ？
コメントアウトした部分を削除する時は日付も削除するし
そんな状態にはならない
- 87 名前：仕様書無しさん mailto:sage [2012/03/16(金) 20:15:05.66 ]
- >>86
>普通はコメントアウトして残すだろ？
普通は…な…。
- 88 名前：仕様書無しさん mailto:sage [2012/03/16(金) 20:20:45.49 ]
- 正月から大変なんだなw
- 89 名前：仕様書無しさん mailto:sage [2012/03/17(土) 01:07:19.71 ]
- >85
そもそもSubversionとか使ってないの？
- 90 名前：仕様書無しさん mailto:sage [2012/03/17(土) 01:09:31.26 ]
- そこかよw
- 91 名前：仕様書無しさん mailto:sage [2012/03/17(土) 07:04:29.13 ]
- 全体の設計があきらかにアレなコードで
いちいち修正をコメントで残されてもなー
- 92 名前：仕様書無しさん mailto:sage [2012/03/17(土) 10:36:26.07 ]
- ありきたりだけど、コメントが疑問系のやつ
一回それに対する回答コメントがあってワラタ
- 93 名前：仕様書無しさん mailto:sage [2012/03/18(日) 07:15:33.32 ]
- LINQの使い方を知って以来、foreachまみれのソースは基本的にイラつく
- 94 名前：仕様書無しさん mailto:sage [2012/03/18(日) 19:33:17.18 ]
- foreach？gotoでループを表現しているコードをいじらされるよりだいぶマシだな
- 95 名前：仕様書無しさん mailto:sage [2012/03/19(月) 01:56:27.12 ]
- >>14
ちゃんと意味のある関数ならいいけどな。
- 96 名前：仕様書無しさん mailto:sage [2012/03/19(月) 01:57:43.43 ]
- >>20
C#はなるんじゃなかった？
- 97 名前：仕様書無しさん mailto:sage [2012/03/19(月) 11:49:33.84 ]
- C#にはポインタはありません（すくなくとも表面的には）
それよりもここのタイトルの「イラッつと」って書き方にいらっと来た。
- 98 名前：仕様書無しさん mailto:sage [2012/03/19(月) 13:33:57.93 ]
- >>97
あからさまに仕様上あるわけだが＞C#のポインタ
そんなことより、スレタイに関しては >>5 で既出なんだが
ホントに直近のレスだけしか見てないんだなあ(´・ω・｀)
- 99 名前：仕様書無しさん mailto:sage [2012/03/19(月) 13:56:14.92 ]
- 何のひねりもないﾏｼﾞﾚｽにイラッつとした
- 100 名前：仕様書無しさん [2012/03/19(月) 15:44:51.04 ]
- スレタイの「イラッつと」は「イラッと」の間違い？
なんかいらっと来た
- 101 名前：仕様書無しさん mailto:sage [2012/03/19(月) 18:12:57.12 ]
- if (0 == hoge)
|
 |
