По колено в коде — Говнокод.ру

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

+136

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

public int GetModuleId(int userId)
{
    return moduleIdGet(userId);
}

protected int moduleIdGet(int userId)
{
    int moduleId;
    // calculate moduleId
    // ...

    return moduleId;
}

Дал открытый доступ, но в то же время как бы сохранил защищённый.

wissenstein, 29 Января 2014

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

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

+120

1
2
3
4

removeTSdbTables(con, yesIknowWhatIamDoing=FALSE)
dropTStable(con, Table, yesIknowWhatIamDoing=FALSE)

The arument yesIknowWhatIamDoing defaults to FALSE and must be set to TRUE or the function will return an error.

http://www.inside-r.org/packages/cran/TSsql/docs/dropTStable

someone, 28 Января 2014

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

ActionScript / Говнокод #14432

−121

01
02
03
04
05
06
07
08
09
10
11

private function taskResizer_resizeFinished(event:ResizerEvent):void {
            var width:int = event.width;
            var height:int = event.height;
            task.changeSize(width, height);
            // Wake up bindings
            var zones:* = zones; // AHAHAHHAHAHA
            this.zones = null;
            this.zones = zones;
            taskResizer.width = width;
            taskResizer.height = height;
        }

Апдейт биндингов и контрольный выстрел в (шестой) строке.

maleka, 28 Января 2014

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

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

+155

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

try
{
  return ($image = $this->row->image()->first())
    ? $image->{$this->imageAlias}
    : dummyThumbnail($this->imageAlias)
  ;
}
catch(\Exception $e)
{
  if ($e->getMessage() === 'Method [image] is not defined on the Query class.')
  {
      $val = parent::value();
      return (is_string($val) and \Str::contains($val, 'http://'))
        ? $val
        : $this->row->getImageSrc($this->name, $this->imageAlias)
      ;
  }

  throw new \Exception($e->getMessage(), $e->getCode());
}

mobileka, 28 Января 2014

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

Java / Говнокод #14429

+72

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

@Entity
public class MyObject {
    @Column
    private int type;

    @Id
    private long id;

    @Column
    private String name;

    // и ещё другие поля, а также геттеры-сеттеры для них, в общем, обычная сущность
}

// managed bean в jsp 1.2 (legacy проект)
public class MyList {
    private List<MyObject> oList;
    private SimpleDateFormat filterDateFormat;

    public MyList() {
        filterDateFormat = new SimpleDateFormat("dd-MM-yyyy HH:mm:ss");
    }

    public List<MyObject> getMyList() {
        if (oList == null) {
            oList = DAO.getDAO().findAllMyObjects();
            String name = ...; // берётся из формы
            if (name != null && name != "") {
                oList = getObjectsByName(name, oList);
            }
            Integer type = ...; // тоже берётся из формы
            if (type != null) {
                oList = getObjectsByType(type, oList);
            }
            // и здесь ещё четыре куска такого же говнокода для других свойств MyObject
        }
        return oList;
    }

    private List<MyObject> getObjectsByType(Integer type, List<MyObject> oList) {
        List<MyObject> queriesByType = new ArrayList<MyObject>();
        for (MyObject o : oList) {
            if (o.getType() == type) {
                queriesByType.add(o);
            }
        }
        return queriesByType;
    }

    private List<MyObject> getObjectsByName(String name, List<MyObject> oList) {
        List<MyObject> queriesByName = new ArrayList<MyObject>();
        for (MyObject o : oList) {
            if (o.getName() == name) {
                queriesByName.add(o);
            }
        }
        return queriesByName;
    }

    // и ещё четыре таких же говнометода для других свойств MyObject
}

Наглядное руководство, как не надо работать с JPA

evg_ever, 28 Января 2014

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

Perl / Говнокод #14428

−103

1
2
3
4
5
6

use Carp qw(carp cluck croak confess);

carp "warning";
cluck "warning with backtrace";
croak "fatal error";
confess "fatal error withbacktrace";

Говорящие имена функций такие говорящие придирчивые, кудахтающие, каркающие, исповедывающиеся...

bormand, 28 Января 2014

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

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

+162

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

foreach ($foo as $key1=>$value) {
    if ($foo[$key1][parent] == 0 ) {
        echo '<li><a href="/index.php?cPath='.$key1.'">'.$foo[$key1][name].'</a>';
        foreach ($foo as $key2=>$value) { if ($foo[$key2][parent] == $key1){echo '<ul>'; break; }};
        foreach ($foo as $key2=>$value) {
            if ($foo[$key2][parent] == $key1) {
                echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'">'.$foo[$key2][name].'</a>';
                foreach ($foo as $key3=>$value) { if ($foo[$key3][parent] == $key2){echo '<ul>'; break;} };
                foreach ($foo as $key3=>$value) {
                    if ($foo[$key3][parent] == $key2) {
                        echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'_'.$key3.'">'.$foo[$key3][name].'</a>';
                        foreach ($foo as $key4=>$value) { if ($foo[$key4][parent] == $key3){echo '<ul>'; break;} };
                        foreach ($foo as $key4=>$value) {
                            if ($foo[$key4][parent] == $key3) {
                                echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'_'.$key3.'_'.$key4.'">'.$foo[$key4][name].'</a>';
                                foreach ($foo as $key5=>$value) { if ($foo[$key5][parent] == $key4){echo '<ul>'; break;} };
                                foreach ($foo as $key5=>$value) {
                                    if ($foo[$key5][parent] == $key4) {
                                        echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'_'.$key3.'_'.$key4.'_'.$key5.'">'.$foo[$key5][name].'</a>';
                                        foreach ($foo as $key6=>$value) {if ($foo[$key6][parent] == $key5) {echo '<ul>'; break;} };
                                        foreach ($foo as $key6=>$value) {
                                            if ($foo[$key6][parent] == $key5) {
                                                echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'_'.$key3.'_'.$key4.'_'.$key5.'_'.$key6.'">'.$foo[$key6][name].'</a>';
                                            }
                                        }
                                        foreach ($foo as $key6=>$value) { if ($foo[$key6][parent] == $key5){echo '</ul>'; break;} };  
                                        echo '</li>';                                      
                                    }
                                } 
                                foreach ($foo as $key5=>$value) { if ($foo[$key5][parent] == $key4){echo '</ul>'; break;} };
                                echo '</li>';
                            }
                        }
                        foreach ($foo as $key4=>$value) {if ($foo[$key4][parent] == $key3) {echo '</ul>'; break;} };
                        echo '</li>';                          
                    }
                } 
                foreach ($foo as $key3=>$value) { if ($foo[$key3][parent] == $key2){echo '</ul>'; break;} }; 
                echo '</li>';              
            }
        }
        foreach ($foo as $key2=>$value) {if ($foo[$key2][parent] == $key1) {echo '</ul>'; break;} };
        echo '</li>';
    }        
}

Правильное понимание рекурсии, привитое картинками вконтакте.

Dotterian, 28 Января 2014

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

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

+10

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
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80

align.h: 
#pragma once
#include "stdint.h"
namespace tblib 
{
  template <typename T>
  struct check_align
  {
    uint8_t c;
    T t;
    check_align();
    check_align(const check_align&);
    check_align&operator=(const check_align&);
  };

  template <typename T>
  struct align_value
  {
    enum { value = sizeof(check_align<T>)-sizeof(T) };
  };

  template <int N>
  struct align_by_size;

  template <> 
  struct align_by_size<1> { uint8_t guts; };

  template <> 
  struct align_by_size<2> { uint16_t  guts; };

  template <> 
  struct align_by_size<4> { uint32_t guts; };

  template <> 
  struct align_by_size<8> { uint64_t guts; };

  template <typename T>
  struct align
  {
    align_by_size<align_value<T>::value> guts;
  };
};
 класс массива (начало), файл tbarr.h: 
#pragma once

#include "stdint.h"
#include "assert.h"
#include <algorithm>
#include "tbslice.h"
#include "align.h"

// FUCK THE EXCEPTIONS

template <typename T>
void construct (T* first, T* last) {
  while (first!=last)  {
    new(first) T;
    ++first;
  }
}

template <typename T>
void destroy (T* first, T* last)  {
  while (last!=first)  {
    --last;
    last->~T();
  }
}

namespace tblib
{  
  template <typename T, int N>
  class base_array
  {
    union
    {
      uint8_t memory [sizeof(T[N])];
      align<T> aligner;
    };
  public :

мой выравниватель
намного красивее бустовского, не так ли?

TarasB, 27 Января 2014

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

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

+155

01
02
03
04
05
06
07
08
09
10
11

var partner_id = location.href;
        //Определить значение четвертого сегмента
        var i = 1;
        while (i <= 5) {
            partner_id = partner_id.substr(partner_id.indexOf('/') + 1);
            i++;
        }
        var pos = partner_id.indexOf('/');
        if (pos > 0) {
            partner_id = partner_id.substr(0, pos);
        }

Урлопарсинг от Папке.

ragnar, 27 Января 2014

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

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

+127

1
2
3
4
5
6
7
8

// ------- CLICKING STUFF IN CONTAINERS -------
	if ((!( src in usr.contents ) && (((!( isturf(src) ) && (!( isturf(src.loc) ) && (src.loc && !( isturf(src.loc.loc) )))) || !( isturf(usr.loc) )) && (src.loc != usr.loc && (!( istype(src, /obj/screen) ) && !( usr.contents.Find(src.loc) ))))))
		if (istype(usr, /mob/living/silicon/ai))
			var/mob/living/silicon/ai/ai = usr
			if (ai.control_disabled || ai.malfhacking)
				return
		else
			return

Взято из кода обработки нажатия одной опенсорсной онлайн-игрушки. Зачем, за что и почему?

ACCount, 27 Января 2014