Блог

У меня проблема с функцией remove ()

Сначала посмотрите на пример, и вы увидите проблему

 cout << "Please enter the phone number to remove" << endl << "Phone number: ";
string rphNumber;
cin >> rphNumber;
ifstream ifile("db/"   rphNumber   ".txt");
if(ifile)
  remove(("db/"   rphNumber   ".txt").c_str()); // the problem here
else
  cout << "failure" << endl;
  

Проблема в этой строке (путь к файлу), всегда функция возвращает -1, хотя файл существует

 remove(("db/"   rphNumber   ".txt").c_str());
  

Ваша проблема может заключаться в том, что у вас все еще есть ifile open в тот момент, когда вы пытаетесь его удалить. Некоторые операционные системы не позволяют удалять открытые файлы. Другая возможность заключается в том, что строка rphNumber может содержать новую строку в конце, которую вам нужно удалить перед сборкой имени файла. (Я не помню, cin делает это или нет.)

Ваша проблема определенно заключается в том, что вы пытаетесь выяснить, будет ли работать операция файловой системы. Вы не можете этого сделать. В промежутке между выполнением теста и фактической попыткой выполнить операцию другой процесс может изменить ситуацию так, что операция не будет работать, даже если ваш тест сказал, что это будет. Кроме того, возможность открыть файл — это не то же самое, что возможность удалить файл; на вашем жестком диске, вероятно, много файлов, которые вы можете открыть, но не удалить (например, /dev/null ).

Вам нужно просто выполнить операцию с файловой системой. Он сообщит вам, сработало это или нет, с его возвращаемым значением. Затем, когда это не работает, вы смотрите errno , чтобы выяснить, почему. Служебная функция C strerror (include <cstring> ) преобразует errno значение в удобочитаемое сообщение об ошибке.

Собрав все вместе, вот правильный способ написания вашей программы:

 cout << "Please enter the phone number to remove.nPhone number: ";
string rphNumber;
cin >> rphNumber;
string fname("db/"   rphNumber   ".txt");

if (remove(fname.c_str()))
    cout << "Failed to delete '" << fname << "': " << strerror(errno) << 'n';
else
    cout << ''' << fname << "' successfully deleted.n";
  

Кстати, никогда не используйте endl ; если 'n' не работает, это означает, что ваши streambufs настроены неправильно.

Если remove сбой, он будет установлен errno , а также вернется -1 . Я не совсем понимаю, почему вы уверены, что это сбой, поскольку вы фактически не сохраняете возвращаемое значение в переменной.

Но, предполагая, что он возвращает -1, errno распечатайте, чтобы вы могли определить, какова фактическая ошибка, что-то вроде:

 int rc = remove(("db/"   rphNumber   ".txt").c_str());
if (rc < 0)
    perror ("could not remove file");