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

Код

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

Цитата(LPBOY @ 3.8.2005, 03:18)

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

я это сделал чтобы переводить было легче.

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

если написать 1: if(!chdir(ent->d_name)) printf("ошибка"); то перед каждом открытием новой папки будет выводиться слово "ошибка", затем название папки. конечно очень хочется узнать в чем тут проблема.

Выводится "ошибка", потому что ты сам это выводишь. Функция chdir возвращает 0 в случае успешного завершения,
поэтому инструкция if(!chdir(ent->d_name)) printf("ошибка"); выводит строку "ошибка" при каждом успешном завершении
chdir. Логичней написать if(chdir(ent->d_name)) printf("ошибка");

Программа работает, но в некоторых местах можно немного улучшить.

1) строку if(ent->d_name[0]=='.' && (ent->d_name[1]==0 || (ent->d_name[1]=='.' && ent->d_name[2]==0)))continue;
можешь спокойно заменить на if(ent->d_name[0]=='.') continue; Дело в том что если первый символ точка, то это
в любом случае или "."(текущая папка) или ".."(предыдущая) делать дополнительные проверки на ноль и вторую точку незачем.
Имя обычной папки не может начинаться с точки.

2) while((ent=readdir(d))!=NULL) это то же самое, что while(ent=readdir(d))

3) строку 16 int i,a; замени на int i; ;)

уй мля. у меня че-то совсем крыша поехала. вот что значит не спать по ночам. извините что потревожил Вас по пустякам.


3) ну это обычная опечатка.

огромное Вам спасибо!
Добавлено @ 05:21

Цитата(LPBOY @ 3.8.2005, 05:02)

while(ent=readdir(d))

выдает предупреждение.

Цитата(mclight @ 3.8.2005, 05:19)

выдает предупреждение.

Какое предупреждение?

не помню точно какое. я все же оставил прежний вариант. на всякий. вот а вообще спасибо. просто не было времени разбираться. код был скопирован с РНР. и соответственно при переводе на С++ были проблемы. а времени ужас как не было. так что огромное спасибо
!

Цитата(LPBOY @ 1.8.2005, 12:42)

Массив и указатель это разные вещи.

Поработав в с/с++ начинаешь терять ориентацию между реальностью и указателями.
В общем указателями реализовывается массив. Единственнон удобство статического массива - что за тебя освобождает память компилятор. Это безусловно удобно. Но принцип действия - как и у указателя.
Ну и конечно не надо забывать, что массива как типа нету. Нету типа array и тут не попишешь. Это не паскаль.
Ведь

Ni@m
Вообще говоря, тип массива есть. Но он сохраняется только в пределах видимости переменной.

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

Цитата(Ni @ 4.8.2005, 10:56)

В общем указателями реализовывается массив.

Ничего подобного. Имеется стандартное преобразование имени массива в указатель на его первый элемент,
но не надо путать массив и указатель - это совершенно разные типы.

Цитата(Ni @ 4.8.2005, 10:56)

Ну и конечно не надо забывать, что массива как типа нету. Нету типа array и тут не попишешь. Это не паскаль. Ведь

Авторы Стандарта С++ считают по-другому:
"8.3.4
In a declaration T D where D has the form
D1 [constant-expression]
and the type of the identifier in the declaration T D1 is "derived-declarator-type-list T",
then the type of the identifier of D is an array type."

Откомпилируй, например, такую программу:

Код

#include <iostream> #include <typeinfo> #include <conio.h> using namespace std; int main() {     cout<<boolalpha;     cout<<bool(typeid(int[])==typeid(int*));     getch();     return 0; }

Выдает false, кроме старых версий VC, у которых глюк по этому поводу.

Это сообщение отредактировал(а) LPBOY - 4.8.2005, 11:49

И еще -- readdir возвращает указатель на отведенную внутри нее память. Ты должен ее освобождать в конце каждого оборота цикла.

Это сообщение отредактировал(а) comcon1 - 4.8.2005, 11:57

Цитата(LPBOY @ 4.8.2005, 11:47)

Ничего подобного. Имеется стандартное преобразование имени массива в указатель на его первый элемент, но не надо путать массив и указатель - это совершенно разные типы.

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

Цитата(Void @ 4.8.2005, 11:14)

Вообще говоря, тип массива есть. Но он сохраняется только в пределах видимости переменной.

Попрошу разьеснить что значит в пределах видимости переменной в данном случае. Ведь с указателем таже ситуация. Или я уже запутался

Цитата(Ni @ 4.8.2005, 15:24)

Попрошу разьеснить что значит в пределах видимости переменной в данном случае. Ведь с указателем таже ситуация. Или я уже запутался smile

Это значит, что вне области видимости (например, при передаче в качестве параметра функции) тип T[N] неотличим от T*. Но в ее пределах это разные типы и даже массивы разного размера - разные типы. Например, специализации шаблона типом int[5] и int[10] различаются.