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

前提・実現したいこと

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

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

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

該当のソースコード

C++
1
2#include <stdio.h>
3#include <dirent.h>
4#include <sys/types.h>
5#include <stdib.h>
6#include <string.h>
7
8int selects(struct dirent *dir){
9     if(dir->d_name[0])=='*.INI'){
10            return (0);
11     }
12}
13
14int main(int argc,char **argv){
15
16     DIR *dr;
17     const char *dir=*(argv+1);
18     struct dirent **fileList;
19     dr=opendir(dir);
20     int i=scandir(dir,&fileList,selects,NULL);
21    
22    
23
24
25
26
27
28}
29
30
31
32
33
34
35
36
37

試したこと

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

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

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

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

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

というイメージです。

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

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

よろしくお願いします。

y_waiwai

2020/08/31 11:33

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

pepperleaf

2020/08/31 11:52

目についたところで、 > if(dir->d_name[0])=='*.INI'){ あと、C++じゃなくて、 Cって気がする。

tiitoi

2020/08/31 13:38 編集

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

tamatamagoo

2020/08/31 14:44

>y_waiwaiさんご回答いただきありがとうございます。お返事が遅くなり申し訳ありません。このソースをコンパイルすると、selects関数に関するエラーがでます。今すぐに確認できる状況ではないので、明日確認後修正させていただきます。

tamatamagoo

2020/08/31 14:45

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

tamatamagoo

2020/08/31 14:46

＞tiitoiさんご回答いただきありがとうございます。お返事が遅くなり申し訳ありません。使用OS　windows10 です。 OSにディレクトリの走査が依存いていることは知りませんでした。ありがとうございます。

tiitoi

2020/08/31 15:03 編集

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

tamatamagoo

2020/08/31 22:28

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

行動規範の内容に同意します

回答2件

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

C++
1#include <filesystem>
2#include <iostream>
3#include <string>
4
5int main() {
6  /* カレントdir.にある～.txt を列挙する */
7  for ( auto& entry : std::filesystem::directory_iterator(".") ) {
8    if ( entry.path().extension() == ".txt" ) {
9      std::cout << entry.path().filename().string() << std::endl;
10    }
11  }
12}

投稿2020/09/01 01:52

編集2020/09/01 01:53

episteme

総合スコア16614

tamatamagoo

2020/09/01 04:58

epistemeさんご回答ありがとうございます。 std::filesystem 初めて知りました。 OSを選ばない分汎用性が高そうですね。こちらでも行ってみたところ<filesystem>が存在しないとエラーがでてしまったのでダウンロードして再度挑戦してみます！

kazuma-s

2020/09/01 20:08

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

行動規範の内容に同意します

ベストアンサー

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

C
1#include <stdio.h>   // printf
2#include <stdlib.h>  // free
3#include <string.h>  // strrchr, strcasecmp
4#include <dirent.h>  // scandir
5
6int filter(const struct dirent *ep)
7{
8	const char *p = strrchr(ep->d_name, '.');
9	return p && !strcasecmp(p, ".ini");
10}
11
12
13int main(int argc, char *argv[])
14{
15	if (argc != 2) { puts("argc"); return 1; }
16
17	struct dirent **entry;
18	int n = scandir(argv[1], &entry, filter, NULL);
19	if (n < 0) { perror("scandir"); return 2; }
20
21	if (n > 0) printf("%s", entry[0]->d_name);
22	for (int i = 1; i < n; i++) {
23		printf(",%s", entry[i]->d_name);
24		free(entry[i]);
25	}
26	printf("\n");
27	free(entry);
28}

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

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

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

C
1#include <stdio.h>   // printf
2#include <stdlib.h>  // free
3#include <string.h>  // strrchr, strcasecmp
4#include <dirent.h>  // opendir, readdir, closedir
5
6int main(int argc, char *argv[])
7{
8	if (argc != 2) { puts("argc"); return 1; }
9
10	struct dirent **entry;
11	DIR *dp = opendir(argv[1]);
12	if (!dp) { perror("opendir"); return 2; }
13	const char *sep = "";
14	struct dirent *ep;
15	while (ep = readdir(dp)) {
16		const char *p = strrchr(ep->d_name, '.');
17		if (p && !strcasecmp(p, ".ini")) {
18			printf("%s%s", sep, ep->d_name);
19			sep = ",";
20		}
21	}
22	printf("\n");
23	closedir(dp);
24}