1. Java / Говнокод #7861

    +93

    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
    15. 15
    16. 16
    17. 17
    <%if(myaccount.getCountry().equals( "AF" )){%> <font color="grey"><label for="country">Afghanistan</label></font>
    	<%}else if(myaccount.getCountry().equals( "AL" )){%> <font color="grey"><label for="country">Albania</label></font>
    	<%}else if(myaccount.getCountry().equals( "DZ" )){%> <font color="grey"><label for="country">Algeria</label></font>
    	<%}else if(myaccount.getCountry().equals( "AS" )){%> <font color="grey"><label for="country">American Samoa</label></font>
    	<%}else if(myaccount.getCountry().equals( "AD" )){%> <font color="grey"><label for="country">Andorra</label></font>
    	<%}else if(myaccount.getCountry().equals( "AO" )){%> <font color="grey"><label for="country">Angola</label></font>
    	<%}else if(myaccount.getCountry().equals( "AI" )){%> <font color="grey"><label for="country">Anguilla</label></font>
    	<%}else if(myaccount.getCountry().equals( "AQ" )){%> <font color="grey"><label for="country">Antarctica</label></font>
    	<%}else if(myaccount.getCountry().equals( "AG" )){%> <font color="grey"><label for="country">Antigua and Barbuda</label></font>
    	<%}else if(myaccount.getCountry().equals( "AR" )){%> <font color="grey"><label for="country">Argentina</label></font>
    	<%}else if(myaccount.getCountry().equals( "AM" )){%> <font color="grey"><label for="country">Armenia</label></font>
    	<%}else if(myaccount.getCountry().equals( "AW" )){%> <font color="grey"><label for="country">Aruba</label></font>
    	<%}else if(myaccount.getCountry().equals( "AU" )){%> <font color="grey"><label for="country">Australia</label></font>
    	<%}else if(myaccount.getCountry().equals( "AT" )){%> <font color="grey"><label for="country">Austria</label></font>
    	<%}else if(myaccount.getCountry().equals( "AZ" )){%> <font color="grey"><label for="country">Azerbaijan</label></font>
    	<%}else if(myaccount.getCountry().equals( "BS" )){%> <font color="grey"><label for="country">Bahamas</label></font>
        <%}else if ...

    Индусская проверка принадлежности аккаунта к определённой стране...

    P.S. Всё не влезло - стран у нас много =)

    Запостил: vetal, 14 Сентября 2011

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

    • ava SmackMyBitchUp 14.09.2011 17:24 # +1

      Они не то, чтобы в JSP не шарят. Вот почему бы не реализовать еще один метод, который возвращает полное название страны?
      Дак нет же, лепим. А, главное, что скорее всего лепит это все то быдло, которое в пхп пируэты любит выделывать.
      Ответить

    • ava roman-kashitsyn 14.09.2011 17:30 # +2

      вызовы любых методов в jsp меня обычно настораживают (если код не 8-летней давности)...
      Ответить

      • ava Lure Of Chaos 14.09.2011 18:05 # 0

        вьюшка не должна просить сгенерить ей данные?
        Ответить

      • ava SmackMyBitchUp 14.09.2011 18:09 # 0

        <label for="country"><%= myaccount.getCountry()%> </label>
        Так можно?
        Ответить

        • ava vetal 14.09.2011 18:12 # 0

          <label for="country"><%= myaccount.getCountry()%> </label> - так то можно, но getCountry() у них возвращает сокращенное название страны, типа "AF", "AL" и т.д... поэтому и 230 IF'ов =)
          Ответить

          • ava inkanus-gray 14.09.2011 18:27 # 0 

            <label for="country"><%= shortNameToLongName(myaccount.getCountry())%> </label>
            где shortNameToLongName() — функция, расшифровывающая название. А так?
            Или даже так: 
            <label for="country"><%= countryNames[myaccount.getCountry()]%> </label>
            где countryNames[] — массив полных названий.

            Скрипач 230 ифов на странице не нужны.
            Ответить

            • ava vetal 14.09.2011 18:37 # +2

              Понятно, что 230 ифов на странице не нужны.. поэтому этот код и на ГовноКоде :D
              Ответить

              • ava SmackMyBitchUp 14.09.2011 19:34 # +1

                Самое интересное - человек скорее всего даже не засомневался в своих действиях))
                Ответить

                • ava Lure Of Chaos 14.09.2011 21:19 # +5

                  еще и матерился, мол, какая скучная, тупая работа программера - тыщу раз нажимать Cut и еще тыщу Paste
                  Ответить

        • ava roman-kashitsyn 14.09.2011 18:55 # 0 

          <label for="country><c:out value="${myaccount.country}"/></label>
          если в country сокращение, то нужно занести в контроллере в атрибуты полное название и будет 
          <label for="country><c:out value="${fullCountryName}"/></label>
          Ответить

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