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

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

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

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

+165

// Color and text
// -- Безупречный
if (nScore >= 90)
{
var strText = "Отличный пароль! Главное не забыть его :) ";
var strColor = "#0ca908";
}
// -- Очень хороший
else if (nScore >= 80)
{
var strText = "Очень хороший";
vstrColor = "#7ff67c";
}
// -- Хороший
else if (nScore >= 70)
{
var strText = "Хороший";
var strColor = "#1740ef";
}
// -- Давольно нормальный
else if (nScore >= 60)
{
var strText = "Достаточно неплохо";
var strColor = "#5a74e3";
}
// -- Нормальный
else if (nScore >= 50)
{
var strText = "Нормально";
var strColor = "#e3cb00";
}
// -- Слабый
else if (nScore >= 25)
{
var strText = "Слабенько";
var strColor = "#e7d61a";
}
// -- Очень плохой
else
{
var strText = "Ужас. (qwerty и то лучше :) ) ";
var strColor = "#e71a1a";
}

else if (nScore >= 25)
{
var strText = "Слабенько";
var strColor = "#e7d61a"; // -- Слабенько?WTF???????
}

Взято с блога великого кодера darkoff.ru

Запостил:

BlincAttack, 25 Июля 2010

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

This is obvious 25.07.2010 10:04 # −6

показать все, что скрытоТ.е написав 90 раз одну и ту же букву, по мнению автора, - это отличный пароль ?
Ответить
- Kornew 25.07.2010 12:02 # +1
  
  я лично тут не вижу определения переменной nScore, - мб она получается из функции какой-то, которая выделяет последовательности символов, чот типа такого примера:
  afafafafafafafaf -> af x8 => nScore = 8;
  afeytd as to as jrge -> as x2, +12 => nScore = 14;
  
  вообщем: nScore - это не обязательно длинна строки ;-)
  Ответить
  - This is obvious 25.07.2010 16:50 # 0
    
    Тогда логика автора мне не понятна :3
    Ответить
    - Kornew 25.07.2010 19:20 # −1
      
      логика:
      - юзверь вводить Строку (пароль)
      - некая функция анализирует пароль, по какому-то алгоритму, и выдаёт Оценку: nScore
      - тут представлена градация оценки пароля (отличный, хороший, плохой т.п.), и цвет текста, которым будет написана эта надпись
      
      как такового гавнокода тут не видно....
      - разве что излишние оценки:
      "Отлично",
      "почти отлично",
      "почти почти отлично",
      "чуть лучше чем довольно хорошо",
      "довольно хорошо",
      "чуть хуже чем довольно хорошо",
      "хорошо", ....
      
      ну и цвет, который аффтор выделил коментом: это цвет децкой неожиданности, аффтор этим хотел сказать, что "пароль - гавно!" =Ъ
      Ответить
      - This is obvious 25.07.2010 19:32 # 0
        
        Хотелось бы взглянуть на алгоритм :-D
        Ответить
        
        Stalker 25.07.2010 19:45 # +1
        
        BlincAttack ссылку дал же:
        
        http://darkoff.ru/2010/06/проверка-сложности-пароля-на-javascript/
        Ответить
- TarasB 25.07.2010 13:18 # 0
  
  Где здесь вообще написано про связь nScore и длины строки?
  Ответить
BlincAttack 25.07.2010 18:20 # 0

http://darkoff.ru/2010/06/%D0%BF%D1%80%D0%BE%D0%B2%D0%B5%D1%80%D0% BA%D0%B0-%D1%81%D0%BB%D0%BE%D0%B6%D0%BD%D0%BE%D1% 81%D1%82%D0%B8-%D0%BF%D0%B0%D1%80%D0%BE%D0%BB%D1%8F-%D0%BD%D0%B0-javascript/

вот оригинал, посмотрите чему равна переменная=)
Ответить
- Stalker 25.07.2010 18:43 # 0
  
  Бу-га-га :))) Смайлы в исходнике :)
  А с цветами прикол лично мне не понятен. От балды?
  Ответить
- 3.14159265 26.07.2010 09:38 # +1
  
  ИМХО какая-то тупая логика - я б энтропию попытался определить
  и помимо этого сжал бы его каким-нибудь простым RLE
  да и вообще - там можно упростить логику - сделав массив с весами
  и не писать таких говеных условий
  // -- Letters are all lower case
  if (nUpperCount == 0 && nLowerCount != 0)
  nScore += 10;
  // -- Letters are upper case and lower case
  else if (nUpperCount != 0 && nLowerCount != 0)
  nScore += 20;
  Ответить
  - Stalker 26.07.2010 11:22 # 0
    
    +1 к массиву с весами для условий.
    А вот с энтропией хз. "qwertyyy" и "q1e%tYYY" будут иметь равную энтропию, ведь так?
    Ответить
    - 3.14159265 26.07.2010 12:49 # 0
      
      ну вообще-то я имел ввиду энтропию, с разными вероятностями, (существуют ведь разные виды энтропии. есть навороченые - вероятность с учетом предыдущего символа 2-х,3-х)..
      а-ля хаффман-сжатие с разными весами...
      
      у гугла вроде js-проверка выполняется через аякс и логика лежит на серваке
      
      хотелось бы увидеть нормальную проверку
      а то гуглил и нашел практически идентичиный пример, только с другими весами xD ))
      http://megasoft.3dn.ru/forum/28-1309-1
      Ответить
Анонимус 26.07.2010 01:12 # +1

кто такой дуракофф? очередной вебкил?
Ответить
- BlincAttack 26.07.2010 02:01 # +1
  
  Прямо в точку, это очередной вебкилл ^_^
  Ответить
  - Анонимус 26.07.2010 03:09 # +3
    
    Пожалуй самый жирный высер товарища дуракоффа -- "Нам нужен Delphi компилятор!".
    Ответить
    - Bjarne_Stroustrup 26.07.2010 07:32 # +2
      
      Видимо, его чмырили в детстве:
      
      MessageBox(0,'Введи хп, козел','Ошибка в ДНК!',MB_OK + MB_ICONERROR);
      Ответить
      - 3.14159265 26.07.2010 09:32 # 0
        
        >>>'Введи хп, козел'
        точно очередной
        
        Delphi-кодинг - подумал что Быдло-кодинг )))
        честное программерское
        Ответить
Lure Of Chaos 26.07.2010 15:11 # 0

количество оценок намекает на способ, "холодно-горячо" ))
Ответить
bugmenot 31.07.2010 03:03 # 0
На дельфях писаный каканый автором "билдер" был эпичнее - #3750.
```
Функции для чтения и записи текста в фаил на Delphi 
Вот написал на делфи функции SaveToFile() и LoadFromFile().
```
И вообще высеры в дельфевом разделе доставляют

А этот алгорифм оценки сложности бородатъ.
Ответить

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

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

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

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