[C++] フォルダオープンから特定ファイル読み込み

Question

前提・実現したいこと

１つのフォルダに入っているすべてのテキストファイルをcsvに出力するツールの作成。

発生している問題・エラーメッセージ

1 : int (*)(dirent*)’ から ‘int (*)(const dirent*)’ への無効な変換です
2 : エラー: expected ‘;’ before ‘}’ token

該当のソースコード

#include <stdio.h>
#include <dirent.h>
#include <sys/types.h>
#include <stdib.h>
#include <string.h>

int selects(struct dirent *dir){
     if(dir->d_name[0])=='*.INI'){
            return (0);
     }
}

int main(int argc,char **argv){

     DIR *dr;
     const char *dir=*(argv+1);
     struct dirent **fileList;
     dr=opendir(dir);
     int i=scandir(dir,&fileList,selects,NULL);
    
    





}

試したこと

グーグルにて使えそうな関数を検索
selects関数なしだとコンパイルは通りました。

補足情報（FW/ツールのバージョンなど）

C++はじめたての初心者です。

丸投げのような形の質問になっておりました。申し訳ございません。

上記ソースは「.ini」ファイルを仮に当てはめています。
流れとして、
①フォルダ読み込み
②ファイル抽出
③ファイルの一行ずつ読み込む
④","を後ろに入れながらファイル出力

というイメージです。

まず第１，２段階のフォルダオープンからのファイル抽出ができません。

また、そのあとの流れもこのままでよいかわからず、手詰まりになっています。

よろしくお願いします。

Comments

[y_waiwai]

提示のコードではどういう不具合があるんでしょうか

[pepperleaf]

目についたところで、
> if(dir->d_name[0])=='*.INI'){

あと、C++じゃなくて、 Cって気がする。

[tiitoi]

ディレクトリ走査は OS 依存です。
使用している OS を追記してください

[tamatamagoo]

>y_waiwaiさん

ご回答いただきありがとうございます。
お返事が遅くなり申し訳ありません。

このソースをコンパイルすると、selects関数に関するエラーがでます。今すぐに確認できる状況ではないので、明日確認後修正させていただきます。

[tamatamagoo]

＞pepperleafさん
ご回答いただきありがとうございます。お返事が遅くなり申し訳ありません。
確かにC言語ですね。。。
if文の中身がやはり間違っているのでしょうか、、、

[tamatamagoo]

＞tiitoiさん

ご回答いただきありがとうございます。お返事が遅くなり申し訳ありません。

使用OS　windows10

です。
OSにディレクトリの走査が依存いていることは知りませんでした。
ありがとうございます。

[tiitoi]

Windows で質問のコードはそもそもコンパイルが通るのでしょうか？
dirent.h は POSIX 系の OS でしか使えないと思っていましたが、開発は WSL 上で行っていたりするのでしょうか？

[tamatamagoo]

>tiitoiさん
開発はlinuxで行っています。
Windowsでも通りませんでした。。。

Answer 1

C++17 なら std::filesystem 使います僕だったら。 (OSを選ばんので)

#include <filesystem>
#include <iostream>
#include <string>

int main() {
  /* カレントdir.にある～.txt を列挙する */
  for ( auto& entry : std::filesystem::directory_iterator(".") ) {
    if ( entry.path().extension() == ".txt" ) {
      std::cout << entry.path().filename().string() << std::endl;
    }
  }
}

Comments

[tamatamagoo]

epistemeさん

ご回答ありがとうございます。
std::filesystem 初めて知りました。

OSを選ばない分汎用性が高そうですね。
こちらでも行ってみたところ<filesystem>が存在しないとエラーがでてしまったのでダウンロードして再度挑戦してみます！

[kazuma-s]

g++ -std=c++17 でコンパイルしましたか？
または、g++ --version でコンパイラのバージョンを調べてみてください。

Answer 2

これ思った通りに動きますか？

#include <stdio.h>   // printf
#include <stdlib.h>  // free
#include <string.h>  // strrchr, strcasecmp
#include <dirent.h>  // scandir

int filter(const struct dirent *ep)
{
	const char *p = strrchr(ep->d_name, '.');
	return p && !strcasecmp(p, ".ini");
}


int main(int argc, char *argv[])
{
	if (argc != 2) { puts("argc"); return 1; }

	struct dirent **entry;
	int n = scandir(argv[1], &entry, filter, NULL);
	if (n < 0) { perror("scandir"); return 2; }

	if (n > 0) printf("%s", entry[0]->d_name);
	for (int i = 1; i < n; i++) {
		printf(",%s", entry[i]->d_name);
		free(entry[i]);
	}
	printf("\n");
	free(entry);
}

FILE *fp = fopen("xxx.csv", "w"); でファイルをオープンして、
printf(...); の代わりに、fprintf(fp, ...); にしてみてください。

追記
コードにバグがありました。free(entry[0]); が抜けています。

scandir の代わりに opendir/readdir/closedir を使っても書けます。

#include <stdio.h>   // printf
#include <stdlib.h>  // free
#include <string.h>  // strrchr, strcasecmp
#include <dirent.h>  // opendir, readdir, closedir

int main(int argc, char *argv[])
{
	if (argc != 2) { puts("argc"); return 1; }

	struct dirent **entry;
	DIR *dp = opendir(argv[1]);
	if (!dp) { perror("opendir"); return 2; }
	const char *sep = "";
	struct dirent *ep;
	while (ep = readdir(dp)) {
		const char *p = strrchr(ep->d_name, '.');
		if (p && !strcasecmp(p, ".ini")) {
			printf("%s%s", sep, ep->d_name);
			sep = ",";
		}
	}
	printf("\n");
	closedir(dp);
}

Comments

[tamatamagoo]

kazema-sさん
ご回答いただきありがとうございます。

現在確認できないため、明日確認後ご報告させていただきます。

よろしくお願いします。

[tamatamagoo]

kazema-sさん

コンパイル通りました。
対象ファイルが標準出力されました。

ありがとうございます！

[tamatamagoo]

追加で質問がございます。

const char *p = strrchr(ep->d_name, '.');

などにあります　

ep->d_name　や　entry[i]->d_name

は構造体に格納されたファイル名を対象にしているということでしょうか。

[kazuma-s]

はい、その通りです。