Така де - или кода е идиотски или аз съм идиот

Съдържаниет е скрито
Влезте за да го видите

Съдържаниет е скрито
Влезте за да го видите

 

  while (!done && ((res = ast_waitfor(chan, -1)) > -1) && fs && (f = ast_read(chan))) {

   if (digit) {

     .....

    }

    if ((f->frametype == AST_FRAME_DTMF)) {

      int got = 1;

      switch(f->subclass) {

      case '1':

        ast_set_flag(&flags, DFLAG_PAUSE);

        mode = DMODE_PLAY;

      break;

      ....

      default:

        got = 0;

     }

     if (!got) {

       switch(f->subclass) {

       case '#':

          done = 1;

       continue;

       break; ////Този break имам предвид

...........

Съдържаниет е скрито
Влезте за да го видите

се си мислех, че съм виждал идиотски код, ама тва надхвърли очакванията ми. не е въпроса какво прави break, а по-скоро какво прави continue в switch конструкция, щото той си е само за цикли.

дори го пробвах това. GCC-то плюе: error: continue statement not within a loop

continue e за while цикъла, а break e за switch-a

иначе кода ми се вижда малко "индийски", ама явно не си дал сичко

в случая break-a е по-скоро като coding standard, не че е нужен, но е полезен

Съдържаниет е скрито
Влезте за да го видите

да прав си ся го видях. при скролирането на екрана не съм видял while. пак и бирата си е спестил скоби

Съдържаниет е скрито
Влезте за да го видите

Немате навик да гледате dead listings маму деба...

Съдържаниет е скрито
Влезте за да го видите

Съдържаниет е скрито
Влезте за да го видите

кодът е писан от млад индийски хакер сигурно :)

Не си гледал достатъчно идиотски код, очевидно.

Съдържаниет е скрито
Влезте за да го видите

Съдържаниет е скрито
Влезте за да го видите

Ти явно си минал през чичко Джонсън ама нищо не си научил.

1) Трябва да се прави разлика между дефект и проявление на дефекта.
2) Трябва да се оправя дефекта, а не проявлението му.

В случая дефекта е в continue, а проявлението му е в недостижимия break. Доста компилатори ще изпсуват както каза eddhunter.

Ако някой в тази ситуация махне break-а ще ида ще го наритам и ще му изнеса едночасова лекция как трябва да се пише код. Кода без break-a става направо опасен, защото утре някой друг ще реши да махне и continue-то и гледай какво става тогава.

Това continue най-добре да го изнесат зад switch-а в един:
if (1 == done)
{
continue;
}

Мда, може да реши да махне continue-то и да не му е нужен break-a също така, не мислиш ли? Може да иска да мине и през следващото условие.
Явно работиш в споменатата фирма по-дълго, отколкото е здравословно.