"PHP" / Говнокод #470 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 guest, (Updated )

Комментарии (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; нужно оставлять всегда, даже вот в таких вот ситуациях. Так что здесь всё правильно.
    Ответить

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

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?


    8