scandir関数を使ったファイルの検索について

Question

早速本題に入らせて貰います

int is_event_device(const struct dirent *dir)
{
	return strncmp("event", dir->d_name, 5) == 0;
}

struct dirent **namelist;
int i, ndev;

ndev = scandir("/dev/input", &namelist, is_event_device, alphasort);

大分省いてソースを書きましたが
このようなC言語のソースがあり、コンパイルも通り
問題なく動作することが確認できました

これを元に既にあるC++のプログラムに組み込もうと思い
以下のように変更しました

int クラス名::is_event_device(const struct dirent *dir)
{
	return strncmp("event", dir->d_name, 5) == 0;
}

変更といいましても、使用したいクラス内に組み込んだだけなので
クラス関数(メソッド?)として追加しただけなのですが
これをコンパイルするとエラーになってしまいました。
エラーはint型をint ()(struct dirent)型に変換できないという内容でした

これをクラス外で宣言してソースの「クラス名::」の部分を消すだけで
コンパイルが通るようになりました。
出来ればこれをクラス関数として使いたいため解決法が分かる方居ましたら
教えていただけませんでしょうか？

出来ればこういうエラーになってしまう原因なども分かりましたら教えていただけると嬉しいです。
関数とクラス関数は似ているようで別物なのでしょうか

以上、よろしくお願いいたします

----------追記--------------
クラス宣言

//ファイル名：クラス名.h
namespace 名前
{

//----------------------------------------------------------------------------
// クラス宣言.
//----------------------------------------------------------------------------
class クラス名
{
//----------------------------------------------------------------------------
// 定数定義.
//----------------------------------------------------------------------------


//----------------------------------------------------------------------------
// メンバ関数.
//----------------------------------------------------------------------------
// 公開メンバ関数.
public:


// 非公開メンバ関数.
private:
	int is_event_device(const struct dirent *dir)

//----------------------------------------------------------------------------
// メンバ変数.
//----------------------------------------------------------------------------
// 非公開メンバ変数.
private:

};

}  // namespace 名前

エラー文は
「cannot convert '名前::クラス名::is_event_device' from type 'int (名前::クラス名::)(const dirent*) to type 'int ()(const dirent)'」
です

エラーが出ているのは
ndev = scandir("/dev/input", &namelist, is_event_device, alphasort);
のところです

Comments

[Chironian]

C++のクラスの宣言も記載下さい。エラーメッセージはコピペして下さい。エラーメッセージが出ている行も示して下さい。

[tuyudaku]

申し訳ありません、補足したのでよろしくお願いいたします。コード内に日本語で書いてある部分は会社のソースのため特定されそうな部分は置き換えてあります。分かりづらくて申し訳ありませんが、よろしくお願いいたします...

Answer 1

（追記２）
試してみてコンパイルが通ったので、コードを載せておきます。

class scandirTest
{
public:
    void callScandir(void){
        struct dirent **namelist;
        int i, ndev;
        ndev = scandir("/dev/input", &namelist, is_event_device, NULL);  
    }
private:
    static int is_event_device(const struct dirent *dir){
        return strncmp("event", dir->d_name, 5) == 0;
    }
};

int main(void){
    class scandirTest *p = new scandirTest();
    p->callScandir();
}

（追記１）
すみません、良く調べずに回答してしまいました。
is_event_deviceはscandirのコールバックとして登録するのですね。
出直します・・・。

以下は忘れてください。

クラスの宣言部分を追記いただいた方が、より良い回答が得られると思います。

勘ですが、そもそもis_event_device関数に渡している変数が間違っていませんか？
エラーメッセージの解釈が正しいのであれば、渡されている変数の型はint型だと言っています。
しかしis_event_device関数はstruct dirent *型を受け取りたいはずです。
scandir関数の戻り値ndevを渡してたりしないでしょうか。
*namelistを渡していますか？

Comments

[tuyudaku]

そこが間違っている場合、C言語のコンパイラでも通らないと思っているのですが
やはり言語が違うと変わってくるのでしょうか...
自分でも試行錯誤しながら回答を待ちたいと思います

回答ありがとうございました

[tuyudaku]

いえ、わざわざありがとうございます
単純そうで意外と深い部分の問題な気がしてとても困っています

回答ありがとうございました

[ttyp03]

試してみました。
ただ、出力されるエラーが異なり、static関数でないとダメと言われたので、static宣言しましたところ、コンパイルは通りました
インスタンス化されていないものをコールバックの対象にはできないでしょうから当然ですね。
こうしました↓
static int is_event_device(const struct dirent *dir)

[ttyp03]

コードを追記しましたので参考にしてみてください。

[tuyudaku]

とりあえずstaticにして試してみましたが
「cannot declare member function 'static int 名前::クラス名::is_event_device(const dirent*)' to have static linkage」
というエラーが出てしまいました。

何が違うのでしょうか...

皆様のアドバイスを参考にもう少し試行錯誤してみたいと思います

回答ありがとうございました。

Answer 2

他の方が説明されていますが、

変更といいましても、使用したいクラス内に組み込んだだけなのでクラス関数(メソッド?)として追加しただけなのですが

「関数」を「メンバ関数」という別の物へかえてしまったのですから大問題です。「だけ」じゃありません。

エラー文は「cannot convert '名前::クラス名::is_event_device' from type 'int (名前::クラス名::)(const dirent*) to type 'int ()(const dirent)'」です

といいますが、(名前::クラス名::)は(名前::クラス名::*)のはずです。
エラーメッセージは正確に載せるべきです。
といいますか、このエラーメッセージに「メンバ関数へのポインタ」は「関数へのポインタ」に変換できませんと、はっきり書いてあります^^
クラスのメンバ関数へのポインタについては、C++の基礎なので、参考書をみてください。

Answer 3

こんにちは。

出遅れました。ttyp03さんが回答されている通りstatic関数でないことが原因です。

非staticなメンバ関数は、内部的に this をパラメータで受け取っています。（コンパイラがこっそり付けてます。呼び出し側と呼び出される側両方にです。）
C言語にはそのような機能はありませんので、C言語のコールバック関数を呼ぶことを想定しているライブラリは、当然、thisを渡さないで呼び出しますから、引数リストがずれて異常動作します。それが自明ですので、事前にコンパイラがエラーにしているということなのです。

Comments

[tuyudaku]

とりあえずstaticにして試してみましたが
「cannot declare member function 'static int 名前::クラス名::is_event_device(const dirent*)' to have static linkage」
というエラーが出てしまいました。

何が違うのでしょうか...

皆様のアドバイスを参考にもう少し試行錯誤してみたいと思います

回答ありがとうございました。

[Chironian]

staticを付ける場所が違います。
クラスを宣言している側に付けて下さい。
関数定義側には付けてはいけません。

[tuyudaku]

はい、ごめんなさい
私もそれに気付いてたった今修正したところでした...

わざわざ回答ありがとうございます

分かっていたことですが、自分自身の勉強不足ぶりが遺憾なく発揮されて
落ち込むばかりです...

ChironianさんのC++講座がかなり興味深かったので
これを機に暇な時間(仕事中)に読み進めたいと思います
買って全然読んでない参考書も読み進めたいです...

なんとか問題は解決できそうです
ありがとうございました！