nobb_hero 2013-09-18 07:53:15

[C++] 一社相伝の伝統を受け継いだ秘伝のソース このエントリーをはてなブックマークに追加

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

Setterの役割なのにIsPrintCan・・・
全く役割が違うのにオーバーロード・・・
挙句の果てに、コメントが[in] or [out]しか違いがない・・・
この関数を呼んでいる箇所にも もちろん コメントなんてありません!

//印刷設定可[in]
void IsPrintCan(BOOL bPrintCan)
{
    //メンバ変数へ代入
};

//印刷設定可[out]
bool IsPrintCan()
{
    //メンバ変数から判定してbool値を返す
};

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

コメント(5)

#1 tkcomcom1is 2013-10-02 14:49:31  

完全にネタですが、8行目が

void IsPrintCan(bool* bPrintCan)

だったら、もう発狂モノだなとふと思ってしまった……。多分絶対しないだろうけど……。

#2 aokomoriuta 2015-05-10 06:14:39  

いやこれは普通にありでしょう。プロパティですし。 これは「IsPrincCanって名前がなんやねん」ってとこがツッコミどころじゃないんですかね。

#3 shanxia1218 2015-05-23 13:07:15  

↑機能として用意するのはありだけど、IsPrintCan()なんて名前でメンバ変数更新されたらたまらんよ。

#4 9YdxOr 2015-06-03 06:45:11  

bool ≠ BOOL は気になる。bool * はあり得る:

void IsPrintCan([out, retval] bool * bPrintCan)
#5 utubyou009 2018-08-07 13:25:42  

in が BOOL で out が bool? まあ、せめて、

bool IsPrintCan() const{...}

ぐらいはしましょう。

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

Twitter認証

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

[その他] HOW TO ABC..

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

実際にあった某システムの超重要なマスター...

create table item_master (
 A varchar(2...

鑑賞する »

[Java] is禁止令

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

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

// Mod yamada Start

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

鑑賞する »

[C] 恐怖のスパゲッティ

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

Pastebin.com で見つけた恐怖...

#include <stdio.h>

int main() {
    ...

鑑賞する »