народ помогите!

Код

bool is_dir(char dirname[]) { WIN32_FIND_DATA w32fd; HANDLE h = FindFirstFile(dirname, &w32fd); return (h!=INVALID_HANDLE_VALUE)&&((w32fd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)!=0); } void PrintTree(int level=1) { DIR *d; struct dirent *ent; int i, attrib; d=opendir("."); if(!d) return; while((ent=readdir(d))!=NULL){      if(ent->d_name[0]=='.' && (ent->d_name[1]==0 || (ent->d_name[1]=='.' && ent->d_name[2]==0)))continue;            char dirname[]=TEXT(ent->d_name);      if(!is_dir(dirname))continue;

получается -
Cannot convert 'char *' to 'char[]'

mclight, Где получается?
Компилер обычно строку называет

Делай так:

Код

if(!is_dir((char*)dirname))continue;

19
Добавлено @ 22:54
19

Код

char *dirname=TEXT(ent->d_name);

А так не годится?

пытался
скорее всего это и есть правильное решение. видимо я структуру не правильно настроил. что-ж Большое спасибо!

Цитата(mclight @ 31.7.2005, 22:44)

народ помогите!

mclight, в чем дело? До сих пор мучаешься с этой задачей (PrintTree)?
А ведь я тебе где-то в старых темах написал ее решение. Что не подошло?

По поводу строки 19: ты пытаешься присвоить один массив другому, а это в С++ сделать нельзя .
Наверно надо ипользовать цикл (или memmove).

Код

char dirname[13]; for( int i=0 ; i<13 ; ++i)     dirname[i] = ent->d_name[i];

Кстати ты мешаешь в коде вызовы функций WinAPI с древними библиотечными функциями Борланда, как то странно получается...

Цитата
ты пытаешься присвоить один массив другому

Цитата
Наверно надо ипользовать цикл

Часто бывает, что функции возвращают указатель на выделенную память, не так как strcat, а как realloc.
Обязательно надо высвободить память. Утечки сейчас никого не красят. Но мне непонятно, как в этом случае пройтись циклом??

А почему нельзя присвоить один массив дркгому??
попробуй это:

Код

int a[] = { 1, 2, 3, 4, 5 }; int *b; b = &a[2]; printf("%d,%d",a[0],b[-2]); b = a; printf("%d,%d",a[0],b[0]);

Прошу прощения - предыдущий пост был моим. Забыл зарегаться.

Это сообщение отредактировал(а) Ni@m - 1.8.2005, 11:37

Цитата(Ni @ 1.8.2005, 11:36)

А почему нельзя присвоить один массив дркгому??

По правилам С++ этого делать нельзя. Придумал не я, все претензии к Страуструпу
(Вообще я говорю о массиве как о типе.)

В твоем коде ты присваиваешь указателю b (а не массиву) адрес 3-го элемента массива a.
Никакого присваивания одного массива другому тут нет.
Массив и указатель это разные вещи.

У mclight'а был один массив внутри структуры dirent, который объявлен след. образом
(в dirent.h):

Код

struct dirent {     char  d_name[13]; };

Затем он создал второй char dirname[] и попытался скопировать в него массив d_name.
Самый естесственный способ сделать это с помощью цикла.

Хотя вместо

Код

char dirname[]=TEXT(ent->d_name);      if(!is_dir(dirname))continue;

проще было бы написать сразу

Код

if(!is_dir(ent->d_name))continue;

промежуточные массивы тут даже не нужны...

Цитата(Ni @ 1.8.2005, 11:36)

Часто бывает, что функции возвращают указатель на выделенную память, не так как strcat, а как realloc. Обязательно надо высвободить память. Утечки сейчас никого не красят. Но мне непонятно, как в этом случае пройтись циклом??

А в чем проблема?

можно и так если использовать только приметивы с:

Код

char dirname[strlen(TEXT(ent->d_name))]; strcpy(dirname, TEXT(ent->d_name));

конечно лучше не мешать приметивы разных в концептуальном плане библиотек - а то программа может получиться больших размеров - оптимально использовать либо приметивы самого языка(string.h, cstring....), либо функции и методы свойственные для конкретной реализации библиотеки (MFC, VCL), или же приметивы самой ОС (API)

Цитата(RTcoder @ 1.8.2005, 13:17)

можно и так если использовать только приметивы с:

Согласен, но тут есть несколько моментов

1)
Точнее будет так:

Код

char* dirname = new char[strlen(ent->d_name)+1]; strcpy(dirname, ent->d_name);

2) Этот частный случай работает только для массивов, содержащих asciiz-строки.

3) Внутри функции strcpy все равно крутится все тот же цикл

P.S. использовать макрос TEXT для указателей нет никакого смысла TEXT(ent->d_name).
Это приведет к ошибке компиляции если определен Юникод.

Это сообщение отредактировал(а) LPBOY - 1.8.2005, 14:35

спасибо. а твой иходник LPBoy тогда не пошел.
Он выводил одну папку. и все. а мне надо было все. я не стал дальше разбираться. что-там к чему. времени не было. но все равно огромное спасибо!

Это сообщение отредактировал(а) mclight - 3.8.2005, 01:56

Народ конечно извините. я Вас совсем замучил. получилось все!
дело было не в промежеточных массивах. все что вы предлагали я и раньше делал. вот только программа выводила только первую папку. ане все.

Код

if(!chdir(ent->d_name))continue;

исправил на

Код

chdir(ent->d_name);

правда совсем не понял почему так получилось. прежде чем открыть след папку выводится сообщение об ошибке
если написать

Код

if(!chdir(ent->d_name)) printf("ошибка");

то перед каждом открытием новой папки будет выводиться слово "ошибка", затем название папки.

конечно очень хочется узнать в чем тут проблема.

Это сообщение отредактировал(а) mclight - 3.8.2005, 02:11

Цитата(LPBOY @ 1.8.2005, 11:00)

Кстати ты мешаешь в коде вызовы функций WinAPI с древними библиотечными функциями Борланда, как то странно получается...

можешь объяснить что и где?

Цитата(mclight @ 3.8.2005, 01:25)

хочется узнать в чем тут проблема.

Я не телепат Покажи код полностью.

Цитата(mclight @ 3.8.2005, 02:12)

можешь объяснить что и где?

Это не ошибка, просто обычно придерживаются какого-то одного стиля.
Функция is_dir написана средствами WinAPI (т.е. ориентирована на Windows),
а в остальной программе используются функции библиотеки борланда (ориентированной на MS-DOS).

Цитата(mclight @ 2.8.2005, 23:58)

Он выводил одну папку. и все. а мне надо было все. я не стал дальше разбираться. что-там к чему. времени не было.

Не, мой исходник выводит ВСЕ папки и файлы. Не знаю почему у тебя выводит только одну...
Может ты имя папки задавал больше 8-ми символов, с такими именами функции MS-DOS
не работают. Попробуй с папкой с коротким именем или если имя длинное, например
program files, то надо писать так progra~1. И слэш надо использовать не одинарный, а двойной.
Например: c:\\dir

Код

#include <stdio.h> #include <dir.h> #include <dirent.h> bool is_dir(char dirname[]) { WIN32_FIND_DATA w32fd; HANDLE h = FindFirstFile(dirname, &w32fd); return (h!=INVALID_HANDLE_VALUE)&&((w32fd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)!=0); } void PrintTree(char *entrydir, int level=1) { DIR *d; struct dirent *ent; int i,a; chdir(entrydir); d=opendir("."); if(!d) exit; while((ent=readdir(d))!=NULL){      if(ent->d_name[0]=='.' && (ent->d_name[1]==0 || (ent->d_name[1]=='.' && ent->d_name[2]==0)))continue;       if(!is_dir(ent->d_name))continue;      for(i=0;i<level;i++)printf("...");      printf("%s\n",ent->d_name);       chdir(ent->d_name);      PrintTree(ent->d_name, level+1);      chdir("..");      } closedir(d); } void main(int argc,char *argv[]) {    if (argc != 2)    {      printf("please enter dir\n");      exit(1);    }    PrintTree(argv[1]); }

а именно строка 28

Это сообщение отредактировал(а) mclight - 3.8.2005, 03:38