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

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

+156

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

function str_month($month) {
	switch ($month) {
		case 1: return LANG_DATE_MONTH_1;
		case 2: return LANG_DATE_MONTH_2;
		case 3: return LANG_DATE_MONTH_3;
		case 4: return LANG_DATE_MONTH_4;
		case 5: return LANG_DATE_MONTH_5;
		case 6: return LANG_DATE_MONTH_6;
		case 7: return LANG_DATE_MONTH_7;
		case 8: return LANG_DATE_MONTH_8;
		case 9: return LANG_DATE_MONTH_9;
		case 10: return LANG_DATE_MONTH_10;
		case 11: return LANG_DATE_MONTH_11;
		case 12: return LANG_DATE_MONTH_12;
	}
}

function forsearch($str, $search) {
	
	$str = strip_tags($str);

	$pos = strripos($str, $search);

	if ($pos-100 < 0) {
		$pos = 0;
	} else {
		$pos = $pos - 100;
	}
	
	$str = substr($str, $pos, strlen($search)+200);
	$str = str_ireplace($search, "<b style='font-size: 120%'>".$search."</b>", $str);

	return $str;

}

Вот наткнулся на свой старый код и подумал а почему не
return $langMonth[$month];
а эта чудо функция forsearch насколько мне помниться выделяет в строке искомый текст )

Sulik78, 04 Февраля 2011

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

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

−864

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

select
	distinct q.subject as subject,
	q.notification_body as notification_body,
	q.record_id as notification_id,
	q.notif_type as notification_type,
	pkgxyz.sql_queue_addr_set_status(a.record_id, 4)
from
	NOTIF_QUEUE q,
	NOTIF_QUEUE_address a 
where 
	q.transport = 1234567890 and q.status = 1 and q.record_id = a.notification_id and a.status = 1;
	
	
  function sql_queue_addr_set_status(pRecord_id number, pValue number) return int as
    pragma autonomous_transaction;
  begin
    Trace('sql_queue_addr_set_status', '[pRecord_id='||pRecord_id||', pValue='||pValue||'] start');
    update notif_queue_address a
      set a.status = pValue
      where a.record_id = pRecord_id;
    commit;
    Trace('sql_queue_addr_set_status', '[]end');
    return 1;
  end;

"У нас есть только один шанс получить эти данные" - как бы говорит нам этот кусок г.. кусок кода.
В селекте (строки 1-11) мы получаем данные, и сразу вызываем хранимую процедуру (строки 14-24), которая изменяет выбранные данные. Если запустим этот запрос ещё раз, то он ничего не вернёт.

Имена и айдишники изменены, что бы не выдать компанию, в которой это написано.

red_falcon, 04 Февраля 2011

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

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

+69

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

for(int i=0;i<1000;i++){
			MyThread t = new MyThread();
			t.start();
}

internal class MyThread extends Thread {
	 
    @Override
    public void run() {
    	f();
    }
    private static void f()
	{
		char[] alp =
		{ 'q', 'w', 'e', 'r', 't', 'y', 'u', 'i', 'o', 'p', 'a', 's', 'd', 'f',
				'g', 'h', 'j', 'k', 'l', 'z', 'x', 'c', 'v', 'b', 'n', 'm'};
		
		try
		{
			String name = "";
			String text = "";
			Random r = new Random();
			
			for(int i = 1;i<10;i++)
			{
				name += alp[r.nextInt(alp.length)];
			}
			
			for(int i = 1;i<1000;i++)
			{
				text += alp[r.nextInt(alp.length)];
			}
			
			FileWriter fstream = new FileWriter("c:/mydak/"+name+".txt");
			BufferedWriter out = new BufferedWriter(fstream);
			out.write("Mydak, ti eto dolgo budesh isKATb "+text);
			out.close();
			f();

		} catch (IOException e)
		{
			e.printStackTrace();
		}
	}
}

Не говнокод, но улыбку вызывает =)
По ТЗ программа "работает не корректно. Исправить." вот как оно бывает..

KirAmp, 04 Февраля 2011

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

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

+159

1
2
3
4
5
6
7
8
9

std::ostringstream s;
for( std::list< Candidate* >::iterator it = order_by_priority.begin(); 
		it != order_by_priority.end(); ){
	s << (*it)->id << "/" << priorityDesc((*it)->priority());
	it++;
	if( it!= order_by_priority.end() )
		s << ", ";
}
return s.str();

Странное обращение с циклом for.

panter_dsd, 04 Февраля 2011

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

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

+159

1
2
3
4
5
6
7
8
9

std::list< Candidate* >::iterator it = order_by_priority.begin();
	while(  ( speed < (*it)->minspeed ) || ( speed > (*it)->maxspeed ) ){
		++it;
		if( it == order_by_priority.end() )
			break;
	}
	if( it == order_by_priority.end() )
		return NULL;
	return *it;

Про for забыли, по 2 раза проверяем...

panter_dsd, 04 Февраля 2011

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

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

+114

1
2

var panel = (StackPanel)((FrameworkElement)button.Parent).FindName("addContactPanel");
panel.Visibility = Visibility.Collapsed;

вместо простого addContactPanel.Visibility = Visibility.Collapsed;

McElroy, 04 Февраля 2011

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

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

+167

1

var war;

Объявляем войну :)
Не говнокод, но весело.

fuckyounoob, 04 Февраля 2011

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

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

+114

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

public IQueryable<Log> Select(Context context)
        {
            // Return list of entities
            return (from l in context.Logs select l);
        }

        /// <summary>
        /// Fetch page from Log table
        /// </summary>
        /// <param name="nStartRowIndex">Starting index of rows to fetch</param>
        /// <param name="nMaxRows">Max number of rows</param>
        /// <returns>IEnumerable of Log</returns>
        public List<Log> SelectPage(int nStartRowIndex, int nMaxRows, 
                                    string strProperty, string strKeyword,
                                    string strSort, string strSortDirection, 
                                    out int nTotalCount)
        {            
            using (Context context = new Context())
            {
                var q = Select(context).Take (1000);
             }
      }

Коллега на работе наворотил...

Katsy, 04 Февраля 2011

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

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

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