Метки, для упрощения структуры программы

[wandererindreams]

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

#include <stdio.h>
#include <conio.h>
int read_number(int &number);                            //считывает максимальное число для составления рядов Лэнгфорда
int make_series(const FILE* f_series, const int number, const int &number_of_series = 3);  //находит ряды Лэнгфорда

int output_series(const FILE* f_series);                 //выводит на экран ряды Лэнгфорда, записанные в файле f_series
void erors(const &error);                                //обрабатывает ошибки (угаданные заранее), возникающие в программе
//---------------------------------------------------------------
/*
 error    - номер ошибки (0 - ошибок нет, работаем дальше)
 number   - максимальное число для составления ряда Лэнгфорд (его вводит пользователь)
 char0    - вспомогатеьная переменная
 f_series - файл, куда записываются найденные ряды Лэнгфорда
*/
main()
{
 int error = 0,
     number = 0,
     number_of_series = 2;
 char char0 = 'n';
 FILE* f_series = NULL;

 clrscr();
 printf("Данная программа находит последовательности Ленгфорда для любого целого числа\n");
 printf("от 3 до 12 с условием, что последовательность должна начинаться с наибольшего\nчисла.\n");

 f_series = fopen("\\series", "w");

 do                                              //цикл, пока не будет ошибки или пользователь не захочет закончить работу
 {
  error = read_number(number);                   //пользователь вводит максимальное число Лэнгфорда
  if(error) goto m_error;
  error = make_series(f_series, number); //находятся ряды Лэнгфорда, начинающиеся с масимального числа number
  if(error) goto m_error;
  error = output_series(f_series);
  if(error) goto m_error;
  printf("Завершить работу программы? (y/)n: ");
  char0 = getchar();
  while(getchar() != '\n');
 }
 while(char0 != 'y');

 m_error:                                        //выход сюда, если возникла ошибка
 if (error) erors(error);
 getch();
}
//---------------------------------------------------------------

Смысл в том, что я хочу знать нормально ли выполнила функция свою работу, или нет, и если нет, то прекратить выполнение программы. Но если убрать метки, то такое можно сделать только вложенными условиями, которые выглядят не очень читабельно даже в такой небольшой программе. Вопрос: может такой вариант использования меток вполне приемлем? Если нет, то почему и как получше переписать программу.

[veniside]

> ее читаемость будет гораздо хуже

куда уж хуже )
У вас же есть уже цикл, чем break не устраивает?

do                                              //цикл, пока не будет ошибки или пользователь не захочет закончить работу
{
    if (error = read_number(number)) break;                   //пользователь вводит максимальное число Лэнгфорда
    //
    if (error = make_series(f_series, number)) break; //находятся ряды Лэнгфорда, начинающиеся с масимального числа number
    //
    if (error = output_series(f_series)) break;
    //
    printf("Завершить работу программы? (y/n): ");
    char0 = getchar();
    //
    while (getchar() != '\n') ;
}
while(char0 != 'y');

if (error) erors(error);

 m_error:                                        //выход сюда, если возникла ошибка

сюда он попадёт и при обычном выходе из цикла (по нажатию 'y'). Видите, уже сами запутались с этими метками. Фтопку их )

[wandererindreams]

У вас же есть уже цикл, чем break не устраивает?

Согласен, если это законченная программа, то да, но стоит мне хоть что-то добавить после цикла do/while и break передаст управление НЕ в конец программы.

сюда он попадёт и при обычном выходе из цикла (по нажатию 'y'). Видите, уже сами запутались с этими метками.

Я не запутался. До обычного нажатия 'y' будут выполнены все функции цикла, а я хочу выйти досрочно, без выполнения, если например чтение данных вызвало ошибку.

[G-Kyller]

Есть специальные операторы try-catch которые отлавливают ошибки, т.е. пока все хорошо выполняется TRY{}, как только появляется ошибка(в блоке try), программа переходит в CATCH{}.
Таким образом можно отлавливать все ошибки, либо создать свои типы ошибок и отлавливать их, добавляя разные действия на разные ошибки.
P.S. Читабельность при этом никак не ухудшается, а наоборот.

[wandererindreams]

Сообщение от G-Kyller

Есть специальные операторы try-catch которые отлавливают ошибки, т.е. пока все хорошо выполняется TRY{}, как только появляется ошибка(в блоке try), программа переходит в CATCH{}.

Поискал в интернете. Если правильно понял, то операторы try-catch введены в С++, мне же нужно написать именно на С.

[veniside]

> break передаст управление НЕ в конец программы

ну и что? у вас же там error всё-равно проверяется:

if (error) {
  // всё, приплыли
}
else {
  // кажется, пронесло
}

> Я не запутался

я имел в виду, что вот этот комментарий:

//выход сюда, если возникла ошибка

не соответсвует действительности.

[wandererindreams]

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

> break передаст управление НЕ в конец программы

ну и что? у вас же там error всё-равно проверяется:

if (error) {
  // всё, приплыли
}
else {
  // кажется, пронесло
}

Вы предлагаете после цикла написать if-else от которого я и пытаюсь избавиться, но в любом случае я могу после цикла написать переход по метке в случае ошибки, так что от меток в цикле в любом случае вы меня избавили =).
У меня правда опыта мало, может получиться избежать глубокого вложения условий, буду пытаться. А на основной вопрос пока никто не ответил: использование меток, как переход в конец программы при возникновении ошибки, является плохим, плохо модифицируемым способом или нет? Вроде нет, тут они работают вроде оператора break. Можно и сам break использовать, но тогда будет полный выход из main, без захода в функцию errors(), которая подскажет где именно произошла ошибка.

[Utkin]

А почему Вы пытаетесь уйти от if-then-else? Как раз в сравнении с метками это основной и вполне логичный механизм управления логикой вычислений.

[wandererindreams]

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

А почему Вы пытаетесь уйти от if-then-else? Как раз в сравнении с метками это основной и вполне логичный механизм управления логикой вычислений.

Потому, что первый вариант читается лучше, чем второй.

 {
  if(error = read_number(number)) goto m_error;
  if(error = make_series(f_series, number)) goto m_error;
  if(error = output_series(f_series)) goto m_error;
  ......
  m_error: errors();
 }

 {
  if( !(error = read_number(number)) ) {
   if( !(error = make_series(f_series, number)) )  {
    if( !(error = output_series(f_series)) ) {
    } else errors(); break;
   } else errors(); break;
  } else errors(); break;
 }

Я как-то написал не очень большую программу (в турбо паскале), где тело условия занимало больше одного экрана, а учитывая что условий несколько, было не удобно высчитывать к какому if относится else (конечно комментарии в помощь, но все же). А в первом варианте та же линейная структура, более компактная и без лишнего нагромождения скобок.
Может конечно я плохо продумал программу, но в любом случае первый вариант смотрится удобнее второго.

[Utkin]

Есть еще swith-case...