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

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

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

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