Говнокод #6303 — JavaScript

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

JavaScript / Говнокод #6303

+165

function securityMode(c_f)                                                                                                                                   
{                                                                                                                                                            
        var security_mode;

        changed = c_f;

        hideWep();

        document.getElementById("div_security_shared_mode").style.visibility = "hidden";
        document.getElementById("div_security_shared_mode").style.display = "none";
        document.getElementById("div_wpa").style.visibility = "hidden";
        document.getElementById("div_wpa").style.display = "none";
        document.getElementById("div_wpa_algorithms").style.visibility = "hidden";
        document.getElementById("div_wpa_algorithms").style.display = "none";
        document.getElementById("wpa_passphrase").style.visibility = "hidden";
        document.getElementById("wpa_passphrase").style.display = "none";
        document.getElementById("wpa_key_renewal_interval").style.visibility = "hidden";
        document.getElementById("wpa_key_renewal_interval").style.display = "none";
        document.getElementById("wpa_PMK_Cache_Period").style.visibility = "hidden";
        document.getElementById("wpa_PMK_Cache_Period").style.display = "none";
        document.getElementById("wpa_preAuthentication").style.visibility = "hidden";
        document.getElementById("wpa_preAuthentication").style.display = "none";
        document.security_form.cipher[0].disabled = true;
        document.security_form.cipher[1].disabled = true;
        document.security_form.cipher[2].disabled = true;

        // etc...                                                                                                                                                           
}

Китайский говнокод, который пока даже страшно рефакторить.

Запостил:

SadKo, 10 Апреля 2011

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

Lure Of Chaos 10.04.2011 18:27 # 0

не бойся, товарищ! "Садко" непотопляем!
Ответить

wvxvw 10.04.2011 21:37 # +4

Я бы сделал так:

document.getElementById("div_security_shared_mode").style.visibility = "hidden";
document.getlEementById("div_security_shared_mode").style.display = "none";
document.getleEmentById("div_wpa").style.visibility = "hidden";
document.getlemEentById("div_wpa").style.display = "none";
document.getlemeEntById("div_wpa_algorithms").style.visibility = "hidden";
document.getlemenEtById("div_wpa_algorithms").style.display = "none";
document.getlementEById("wpa_passphrase").style.visibility = "hidden";
document.getlementBEyId("wpa_passphrase").style.display = "none";
document.getlementByEId("wpa_key_renewal_interval").style.visibility = "hidden";
document.getlementByIEd("wpa_key_renewal_interval").style.display = "none";
document.getlementByIdE("wpa_PMK_Cache_Period").style.visibility = "hidden";
document.getlementyBIdE("wpa_PMK_Cache_Period").style.display = "none";
document.getlementyIBdE("wpa_preAuthentication").style.visibility = "hidden";
document.getlementyIdBE("wpa_preAuthentication").style.display = "none";

по-моему так красивше.

Ответить

SadKo 10.04.2011 22:12 # +1

А если какой-нить div_wpa в одной сборке есть, а в другой его нет :)?
Ответить
- inkanus-gray 10.04.2011 23:36 # 0
  
  А метод getlementyIdBE класса document нисколько не удивляет?
  Ответить
  - SadKo 10.04.2011 23:57 # 0
    
    Хе, а точно. Хорошо прикололся.
    Ответить

inkanus-gray 11.04.2011 00:10 # +1

function MakeMeUnseenThis (x)
{ 
if (var i = document.getElementById(x)) {
  if (i.style) {
    i.style.visibility = "hidden";
    i.style.display = "none";
    i.style.opacity = "0.0";
    i.style.color = "white";
    i.style.background.color = "white";
    if (i.style.zoom)
      i.style.zoom = "1";
  } else {
    alert("Нежданчик!");
  }
} else {
  alert("Не нашёл аднака!");
}
}

function securityMode(c_f)                                                                                                                                   
{                                                                                                                                                            
        var security_mode;

        changed = c_f;

        hideWep();

        MakeMeUnseenThis("div_security_shared_mode");
        MakeMeUnseenThis("div_wpa");
        MakeMeUnseenThis("div_wpa_algorithms");
        MakeMeUnseenThis("wpa_passphrase");
        MakeMeUnseenThis("wpa_key_renewal_interval");
        MakeMeUnseenThis("wpa_PMK_Cache_Period");
        MakeMeUnseenThis("wpa_preAuthentication");
        document.security_form.cipher[0].disabled = true;
        document.security_form.cipher[1].disabled = true;
        document.security_form.cipher[2].disabled = true;

        // etc...                                                                                                                                                           
}

Ответить

bugmenot 11.04.2011 00:42 # 0

да ладно, этож el cheapo(r)(tm) точка доступа, она всё равно не стоит времени, затраченного на настройку
Ответить
- SadKo 11.04.2011 07:34 # 0
  
  Вообще, эта прошива подойдёт не к одному роутеру.
  Ответить
  - bugmenot 11.04.2011 07:57 # +1
    
    > не к о
    ня! я нихуя не понял
    сама идея конфигурирования железок из httpd порочна
    Ответить
    - eth0 11.04.2011 08:54 # 0
      
      telnet-only?
      Ответить
      - bugmenot 11.04.2011 09:32 # +1
        
        лучше бы ssh, для in-band то
        Ответить
    - SadKo 11.04.2011 11:25 # 0
      
      Объясните это хомякам.
      Ответить
      - bugmenot 11.04.2011 12:54 # 0
        
        Зачем объяснять? Всего лишь нужен нормальный шелл и набор типовых шаблонов конфигов. Обезьянка видит, обезьянка делает.
        Ответить
        
        GhOsTMZ 11.04.2011 13:27 # 0
        
        Помнится была такая практика у Zyxel, когда не было веб-морды, а был телнет-доступ, в которой была достаточно удобная менюшка. А результат какой? Куча слез от владельцев этих железов и море счастья от появления NetFriend.
        Ответить
        
        bugmenot 11.04.2011 14:37 # 0
        
        Зухель, пфф, сами виноваты, если сделали такие девайсы, которые надо ежедневно в консоль дрочить
        А вот фронтенд это уже правильный, масштабируемый подход
        
        что-то мне подсказывает, что у зухеля даже удобная менюшка была вырвиглазной
        http://bit.ly/aironet
        Ответить

Говнокод: по колено в коде.

JavaScript / Говнокод #6303

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

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