1read 100read
2012年5月プログラマー19: イラッつとするコーディングスタイル (898)
TOP カテ一覧 スレ一覧 2ch元 削除依頼 ▼
札幌のITは終わった。。。 (532)
.NETのような万能フレームワークが嫌いな人って必ず居る (127)
二重派遣&偽装請負の利用がやめられない日立製作所 (213)
中小零細ワンマン社長の生態を語ろう (682)
例外を正しく使えないプログラマ多いね。 その7 (703)
スペランカーごっこ 7周目 (335)
イラッつとするコーディングスタイル
- 1 :12/03/10 〜 最終レス :12/05/30
- 文法的には何ひとつ間違ってはいないし、本人なりに見やすくしようとする意図は汲み取れるのだが、
どうにも気持ち悪くて、「修正してやる!」と叫びながらキーボードを激しく連打したくなる
そういう薄気味悪いコーディングスタイルを発見したら書き込むスレッド
- 2 :
- for(i = 0;i < IMAX;i++){・・・}
- 3 :
- #define { begin
#define } end
- 4 :
- pBase = (条件) ? new Child1() : new Child2();
- 5 :
- スレタイがイラッつとする
- 6 :
- ☆
× ' . ×
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 :
- #define private public
- 8 :
- String hoge
=
null
;
hoge
=
getSQL(
flg1
flg2
)
;
- 9 :
- >>8の9行目訂正
flg1,
- 10 :
- GNUの字下げスタイルはなんか嫌だ
- 11 :
- コードの書き方にこだわるようじゃ3流だな。
こだわるべきはアルゴリズムだろ。
そこがきちっとしていればコードが短くなる。コードが短くなれば自然と読みやすくなる。
早くその領域に達するといいな。
- 12 :
- 他人のコードを読んだことないの?
- 13 :
- >>11
既知のパターンで書かれているかも重要
パターンとパターンをつないでいくような作りであると、読みながら何をしたいのかが分かる。
- 14 :
- >11
コードを短くするにはアルゴリズムではなく
関数にすることが重要。
どんなに長いコードでもたった一行にできる。
しかもすごく読みやすくなる。
- 15 :
- というか設計だな
- 16 :
- >>1の条件に該当するヤツは
一行が長くならないようにコメントを適当に改行してるんだけど
単語の途中を改行でぶった切られてのぐらいしか思いつかない
あとはイラッっとするので
> 本人なりに見やすくしようとする意図は汲み取れるのだが
で該当したものはない
- 17 :
- 古来から言われていることだが、ポインタ変数と普通の変数をいっぺんに宣言するとまぎらわしい。
- 18 :
- >>17
int* ptr_A, B;
こんな感じ?
- 19 :
- K&R流では
int *pa, b;
だね
*(アスタリスク)の前に空白を入れるのか後に入れるのかという違いだけど、
>>18のスタイルの発祥はどこなんだろ?
- 20 :
- >>18
>int* ptr_A, B;
これで両方ポインタになるべきだと
俺は思うのだが
- 21 :
- クラスのプロトタイプ宣言ファイルにそのクラスが使う複数の構造体の宣言を一緒に入れる
クラスのプロトタイプ宣言のなかにそのクラスが使う構造体の宣言をごっそり入れる
どう思う?
- 22 :
- >>20
ていう思い違いをしやすいから*は識別名にくっつけろ、という話じゃね?
- 23 :
- いやそうじゃなく文法自体が不適切なんじゃないのかって話
- 24 :
- >>23
そりゃあ「コンパイラに優しい言語仕様」を目指したわけだし
てかそもそもそういうスレじゃないし
- 25 :
- if(hoge != null) {
if(hoge.length() > 0) {
// 糞処理
}
}
ネスト深くせずreturnしてほしいわ
if(hoge != null && hoge.lenght() > 0) { return; }
// 糞処理
- 26 :
- >>25
バカっ☆
- 27 :
- 定数宣言で=の位置を縦に揃えるのがあまり好きじゃない
- 28 :
- Linux環境でC++ソースの整形コマンドってなかったっけ?
- 29 :
- >27
俺は逆に、揃えてないほうが不快。
- 30 :
- 揃ってると見やすいけど修正しづらい
- 31 :
- 後で長い名前の定義を足した時に揃え直すか迷う
- 32 :
- >>27 and >>29,30
縦揃えにTabを使わないのが問題なんじゃね?
自分はタブ幅4(:se ts=4 sw=4 ai)だけど、全く苦にならない
- 33 :
- 俺はインデント以外でタブ文字使われるのが大嫌い
- 34 :
- キナ臭くなってまいりました
- 35 :
- タブサイズを8以外にしてるのがイラっとするわ。
- 36 :
- indentのデフォルトは2じゃね?
- 37 :
- =の位置で簡単に揃えられない
エディタを使う奴が無能なだけ。
- 38 :
- そもそも変数名の長さがまちまちなところに問題の根本が潜んでいる気がする
- 39 :
- だからコボラーはCOL001,COL002,COL003…みたいな変数にしてたんだろ
- 40 :
- エディタ表示部が勝手に検知して勝手に揃えて表示するというのはありなような気はする
画面表示がディスク上のファイルのバイナリ列と完全に一致しなければならないわけじゃないしな
- 41 :
- 2行以上のコメントを//で書いてあるとイラッとくる
- 42 :
- >>39
変数の長さを規約で決めてしまうのか
ソースを見やすくするっていう規約の存在意義を考えるとアリだな
- 43 :
- >>36
インデントは2でも4でも8でもいいけど、タブのサイズを4とかにしてタブでインデントしてるヤツは
イラっとするわ。
- 44 :
- >41
それは//のほうが正しいだろ。
/*〜*/で普段のコメントが書いてあると、一部の処理を殺して試したいときに、普段のコメントがそれを邪魔する。
関数の中身に書くコメントは//じゃないと邪魔だ。
JavaDoc形式については別だぞ。
- 45 :
- for(i = 0; i < max; i++); ←※
{
// 云々
}
こーゆーミスするなら中括弧の前に改行入れんじゃねえよハゲ
- 46 :
- てへぺろ;-P
- 47 :
- >>43
MS「(;゚Д゚)エッ…」
- 48 :
- >>43
なんで?
- 49 :
- >>43
なんで?
- 50 :
- >>43はRuby使いなんじゃね?
Rubyには標準ライブラリを含めてインデント2で書かれたコードが多い
- 51 :
- インデント2は明らかに少なすぎ。
デザイン的に空間が分かれてるように見えない。
- 52 :
- >>43
チカンすればいいじゃん
- 53 :
- >>52
おまわりさんこっちです
- 54 :
- 生ポインタとusingを禁止で全て書き直せとのお達し
配列長が必要なので、shared_arrayは使えない
vector<Hoge*>* hoge;
↓
boost::shared_ptr<std::vector<boost::shared_ptr<Hoge>>> hoge;
マジキチ
下手すりゃdelete漏れを探すよりもカオスなことになりそうだぜ
- 55 :
- >>47-49
タブサイズは8に決まってるだろ。
タブサイズを8以外にしてるやつは迷惑だわ。
インデントを8以外にしたいときにはスペース使え。
>>52
どっちにしても面倒だし、チカンするくらいならエディタの設定を変えたほうが速いだろ。
- 56 :
- インデントはタブサイズの設定関係ないだろ
インデント以外のレイアウトにタブ使うヤツがクソ野郎
- 57 :
- みんなそれぞれ意見がバラバラなのが面白いぜw
- 58 :
- コーディングスタイルはしばしば宗教に例えられる
- 59 :
- コーディングルール・コーディングスタイル議論は山のようにあって正直ウンザリなので、
このスレでは「イラッつとする」かどうかのみで判断した感情的なレスをお願いします
- 60 :
- >>56
関係あるだろ。
タブサイズ4前提でインデントしてて、8に設定してあるエディタで見ると崩れるやつとかいる。
- 61 :
- いまやってるphpのシステムで関数の引数を
function hoge($arg1, $arg2, $arg3)
{
$weight = arg1;
$height = arg2;
$age = arg3;
}
と必ず$arg1, $arg2…みたいな意味の無い名前の変数でうけて、関数の中で意味のある
名前の変数に移してるんだけど、
普通に
function hoge($weight, $height, $age)
でいいじゃないか。
なんか意味あるのか。
- 62 :
- コードを追うと突如現る謎の空白行
ふと右を見ると変数の頭文字らしきものがニョキっと生えてる
タブ8とか3階層ネストするだけで宇宙に行ってまうわ
気持ち悪いったらありゃしない
- 63 :
- Full HDなモニタ買ってもらえ。
- 64 :
- >>61
Perl厨のせいなんじゃね?
Perlには仮引数がないから
sub hoge {
$weight = shift;
$height = shift;
$age = shift;
}
ってやるよ。
- 65 :
- >>60
すまん、インデントの意味を間違えてた
段落を意図したインデントのみタブを使えって言いたかった
こういうこと言いたかったんです
ttp://ameblo.jp/argv/entry-10000648280.html
- 66 :
- >>65
正しい日本語使おうな。
たしかに、タブとスペース混ぜられたり、後ろに不要な空白残したりされると殴りたくなる。
てめーのことだぜ先輩!
- 67 :
- いえ、わたくしは、イライラしながらフォーマッタでポチポチ揃えてる側の人間ですが。
- 68 :
- 昔はタブサイズは8にするべきだって思ってたんだけどね
今は1か2がちょうどよく思えてきたよ
- 69 :
- タブのサイズを1に設定してタブを使うなら、ふつーにスペース使ったほうがよくね?
- 70 :
- bool hoge(){
if( fuga() != false ){ return false; }
else{ return true; }
}
なぜreturn !fuga();としないのだ……。
- 71 :
- /* 2011.3.11 なんかエラーになるのでとりあえず外す
return false;
*/
}
return true;
- 72 :
- >>70
A:論理値をリテラルと比較するような阿呆だから
- 73 :
- >>71
3.11・・・
- 74 :
- >>73
rev.666 2011-03-11 15:47
ほぼ100%職場おわるので中間コミット
SyntaxError出るけどこれ以上はやばいのでかんべんしてください
去年下請けと組んでやった案件の作業ログにこんなのあったの思い出したわ
- 75 :
- 修正したり追加した行に日付と名前が書いてあるのは
お前の名前分かっても意味無いんじゃ、って思うな。
- 76 :
- 日付は何故そう修正したか雰囲気がわかったりするから無いよりはマシ
それよりはまともなコメントを書けよハゲって話なんだが
担当者を入れるのは責任問題の押し付け合いをするためのものと理解している
- 77 :
- 日付と名前は必須だろ
どこにバグがあるか特定する時に一番役に立つ
- 78 :
- 修正履歴なんて入れてないでソース管理ツール使えよって感じだけど、ドカタの現場だとただのファイル共有ツールって認識だし使っても同じか。
- 79 :
- わざわざソース管理ツールをインストールするの面倒じゃん
おまえらすぐ管理ツールを変更しちゃうから古いソースを見る時に大変なんだよ
- 80 :
- やっぱり新しいツールや技術についていけない無能に合わせるしかないよな
- 81 :
- 名前がファミリーネームどころかファーストネームですらなく、
親しい間でなければ使わないような愛称
鼻穴に5センチほど割り箸突っ込んでグググと水平に近づけて
後遺症が残らない程度に苦痛を与えることで反省を促したい
- 82 :
- 俺もちょっと前にPHPで
return hoge ? false : true ;
って書いてた。恥ずかしい
- 83 :
- return func();
とか、気持ち悪くないか?
- 84 :
- 返却値の柔軟性を奪っておいた方が後々不具合が少ない気はしないでもない。
- 85 :
- /*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 :
- >>85
え?なんで修正した箇所のソースが残ってないの?
普通はコメントアウトして残すだろ?
コメントアウトした部分を削除する時は日付も削除するし
そんな状態にはならない
- 87 :
- >>86
>普通はコメントアウトして残すだろ?
普通は…な…。
- 88 :
- 正月から大変なんだなw
- 89 :
- >85
そもそもSubversionとか使ってないの?
- 90 :
- そこかよw
- 91 :
- 全体の設計があきらかにアレなコードで
いちいち修正をコメントで残されてもなー
- 92 :
- ありきたりだけど、コメントが疑問系のやつ
一回それに対する回答コメントがあってワラタ
- 93 :
- LINQの使い方を知って以来、foreachまみれのソースは基本的にイラつく
- 94 :
- foreach?gotoでループを表現しているコードをいじらされるよりだいぶマシだな
- 95 :
- >>14
ちゃんと意味のある関数ならいいけどな。
- 96 :
- >>20
C#はなるんじゃなかった?
- 97 :
- C#にはポインタはありません(すくなくとも表面的には)
それよりもここのタイトルの「イラッつと」って書き方にいらっと来た。
- 98 :
- >>97
あからさまに仕様上あるわけだが>C#のポインタ
そんなことより、スレタイに関しては >>5 で既出なんだが
ホントに直近のレスだけしか見てないんだなあ(´・ω・`)
- 99 :
- 何のひねりもないマジレスにイラッつとした
- 100read 1read
- 1read 100read
TOP カテ一覧 スレ一覧 2ch元 削除依頼 ▲
最強のテキストエディタってなんだ? (381)
(´ー`)男の子と女の子が仲良く会話するスレ('-'*) (228)
[ 99BASIC ] 日曜プログラマー [ DarkBASIC ] (164)
【天才マ】ひげぽん Part2【Mona】 (525)
★☆☆ アスペルガーだけどプログラマ (648)
プログラマ業界で生きていくための最低条件 (745)
--log9.info------------------
誰かこの疑問といてください2 (612)
Answer×Answer=アンアン的にクイズ作成 (628)
漢字のヒントから人物を当てるクイズ (880)
頭いいやつ頼む (145)
【突攻可】コミュケーション苦手な出題者用20質 part11 (616)
セリフから漫画(キャラ)を当てるスレ (133)
何でもいいからトリビアを勝手に作れ!! (151)
どんなクイズでも自由に出題していいスレ27 (395)
【天空の城ラピュタ】バルスの教え方 (235)
ダジャレからなぞなぞを作るスレ (283)
1+1はいくつですか (113)
一行リレー小説「アメリカ横断ウルトラクイズ」 (139)
こんなクイズ番組は賞金が100円だ! (344)
【IQテスト?】就活で企業から聞かれた問題 (702)
みんなでアタック25の予選筆記対策しようぜ! (395)
【?】スレッドを立てるまでもない質問・相談【!】 (666)
--log55.com------------------
三菱東京UFJ銀行 MUFG22
◆◆◆M&A実力ランキング!!◆◆◆
ゆうちょ銀行 貯金26冊目
【不倫】鹿児島銀行【温床】
ドイツ銀行グループについて語ろう!
■消費者金融で金借りまくって自己破産■
【糞尿】じぶん銀行★26【末期】
☆★連帯保証人制度は法律でなくせ!2★★
-