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

@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) это фактически язык С (если не вдаваться в подробности).

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,705
	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