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

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

+162

1
2

themeStr= (false)? "<br><b>" + "Вопросы по теме: " + theme + "<\/b><br>" : "";
  qNumberStr = (!false && (!false || afterAnswers))? "<br><b>" + qNumbers[iQuestion] + "<\/b><br>" : "";

Это же какой полет мысли!

Daiver, 23 Мая 2013

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

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

+157

1
2
3
4
5
6
7

.td-basket .region>.td-list .td-a>p+div>p:first-child+p,
.td-basket .region>.td-list>div>img+div>p+p>span,
.td-basket .region>.td-list .td-a>p+div+div>p:first-child+p,
.td-basket .region>.td-list h2
 {
   font-size: 16px;
}

Ниндзя-верстак

dsxack, 22 Мая 2013

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

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

+14

1
2
3
4
5

uint8_t* head = (uint8_t*) Buffer::Data(buffer);
uint8_t* tail = head + Buffer::Length(buffer) - 1;

// xor swap, just because I can
while (head < tail) *head ^= *tail, *tail ^= *head, *head ^= *tail, ++head, --tail;

https://github.com/bnoordhuis/node-buffertools/blob/master/buffertools.cc#L157

movaxbx, 22 Мая 2013

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

SQL / Говнокод #13032

−165

1
2
3
4
5

delete
    from liaison
where
    type = 'UsersGroup' and
    `group` = 'static_' || (select id from usergroup where name = :grp)

Час назад вынес этим запросом все связи между группами и юзерами в боевой базе ;)

P.S. Почему в mysql все дерьмо, работающее совсем не так, как оно работает почти во всех остальных СУБД, включено по умолчанию?

bormand, 22 Мая 2013

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

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

+150

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

function deleteDublicateItems(){
    $q = '
    select COUNT(*), id, vk_id
    from `items`
    group by `vk_id`
    having COUNT(*) > 1';
    $sql = mysql_query($q);
    
    if (mysql_num_rows($sql)){	
	while($row = mysql_fetch_assoc($sql)){
	    $sql2 = mysql_query('select * from `items` where `vk_id` = "'.$row['vk_id'].'" and `id` != "'.$row['id'].'"');
	    
	    if (mysql_num_rows($sql2)) {
		while($row2 = mysql_fetch_assoc($sql2)) {
		    $sql3 = mysql_query('select * from `images` where `item_id` = "'.$row2['id'].'"');
		    
		    if (mysql_num_rows($sql3)) {
			while ($row3 = mysql_fetch_assoc($sql3)) {
			    @unlink( ROOT . DS . 'uploads' . DS . 'images' . DS . $row3['name'] . '.' . $row3['ext']);
			    mysql_query('delete from `images` where `id` = "'.$row3['id'].'"');
			}
		    }		    
		    mysql_query('delete from `items` where `id` = "'.$row2['id'].'"');
		}
	    }
	}
    }
}

удаление дубликатов

Serious_Andy, 22 Мая 2013

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

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

+147

1

<cbimport_import_check_checkimport>

надо вдуматся что бы понять
нода из конфигурации модуля под маженто

vesan, 21 Мая 2013

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

Ruby / Говнокод #13029

−85

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

def self.process(site, object = nil, &block)
    dont_unblock = false
    block_res = true

    site.lock!
    if site.assets_blocked?
      dont_unblock = true
      raise Asset::BlockedException
    end
    site.assets_blocked = true
    site.save!

    block_res = yield if block_given?

    return false unless block_res
    return object.save if object

    true
  ensure
    site.update_attribute(:assets_blocked, false) unless dont_unblock
  end

  # во всех местах, где с ассетами делалось хоть что-нибудь, копипастился такой кусок:

      result =
      begin
        @asset = @site.assets.new
        @asset.process do
          @asset.attached = params[:file]
          @asset.folder_path = params[:path]
        end
      rescue Asset::BlockedException
        render json: { errors: [I18n.t("layout.assets.errors.assets_blocked")] } and return
      end

По мотивам http://govnokod.ru/13027: транзакции изобретены не нами, давайте сделаем их сами!

whitequark, 21 Мая 2013

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

Ruby / Говнокод #13027

−89

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

def set_path
      if name && (!path || path == "/")
        self.path = self.parent.present? ? "#{self.parent.path}/#{name}" : "/#{name}"
      elsif !new_record? && name && path && name_was != name
        parts = path.split("/")
        parts.pop
        self.path = [parts.join("/"), name].join("/")
      elsif !new_record? && name && self.parent_id_changed?
        self.path = self.parent.present? ? "#{self.parent.path}/#{name}" : "/#{name}"
      elsif new_record? && name && path
        self.path = [path, name].join("/")
      end

      if path && self.parent.blank?
        parts = path.split("/")

        self.name = parts.pop

        parent_path = parts.join("/")
        if parent_path.blank? || parent_path == "/"
          self.parent = nil
        else
          possible_parent = site.asset_folders.find_by_path(parent_path)
          self.parent = possible_parent.present? ? possible_parent : self.class.create(path: parent_path, site: site)
        end
      end
      true
     end

Как не нужно работать с путями в Rails-приложении. Это и еще примерно 500 строк было заменено на 11 строк кода, включая пустые.

whitequark, 20 Мая 2013

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

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

+154

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

<script type="text/javascript">
	// <![CDATA[
	function update() {
		document.calculate.totalsales.value = parseFloat(document.calculate.shoessingle.value) * parseFloat(document.calculate.socktoshoe.value) * parseFloat(document.calculate.singlepair.value);
	}
	// ]]>
	
</script>
<script type="text/javascript">
	// <![CDATA[
	function amargin() {
		document.calculate.avgmarg.value = parseFloat(document.calculate.singlepair.value) /2;
	}
	// ]]>
	
</script>


<script type="text/javascript">
	// <![CDATA[
	function gmargin() {
		document.calculate.grossmargin.value = parseFloat(document.calculate.totalsales.value) /2;
	}
	// ]]>
	
</script>

отакота...

padonak, 20 Мая 2013

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

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