- 1
- 2
- 3
String testName;
//...
std::swap(testName, _testName);-
Лучший говнокод
- В номинации:
-
- Лучший рейтинг
- Самый комментируемый
- За время:
-
- за сегодня
- за неделю
- за месяц
- за всё время
-
C++ / Говнокод #12756
+15
String из thirdparty-библиотеки, а swap везде в нашем коде. По очевидным причинам получаем подение производительности.
LispGovno,
16 Марта 2013
Комментарии (41) -
C++ / Говнокод #12610
+80
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
- 35
- 36
- 37
- 38
/* Попытка обьединить четыре цикла в один. Первый - Присвоение элементам массива значения. Второй - Поиск минимального значения. Третий - Поиск максимального значения. Четвертый - Вывод содержимого массива. */ #include <iostream> #include <cstdlib> using namespace std; int main() { int array[100]; int flag; int min = 0; int max = 0; for (flag=0;flag<100;flag++){ array[flag]=rand(); if ( ! min && ! max){ //Инициализация переменных, для первой итерации. max = array[0]; min = array[0]; } if (array[flag] >=max) max = array[flag]; if (array[flag] <= min) min = array[flag]; cout << array[flag] << "\n"; } cout << "\nМаксимальное значение: " << max; cout << "\nМинимальное значение: " << min << '\n'; return 0; }Писал я, пытаясь переделать пример из книжки, в котором для этой же задачи использовалось 4 цикла.
Просьба обьективно обгадить сей код, дабы мне было над чем подумать.
P.S только учусь.
shim,
18 Февраля 2013
Комментарии (41) -
Java / Говнокод #12583
+79
- 1
- 2
- 3
public class StringToObjectMap extends HashMap<String, Object> { public StringToObjectMap(Map<? extends String, ? extends Object> map)Нет слов выразить мою печаль.
nafania217518,
13 Февраля 2013
Комментарии (41) -
C++ / Говнокод #12488
+19
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
struct Base { }; struct Class : Base { int mem{ 0 }; // init non-static data member Class(int i) : Base{} // init base class , mem{i} // init member { int j{i}; // init local var int k = int{0}; // init temporary f( { 1 } ); // init function arg int* p = new int{1}; // new init // int k(int()); // most vexing parse, declares function int k{ int{} }; // ok, declares variable int i[4]{ 1,2,3,4 }; // init array } Class f(int i) { return { i }; // init return value } }; Class c{1}; // init global var LispGovno,
28 Января 2013
Комментарии (41) -
1C / Говнокод #12383
−119
- 001
- 002
- 003
- 004
- 005
- 006
- 007
- 008
- 009
- 010
- 011
- 012
- 013
- 014
- 015
- 016
- 017
- 018
- 019
- 020
- 021
- 022
- 023
- 024
- 025
- 026
- 027
- 028
- 029
- 030
- 031
- 032
- 033
- 034
- 035
- 036
- 037
- 038
- 039
- 040
- 041
- 042
- 043
- 044
- 045
- 046
- 047
- 048
- 049
- 050
- 051
- 052
- 053
- 054
- 055
- 056
- 057
- 058
- 059
- 060
- 061
- 062
- 063
- 064
- 065
- 066
- 067
- 068
- 069
- 070
- 071
- 072
- 073
- 074
- 075
- 076
- 077
- 078
- 079
- 080
- 081
- 082
- 083
- 084
- 085
- 086
- 087
- 088
- 089
- 090
- 091
- 092
- 093
- 094
- 095
- 096
- 097
- 098
- 099
- 100
Функция РазобратьКодыНеисправностей(СтрокаКодов, ТипКода) СпрКодов = СоздатьОбъект("Справочник.КодыГарантийныхНеисправностей"); ГруппаТ1 = СпрКодов.НайтиПоКоду("0",0); ГруппаТ2 = СпрКодов.НайтиПоКоду("00",0); ГруппаТ3 = СпрКодов.НайтиПоКоду("000",0); Если СтрокаКодов <> "" Тогда Разд1 = Найти(СтрокаКодов, ";"); Разд2 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд1), ";") + Разд1; Разд3 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд2), ";") + Разд2; Разд4 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд3), ";") + Разд3; Разд5 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд4), ";") + Разд4; Разд6 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд5), ";") + Разд5; Разд7 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд6), ";") + Разд6; Разд8 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд7), ";") + Разд7; Разд9 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд8), ";") + Разд8; Разд10 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд9), ";") + Разд9; Разд11 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд10), ";") + Разд10; Если ТипКода = "Т1" Тогда Если Разд1 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ1); Если СпрКодов.НайтиПоКоду("0/" + Лев(СтрокаКодов,Разд1-1),2) = 1 Тогда ПолучТ1 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ1; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т2" Тогда Если Разд2-Разд1 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ2); Если СпрКодов.НайтиПоКоду("00/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд1), Разд2 - Разд1 - 1), 2) = 1 Тогда ПолучТ2 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ2; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т3_1" Тогда Если Разд3-Разд2 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ3); Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд2), Разд3 - Разд2 - 1), 2) = 1 Тогда ПолучТ3_1 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ3_1; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т3_2" Тогда Если Разд4-Разд3 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ3); Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд3), Разд4 - Разд3 - 1), 2) = 1 Тогда ПолучТ3_2 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ3_2; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т3_3" Тогда Если Разд5-Разд4 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ3); Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд4), Разд5 - Разд4 - 1), 2) = 1 Тогда ПолучТ3_3 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ3_3; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т3_4" Тогда Если Разд6-Разд5 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ3); Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд5), Разд6 - Разд5 - 1), 2) = 1 Тогда ПолучТ3_4 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ3_4; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т3_5" Тогда Если Разд7-Разд6 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ3); Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд6), Разд7 - Разд6 - 1), 2) = 1 Тогда ПолучТ3_5 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ3_5; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т3_6" Тогда Если Разд8-Разд7 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ3); Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд7), Разд8 - Разд7 - 1), 2) = 1 Тогда ПолучТ3_6 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ3_6; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т3_7" Тогда Если Разд9-Разд8 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ3); Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд8), Разд9 - Разд8 - 1), 2) = 1 Тогда ПолучТ3_7 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ3_7; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т3_8" Тогда Если Разд10-Разд9 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ3); Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд9), Разд10 - Разд9 - 1), 2) = 1 Тогда ПолучТ3_8 = СпрКодов.ТекущийЭлемент(); Возврат ПолучТ3_8; КонецЕсли; КонецЕсли; ИначеЕсли ТипКода = "Т3_9" Тогда Если Разд11-Разд10 > 1 Тогда СпрКодов.ИспользоватьРодителя(ГруппаТ3);ещё один высер от автодилера.
1. начнём сначала, там есть строки:
...
ГруппаТ1 = СпрКодов.НайтиПоКоду("0",0);
..., обращаю внимание автора, что функция НайтиПоКоду не возвращает элемент справочника, она лишь говорит есть он или нет, для позиционирования на элементе не хватает строк типа ГруппаТ1 = СпрКодом.ТекущийЭлемент(); и т.д.
2. дальше. в функцию передаётся параметр СтрокаКодов - это строка с запятыми в качестве разделителей. я конечно понимаю, что в 1С нет встроенной функции для получения массива частей строки типа split в java, но написать свою вообще не сложно, но видимо у автора не хватило мозг..., извините, времени.
3. теперь смотрим на подобные (их несколько в функции) блоки кода типа:
СпрКодов.ИспользоватьРодителя(ГруппаТ1);
Если СпрКодов.НайтиПоКоду("0/" + Лев(СтрокаКодов,Разд1-1),2) = 1 Тогда
ПолучТ1 = СпрКодов.ТекущийЭлемент();
Возврат ПолучТ1;
КонецЕсли;
строка СпрКодов.ИспользоватьРодителя(ГруппаТ1); не имеет смысла, т.к. ГруппаТ1 содержит значение 1 (см. п.1), а не ссылку на элемент справочника, код работает только благодаря второму параметру - 2, т.к. он заставляет функцию НайтиПоКоду искать по всему справочнику, используя полный код.
VladC,
02 Января 2013
Комментарии (41) -
C++ / Говнокод #12160
+21
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
- 35
- 36
- 37
- 38
- 39
- 40
- 41
- 42
- 43
- 44
- 45
- 46
- 47
- 48
- 49
- 50
- 51
- 52
- 53
- 54
- 55
- 56
- 57
- 58
- 59
- 60
- 61
//Было: int foo()// ; { ; int a ; float b ; //В правой части старые typedef int testing ; //полезные комментарии std::cout << "Ho-ho-ho" ; a=0 ; b=1.0f ; if (a==b)// ; { ; std::cout <<"Yet another caption"; std::cout <<"Hi-hi-hi" ; bar() ; ; } ; using namespace std ; for (int i =0;i<10;i++)// ; { ; ; std::cout <<"Hello, world!" ; std::cout <<"Yet another line" ; } ; return 0 ; ; } ; //Кто-то что-то решил поменять //Стало: int foo()// ; { ; int a ; float b ; //В правой части старые typedef int testing ; //полезные комментарии std::cout << "Ho-ho-ho" ; a=0 ; b=1.0f ; if (a==b)// ; { ; std::cout <<"Yet another caption"; std::cout <<"Hi-hi-hi" ; bar() ; ; } ; double c; //Слева комментарии требующие //проверки. Wtf???Зачем здесь с??? using namespace std ; for (int i =0;i<10;i++)// ; { ; ; std::cout <<"Hello, world!" ; std::cout <<"Yet another line" ; } ; return 6666 ; ; } ; //Быстро определили по линии из точек c //запятой место в котором произошло изменение. //Если согласились с ним восстанавливаем линию.Бесплатная распределенная говносистема контроля версий. Git и SVN нервно курят в сторонке.
igumnovf,
20 Ноября 2012
Комментарии (41) -
JavaScript / Говнокод #12002
+160
- 1
new Array(100).map(function(x) { return Math.random() * 1000; });Угадайте результат.
wvxvw,
25 Октября 2012
Комментарии (41) -
PHP / Говнокод #11422
+72
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
function valid_gender($str){ switch($str){ case "male": return "male"; break; case "female": return "female"; break; default: return "male"; break; } }Ультимативная проверка переменной на половую принадлежность. Also return'ам нынче доверия нет!
swiftfooth,
16 Июля 2012
Комментарии (41) -
Си / Говнокод #11392
+141
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
int main() { int i = 0; goto A; for (;i<3; ++i) L:{ { int x; x += i; if (0) { A: x = 0; goto B; } printf("X: %d ",x); } { int y; y -= i; if (0) { B: y = 10; goto L; } printf("Y: %d\n",y); } } }Отсюда: http://rsdn.ru/forum/cpp/4812647.flat.aspx Автор походу уверен в гениальности кода.
slavap,
10 Июля 2012
Комментарии (41) -
PHP / Говнокод #9162
+161
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
function timestampToDate($time) { $time = explode(" ", $time); $mysqlTime = explode("-", $time[0]); $month = array("нулября","января","февраля","марта","апреля","мая","июня","июля","августа","сентября","октября","ноября","декабря"); $num = $mysqlTime[1]; $num = intval($num); $date = $mysqlTime[2]." ".$month[$num]." ".$mysqlTime[0]; return $date; }Задолбали эти нулябри в коде. Стандартные функции? Не, не слышал.
b4075328,
18 Января 2012
Комментарии (41)
Follow us!