純粋仮想関数=参照されていないローカル変数の警告

Question

今後の読者のために結論

関数内で抽象クラスは、（少なくともVS2013アップデート3で、それはバグかもしれません）良いアイデアではありません。詳細は編集3とLeonの答えを参照してください。

オリジナルポスト

私はこの警告をやったことがなかったと私は何が起こっているのか見当がつかない。私は警告をエラーとして扱うように私のプロジェクトを設定しました。

class BaseFoo 
{ 
public: 
    virtual ~BaseFoo() = default; 

    virtual void doIt(const std::string& x, const uint8 y) = 0; 
}; // class BaseFoo 

class Foo : public BaseFoo 
{ 
    void doIt(const std::string& x, const uint8 y) override 
    { 
    } 
}; // class Foo 

class Executer 
{ 
public: 
    void doIt(BaseFoo* f) 
    { 
     f->doIt("hello", 0); 
     f->doIt("world", 1); 
    } 
}; // class Executer 

Executer exec; 
BaseFoo* f = new Foo(); 
exec.doIt(f); 
delete f; 

MSVC（VS2013アップデート3）は言う：C4101警告

： 'をBaseFoo ::のdoIt'：参照されていないローカル変数

がどのように純粋仮想関数は、「ローカル参照されていないことができます変数 "？私はジャガイモでなければならないか、コンパイラのバグを見つけましたか？

編集：私はFoo::doIt機能（例えばSTD :: coutの）にコードを配置する場合

、何も変わりません。

BaseFoo::doItを非純粋な「単純な」仮想関数に変更すると、何も行われず警告が消えます。

編集2：コンパイル可能な単一のファイル

だから私は、コンパイル可能主な機能（私が知っている、ARGV、argcの、MEH ... :)）に "コピー＆ペースト" のコードをしました。レベル3の警告

• ：：私はまた、プロジェクトの設定をコピーしたエラーなど/ W3
• トリート警告：はい/ WX
• 最適化：有効/ Gmの
：/ OD
• 最小限の再構築を無効
• RTTI：いいえ/ GR-

そして、ここではコードです：

#include <iostream> 
#include <string> 

typedef unsigned __int8 uint8; 

int main() 
{ 
    class BaseFoo 
    { 
    public: 
     virtual ~BaseFoo() = default; 

     virtual void doIt(const std::string& x, const uint8 y) = 0; 
    }; // class BaseFoo 

    class Foo : public BaseFoo 
    { 
     void doIt(const std::string& x, const uint8 y) override 
     { 
      std::cout << x << y; 
     } 
    }; // class Foo 

    class Executer 
    { 
    public: 
     void doIt(BaseFoo* f) 
     { 
      f->doIt("hello", 0); 
      f->doIt("world", 1); 
     } 
    }; // class Executer 

    Executer exec; 
    BaseFoo* f = new Foo(); 
    exec.doIt(f); 
    delete f; 

    return 0; 
} 

編集3：

ポイントは、私は、main関数からクラスを移動した場合、警告は消え、レオンのために行きます。しかし、主な機能の中でこれらのクラスを定義するとどうして私が間違っているのか分かりません。だから、次のコードは警告なしにコンパイルされます。

#include <iostream> 
#include <string> 

typedef unsigned __int8 uint8; 

class BaseFoo 
{ 
public: 
    virtual ~BaseFoo() = default; 

    virtual void doIt(const std::string& x, const uint8 y) = 0; 
}; // class BaseFoo 

class Foo : public BaseFoo 
{ 
    void doIt(const std::string& x, const uint8 y) override 
    { 
     std::cout << x << y; 
    } 
}; // class Foo 

class Executer 
{ 
public: 
    void doIt(BaseFoo* f) 
    { 
     f->doIt("hello", 0); 
     f->doIt("world", 1); 
    } 
}; // class Executer 

int main() 
{ 
    Executer exec; 
    BaseFoo* f = new Foo(); 
    exec.doIt(f); 
    delete f; 

    return 0; 
} 

Answer 1

私は問題はあなたが誰もがBaseFooを継承し、そのdoIt()メソッドをオーバーライドするためにはチャンスを残さない、関数にローカルとしてあなたのクラスを宣言することだと思います。

Answer 2

はこれを試してみてください：

class BaseFoo 
{ 
public: 
    virtual ~BaseFoo() = default; 

    // As a pure-virtual is intrinsically a "do-nothing" 
    // the name of the parameters doesn't matter for the compiler 
    virtual void doIt(const std::string& /* x */, const uint8 /* y */) = 0; 
}; // class BaseFoo 

class Foo : public BaseFoo 
{ 
    // Other compilers are even Nazi-er than VS and emit a whinge... 
    // errr, pardon my mouth, I meant to say... 
    // a warning for any unused parameter ('g++ -Wall', I'm looking at you)   
    // The same works too for keeping them happy and silent. 
    void doIt(const std::string& /* x */, const uint8 /* y */) override 
    { 
    } 
}; // class Foo