#俺が見たクソコード選手権 が面白い件について

このエントリーをはてなブックマークに追加
1番組の途中ですがアフィサイトへの転載は禁止です

個人的にはこれが一番衝撃的だった。 きっと何かトラウマがあったんだと思う。
x = y; x = y; // 不安だからもう一回 #俺が見たクソコード選手権

if (1 == 2) {...}
これだわ・・・哲学やねんなもう・・・
#俺が見たクソコード選手権

x = y;
y = x;
if( x == y && y == x ){
//どっちも一緒!
}

#俺が見たクソコード選手権

togetter
http://togetter.com/li/544905

http://realtime.search.yahoo.co.jp/search?tt=c&ei=UTF-8&fr=sfp_as&aq=-1&oq=&p=%23%E4%BF%BA%E3%81%8C%E8%A6%8B%E3%81%9F%
E3%82%AF%E3%82%BD%E3%82%B3%E3%83%BC%E3%83%89%E9%81%B8%E6%89%8B%E6%A8%A9&meta=vc%3D
2番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:33:38.58 ID:Y9JiHDei0
   / :/  ...:/:′::/ :.:.:.....:./.:/:!:.:.:.i:..!:.:.....:{:.:.:.:.:.:ハ
.  /.〃/:...../:′'.::|:: i .::.:.:.:| :i:_{__|:.|:.:.:.i :|:.:.../  ̄`ヽ
  '://:′::/斗:十 |::.::.::.:.:.:.: :}}ハ ::ハ:{:≧ト|:::/   も   |
 {//::{: /|i:八::{=从:{ i::::: :N孑弐{ミト∨:::|::′  ち   |
.  i :从 ::::{イァ:う{ミト爪ト::::. ! ん):::::ハヽト、:{:|    ゅ   |
.  |.::| : \《 { ::::::: }  ヽ\{ { ::::::::: リ | :::ヽ!    も   |
.  | ::!::|ハト.乂__ノ       ー '  | :::<    ち   .!
 八::| :|::::i /i, ,     ,     /i/ , }:::}i::人   .ゅ ノ
  (__):::l:::::.                 i.:/::::::::厂「{:::::::{
 / :{ | :V:入     〜(       }/}::::}/::::::l.|:::::::|
 { ::|人::∨::::>...   `      . ィ升|:::/::::::::八::::::{
3番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:34:25.15 ID:f5LE2J/gP
糞コードなんてない
4番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:35:07.88 ID:7fwJvFSq0
クソコードってそうじゃないよね
5番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:35:28.69 ID:dkx8rd15P
ただ間違えてるだけじゃん
6番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:36:26.39 ID:j21CKJks0
while(!0) {;};
7番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:36:34.35 ID:QJIzkZoY0
糞のレベルが低すぎて惹かれない
8番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:36:49.16 ID:lDFf3mQoi
面白くないんだが
9番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:37:19.96 ID:jPy+Rb4d0
ただのおふざけじゃん
10番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:37:23.05 ID:KsiIU5DDO BE:1982783399-PLT(18890)

面白いことやろうとすると即アフィられる世の中
11番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:38:03.00 ID:VKYgM1IC0
前にどっかでランキング形式のサイトがあったよな
12番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:38:07.58 ID:nfGJGXth0
レベル低すぎてつまらん
さすがにここまでわかりやすい糞コードじゃないけど、
俺も似たような事は書いてるかもな・・・
大学時代につくった俺の関数一個だけで10万行のゲームも今見ると相当糞だと思う
糞コードより糞コメントの方がいいんじゃねえの
16番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:38:41.06 ID:h9WH6KZQ0
そういうことじゃないだろ
17番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:38:42.08 ID:s62VJjtkP
プログラム学びたての奴ってこういうネタで自分アピールするのが大好きなんだよね(´・ω・`)
18番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:41:53.87 ID:VKYgM1IC0
ttp://unkode-mania.net/
これだった
19番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:42:10.28 ID:N8gdfnxAT
コンパイラが最適化してくれるからかまへんかまへん
if(true){
処理
}

これとかは消し忘れだと思うんだよね
ちゃんとチェックせえやって言うならその通りだけど
まぁ、ありがちなのはfor使わず、全部ifで書く馬鹿だな
100行ぐらいの奴を俺はみたことあるぞ。
22番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:43:42.05 ID:86ETixCk0
>>1のって、デバッグ的な意味で突っ込んで置いて消し忘れただけに見える
>>20
は?
>>21
10個くらいifがネストしててびっくりしたことがある
めちゃくちゃだわ
25番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:46:02.24 ID:A0DOlV8bO
あれこれ付け足してくとこんな感じになってる事はあるな
こんなあからさまなのは気付いた時点で直すけど
26番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:46:28.71 ID:WebZ495a0
>>21
そっちのほうが早いよ
バイナリで見たか?
あ読めないかゆとりには
27番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:46:28.94 ID:hwWgYLMX0
>>15
コメントで履歴管理とかな
28番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:46:55.06 ID:Ta+8lZmL0
githubでPHPで検索したらたくさん見れるよ
バイオス見ろよ
SCSIとかも疑っていけ
面白いクソコードねーのかよ
public変数とか必要になったらその場でそこに書く。
//When I wrote this, only God and I understood what I was doing
//Now, God only knows
33番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:49:05.49 ID:mq4RY0dHO
>>20
処理次第ではそのif文大事かもな
まあ素直にブロックにするかメソッド切れって話かもしれんが
34番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:49:13.46 ID:PJOm1Ymf0
//おまじない
35番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:49:22.80 ID:14AHjDOi0
void main(void) {
俺は読むのを止めた
36番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:49:36.30 ID:HnGf5F0jP
/* 気持ち遅らせる */
int i;
for (i = 0; i < 10000; i++)
;
>>26
出たな、速さバカ。可読性こそ正義。
>>26
こういう馬鹿がいるから、コードがドンドンゴミ化していくんだよな(笑)
とあるコードのテストを担当したの時のことだが

テストを開始した時点で五千行ぐらいのコードだったのが、
テストが通らない⇒不具合報告⇒修正⇒再テスト依頼、を繰り返し
テストが一割ほど終わったところまで来たら、コードは五万行ぐらいに膨れ上がっていたな
もうバグだらけでそれ以上はどうにもならん感じだった
そこで担当を離れたからその後は知らんのだけど

ちなみに俺が作れば二千行ぐらいで出来そうな簡単な案件で
プログラミング担当の人は、海の向こうの職場で勤務してるらしい会ったこともない中国人だった
こういうのってどこで学ぶの?
皆独学なん?
if(time<100){処理}
else if(time<200){処理}
else if(time<300){処理}
・・・ってtimeが2000になるまで延々と続いたコードがチーム制作で上がってきた時は思わず絶望した

これ駄目なん?
これがクソならどんな書き方すればいいんだよと
>>40
俺は情報処理系の大学で基本は覚えたけど、結局は独学だよなぁ。
43番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:53:23.48 ID:rXq3n9i+0
>>32
声出して笑った
会社で笑かすなw
てめーらデスクワーカーは糞コード作っても笑い話ですむけどな、
MCオペレーターは糞コード次第で物理的にダメージ負うんだよコラ
こういうつまらない糞まとめでアフィしてる奴氏なねえかなあ
47番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:56:38.61 ID:HnGf5F0jP
>>45
おう深すぎてエンドミルはじけ飛んだわ
>>47
何故初期Z位置を100.以外にしてプログラムを出力するのか
49番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 20:59:27.29 ID:ctUl1N0c0
>>36
味わい深い
50番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:01:36.36 ID:aqldO5NLP
クソコードどころか初めてプログラミング言語に触れた中学生レベルじゃねえか
今は動いてるけど、数年後には動かないだろうコードを書く事もある。
仕様書や設計書なんて当てにならないから、
多少遅くても「気持ちが伝わるように」って意識しながら書いてるわ。
今後、何処の誰が触るか分からんからね。
52番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:03:02.50 ID:lq5/MrSgP BE:222507252-PLT(12001)

大量のifとprintが並んでたら見るの嫌になるな
/**
* 常に returns true
*/
public boolean isAvailable() {
return false;
}

これに近い事はまれに良くある
54番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:03:50.21 ID:VtGkaHeZ0
インデントがこのタイプじゃないと何となく気持ち悪い
http://minnie.tuhs.org/cgi-bin/utree.pl?file=2.11BSD/src/bin/su.c
55番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:04:02.29 ID:PJOm1Ymf0
>>40
嫌儲で教えてもらった本。あとやっぱり英語だといろいろ出てるね
リーダブルコード
http://www.oreilly.co.jp/books/9784873115658/
56番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:04:05.00 ID:mq4RY0dHO
>>49
組み込みならたまにあった
最近は知らんが
57番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:04:49.53 ID:spF2gErI0
これはネタコードであって糞コードじゃない
本物の糞コードは笑えない
58番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:05:30.45 ID:teRgMDsL0
求めてた処理できてるのにクソなコードを上げるべきだろ
なんだよ>>1のは。無理して作っただろ所詮ツイッター
59呉はじめ ◆KUREKO1w06 :2013/08/06(火) 21:06:16.33 ID:na/JKyUo0
> if (1 == 2) {...}



もっとマシな方法はあるだろうけど、
デバッグ用のコードなのでは?
動作確認の時には1=1にするという。
60番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:06:25.85 ID:yBP0aPZUO
他人のコード見る機会がないからヤバいコードがどんなのか想像出来ん
61番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:07:20.35 ID:FrWtg7N70
getが5段階くらい深い階層のは殺したくなる
>>49
Cのファームww
> x = y; x = y; // 不安だからもう一回 #俺が見たクソコード選手権

これって割り込みが起きることを恐れてるのかね?
64番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:09:38.36 ID:4i/vLr6yP
書いた人にしか何が書いてあるのかわからん
書いた人もしばらくすると何を書いたのかわからん
ってのが糞コード
コンパイラの最適化で消え去るようなコードは糞とは言わん。ただのゴミだ。
66番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:10:27.17 ID:wphxncZ00
>>58
だよな
67番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:10:31.06 ID:pzxnOgEuT
for(i=0; ; i++){

}
68番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:10:56.70 ID:QyueVf6X0
最近のリソース考えないゆとった奴らにしたら昔のソースなんか全部糞コードなんだろうな
69番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:11:39.76 ID:NrY3MlgZ0
初心者の頃に書いたのはこういうコードだわ

$a = 5;
$b = 0;
if ($a == 0){
$b = false;
}else{
if ($a == 1){
$b = true;
}else{
if ($a == 2){
$b = false;
}else{
if ($a == 3){
$b = false;
}else{
if ($a == 4){
$b = true;
}else{
if ($a == 5){
$b = true;
}else{
$b = false;
}
}
}
}
}
}
>>55
いい本教えてくれてありがとう。速攻でポチったわ
71番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:12:30.07 ID:LOcypAO50 BE:563630933-2BP(2172)

static int IWIN = 0;

if( IWIN ) {
marryHer();
}
int a = 10;
int b = 20;

a = b;

aとbの値は何か これで「分からない」と言える人のほうが良いらしい
73番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:13:41.51 ID:3uPjxSz30
while (true)
74番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:15:03.22 ID:mq4RY0dHO
>>71
IWINにどっかで値代入してんじゃないの?
if (1 == 2) { ... } というコードになる状況を考えてみた

もとは、if (honyarara == 2) { .. } という当たり前のコードだったのだが
バージャンアップで仕様が変わり、honyararaの値は常に1のままで変化しない仕様となった

ところがテストをしてみると、変数honyararaの値を1に初期化する処理をする前に
if (honyarara == 2) { .. } というコードを実行することがあることが分かってパニックに
うわこれ面倒くせーってことで、if (1 == 2) { ... } になった

if文ごと消すのは、else節がぶら下がっていて
if (1 == 2) { ... } else if (...) { ... } else { ... } こんな風になっているので却下された
76番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:15:26.11 ID:eXfE5GqT0
go to 使いまくるアホは死ね
77番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:15:46.77 ID:e96Pu05c0
動いてるけど糞コード書けよ
78番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:16:01.49 ID:XAei6nmT0
>>69
インデント加えるとおっぱいに見える素晴らしいコードじゃないか
int *p = NULL;
*p = 0;
80番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:17:15.29 ID:XAei6nmT0
いつだったかスイーツ(笑)が流行ってた頃に見た
"スイーツ(笑)は関数"ってスレタイは笑えた
81番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:17:27.39 ID:LoXEyDF80
まず変数名が失格
創作するなら細部まで気を使わないと
82番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:17:48.12 ID:LnSmZmZ80
#define PI 3 // ゆとり
83番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:18:08.38 ID:6XehgRvb0
phpがわからないってコメントと一緒にphpの部分のソースが全部コメントアウトされてて
ぱっと見似たような動きをする様にhtmlが書かれていた時は凄い絶望感だった
if (i=1){処理A}
if (i=2){処理B}
if (i=3){処理A}
if (i=4){処理B}
・・・みたいなコードなら見たことある
>>32
わろた
86番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:20:50.56 ID:qntKPSYN0
>>84
変にまとめるよりそっちのほうがぱっと見分かりやすい
87番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:21:44.55 ID:/0WaTYfq0
見たんじゃなくてボクの考えた最悪にクソなコードだろ
アフィカス死ねや
88番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:22:08.75 ID:mq4RY0dHO
>>84
処理Aと処理Bをiインクリメントしながら繰り返したかったんかな?
>>86
コードは効率よりも見通しの良さとメンテナンス性だもんな
>>86
分かりやすいわけねーだろ
パッと見だと1行目と2行目の処理Aは違う事をやっていると判断するだろうし、
じっくり見比べないと同じ事をやっていることが分からない
昔は配列だのループだのは綺麗にまとめないと糞コード扱いだったけど
今シンプルに列挙してあるコードは保守性最強だからな
限度はあるが
92番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:23:33.34 ID:DjZKsESP0
1==1
93番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:24:07.25 ID:ASVUIuFR0
>>84
Cなら全部実行されるな
94番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:24:21.46 ID:RxSguyi1P
全くわからん
C言語とか考えた奴マジキチだわ
よくよく考えりゃパソコンの中身が
どうして作れるのか、
微細すぎる集積回路とかマジキチレベルやわ
95番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:25:47.77 ID:jOKzYPwV0
コードではないが「ええっ!Cでnew出来ないの!?」という発言は聞いたことある
>>86
効率悪いだろ
コンパイラ次第では全て判定されるんだぞ
>>86
1行目と3行目の処理Aの間違い。

あと処理Aの内容を変えたいときに1行目だけ変えて
3行目の方を変え忘れるという自体が簡単に予想できる。

>>89
見通しも悪いしメンテナンス性も悪いわアホ
98番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:27:10.60 ID:QyueVf6X0
>>94
どうやったら論理回路での四則演算出来るのか常々気になってた

そこら辺解説してるサイトて無いかな
99番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:27:29.19 ID:mq4RY0dHO
>>96
判定されなきゃまずいだろ
代入してんじゃん
100番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:27:40.95 ID:qntKPSYN0
>>90
それは処理Aと処理Bの量による
少ない場合はこの書き方もアリかなと思うけど
グローバル変数使いまくって膨大な関数群から引数を撤廃した奴
そんなにスタックの消費を減らしたいのか
コード読みにくくてしかたねーぞ

あと9000行ある関数書いた奴は腹切って死ね
>>98
1+1=2を延々と繰り返して計算するんだよ
いつかは目的の数に辿り着くだろ
>>100
>>97にも書いたが少なくてもナシ。
バグの原因になるからこんな書き方は絶対ダメだ。
104番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:29:49.25 ID:NUnhG1jr0
こういうの言う奴に限ってIOマップドメモリとかわかってない
105番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:30:02.37 ID:4i/vLr6yP
>>84だけ見ると
処理Aや処理Bには特定の場合のみiをインクリメントする処理が含まれているのではないかと
プロフェッショナルな俺は思っちゃうね
106番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:30:14.02 ID:dSAiFjmYP
{
ぼにゃららららら
}


{ぼにゃららららら}

プロとしてはどっちがええのん
>>97
処理AとかBの部分をダラダラ書いてると思ってるのか
おれはてっきり関数化した部分だと思ってた
>>104
は?
109番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:31:59.26 ID:mq4RY0dHO
>>105
処理A処理Bでインクリメントしても代入されて意味なくね
110番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:32:18.53 ID:ASVUIuFR0
int m_flag (コメントなし)
の絶望感は異常
>>88
だいぶ昔だから忘れたが整数をiに代入して偶数奇数で違う処理をやるコードだったはず
112番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:34:37.59 ID:4i/vLr6yP
>>109
>>111
なるほど、わかりにくいわw
113番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:37:46.59 ID:SIwk0NXU0
>>1
何も抽出したく無い時のクエリ文にwhere 1=0って書いてるんだがセーフ?
お前らこのソースコードどう思うよ

多分日本で一番有名なプログラマが書いたもので、
20年ぐらいメンテされているぞ
https://github.com/ruby/ruby/blob/trunk/parse.y
115番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:40:34.62 ID:14AHjDOi0
>>110
public class foo {
int m_flag;
//…<<数百行のコード>>
//…<<多くの個所で参照・代入>>
}
}
>>114
ガキにそこら辺のツールで解析されて
最適化されてドヤ顔されるアホ言語の作者がどうしたって?
117番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:41:18.00 ID:SIwk0NXU0
>>106
前者

行数が増えて仕事した気になる
{}のみをコメントアウトする事ができる(差分確認が楽)
118番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:42:02.15 ID:4P6pN5hBT
>>101
手動最適化された昔のコードにそういうのがあったわ
今でもそのまま使われてるけど
119番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:43:16.43 ID:9cNiUxg60
>>113
何も抽出したくないのにクエリを発行するってのは妥当か?
120番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:44:16.45 ID:ZjRz+Clx0
コメントに死ねよ●●!とか書いてあったのをみてふいたことある
if (1 == 2) {...}

これはありうるだろ。コメントしたい時にたまに使う
>>114
rubyって何で人気あるの?
ザ・ガラパゴス言語じゃん……

今の人気だって半分くらいはrailsのおかげでしょ
>>123
人気あるの?
俺は好きだけど
125番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:48:52.11 ID:SIwk0NXU0
>>119
仕様上何かしら設定しておかないといけなくて全件出力されるから仕方ないんだ
よそから買ってきた製品だからどうにもならん
126番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:51:01.20 ID:BT9FU67o0
int* test(void){
int a=10;
return(&a);
}
127番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:52:16.86 ID:9A78+mup0
String myStr
128番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:52:18.11 ID:N80bTlgd0
>>75
あるわ
//**月**日のサビ残180分
130番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:53:19.26 ID:Z0UkUu9e0
レス読む限りこの板にも糞コーダーがいるようだな
嫌儲コードレビュー部の創設を提案する
131番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 21:57:20.05 ID:SgqZQOn+0
isError() ってメソッドの戻り値が、正常の場合に true を返すとか、
AbstractDao ってクラスが、new AbstractDao() って直接インスタンス生成してから使うことを想定しているとか
毎行コメント書く奴ならいた
注意しても治らないから定期的に消してたわ
133番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:01:13.02 ID:4P6pN5hBT
>>36
気の利いたコードだと、変数にvolatileがついてたりする
VBAしかわからんのに他人が作ったプログラムの仕様変更頼まれた時の傑作があるわ
C#なんてわからんくて適当に変更したらエラー吐き出されるようになって
回りこんでエラー吐かないようにして最後にそこの帳尻合わせるっていうパワープレイでなんとかしたよ
それでなんとかなってるのが奇跡
135番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:04:27.58 ID:BT9FU67o0
割り込みステータスレジスタをポーリングで監視する糞コード
死んでどうぞ
>>134
// Magic. Do not touch.
137アフィサイト転載禁止 ◆EJb4S7IrTY :2013/08/06(火) 22:07:38.88 ID:SZrO2exq0
どんな馬鹿でも理解できるソースこそが最上

コミュニケーションツールなのだから
138番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:07:41.98 ID:nfGJGXth0
>>36
良いなぁ
ほのぼのするw
x = y;
x = y; // 不安だからもう一回
>>95
俺が聞かれたら、できる、って答えるなw
>>21
古いシェーダーコードならありだな。
142番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:16:04.59 ID:dSAiFjmYP
>>117
子供のころ大嫌いな作文を句読点と改行で文字数水増しみたいだな
> x = y; x = y; // 不安だからもう一回
何らかのマルチスレッドプログラムにおけるレジスター割り当てのテストだろう

> if (1 == 2) {...}
テスト時のみ動作する処理を本番系でスキップする為に容易な方法

> x = y;
> y = x;
> if( x == y y == x ){
> //どっちも一緒!
> }

これがどっちも一緒に見えるおまえの目は節穴か?
if( (x == y y ) == x ){
ってことだ
144番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:23:21.01 ID:y7wAA5/rP
>>143
なんで&&抜いて動作しないコードにしてるの?
145番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:31:43.80 ID:NJzdVWt20!
int を使うな short と long を使え!
146番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:32:59.04 ID:E5nDTyzN0
>>98
実行部分はこれとか?

大きな組み合わせ回路としてのALU
http://www.ie.u-ryukyu.ac.jp/~wada/digcir10/alu.html
加算器は別に調べれば論理回路出来るはず

命令の解読はモロに論理回路だし
147番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:38:30.49 ID:dSAiFjmYP
オブジェクト指向ってgdgd長文ならべて結局よくわからんが結論になる
148番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:42:40.85 ID:SIwk0NXU0
>>36
組み込みで何かしらwaitしたいがディスパッチはさせたくないジレンマから生まれたんだろきっと
149番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:46:46.61 ID:itJRXCO9P
とりあえずperlは消滅しろ
高級言語は糞コードでも動くから良いんじゃないか。
151番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:48:13.47 ID:gh+/f0AO0
#include <stdio.h>
#include <malloc.h>
main(togo,toog)
int togo;
char *toog[];
{char *ogto, tgoo[80];FILE *ogot; int oogt=0, ootg, otog=79,
ottg=1;if ( togo== ottg) goto gogo; goto goog; ggot:
if ( fgets( tgoo, otog, ogot)) goto gtgo; goto gott;
gtot: exit(); ogtg: ++oogt; goto ogoo; togg: if ( ootg > 0)
goto oggt; goto ggot; ogog: if ( !ogot) goto gogo;
goto ggto; gtto: printf( "%d goto \'s\n", oogt); goto
gtot; oggt: if ( !memcmp( ogto, "goto", 4)) goto otgg;
goto gooo; gogo: exit( ottg); tggo: ootg= strlen(tgoo);
goto tgog; oogo: --ootg; goto togg; gooo: ++ogto; goto
oogo; gott: fclose( ogot); goto gtto; otgg: ogto= ogto +3;
goto ogtg; tgog: ootg-=4;goto togg; gtgo: ogto= tgoo;
goto tggo; ogoo: ootg-=3;goto gooo; goog: ogot= fopen(
toog[ ottg], "r"); goto ogog; ggto: ogto= tgoo; goto
ggot;}
152番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:48:38.57 ID:OFP71Tee0
perlはほんと読めないねぇ・・・
153番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 22:49:07.69 ID:SIwk0NXU0
>>145
数バイトのメモリと処理速度どっちが大切かよく考えろ
154番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 23:11:13.06 ID:NJzdVWt20!
>>153
いや、昔の話だが int は PC と unix でバイト数が違うから移植が面倒だったから・・
今は引退したからどうなってるか知らんが
>>154
intどころかshortもlongもサイズが処理系依存バリバリの大混沌時代ですが。
156番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 23:29:35.67 ID:7KuWStsj0
// 宣言しといてなんなんですけど、デバイスコンテキストってなんですか
157番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 23:38:05.72 ID:U2HBKObs0
お前ら何の暗号で会話してんだよ
N88-BASICでやってくれよ
158番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 23:43:36.12 ID:SIwk0NXU0
>>154
それが目的ならuint16とか使え
159番組の途中ですがアフィサイトへの転載は禁止です:2013/08/06(火) 23:51:07.57 ID:Sxh6Ul7P0
void main ( ) {

3000行位

}
()とか;とか=の前後に半角スペース入れる場合と入れない場合があるみたいだけど
スペース入れるか入れないか全部覚えてるの?
うっかり入れ忘れとかしないの?
全然糞じゃないし
最近objective-c始めたんだけど何アレ
ちゃんと書いてもクソと化すんだけど
163番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:07:28.72 ID:N80bTlgd0
関数名に日本語使う奴
164番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:17:51.76 ID:fEBQBwq10
二つ目の恒偽条件は
ifが実装されてるのに複数行コメントがないマイナー言語で
コメントアウトに使える
結構有用なんだぜ

メジャーな言語でやるのは阿呆だが
165番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:21:25.44 ID:xNfI9yCJ0
クッソつまらん創作全開のコード挙げて自分はマトモですアピールしてるだけかよ
166番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:24:49.43 ID:sSRTseiM0
if(A){}
else if(B){}
else if(C){}
else if(D){}
これが数行続くときの綺麗な書き方が知りたい
どう頑張ってもごちゃごちゃする
167番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:26:26.60 ID:kzFdfi2Q0
何が面白いかわからん
ガンダムで例えろ
168番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:31:17.21 ID:V6xtqrKM0
>>166
大量の分岐にしないようにすりゃいい
>>167
キラじゃなくても書き直したくなるコード
>>166
a?b=c:d:
とか使えるなら使うといい。
三項演算子ってコードがシンプルになるけど可読性に欠ける印象
172番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:34:12.89 ID:4Pdxx0TN0
>>160
今時のIDEなら全部自動でやってくれる

>>166
ポリモーフィズムとか使って違う場所に散りじりにしちゃえばいい
一箇所にまとめて書くからきたない気がするだけ
173番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:35:14.46 ID:D7WUKHWqO
>>166
その分岐が一回ならそのまま
二回以上あるなら言語にもよるけどStrategyパターン使えるか考える
174番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:39:36.10 ID:eJ34enNI0
#+++++++++++
#危険 触るな


・・・
#-----------
175番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:43:33.76 ID:m3Xw62vk0
>>162
単に慣れの問題だ。
objective-cそのものより、x-codeで画面作るやり方が
最初わけわからなかったが、1か月もすれば慣れた。
176番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:43:40.10 ID:zEe5Kdc+O
単に後から書き換える用だろ
c#使いでこんどpythonを触るのですがやっといた方がいいことあれば教えてください。
178番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:46:23.33 ID:zEe5Kdc+O
>>157
10 X=Y
20 X=Y
179番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 00:47:57.46 ID:m3Xw62vk0
>>177 「型」っていう概念を頭から追い出す。
>>41
switch caseかな、遅くなるが
181番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 01:00:04.07 ID:gPnRH280P
変なとこでインデントしてるやつwww
>>179
型がないの?
183番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 01:04:53.86 ID:FC+tee010
#define private public
#include "○○○.h"


おい、馬鹿やめろ
184番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 01:10:00.99 ID:ItN26rx20
>>121
お前とは仕事したくないわ

つかperlも読めない奴とかとも仕事したくないわ

つか仕事したくないわ
PHP死ね
if( 条件 ) {
}
ってカッコのつけかたやめてくれ。条件だけをコメントアウトして
書きなおしたりすんのがメンドい

if( 条件 )
{
}
こっちだと、
// if( 条件 )
if( 条件2 )
{
}
ってすぐに書きなおせて楽チン
186番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 01:21:32.58 ID:D7WUKHWqO
>>185
コメントで残すな、消せ
187番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 01:22:27.10 ID:/R/WkLuy0
言い訳がましいコメント…ファックだね
真のプログラマはコメントなんて書かない
188番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 01:27:37.15 ID:eJ34enNI0
>>187
このお盆はお前のケツ拭いて回ってる奴の供養をしとけよ
>>185
有料の中ではsublimetext2よいよね
1kステップ使ってフラグ立ててるのに、よく見たらフラグ使ってないとか、
100ステップくらいの処理をコピペしまくって、3スクロールしたら既視感あるコード出てくるとかが精神的にくる
>>187
お前の仕事の引き継ぎはお断りだ
192番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 02:16:46.66 ID:HNB40w7P0
そういえば一時期、コメントなんて書かない方がいい
みたいな主張をする一派がいたっけ
>>166
switch文使えばいんじゃね?
else if(B){} の状態になった時を整数で表せるようにしとけばすっきりするんじゃね?
194番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 02:31:06.65 ID:4HZHoW1H0
>>40
IT勤めてても誰も教えてくれんしなあ
独学です
アドバイスくれるのはレビューの時ぐらいかなあ
195番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 02:36:54.00 ID:to/4xOcp0
// 暫定対応 // 暫定なら後で直せ // は?まだ暫定中だけど?

コメントで煽り合うなカス
swich使うと遅くなるってのはどっから出てきた話なんだ
よくわからん
組み込みでもないかぎり、文法レベルでの速さとか
メモリとか気にする時代はとっくに終わってるだろ
アルゴリズムで1000倍違うとかならともかく
気にしてたら今時の言語使うと発狂するぞ
>>192
丁寧にコメントつけるつもりか知らないが
コードで書いてることを、一行一行そのまま日本語で
書く奴は馬鹿だと思う

無駄だし読みにくいし保守性落とす
199番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 03:29:11.89 ID:FMxWT1QU0
ウンコードというか・・・
バージョン管理がされてなくて、ソースコードが入ったフォルダを日付入りの名前にして管理(´・ω・`)
それプラス・・・
バージョン管理システムに入ってないから、コードの修正はコメントで残すルール・・(´・ω・`)

// 2013.8.6 権兵衛さん修正ここから
//// 2013.7.1 名無しさん 修正ここから
//// ウンコード;
//// ウンコード;
//アホコード;
//アホコード;
//// 2013.7.1 修正ここまで
池沼コード;
池沼コード;
// 2013.8.6 権兵衛さん修正ここまで
200番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 03:31:16.02 ID:FMxWT1QU0
http://unkode-mania.net
ウンコードマニアというサイトがあったお(´・ω・`)
201番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 03:39:56.63 ID:xYKAHzhv0
>>196
今の時代速度なんて二の次だからへーきへーき
>>196
逆、switchの方が2倍近い速度
http://d.hatena.ne.jp/YasuyukiMiura/20080524/1211612944

>>201
上のURLのように速度が2倍近く違うから、
ループで常に回り続けてる処理、例えばアクションゲームなんかだと雲泥の差が出るよ
203番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 04:30:07.29 ID:mGw2czZu0
単なるバグだろってのをクソコードとして挙げる奴wwwwwwwww
>>202
今Android端末で検証してみたんだが、
確かにswitch文のが速くなることが多いな
条件分岐2つまでならif文のが絶対速いと思ってたんだが、そうでもなかった
条件分岐2つでもswitchのが速いことがある
少なくともAndroidのJavaでは出来るだけswitch文使った方が良さそうだわ
205番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 04:36:09.07 ID:G6A7e7pO0
プログラマって大半がバカなのに
頭の出来がもろに出る仕事だから大変だよね

だからお前らみたいに性格歪んだ人間が量産される
206番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 04:41:50.02 ID:mGw2czZu0
switchは、比較する値が連続してる場合はコンパイラのオプティマイザがテーブル化するので速くなる。
連続せず乖離した値を比較している場合はifと変わらなくなる可能性も高いんじゃないかな。
switchだから速くなるってより、コンパイラの性能次第って所だね。
>>167
テムレイパーツのプログラムが蓋を開けてみればクソコードだらけで泣きながら捨てて去るアムロ
//unko

これいれたら問題なく動いて消すと動かないみたいなやつあったよな
209番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 04:59:46.68 ID:jcydbx320
>>26
よう老害
>>196
case100個くらい並べるならステートマシンか関数ポインタテーブルか使えって話ならわかる。
211番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 07:51:31.32 ID:4Pdxx0TN0
>>208
どっかでメモリ壊してるだけだろ
212番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 07:52:17.82 ID:N2z4JnUe0
>>196
Cじゃ昔からジャンプテーブル化されないと遅いって散々言われてたけど
>>35
なんで?
たまにやるけど。
214番組の途中ですがアフィサイトへの転載は禁止です:2013/08/07(水) 09:20:58.28 ID:Ud0cBmkP0
>>20
プリプロセッサがないJavaで考案されたバッドノウハウ
if (true) {}とか if (false) {} は、Javaならありだよな。
x = y; x = y; // 不安だからもう一回
これでxとyがauto_ptrだと泣ける
>>155
C/C++ならいまでもサイズの規定はほとんど無いも同然です。
プログラムの知識は全く無いけど
ifで場合分けしてひたすら「何々の値が○○だったら××して、そうじゃなかったら一歩戻ってやり直し」みたいなことを
書き込んでいくんだよね?

フォトショップとか動画編集ソフトでほぼ自動的に人物だけ消して背景を補完する魔法のような処理も
全部が「何々の値が○○だったら××して、そうじゃなかったら一歩戻ってやり直し」の連鎖の結果だと思うと気が狂いそうになる