Говнокод #8788 — C++ — Говнокод.ру

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

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

C++ / Говнокод #8788

+1006

const unsigned BAD_VALUE = (unsigned) -1;
typedef pair<string, unsigned> RomeDecPair;
typedef map<string, unsigned> Table;
Table g_table;
 
void InitTables()
{
  g_table["I"] = 1;
  g_table["II"] = 2;
  g_table["III"] = 3;
  g_table["IV"] = 4;
  g_table["V"] = 5;
  g_table["VI"] = 6;
  g_table["VII"] = 7;
  g_table["VIII"] = 8;
  g_table["IX"] = 9;
  g_table["X"] = 10;
  g_table["XX"] = 20;
  g_table["XXX"] = 30;
  g_table["XL"] = 40;
  g_table["L"] = 50;
  g_table["XC"] = 90;
  g_table["C"] = 100;
  g_table["CC"] = 200;
  g_table["CCC"] = 300;
  g_table["CD"] = 400;
  g_table["D"] = 500;
  g_table["CM"] = 900;
  g_table["M"] = 1000;
  g_table["MM"] = 2000;
  g_table["MMM"] = 3000;
}
.....
int main()
{
  InitTables();
 
  vector<RomeDecPair> tests;
 
  tests.push_back(make_pair("I", 1));
  tests.push_back(make_pair("II", 2));
  tests.push_back(make_pair("III", 3));
  tests.push_back(make_pair("IV", 4));
  tests.push_back(make_pair("V", 5));
  tests.push_back(make_pair("VI", 6));
  tests.push_back(make_pair("VII", 7));
  tests.push_back(make_pair("VIII", 8));
  tests.push_back(make_pair("IX", 9));
  tests.push_back(make_pair("X", 10));
  tests.push_back(make_pair("XI", 11));
  tests.push_back(make_pair("XII", 12));
  tests.push_back(make_pair("XIII", 13));
  tests.push_back(make_pair("XIV", 14));
  tests.push_back(make_pair("XV", 15));
  tests.push_back(make_pair("XVI", 16));
  tests.push_back(make_pair("XVII", 17));
  tests.push_back(make_pair("XVIII", 18));
  tests.push_back(make_pair("XIX", 19));
  tests.push_back(make_pair("XX", 20));
  tests.push_back(make_pair("XXI", 21));
  tests.push_back(make_pair("XL", 40));
  tests.push_back(make_pair("XLII", 42));
  tests.push_back(make_pair("LIX", 59));
  tests.push_back(make_pair("LXXVII", 77));
  tests.push_back(make_pair("XC", 90));
  tests.push_back(make_pair("CX", 110));
  tests.push_back(make_pair("CDXCIX", 499));
  tests.push_back(make_pair("DLXXXIII", 583));
  tests.push_back(make_pair("DCCCLXXXVIII", 888));
  tests.push_back(make_pair("MDCLXVIII", 1668));
  tests.push_back(make_pair("MCMLXXXIX", 1989));
  tests.push_back(make_pair("MMMCMXCIX", 3999));
 
  tests.push_back(make_pair("", BAD_VALUE));
  tests.push_back(make_pair("IIIV", BAD_VALUE));
  tests.push_back(make_pair("IIV", BAD_VALUE));
  tests.push_back(make_pair("IIII", BAD_VALUE));
  tests.push_back(make_pair("IIX", BAD_VALUE));
  tests.push_back(make_pair("XIIII", BAD_VALUE));
  tests.push_back(make_pair("XIIIIX", BAD_VALUE));
  tests.push_back(make_pair("XIIIIX", BAD_VALUE));
  tests.push_back(make_pair("XXXX", BAD_VALUE));
  tests.push_back(make_pair("LL", BAD_VALUE));
  tests.push_back(make_pair("CLC", BAD_VALUE));
  tests.push_back(make_pair("CLL", BAD_VALUE));
  tests.push_back(make_pair("DLD", BAD_VALUE));
  tests.push_back(make_pair("LDD", BAD_VALUE));
  tests.push_back(make_pair("LLI", BAD_VALUE));
  tests.push_back(make_pair("MMMCMXCX", BAD_VALUE));
  tests.push_back(make_pair("AXX", BAD_VALUE));
  tests.push_back(make_pair("LXA", BAD_VALUE));
 
  for (vector<RomeDecPair>::const_iterator it = tests.begin(), end = tests.end(); it != end; ++it)
  {
    CheckConversion(*it);
  }
 
  return 0;
}

Запостил:

bugaga, 10 Декабря 2011

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

gegMOPO4 10.12.2011 13:58 # 0

И где здесь бугага?
Ответить
- bugaga 10.12.2011 16:33 # +1
  
  не заметил?
  Ответить
  - gegMOPO4 10.12.2011 22:38 # 0
    
    Много букв.
    
    Так какая причина удержит меня от установки минуса?
    Ответить
    - lucidfoxGovno 10.12.2011 23:20 # −2
      
      > какая причина удержит меня от установки минуса
      По3иТив4иК
      Ответить
    - bugaga 10.12.2011 23:54 # 0
      
      откуда минус? это установка старшего бита целого числа в единицу, которое будет вычисляться посредством прямого кода вместо дополнительного по замыслу афтара, но суть не только в этом. Сам подход к решению и вообще программированию убивает. Начнем с того что данная задача (перевод из римской системы в арабскую) по определению должна решаться посредством стека при помощи алгоритма ОПН + обработка исключений аля IIIV. Говнокодер так просто пользуется списками и мапами и в то же время не знает о существовании std::stack и вообще понятия алгоритм.
      Ответить
      - gegMOPO4 11.12.2011 00:14 # +1
        
        Самой-то реализации алгоритма нет.
        Ответить
        
        Lure Of Chaos 11.12.2011 17:18 # 0
        
        кстати, жаль. думаю, главные лулзы так и остались за кадром.
        Ответить
        
        gegMOPO4 11.12.2011 19:09 # +2
        
        Нет кода — нету лулзов.
        Ответить
        
        lucidfoxGovno 11.12.2011 19:11 # 0
        
        утром код - вечером лулзы, вечером код - утром лулзы
        Ответить
        
        Lure Of Chaos 11.12.2011 19:47 # +1
        
        ага, как до утки - на вторые сутки
        Ответить
bugaga 11.12.2011 00:51 # 0
тут только 100 строк можно вешать. InitTables распирает. Для чего вообще map, pair в решении этой задачи.
```
unsigned short values[256];
values[ 'I' ] = 1;
values[ 'V' ] = 5;
values[ 'X' ] = 10;
...
```
Ответить
Lure Of Chaos 11.12.2011 15:28 # 0

отрицательные числа не могут быть римскими?
Ответить
- bugaga 11.12.2011 23:58 # 0
  
  тогда понятия целых чисел не существовало, были лишь натуральные. Кстати нуля нету в римской форме записи.
  Ответить
  - ctm 12.12.2011 11:12 # 0
    
    ага, 0 в Индии изобрели и цифры привычные тоже:)
    Ответить
    - Lure Of Chaos 12.12.2011 11:56 # 0
      
      а говорят, что в Риме )
      Ответить
      - gegMOPO4 12.12.2011 13:14 # +1
        
        Кому какие привычнее.
        Ответить
      - ctm 13.12.2011 06:05 # +1
        
        в Риме математика нужна была для подсчета солдат - поэтому и цифры у них такие. 0 им не был нужен.
        Ответить
        
        gegMOPO4 13.12.2011 13:01 # 0
        
        0 в поле не воин.
        Ответить
        
        Lure Of Chaos 13.12.2011 13:18 # +1
        
        0 - боец невидимого фронта!
        потом уже, видимо, понадобилось считать потери и раненых
        Ответить
        
        ctm 13.12.2011 15:00 # 0
        
        вот они и считали их: I, II, III.... а если нет потерь, то пусто оставляли
        Ответить
        
        SmackMyBitchUp 13.12.2011 17:41 # 0
        
        В Паскале, если не ошибаюсь, в нулевом байте строки хранится ее длина
        Ответить
        
        Lure Of Chaos 13.12.2011 19:33 # 0
        
        только не говорите, что в виде римского числа!
        Ответить
        
        SmackMyBitchUp 14.12.2011 18:07 # 0
        
        И не скажу, хотя бы потому, что среди римских чисел нет ноля =\
        Ответить
Xom94ok 11.12.2011 19:51 # 0

Хе-хе, бывший паскалист, что ли, писал? :-)
Ответить
- Lure Of Chaos 11.12.2011 19:54 # 0
  
  наверное, тот же автор, что и http://govnokod.ru/7638
  Ответить
ctm 13.12.2011 15:04 # 0

еще не хватает тестов IL, IC, IXC (они некорректны)
Ответить
guest8 09.04.2019 12:20 # −999

показать все, что скрытоvanished
Ответить

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

C++ / Говнокод #8788

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

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