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

    +158

    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
    function ShowContent(divId, ajaxIndex, url) {
            if (AjaxObjects[ajaxIndex].response != "ok") 
            {
                alert("Неверный пароль");
                /*
                document.getElementById(divId).innerHTML = AjaxObjects[ajaxIndex].response;
                if (enableCache) {
                    jsCache[url] = AjaxObjects[ajaxIndex].response;
                }*/
                AjaxObjects[ajaxIndex] = false;
            }
            else
                window.location.href= "Default.aspx";
        }

    дурацкая функция, ну кто так обозначает сообщение, надо более полно обозначивать в чем причина непереадресации на нужную мне страницу

    Запостил: BlackMonolit, 14 Апреля 2011

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

    • ava guest 15.04.2011 14:16 # 0

      Если для IF...ELSE указаны обе части, то лучше ставить сравнение по "==", а не "!=" и если в первой части использовались скобки, то и х необходимо ставить и для воторй части.
      if(AjaxObjects[ajaxIndex].response == "ok"){
                window.location.href= "Default.aspx";
}else{
                alert("Неверный пароль");
                AjaxObjects[ajaxIndex] = false;
}
      Ответить

      • ava Surendil 17.04.2011 08:59 # 0

        Если считать нормальной ситуацией "пароль не подходит", то лучше оставить всё как есть ^^ А про скобки... что же, это можно объяснить экономией трафика в два байта )

        Сообщение может и нормальное, всё ясно, а вот то, что оно модальное окно, уже не катит (
        Ответить

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