Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

PHP / Говнокод #18481

+142

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

Ситуация такая:
Самописный движок.
MVC но не очень строго.
Получился толстый контроллер, моделей нет, простой ORM чисто по таблицам.
Шаблонизатор поддерживает только пару функций - цикл, вывод переменной, условие. Условие только на ИСТИНА или ЛОЖЬ.
Движок писался по принципу, что каждая страница имеет запись в основной таблице, где есть информация по урл, родительской странице и т.п., ну и название контроллера который за нее отвечает.
т.о. у нас 27 контроллеров для разных типов страниц, и несколько тысяч страниц в базе.
в юии есть такой компонет как урлконтроллер
он расширяется отдельным классом правил, написал два метода, и все ссылки конвертируются в роуты, а роуты в ссылки как родные..
В роутах Ларавел я не нашел как это сделать нормально.
Я начал упорядочивать всё, есть версия где нормальные абстрактные контроллеры, урлменеджер, и ответственности более менее правильно разграничены, но эта версия не полностью рабочая, есть пару небольших глюков.
В новой версии некоторые функции админки неверно отрабатываются. Мне кажется лучшим вариантом было бы, если бы вы начали общую структуру делать, тот же урлменеджер и т.п., а я бы исправил глюки, и уже добивать эту самую вторую версию.
Самый простой контроллер выглядит вот так:
http://pastebin.com/Rt5sN91V
 $info->allowedParent = '*';
это список контроллеров которые могут быть "родителями" для нашей страницы
 $info->allowedChild = FALSE;
Список дочерних страниц, которые могут быть у нашей страницы. В данном случае таких не может быть, а родитель может быть любой.
 $info->unique = TRUE;
это параметр означает, что страница с таким контроллером на сайте может быть только одна. Если уже создали, то новую не создадим.
 templ в базе и т.п. означает контроллер а не шаблон. Как-то так в начале повелось, а до рефакторинга так и недобрались.
 В новой в папке контроллеров я разделил по степени их сложности. Простейшие, это просто наследование от базового, легкие, это где только read и в корне это более-менее сложные.
 Пользователь с рутовыми правами:
mendel
пароль 1709
 админки как таковой там нет.
 При авторизации админа появляется меню "служебные", но админка не там.
По сути админка у нас на каждой внутренней ссылке, в контекстном меню.
 Например если правой кнопкой щелкнуть по логотипу, то откроется менюшка для главной страницы.
 Если на ссылку пункта меню - меню для нее, ссылка на новость - меню новости...
 У всех контроллеров кроме авторизации стандартный CRUD + сортировка (перемещение вверх/вниз относительно соседей в списке), и файлы.
 Все файлы связанные с той или иной страницей привязаны к ней же.
По хорошему надо было делать в той же таблице, чтобы в общем дереве, но они отдельно идут. т.е. если статья называется /статьи/раздел/статья.хтмл
то картинки из нее это /статьи/раздел/статья/картинка.жпг
 У контролера также есть функция добавления/удаления картинок.
 Естественно оно в абстрактном классе идет.
Да, валидаторы у меня в контроллере, не пугайтесь.
Желательно чтобы модели были Eloquent или если есть весомые причины с другим ОРМ, то можно другой. Шаблоны можно хоть базовые, хоть блейд, но не мои.
 И по роутам я бы расширял роуты/реквесты/респонсы, но я с ларавел еще не настолько разобрался чтобы понять где именно надо расшириться... Чую что тут тоже можно в два метода уложиться, но пока не вижу где.

Это Тех задание к предыдущему говнокоду - прошу нормальных спецов прокомментировать.

Я считаю что это полная жопа!

Aios, 15 Июля 2015

Комментарии (0)

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

+142

1

http://pastebin.com/NeWPms7u

Лаконичное решение как разместить OpenCL и C++ в одном файле. Самое главное что можно спокойно отлаживать данный код. Также объявлять общие переменные и функции OpenCL и host коду. Единственное но! Придется задать пару флагов компилятору OpenCL кода для того чтобы сие подделие заработало. Главный принцип заключается в #if(n)def CL_DEVICE. По умолчанию он не объявлен. Также следует включить поддержку C++ в OpenCL. Скоро кстати выйдет OpenCL 2.1, и вполне возможно что такой трюк и там прокатит. Можно попробовать проделать это с шейдерами, но OpenCL подходит больше всего. Жаль что для OpenCL нету GLM, а если бы он щас был... Тогда можно было такое накодить.

acterhd, 14 Июля 2015

Комментарии (0)

Си / Говнокод #18472

+142

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16

int do_execve(const char *filename,
	const char __user *const __user *__argv,
	const char __user *const __user *__envp,
	struct pt_regs *regs)
{
	struct user_arg_ptr argv = { .ptr.native = __argv };
	struct user_arg_ptr envp = { .ptr.native = __envp };
    /* exec mt_debug*/
    int ret;
    int retry = 3;
    do{
        ret = do_execve_common(filename, argv, envp, regs);
        printk(KERN_DEBUG"[exec] %s(%d)\n", filename, retry);
    }while( -999 == ret && retry-- > 0);
	return ret;
}

Ядро от MTK. Они логгируют каждый exec на уровне ядра.

Azq2, 12 Июля 2015

Комментарии (4)

C# / Говнокод #18466

+142

1
2
3
4

if (_availabilityOfThisPersonnel == null)
     _availabilityOfThisPersonnel = new List<Availability>();
if (_availabilityOfThisPersonnel.Any())
    _availabilityOfThisPersonnel.Clear();

vldalx, 10 Июля 2015

Комментарии (1)

C# / Говнокод #18460

+142

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

/// <summary>
        /// Обновление данных о объекте
        /// </summary>
        /// <param name="theObject">ссылка на объект</param>
        public void Update(IPersistentObject theObject)
        {
            IDbCommand updateCommand = null;
            try
            {
                using (updateCommand = CreateUpdateCommand(theObject))
                {
                    updateCommand.Connection.Open();
                    updateCommand.ExecuteNonQuery();
                }
            }
            catch (DbException ex)
            {
                int code = 0;
                if (ex is SqlException)
                {
                    code = ((SqlException) ex).Number;
                }
                if (code == 229)
                {
                    ex.Data.Add("Name", theObject.ClassInfo.Name);
                    ex.Data.Add("Description", theObject.ClassInfo.Description);
                    ex.Data.Add("Action", "UPDATE");
                    throw new InvalidOperationException(
                        String.Format("Ошибка обновления объекта [{0}] - {1} (ID = '{2}')",
                                      theObject.ClassInfo.Name,
                                      theObject.ClassInfo.Description,
                                      theObject.ID), ex);
                }
                if (code == 207 || code == 208)
                {
                    throw new InvalidColumnException(theObject.ClassInfo, ex);
                }
                //theObject заменен на theObject.ID. Нечитабельно, но ... Т. к. при вычислении ToString()
                //для показа объекта иногда задействуются методы загрузки данных
                //названия объекта. Таким образом при высоком уровне изоляции транзакции
                //мы получим зависание если будет попытка загрузить данные заблокированные транзакцией.
                TraceLogger.Log(String.Format("Update {0}({1}){4} - {2}\n{3}", theObject.ID, theObject.ClassInfo,
                                              ex.Message, ex.InnerException, theObject.GetInfoToTraceMessage()),
                                TraceTypeEnum.UpdateStatement);
                throw;
            }
            finally
            {
                if (updateCommand != null)
                {
                    updateCommand.Connection.Close();
                }
            }
        }

смертельный прием using
и добьем соперника используя finally как фаталити

перехват исключения тоже хорош, авторский комментарий добавляет изюминку

vldalx, 10 Июля 2015

Комментарии (6)

C# / Говнокод #18459

+142

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15

public IndividualService Service
        {
            get { return _service; }
            set
            {
                if (value == null || !value.IsDNSeriaHeader)
                    _service = value;
                else
                {
                    if (!IsLoading && Initialized)
                        XtraMessageBox.Show("Услуга по записи о ДН не должна быть заголовком ДН", "Внимание",
                                        MessageBoxButtons.OK, MessageBoxIcon.Warning);
                }
            }
        }

MessageBox на сеттере в EDM (если говорить в терминах Entity Framework)

vldalx, 09 Июля 2015

Комментарии (0)

C# / Говнокод #18454

+142

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

private MonthCalendarModel GetMonthCalendar(int year, int month, string roleId, string doctorId)
{
var days = new List<DayCalendarModel>();
var today = DateTime.Today;
var firstDay = new DateTime(year, month, 1);
int firstDayOfWeek = 0;
if (firstDay.DayOfWeek == DayOfWeek.Monday)
    firstDayOfWeek = 1;
if (firstDay.DayOfWeek == DayOfWeek.Tuesday)
    firstDayOfWeek = 2;
if (firstDay.DayOfWeek == DayOfWeek.Wednesday)
    firstDayOfWeek = 3;
if (firstDay.DayOfWeek == DayOfWeek.Thursday)
    firstDayOfWeek = 4;
if (firstDay.DayOfWeek == DayOfWeek.Friday)
    firstDayOfWeek = 5;
if (firstDay.DayOfWeek == DayOfWeek.Saturday)
    firstDayOfWeek = 6;
if (firstDay.DayOfWeek == DayOfWeek.Sunday)
    firstDayOfWeek = 7;
for(int i= firstDayOfWeek-1; i > 0; i--)
{
    DateTime day = firstDay.AddDays(-i);
    days.Add(new DayCalendarModel{Available = -1, Day = day.Day, Enabled = false, CssClass = string.Empty});
}
DateScheduler dateScheduler = GetDateScheduler(roleId, doctorId);
DateTime date;
for( date = firstDay; date.Month == month; date = date.AddDays(1)) {
    string cssClass = dateScheduler.CheckDate(date, true) ? "free": "disbl";
    bool enabled = (date >= today);
    cssClass = enabled ? cssClass : string.Empty;
	days.Add(new DayCalendarModel(){Available = 3, Day = date.Day, Enabled = enabled, CssClass = cssClass, Date = date.ToString(FormatHelper.DateOnlySerializationFormat)});
}
if(date.DayOfWeek != DayOfWeek.Monday) {
    for(; GetDayOfWeek(date.DayOfWeek) <= 7; date = date.AddDays(1)) {
        days.Add(new DayCalendarModel()
            {Available = -1, Day = date.Day, Enabled = false, CssClass = string.Empty});
        if(date.DayOfWeek == DayOfWeek.Sunday) {
            break;
        }
    }
}

// пропущено ...
}

private int GetDayOfWeek(DayOfWeek dayOfWeek)
{
    if(dayOfWeek == System.DayOfWeek.Sunday)
	{
        return 7;
    }
    return (int)dayOfWeek;
}

полученный List<DayCalendarModel> используется для отображения пользователю календаря с отмеченными днями работы врача

vldalx, 09 Июля 2015

Комментарии (0)

Куча / Говнокод #18453

+142

1

http://habrahabr.ru/post/72959/

Сразу прошу извинить...

1. за хабр ;(
1. за баянизм - ссылка на этот пост уже была на гк.

Знаю по опыту, что объекты в делфи инициализируются мусором.
"В Delphi это не представляет никаких трудностей, так как любой объект еще до передачи управления конструктору инициализируется нулями." - выебать бы маму автора сего пёрла.

Stertor, 09 Июля 2015

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

C# / Говнокод #18451

+142

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

public String ExecuteInDispancerMonitoring
        {
            get
            {
                string st = "";
                IndividualService service = this;
                int iter = 100;
                while (iter != 0)
                {
                    if (service.ParentService != null)
                    {
                        service = service.ParentService;
                        if (service is DispanserMonitoring)
                        {
                            st = service.CalcStrServiceName; // Строковое представление названия услуги
                            if (service.CalcStartDate.HasValue)
                            {
                                st += " c " + service.CalcStartDate.Value.ToShortDateString();
                            }
                            return st;
                        }
                    }
                    else
                    {
                        return st;
                    }
                    iter--;
                }
                return "";
            }
        }

vldalx, 08 Июля 2015

Комментарии (8)

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

+142

1
2
3
4

setExpanded : function(val) {
    val = val ? true : false;
    this._expanded = val;
}

beatmasta, 07 Июля 2015

Комментарии (5)

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