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

Pascal / Говнокод #14257

+132

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

procedure tmythread.execute;
var i:integer;
begin
  bot.postnumber:=topicno; bot.proxy:='220.181.89.72:8000'; bot.login:=copy(vote,1,pos(':',vote)-1); bot.password:=copy(vote,pos(':',vote)+1,length(vote));
  case actiontype of
    ac_dec:
    begin
      bot.comment:=false;
      try
        bot.connect;
      except
        terminate;
      end;
      bot.VoteAgainst;
    end;
    ac_inc:
    begin
      bot.comment:=false;
      try
        bot.connect;
      except
        terminate;
      end;
      bot.VoteOn;
    end;
    ac_deccurrent:
    begin
      bot.comment:=true;
      try
        bot.connect;
      except
        terminate;
      end;
      bot.VoteAgainst;
    end;
    ac_inccurrent:
    begin
      bot.comment:=true;
      try
        bot.connect;
      except
        terminate;
      end;
      bot.VoteOn;
    end;
    ac_decall:
    begin
      bot.comment:=true;
      try
        bot.connect;
      except
        terminate;
      end;
      for i:=0 to clist.Count-1 do
      begin
        bot.postnumber:=clist[i];
        bot.VoteAgainst;
      end;
    end;
    ac_incall:
    begin
      bot.comment:=true;
      try
        bot.connect;
      except
        terminate;
      end;
      for i:=0 to clist.Count-1 do
      begin
        bot.postnumber:=clist[i];
        bot.VoteOn;
      end;
    end;
    ac_decallc:
      begin
      bot.comment:=true;
      try
        bot.connect;
      except
        terminate;
      end;
      for i:=0 to clist.Count-1 do
      begin
      bot.postnumber:=clist[i];
      bot.VoteAgainst;
      end;
    end;
    ac_incallc:
      begin
      bot.comment:=true;
      try
        bot.connect;
      except
        terminate;
      end;
      for i:=0 to clist.Count-1 do
      begin
      bot.postnumber:=clist[i];
      bot.VoteOn;
      end; end; end; end;

Код Того-На-Кого-Не-Будем-Показывать-Пальцем

ВНИМАНИЕ! Некоторые строки были объединены дабы вместиться в пост

kegdan, 22 Декабря 2013

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

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

+143

01
02
03
04
05
06
07
08
09
10
11

string=(char*)malloc(N);
 k=fread(string,sizeof(char),N,f);
 while (k==N)
 {
 free(string);
 N=N*2;
 rewind(f);//возращает в начало файла
 string =(char*)malloc(N);
 k=fread(string,sizeof(char),N,f);
 }
 fclose (f);

Сегодня зачоты ставил...
студенты читают из файла. В буфер. Я придираюсь к тому, что размер файла может быть больше размера буфера....

Исправили...

klavdiev, 22 Декабря 2013

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

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

+128

1
2
3
4
5
6
7
8

//...
for (int i = 0; i < size; i++)
{
    //...
    if (i == size + 2) break;
    //...
}
//...

Увидел у соседа в коде. Осторожность никогда не помешает :)

MoN, 21 Декабря 2013

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

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

+83

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

// java.io.FilterOutputStream
public void close() throws IOException {
    try {
        flush();
    } catch (IOException ignored) {
    }
    out.close();
}

// тестовый код
try (OutputStream os = new BufferedOutputStream(new FileOutputStream("/tmp/little_virtual_fs/1.txt"))) {
    byte[] buf = new byte[2028544];
    os.write(buf, 0, 2028544); // максимальный размер файла, который влезает на нашу фс
    os.write(buf, 0, 5); // и еще немножко
    //os.flush();
}

А сейчас, на арене нашего цирка - очередная жабопроблема!

Как думаете, каков результат выполнения тестовой программы?

Запишет 2028549 байт? Хрен там, места маловато.
Выдаст IOException на write()? Хрен там, буферизация.
Выдаст IOException на close()? Хрен там, его сожрала реализация FilterOutputStream.

Результат: файл не дописался, исключения нет.
Решение: всегда дергать flush() перед close() самому или не юзать буферизованный поток.

bormand, 21 Декабря 2013

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

1C / Говнокод #14253

−134

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

Функция ПолучитьОбратнуюМетрику(Знач Метрика) Экспорт

	ТаблицаКоординат = СоздатьТаблицуКоординат();
	ПреобразоватьМетрикуВТаблицуКоординат(Метрика,ТаблицаКоординат);
	ТаблицаКоординат.Колонки.Добавить("ПорядковыйНомер");
	
	Сч = 1;
	Для каждого СтрокаТаблицаКоординат Из ТаблицаКоординат Цикл
		СтрокаТаблицаКоординат.ПорядковыйНомер = Сч;
		Сч = Сч + 1;
	КонецЦикла;
	
	ТаблицаКоординат.Сортировать("ПорядковыйНомер УБЫВ");
	
	ОбратнаяМетрика = "";
	ПреобразоватьТаблицуКоординатВМетрику(ТаблицаКоординат,ОбратнаяМетрика);
	Возврат ОбратнаяМетрика;

КонецФункции

TamerlanDE, 20 Декабря 2013

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

1C / Говнокод #14252

−104

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

Функция Дистанция(Откуда, Куда) Экспорт

	Пи = 3.1415926535897932;
	
	ОткудаШиротаВРадианах 	= Откуда.Широта * Пи / 180;
	ОткудаДолготаВРадианах 	= Откуда.Долгота * Пи / 180;
	КудаШиротаВРадианах 	= Куда.Широта * Пи / 180;
	КудаДолготаВРадианах 	= Куда.Долгота * Пи / 180;
	Аргумент				= Sin(КудаШиротаВРадианах) * Sin(ОткудаШиротаВРадианах) +
        Cos(КудаШиротаВРадианах) * Cos(ОткудаШиротаВРадианах) * Cos(КудаДолготаВРадианах - ОткудаДолготаВРадианах);
		
	Аргумент  = ?(Аргумент > 1, 1, Аргумент);
	Аргумент  = ?(Аргумент < -1, -1, Аргумент);
		
    Возврат Окр(6378000 * ACos(Аргумент));

КонецФункции

TamerlanDE, 20 Декабря 2013

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

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

+71

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

@Override
    List search(Long ownerId, Long projectId, String docnumber, String ctr1, String ctr2, Long dateFrom, Long dateTo, String contract,
                Double amountFrom, Double amountTo, Double vatAmountFrom, Double vatAmountTo, Double withVatAmountFrom,
                Double withVatAmountTo, Boolean defect, DocumentDefect d, Long vatId, Integer limit, String sortField, String order, String docType) {

        // доступные проекты
        List projects = projectDAO.findAll(ownerId)
        
        if(!projects){
            logger.warn("Ошибка отображения списка документов: нет доступных проектов: ownerId:$ownerId")
            return []
        }
        
        Criteria criteria = currentSession.createCriteria(DocumentView)
            .createAlias("document", "d")//
            .createAlias("d.project", "p")//
            .add(Restrictions.in("p.id", projects.id))//

//            if(projectId)
//                criteria.add(Restrictions.eq("d.project.id", projectId))

        if (docnumber)
            criteria.add(Restrictions.ilike("docNumber", docnumber, MatchMode.ANYWHERE))
        if (ctr1)
            criteria.add(Restrictions.ilike("supplierName", ctr1, MatchMode.ANYWHERE))
        if (ctr2)
            criteria.add(Restrictions.ilike("customerName", ctr2, MatchMode.ANYWHERE))
        if (dateFrom) {
            Calendar date1 = Calendar.instance
            date1.setTime(new Date(dateFrom))
            criteria.add(Restrictions.ge("docDate", date1))
        }
        if (dateTo) {
            Calendar date1 = Calendar.instance
            date1.setTime(new Date(dateTo))
            date1.add(Calendar.DAY_OF_MONTH, 1)
            criteria.add(Restrictions.lt("docDate", date1))
        }
        if (contract)
            criteria.add(Restrictions.ilike("contract", contract, MatchMode.ANYWHERE))
        if (amountFrom)
            criteria.add(Restrictions.ge("amount", amountFrom))
        if (amountTo)
            criteria.add(Restrictions.le("amount", amountTo))
        if (vatAmountFrom)
            criteria.add(Restrictions.ge("vatAmount", vatAmountFrom))
        if (vatAmountTo)
            criteria.add(Restrictions.le("vatAmount", vatAmountTo))
        if (withVatAmountFrom)
            criteria.add(Restrictions.ge("totalAmount", withVatAmountFrom))
        if (withVatAmountTo)
            criteria.add(Restrictions.le("totalAmount", withVatAmountTo))
        if (defect) {
            criteria.add(Restrictions.isNotNull("defect"))
            if (d)
                criteria.add(Restrictions.eq("defect", d))
        }
//        if (vatId)
//            criteria.add(Restrictions.eq("d.", withVatAmountTo))

        if (docType) {
            def dt = DocumentType.values().find { it.link == docType || it.code == docType }
            if (dt)
                criteria.add(Restrictions.like("docLink", dt.code, MatchMode.START))
            else
                logger.error("Неправильное значение параметра фильтрации по виду документа [d_t:$docType]. Допустимые значения ${DocumentType.values().code}")
        }

        // получаем общее кол-во записей без ограничений по странице
        def rowCount = criteria.setProjection(Projections.rowCount()).uniqueResult() as Integer

        List result = criteria.setProjection(null)//
                .setResultTransformer(Criteria.ROOT_ENTITY)//
                .addOrder(Order."$order"(sortField))//
                .setMaxResults(limit)//
                .list()

        [rowCount, result]

    }

поиск ?!

floppy, 20 Декабря 2013

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

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

+64

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

int message2 = -1;
      for (int i = 0; i < 5; i++)
      {
          QFile f;
          f.setFileName(QString("/var/log/messages.") + QString::number(i));
          if (f.exists())
          {
              message2 = i;
              break;
          }
      }

      if (message2 != -1 )
        QFile::copy(QString("/var/log/messages.")+QString::number(message2), QString(NMP_DATA_DIR) + QString("messages.") +  QString::number(message2) +"-"+ _section);

Пытается скопировать все файлы /var/log/messages, messages.0, ...

leotrubach, 19 Декабря 2013

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

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

+143

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

const int index64[64] = {
    0,  1, 48,  2, 57, 49, 28,  3,
   61, 58, 50, 42, 38, 29, 17,  4,
   62, 55, 59, 36, 53, 51, 43, 22,
   45, 39, 33, 30, 24, 18, 12,  5,
   63, 47, 56, 27, 60, 41, 37, 16,
   54, 35, 52, 21, 44, 32, 23, 11,
   46, 26, 40, 15, 34, 20, 31, 10,
   25, 14, 19,  9, 13,  8,  7,  6
};
 
/**
 * bitScanForward
 * @author Martin Läuter (1997)
 *         Charles E. Leiserson
 *         Harald Prokop
 *         Keith H. Randall
 * "Using de Bruijn Sequences to Index a 1 in a Computer Word"
 * @param bb bitboard to scan
 * @precondition bb != 0
 * @return index (0..63) of least significant one bit
 */
int bitScanForward(U64 bb) {
   assert (bb != 0);
   return index64[((bb & -bb) * 0x03f79d71b4cb0a89) >> 58];
}

нашёл на кывте

TarasB, 19 Декабря 2013

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

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

+153

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

/*где-то в начале: error_reporting(0);*/

class PhocaGalleryFile{
	function getCSSFile($id = 0, $fullPath = 0) {
		if ((int)$id > 0) {
			$db = &Factory::getDBO();
			$query = 'SELECT a.filename as filename, a.type as type'
				.' FROM #__styles AS a'
			    .' WHERE a.id = '.(int) $id;
			$db->setQuery($query, 0, 1);
			$filename = $db->loadObject();
			if (isset($filename->filename) && $filename->filename != '') {
				if ($fullPath == 1 && isset($filename->type)) {
					return self::getCSSPath($filename->type). $filename->filename;
				} else {
					return $filename->filename;
				}
			}
		}

		return false;
	}

 . . .
}

Поубивал пустые строки (по 2 штуки между каждой строкой кода) и комменты вида:
/* ===============
== This function is geting a CSSFile fron database .....
===============
*/
Гы! Код для заказчика должен выглядеть наукообразно и непонятно.
И обязательно выдавать Notices и Warnings.

virtual_cia, 19 Декабря 2013

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

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