noiz9 noiz9 2012-08-01 16:48:10

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

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

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

return 0;

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

コメント(14)

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

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

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

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

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

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

#4 math_neko math_neko 2012-08-15 03:21:40  

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

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

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

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

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

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

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

#8 2rxJZ4a 2rxJZ4a 2012-08-16 03:59:28  

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Twitter認証

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

[Java] is禁止令

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

ウンコードの趣旨とは違い、レビューで指摘...

// Mod yamada Start

// 一般的に考えて真偽値を返すメ...

鑑賞する »

[Java] 連番

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

もはや人間が読むものではない。

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[Java] 専用機

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

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

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

鑑賞する »