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

    +5

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    <?
if($login=='')
{
   global $APPLICATION;
   $APPLICATION->ThrowException('Имя входа должно быть заполнено.'); 
   return false;
}

// Получить последнее исключение можно методом 
$APPLICATION->GetException()
?>

    Bitrix. Лучшие стандарты веб-разработки и новое ядро! Создатели логики try-catch-throw наверное перевернулись бы 100500 раз в гробу увидев такую "реализацию". Пруф как "нужно делать": https://dev.1c-bitrix.ru/api_help/main/reference/cmain/throwexception.php

    Запостил: zenn1989, 08 Июня 2016

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

    • ava inkanus-gray 08.06.2016 18:12 # 0

      Из документации трудно понять, как это должно работать. По сути это не исключение, а просто отправка сообщения об ошибке, которое потом можно забрать и проанализировать? Что-то типа этого:
      https://habrahabr.ru/company/piter/blog/270331/

      Или я ошибся?
      Ответить

      • ava zenn1989 08.06.2016 19:15 # +2

        В битриксе вообще сложно что-либо понять ... Скорей всего вы правы, но почему не сделать это по-человечески:
        try {
    // инициируй там че хочешь
    $instance = new SomeClass($params);
    $instance->run();
} catch (Exception $e) {
    // лови и обрабатывай свои сообщения из ошибок ... 
    $msg = $e->getMessage();
}
... 
class SomeClass
{
    private $data;
    public function __construct($data)
    {
        $this->data = $data;
    }

    public function run()
    {
        // выбрасывай ошибки с сообщениями
        throw new Exception('Какое то там сообщение об ошибке');
    }
}
        Ответить

    • ava gost 08.06.2016 20:08 # +5

      > $APPLICATION->GetException()
      Повеяло запашком GetLastError().
      Ответить

      • ava inkanus-gray 08.06.2016 20:29 # 0

        И главное, что только Last. Предпоследнюю ошибку мы вытащить не сможем.
        Ответить

        • ava meinf 08.06.2016 21:11 # +1

          ну для кодовозвратной идеологии это норма, по дефолту есть только последняя ошибка, которую и нужно обрабатывать, предпоследняя ошибка не нужна
          Ответить

          • ava inkanus-gray 08.06.2016 22:01 # +1

            Объект $APPLICATION у нас глобальный. Допустим, у нас есть такой фрагмент кода:
            $y = FirstFunction() + SecondFunction();

            Сгенерировать ошибку могут обе функции, а проверить мы сможем только одну.
            Ответить

            • ava meinf 08.06.2016 22:34 # +3

              ну тут ошибка в том, что в качестве яп выбрано пхп что нужно использовать исключения, а не проверять последнюю ошибку

              изначально речь была про GetLastError, а там где есть этот самый GetLastError было бы что то типа
              x = FirstFunction();
if (x == -1)
    return -1;
y = SecondFunction();
if (y == -1)
    return -1;
y +=x;

              то, что в принципе оба варианта не очень, это уже другая история
              Ответить

          • ava LispGovno 08.06.2016 23:06 # +4

            Забавно что обычно как раз важна первая ошибка, а не последняя в цепочке неделимых действий
            Ответить

            • ava meinf 08.06.2016 23:32 # 0

              пример "цепочки неделимых действий"?
              Ответить

              • ava gost 09.06.2016 06:57 # +5

                Подозреваю, что LispGovno имел ввиду что-то наподобие такого:
                FILE *f = fopen(...);
fread(f, ...);
fclose(f);

                Очевидно, что если у нас зафейлится fopen, нам нужна именно ошибка fopen'a, а не всякие там "Bad descriptor" от fclose.
                Ответить

                • ava Dummy00001 09.06.2016 12:41 # +3

                  да. это частая ошибка в винапи: сискалл обламывается, но GetLastError() никакой ошибки не указывает/дает кривой код. потому что внутри сискалл, винапи ошибку обрабатывает, и код обработки затирает нужную "последнюю ошибку".

                  теже яйца на линухах - но там сискаллов меньше и они глубже тестируются. поэтому на такие ошибки еще не спотыкался. (по правде говоря - на виндах сам не спотыкался, а коллеги.)
                  Ответить

                • ava dxd 09.06.2016 12:56 # 0

                  Надо стэк кодов ошибок держать на такой случай.
                  Ответить

                • ava meinf 09.06.2016 13:03 # 0

                  ага, и что не так?
                  https://snag.gy/9jMduR.jpg
                  Ответить

                  • ava gost 09.06.2016 13:05 # 0

                    >> "цепочки неделимых действий"
                    Dummy00001 выше уже более корректный пример привёл.
                    Ответить

                    • ava meinf 09.06.2016 13:23 # +1

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

                      • ava gost 09.06.2016 13:35 # 0

                        > не проверять успешность fopen в своем коде это ссзб, а не "цепочка неделимых действий"
                        Что вы привязались к моему fopen? Это - концепт, иллюстрация. Никто и не говорит, что fopen-fread-fclose - это неделимая цепочка.
                        Ответить

                        • ava meinf 09.06.2016 16:40 # 0

                          ну я просто попросил конкретный пример, хотя вот сейчас что то вспомнилось, что при использовании OpenSSL вроде бы сталкивался с чем то подобным
                          Ответить

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