ぱっと見て「ヘタだなぁ」と思うコード その6
■ このスレッドは過去ログ倉庫に格納されています
0172デフォルトの名無しさん
2007/10/27(土) 02:34:05レベル低すぎてどうしようもない。ちげーだろ。
if( NULL == hoge_func(hoge_arg1, hoge_arg2, hoge_arg3, hoge_arg4, hoge_arg5, hoge_arg6, hoge_arg7, hoge_arg8) )
{
}
if( hoge_func(hoge_arg1, hoge_arg2, hoge_arg3, hoge_arg4, hoge_arg5, hoge_arg6, hoge_arg7, hoge_arg8) == NULL )
{
}
前者と後者、どっちが読みやすいかって話だ。
そもそも関数の引数が多すぎる、とか言われても困るぞ。
Win32APIとかだと、引数8個程度のなんてゴロゴロあるからな。
あと、後者だと、環境によっては==NULLがエディタの外に出てしまって、見えないことがあることも付け加えておく。
■ このスレッドは過去ログ倉庫に格納されています