noiz9 2012-08-01 07:48:10

[C] すげーちゃんと動く! このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

結合試験2日前に脱退したうんkプログラマーの後を引き継いで、テストしてたらどんなテストしても全部正常終了する!!
けど良く見たら、そいつが作った全ての自前関数のreturn値がががががががが

return 0;

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(16)

#1 imunolion 2012-08-14 05:19:25  

むしろこれが全て通るテストを書いたやつ出てこいw

#2 ncaq 2012-08-14 06:59:11  

テストコードの方に問題ありだろ…

#3 kurt1205 2012-08-14 13:44:22  

こんなコード書くアホがいるはず・・・と思ったら友達がJavaで catch (Exception e) { return True; } で書いてるアホに遭遇したとか言ってたのを思い出した

#4 math_neko 2012-08-14 18:21:40  

全部 0 しか返ってこないのかwこれでテストが通る方がむしろおかしいwww

#5 _vapor_ 2012-08-14 23:58:15  

リアルにこのコードを実装した話を聞いたことがある。

#6 miso_nya 2012-08-15 01:32:38  

なんでこれが通るんだよwww

#7 imai141421356 2012-08-15 06:16:29  

まずテストのアサーションちゃんと作れwwwwwwwwwwwwwwwwwww

#8 2rxJZ4a 2012-08-15 18:59:28  

結合テストの前にコードレビューとか単体テストで潰せよ・・・

#9 splittunnel 2012-08-16 01:42:12  

正常系のテストしかしてないのかよ・・・

#10 okadabasso 2012-11-23 14:19:08  

常に0を返す、と言う仕様 でない限りテストで検出できると思うんですけど。

#11 DameotokoN 2012-11-25 12:57:11  

@okadabasso だから「まずテストのアサーションちゃんと作れ」というわけで。このテストコードも殿堂入りウンコードにすべき。

#12 mikecat_mixc 2013-08-24 02:05:29  

正常終了しても、出力した答えが正しいとは限らない。といっても、アクセス違反などのランタイムエラーにならないのは良いことだ。

値の受け渡しも、引数に渡したポインタで処理していると考えられる。ただ、だったら戻り値をvoidにするべきかもしれないが。

もちろんちゃんと出力を正解と比較してジャッジしているんですよね?

#13 PG_nonen 2014-01-10 09:42:36  

いや、これ多分どの関数も「戻り値0なら正常終了」みたいな感じで関数作っててそれを戻り値でテストしてるみたいなそんなじゃ(酷い)

#14 yanorei32 2014-08-06 10:27:03  

「戻り値0なら正常終了」か「常に0を返し続けるというプログラム」ではない限り・・・

#15 HighSpecOjisan 2017-08-15 04:38:15  

なぜ通ったwwww

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[Java] 専用機

このエントリーをはてなブックマークに追加

山田さん(仮名)専用機らしいことがクラス...

/**
 * My Army Knife
 * 20XX/XX/XX
 *...

鑑賞する »

[C] ×有限ループ ◯無限ループ

このエントリーをはてなブックマークに追加

有限ループのはずが、無限ループに... ...

int i,j;
for(i=0;i<100;i++) {
 for(j=0...

鑑賞する »

[JavaScript] とっても長いswitch/case文

このエントリーをはてなブックマークに追加

配列変数を全く理解できていなかった。

var rnd = Math.floor(Math.random() * 128...

鑑賞する »