Неприятная проблема с массивом строк

@kveigs · Регистрация: 20.03.2010

Студворк — интернет-сервис помощи студентам

Здравствуйте! Пытаюсь записать в массив строк содержимое директории.. Всё содержимое находит, но на выходе весь массив почему-то заполнен именем последней папки директории..

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
WIN32_FIND_DATAA FindFile;
    HANDLE hFile;
    LPSTR szPATH;
char* tmpDirs[20];
    printf("--------------------------------------------------------------------------------");
    char s[10]="\\*.*";
    strcat(main_dir,s);
    for(int i=0; i<20; i++)
        tmpDirs[i]=0;
    hFile = FindFirstFileA(main_dir, &FindFile);
    
        int kFile = 0;
 
        if (hFile != INVALID_HANDLE_VALUE)
        {
            do
            { 
                kFile++;
                if (kFile > 2)
                {
                    tmpDirs[kFile-3] = FindFile.cFileName;
                    printf("[%d]%s\n", kFile-3, tmpDirs[kFile-3]);//Здесь выводит все нормально (5 имен                                  разных папок)
                }
            }
            while(FindNextFileA(hFile, &FindFile) != 0);
        }else
        {
            FindClose(hFile);
            return FALSE;
        }
        FindClose(hFile);
    
    printf("--------------------------------------------------------------------------------");
    printf("\n");
    for(int i=0; i<20; i++)
    {
        printf("%s", tmpDirs[i]);//!!!а здесь 5 (в каталоге у меня 5 папок) раз выводит имя последней папки(((
    }

P.S. Пишу фотопросмоторщик!) Я в Отчаянье!!!

@kravam · 18.07.2011, 15:49

А в консоль выводит? А то я тебе могу дать прогу, она в консоль выводит содержимое директории. Значит, в массив строк перегонишь как-нибудь

@kveigs · 18.07.2011, 16:08 **[ТС]**

В консоль выводит все правильно.. Там с указателями скорее всего проблема((
Прогу давай)))

@kravam · 18.07.2011, 16:12

Тут тебе и на консоль и в массив и снова на консоль и куда угодно

Добавлено через 42 секунды

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
#include <windows.h>
#include <iostream>
#include <vector>
#include <iterator>
using namespace std;
 
int main(){
         
        vector<string> vec; 
 
        WIN32_FIND_DATA FindFileData;
        HANDLE hf;
        hf=FindFirstFile("c:\\*", &FindFileData);
        if (hf!=INVALID_HANDLE_VALUE){
                do{
                        std::cout << FindFileData.cFileName << "\n";
                        vec.push_back (string(FindFileData.cFileName));
                
                }
                while (FindNextFile(hf,&FindFileData)!=0);
                FindClose(hf);
        }
        //
        cout<< "  +++++++++++++++++++++++++++++++++  "<< endl; 
        copy (vec.begin(), vec.end(), ostream_iterator<string>(cout, "\n"));
 
 
        getchar ();
}

Добавлено через 2 минуты
Дай полный код своей проги, я имею ввиду со всеми хидерами и прочая. Короче, компилящийся модуль.

@easybudda · 18.07.2011, 16:21

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <dirent.h>
    
int main(void){
    char ** files;
    size_t count, pos;
    DIR * dir;
    struct dirent * de;
    
    if ( ! ( dir = opendir(".") ) ){
        perror("opendir");
        exit(1);
    }
    
    files = NULL;
    count = 0;
    while ( de = readdir(dir) ){
        if ( ! strcmp(de->d_name, ".") || ! strcmp(de->d_name, "..") )
            continue;
        if ( ! ( files = realloc(files, sizeof(char*) * (count + 1)) ) ){
            perror("realloc");
            exit(1);
        }
        if ( ! ( files[count++] = strdup(de->d_name) ) ){
            perror("strdup");
            exit(1);
        }
    }
    closedir(dir);
    
    if ( ! files ){
        fprintf(stderr, "No files found!\n");
        exit(1);
    }
    
    printf("Files in current directory:\n");
    for ( pos = 0; pos < count; ++pos )
        printf("%s\n", files[pos]);
    
    for ( pos = 0; pos < count; ++pos )
        free(files[pos]);
    free(files);
    exit(0);
}

для разнообразия...

@-=ЮрА=- · 18.07.2011, 16:29

dirent.h — заголовочный файл, входящий в библиотеку C POSIX library для языка Си, не на всех студиях покатит такой кодик...

@easybudda · 18.07.2011, 16:41

Не по теме:

А windows.h - стандартный заголовочный файл языка С, ага... void main() тоже не любой компилятор пропускает, кого это останавливает? ;)

@-=ЮрА=- · 18.07.2011, 16:44

На 6-й версии точно не работает

@kveigs · 18.07.2011, 16:54 **[ТС]**

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include <Windows.h>
#include <stdio.h>
#include<conio.h>
 
void main()
{
    char* main_dir="collection";
WIN32_FIND_DATAA FindFile;
        HANDLE hFile;
        LPSTR szPATH;
char* tmpDirs[20];
        printf("--------------------------------------------------------------------------------");
        char s[10]="\\*.*";
        strcat(main_dir,s);
        for(int i=0; i<20; i++)
                tmpDirs[i]=0;
        hFile = FindFirstFileA(main_dir, &FindFile);
        
                int kFile = 0;
 
                if (hFile != INVALID_HANDLE_VALUE)
                {
                        do
                        { 
                                kFile++;
                                if (kFile > 2)
                                {
                                        tmpDirs[kFile-3] = FindFile.cFileName;
                                        printf("[%d]%s\n", kFile-3, tmpDirs[kFile-3]);//Здесь выводит все нормально (5 имен                                  разных папок)
                                }
                        }
                        while(FindNextFileA(hFile, &FindFile) != 0);
                }else
                {
                        FindClose(hFile);
                }
                FindClose(hFile);
        
        printf("--------------------------------------------------------------------------------");
        printf("\n");
        for(int i=0; i<20; i++)
        {
                printf("%s", tmpDirs[i]);//!!!а здесь 5 (в каталоге у меня 5 папок) раз выводит имя последней папки(((
        }
        getch();
 
}

Только у меня не хочет работать(( Visual Studio странная штука однако)

@easybudda · 18.07.2011, 16:59

-=ЮрА=-, на сколько понял, у ТС с сохранением имён в массив проблема, а не с их получением. Мне же все эти виндовсские чудо-конструкции лень было писать, а пример на работоспособность проверить нужно. Вот и получал имена файлов привычным способом. Тот факт, что ни одна студия ANSI C99 за стандарт вообще не признаёт, ещё не значит, что стандарт плохой...

@asics · 18.07.2011, 16:59

Способ попроще

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#include <iostream>
#include <algorithm>
#include <vector>
#include <string>
 
#include <boost/filesystem/path.hpp>
#include <boost/filesystem/operations.hpp>
 
int main(){
  std::vector<std::string> v;
  boost::filesystem::path p("C:\\folder");
  for(boost::filesystem::directory_iterator s(p), f; s != f; ++s)
    v.push_back(s->leaf());
  for(auto &x : v) std::cout << x << '\n';
  return 0;
}

@kravam · 18.07.2011, 17:04

C++
1
        strcat(main_dir,s);

kveigs, уже неправильно. Хочешь- исправляй, поковыряемся.

@-=ЮрА=- · 18.07.2011, 17:13

Ну тогда можно сказать что не все коды плохи, на которые компиляторы старших версий ругаются...
PS:Как вариант стандартного поиска фалов можно юзать IShellfolder::EnumObjects

@grizlik78 · 18.07.2011, 17:18

В исходной программе ошибка, разумеется, вот здесь:

C++
1
tmpDirs[kFile-3] = FindFile.cFileName;

Здесь просто сохраняется указатель на один и тот же буфер. Ничего удивительного, что в результате много раз повторяется последнее значение.
Надо выделить память и скопировать имя.

C++
1
2
tmpDirs[kFile-3] = new char[MAX_PATH];
strncpy(tmpDirs[kFile-3], FindFile.cFileName, MAX_PATH);

Ну а в конце память надо будет освободить.

@Kastaneda · 18.07.2011, 17:25

без записи в массив, просто вывод на экран содержимого дериктории (ни фига не стандартный)

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
#include <io.h>
#include <iostream>
 
int main () {
    _finddata_t  file;
    intptr_t ptr;
    ptr=_findfirst("*",&file);
    do{
        std::cout<<file.name<<std::endl;
    }while(!_findnext(ptr,&file));
    _findclose(ptr);
    return 0;
}

Добавлено через 1 минуту

Не по теме:

если честно, сам не знал об этом. случайно нагуглилось))

@easybudda · 18.07.2011, 17:32

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <windows.h>
 
#pragma comment(lib, "User32.lib")
    
int main(void){
    char ** files;
    size_t count, pos;
    WIN32_FIND_DATAA wfd;
    HANDLE find;
    
    if ( ( find = FindFirstFileA("*.*", &wfd) ) == INVALID_HANDLE_VALUE ){
        fprintf(stderr, "Shit happens!\n");
        FindClose(find);
        exit(1);
    }
    
    files = NULL;
    count = 0;
    do {
        if ( strcmp(wfd.cFileName, ".") && strcmp(wfd.cFileName, "..") ){
            if ( ! ( files = realloc(files, sizeof(char*) * (count + 1)) ) ){
                perror("realloc");
                exit(1);
            }
            if ( ! ( files[count++] = strdup(wfd.cFileName) ) ){
                perror("strdup");
                FindClose(find);
                exit(1);
            }
        }
    } while ( FindNextFileA(find, &wfd) );
    
    FindClose(find);
    
    if ( ! files ){
        fprintf(stderr, "No files found!\n");
        exit(1);
    }
    
    printf("Files in current directory:\n");
    for ( pos = 0; pos < count; ++pos )
        printf("%s\n", files[pos]);
    
    for ( pos = 0; pos < count; ++pos )
        free(files[pos]);
    free(files);
    exit(0);
}

@kveigs · 18.07.2011, 17:53 **[ТС]**

О БОГИ!!! Она заработала!! спасибо всем, а вам grizlik78 в особенности... Мда С++ это вам не игрушки... Надеюсь что привыкну к нему вскоре))

@Kastaneda · 18.07.2011, 18:02

Не по теме:

Сообщение от kveigs

Мда С++ это вам не игрушки...

Все, что было выше (кроме кода asics'a) это фактически язык С (если не вдаваться в подробности).

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Ниже машинный перевод статьи The Thinkpad X220 Tablet is the best budget school laptop period . Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы,. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,702
	18.07.2011, 15:49
	А в консоль выводит? А то я тебе могу дать прогу, она в консоль выводит содержимое директории. Значит, в массив строк перегонишь как-нибудь 1

@kveigs 16 / 16 / 0 Регистрация: 20.03.2010 Сообщений: 85
	18.07.2011, 16:08 [ТС]
	В консоль выводит все правильно.. Там с указателями скорее всего проблема(( Прогу давай))) 1

@-=ЮрА=- 6614 / 4256 / 401 Регистрация: 08.08.2009 Сообщений: 10,325 Записей в блоге: 24
	18.07.2011, 16:29
	dirent.h — заголовочный файл, входящий в библиотеку C POSIX library для языка Си, не на всех студиях покатит такой кодик... 0

@easybudda
	18.07.2011, 16:41
	Не по теме: А windows.h - стандартный заголовочный файл языка С, ага... void main() тоже не любой компилятор пропускает, кого это останавливает? ;) 0

@-=ЮрА=- 6614 / 4256 / 401 Регистрация: 08.08.2009 Сообщений: 10,325 Записей в блоге: 24
	18.07.2011, 16:44
	На 6-й версии точно не работает 0

@easybudda Модератор 12843 / 7592 / 1766 Регистрация: 25.07.2009 Сообщений: 13,973
	18.07.2011, 16:59
	-=ЮрА=-, на сколько понял, у ТС с сохранением имён в массив проблема, а не с их получением. Мне же все эти виндовсские чудо-конструкции лень было писать, а пример на работоспособность проверить нужно. Вот и получал имена файлов привычным способом. Тот факт, что ни одна студия ANSI C99 за стандарт вообще не признаёт, ещё не значит, что стандарт плохой... 0

@-=ЮрА=- 6614 / 4256 / 401 Регистрация: 08.08.2009 Сообщений: 10,325 Записей в блоге: 24
	18.07.2011, 17:13
	Ну тогда можно сказать что не все коды плохи, на которые компиляторы старших версий ругаются... PS:Как вариант стандартного поиска фалов можно юзать IShellfolder::EnumObjects 1

@kveigs 16 / 16 / 0 Регистрация: 20.03.2010 Сообщений: 85
	18.07.2011, 17:53 [ТС]
	О БОГИ!!! Она заработала!! спасибо всем, а вам grizlik78 в особенности... Мда С++ это вам не игрушки... Надеюсь что привыкну к нему вскоре)) 0