return 1 в void функции !

[интеграл]

Всем привет !

Вступление:
Переделываю не ООП в ООП, на языке C++ . Программа элементарная - телефонная книжка.

Создал класс "Записи", в классе 4 функции: 1)читать(читает записи из файла в 2 массива - имена и телефоны), 2) сортировка по телефонным номерам, 3)сортрировка по именам, 4) показать

class record
{
public:
         void read();
         void sort_n();
         void sort_p();
         void display();
};

void read() - открывает текстовой файл при помощи fstream, а если не может открыть/найти файл выдаёт ошибку и заканчивает программу при помощи return 1; если открывает файл, то считывает содержимое в 2 массива, и закрывает файл.

вот как выглядит проверка файла

void record::read()
{
	fstream  infile("file.txt");
	if (infile.fail())
    {
        cerr << "can't open file\n";
        return 1;
    }
     // далее ......

}

Когда компилирую программу, выдаёт ошибку что void read() возвращает return 1(текстовой файл, в той же папке где .cpp и .h файлы)

Могу разместить проверку открытия .txt файла в main(), но тогда теряется смысл самого void read();

Что делать?
Спасибо !!!

[netrino]

объявите record::read() как возвращающую int и в основной программе проверяйте значение, которая вернёт эта ф-ция. кстати, вывод сообщения об ошибке лучше тоже перенести в вызывающую ф-цию

[интеграл]

тоесть, вы хотите сказать что нужно переписать
int record::read()

Сообщение от netrino

кстати, вывод сообщения об ошибке лучше тоже перенести в вызывающую ф-цию

вот это не совсем понял, хотите сказать что

cerr << "can't open file\n";

стоит перенести в main() ??

Спасибо !

[netrino]

Сообщение от интеграл

вот это не совсем понял, хотите сказать что

cerr << "can't open file\n";

стоит перенести в main() ??

Спасибо !

Угу ) Желательно, чтобы функция не производила побочных действий, она должна делать только свою работу ) Также не очень хорошо, когда имя файла "забито" в программу, обычно оно вводится с клавиатуры, впрочем, если вам по заданию именно так и нужно, то другое дело )

[RUSt88]

return 1 в void функции !

void ничего не возвращает...

[интеграл]

Спасибо за что помогли и обьяснили!!
я поступлю так

int record::read()
{
	fstream  infile("file.txt");
	if (infile.fail())
    {
        return 0;
    }
     // далее считывает данные  ......
     return 1

}

а в main() будет проверка
на возврат 0 будет

cerr << "can't open file\n";
return 1;

на возврат 1
продолжать программу......

думаю что так сработает

[интеграл]

Слушайте, я так подумал ......а зачем всё это делать !!!!

заменить return 1 на exit(1)