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

    +161

    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
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    function setOptionText(the_select1, the_array1, the_select2, the_array2)
{
    the_select1.options.length=the_array1.length;
    for (loop=0; loop < the_array1.length; loop++)
    {
        the_select1.options[loop].text = the_array1[loop];
        the_select1.options[loop].value = the_array1[loop];
        if (loop==0){
            the_select1.options[loop].selected=true;
        }    
    }

    the_select2.options.length=the_array2.length;
    for (loop=0; loop < the_array2.length; loop++)
    {
        the_select2.options[loop].text = the_array2[loop];
        the_select2.options[loop].value = the_array2[loop];
        if (loop==0){
            the_select2.options[loop].selected=true;
        }
    }
}

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

    Запостил: wissenstein, 14 Декабря 2012

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

    • ava guest 14.12.2012 11:10 # 0

      Ха..
      Ответить

    • ava wvxvw 14.12.2012 12:52 # +7

      За глобальную переменную loop тоже можно было автору сделать что-нибудь хорошее.
      Ответить

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