1. JavaScript / Говнокод #11802

    +168

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    function confirmDialog(sMessage) {
    if (confirm(sMessage)) {
        return true;
    } else {
        return false;
    }
}

    Запостил: Lokich, 19 Сентября 2012

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

    • ava Vindicar 19.09.2012 17:40 # 0

      Заглушка на будущее, когда будет своя реализация диалога подтверждения в оверлее?
      Ответить

      • ava aikikode 19.09.2012 18:05 # 0

        Даже если и заглушка, можно было бы написать:
        function confirmDialog(sMessage) {
    return confirm(sMessage);
}
        Ответить

        • ava Steve_Brown 19.09.2012 18:30 # +1

          Заглушка на будущее, когда будут выдаваться дополнительные сообщения?
          } else {
        message_box('ну как хотите');
        return false;
    }
          Ответить

      • ava wvxvw 19.09.2012 19:51 # 0

        Заглушка на тот случай, если и когда появится ХТМЛ6, а в нем можно будет самому создавать диалоговые окна.
        Ответить

      • ava хуита 20.09.2012 06:59 # +2

        Было десяток раз, пинус.
        >Заглушка на будущее, когда будет своя реализация диалога подтверждения в оверлее?
        Хреновая заглушка, true js way было бы передача в качестве ргц каллбак функций onOk и onCancle
        Ответить

      • ava Lokich 20.09.2012 14:16 # 0

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

    • ava Lure Of Chaos 19.09.2012 19:31 # 0

      а может confirm челу не нравится что коротко и непонятно что подтверждает )
      Ответить

    • ava psycho-coder 19.09.2012 21:19 # 0

      Понты?
      Из разряда: "А я сделаю свой confirm! С блек-джеком и ...".
      Ответить

      • ava Stud 02.10.2012 21:45 # 0

        А ты рекомендуешь использовать дефолтные?
        Ответить

        • ava psycho-coder 03.10.2012 09:56 # 0

          Да. Потому что не вижу смысла в этой обертке.
          Ответить

          • ava Stud 03.10.2012 10:35 # 0

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

            • ava psycho-coder 03.10.2012 10:40 # 0

              Если так, то согласен. Но все равно, обычно в таких ситуациях юзают jqui.
              Ответить

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