Название диска

[Prokl]

Код: (C++)

1. void __fastcall TForm1::diski()
2. {  //получаем название диска
3.  
4. char DiskLabel[MAX_PATH];
5. char *DiskName1 = (ComboBox1->Items->Strings[ComboBox1->ItemIndex]).c_str();//+"\\"
6. WORD OldErrorMode;
7. Label4->Caption = (ComboBox1->Items->Strings[ComboBox1->ItemIndex]).c_str();
8. OldErrorMode = SetErrorMode(SEM_FAILCRITICALERRORS); // убираем показ ошибок
9. bool ready = DirectoryExists(DiskName1); // пытаемcя открыть корневую директорию
10. SetErrorMode(OldErrorMode); // восстанавливаем старый режим показа ошибок
11.  
12.           if (ready)
13.           {
14.           GetVolumeInformation(DiskName1, DiskLabel, MAX_PATH, NULL, NULL, NULL, NULL, 0);
15.          
16.  
17.           Edit1->Text =  DiskLabel;//здесь название диска
18.           Button2->Enabled = true;
19.           Timer1->Enabled = false;
20.           }
21.           else
22.           {
23.           Edit1->Text = "не готов";
24.           Button2->Enabled = false;
25.           Timer1->Enabled = true;
26.           }
27.  delete [] DiskName1;
28. }

этот код работает нестабильно, невсегда показывает название диска, часто показывает его неправильно.
вызываю функцию:

Код:

void __fastcall TForm1::ComboBox1Change(TObject *Sender)
{
diski();
}

[Dimka]

Prokl, а в чём ты видишь помощь?

[Prokl]

Даже незнаю, может ошибку какую кто-нибудь подскажет...

[Dimka]

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

Если вообще не понимаешь - взял кусок чужого кода, - то разговаривать, в общем-то, не о чем.

[RXL]

Я отредактировал пост - подсветил две строчки кода.
Prokl, расскажи, что в них делается.

[Антон (LogRus)]

RXL
ох ты ж ёлки-метёлки
я эту строчку то и не заметил

[RXL]

Я ж, в основном, на BCB программирую. Оператор delete там нужен только если ты сам используешь new - в остальных случаях либо все само когда надо уничтожается, либо используются косвенные механизмы, но не delete. А тут new не видать, зато есть получение указателя на внутренний буфер объекта.

[Prokl]

Код:

// для получения метки диска напишите следующий код:
char DiskLabel[MAX_PATH];
char *DiskName = "c:\\"; // имя диска
GetVolumeInformation(DiskName, DiskLabel, MAX_PATH, NULL, NULL, NULL, NULL, 0);
// в DiskLabel находится метка диска

Это послужило мне примером для того кода. Здесь тоже нет "new", а "delete" я вставил уже после и на работу кода это не отразилось.
В ComboBox создается список CD-ROMов в формате - D:\\
значение выбранной строки ComboBox  передается переменной DiskName1.

[Вад]

Код:

// для получения метки диска напишите следующий код:
char DiskLabel[MAX_PATH];
char *DiskName = "c:\\"; // имя диска
GetVolumeInformation(DiskName, DiskLabel, MAX_PATH, NULL, NULL, NULL, NULL, 0);
// в DiskLabel находится метка диска

Это послужило мне примером для того кода. Здесь тоже нет "new", а "delete" я вставил уже после и на работу кода это не отразилось.
В ComboBox создается список CD-ROMов в формате - D:\\
значение выбранной строки ComboBox  передается переменной DiskName1.

На всякий случай. Когда в строковой константе написано "\\" - на самом деле в строке слеш только один, а второй его экранирует (дань escape-последовательностям). Поэтому то, что в комбо-боксе видится как "d:\\", в константе будет "d:\\\\".

[Prokl]

Код:

AnsiString DiskName1 = (ComboBox1->Items->Strings[ComboBox1->ItemIndex]);


GetVolumeInformation(DiskName1.c_str(), DiskLabel, MAX_PATH, NULL, NULL, NULL, NULL, 0);

Вот две строки, которые я изменил и все заработало.
RXL, спасибо

[RXL]

Prokl, нифига не "все"! Строку 27 (в коде в начале темы) убери или получишь плавающий глюк.

[Prokl]

Забыл написать, я ее убрал сразу