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

    • ava guest 28.01.2009 00:28 # 0

      USED:
      <?php

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

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

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

      ?>

      (c) Говнокодеры CREW
      Ответить

    • ava guest 28.01.2009 17:19 # 0

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

      <?php echo 'test'; ?>


      читайте правила...
      )))
      Ответить

    • ava guest 28.01.2009 18:06 # 0

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

    • ava guest 30.01.2009 23:59 # 0

      Давид:
      преступления он не совершил, а конструкцию свитча как положено сохранил , хуже если кто то забывает про дефаулт или про брейк
      Ответить

    • ava guest 02.02.2009 14:47 # 0

      Насколько я помню, по стандарту оформления кода PEAR, break; нужно оставлять всегда, даже вот в таких вот ситуациях. Так что здесь всё правильно.
      Ответить

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