1. PHP / Говнокод #470

    +151

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    <...>
    switch($action) {
    case 'postAjouterDocument' :
    	$titre = read_post_str('text1');
    	$files['fichier'] = isset($_FILES['text2'])?$_FILES['text2']:'';
    	$codeRet = $ClsDocument->Add(GUSERID, $files, $titre, GROUPE );
    	location(PAGE_DOC_DEFAUT."&codeRet=".$codeRet);
    	exit();
    	break;
    case 'postAjouterCommentaire' :
    	<...>
    	exit();
    	break;
    case 'postModifierCommentaire' :
    	<...>
    	exit();
    	break;
    }

    зачем break; после exit(); ?

    Запостил: guest, 27 Января 2009

    Комментарии (5) RSS

    • USED:
      <?php

      /*
      много много кода
      */

      switch ( $error )
      {
      case 'error1':
      echo 'Ошибка 1';
      exit;
      break;
      case 'error2':
      echo 'Ошибка 2';
      exit;
      break;
      default:
      echo 'Нет ашипки';
      break;
      }

      /*
      а тут дальнейшее многострочное продолжение, выполнение которого, очевидно, нужно было избежать
      */

      ?>

      (c) Говнокодеры CREW
      Ответить
    • Угу:
      [quote=(c) Говнокодеры CREW][/quote]

      <?php echo 'test'; ?>


      читайте правила...
      )))
      Ответить
    • [quote=USED]а тут дальнейшее многострочное продолжение, выполнение которого, очевидно, нужно было избежать[/quote]Автор имел ввиду "зачем после exit() ставить break, если все равно после этого выполнение кода завершается" в смысле, убирай не убирай break - на твою нежеланную строку все равно не попадешь..
      [quote]зачем break; после exit(); ?[/quote]Может привычка из c...
      Ответить
    • Давид:
      преступления он не совершил, а конструкцию свитча как положено сохранил , хуже если кто то забывает про дефаулт или про брейк
      Ответить
    • Насколько я помню, по стандарту оформления кода PEAR, break; нужно оставлять всегда, даже вот в таких вот ситуациях. Так что здесь всё правильно.
      Ответить

    Добавить комментарий