Список говнокодов пользователя roman-kashitsyn

Всего: 101

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

+60

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

list($width, $height, $type, $attr) = getimagesize($file1);
     
if($width == $height)
{    
    set_time_limit(270);
    if(filesize($file1) > 102400)  return array(false, "Размер больше 100Кб!");
    if($type == 1)      {$file2 = $file1.'.gif'; $out = $tizer_id.'.gif'; $type = 'gif'; $ext = 'gif';}
    elseif($type == 2)  {$file2 = $file1.'.jpg'; $out = $tizer_id.'.jpg'; $type = 'jpg'; $ext = 'jpg';}
    elseif($type == 3)  {$file2 = $file1.'.png'; $out = $tizer_id.'.png'; $type = 'png'; $ext = 'png';}
    else return array(false, "Это не картинка!");
    /* ... */
}
// no square
else
{
    return array(array('img',false, "Изображение не квадратное!"));
}

Зачем жить, если no square

roman-kashitsyn, 19 Августа 2012

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

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

+74

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

@Override
public boolean equals(Object obj) {
    boolean answer = false;
    if (obj instanceof Date) {
        Date d = (Date)obj;
        answer = (this.value.equals(d));
    } else if (obj instanceof String) {
        Date d = parse((String)obj);
        answer = (this.value.equals(d));
    } else if (obj instanceof Calendar) {
        Calendar c = (Calendar)obj;
        answer = (this.value.equals(c.getTime()));
    } else if (obj instanceof AtomDate) {
        Date d = ((AtomDate)obj).value;
        answer = (this.value.equals(d));
    }   
    return answer;
}

Симметрия отношения эквивалентности? Не, не слышал.

roman-kashitsyn, 05 Августа 2012

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

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

+106

1

IDictionary activeProcesses = (IDictionary)(new Hashtable());

По мотивам

http://govnokod.ru/11512

Код встречен в книге

http://tinyurl.com/eai-patterns

Можно объяснить человеку, не знакомому с тонкостями C#, зачем это нужно, если Hashtable : IDictionary?

roman-kashitsyn, 01 Августа 2012

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

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

+67

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

function trim_urls($urls)
{
    $out = '';
    $arr = explode("\n", $urls);
    foreach ($arr as $v)
    {    
        $u = strip_tags(trim(preg_replace("/:.*$/i", "", preg_replace("/\/.*$/i", "", preg_replace("/^www\./i", "", preg_replace("/^http:\/\//i", "", preg_replace("/^https:\/\//i", "", $v)))))));
        if(!empty($u))
        {    
            $out .= $u;
            $out .= "\r\n";
        }    
    }    
    return $out;
}

Русская матрёшка.

roman-kashitsyn, 18 Июля 2012

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

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

+74

1
2
3
4
5
6
7
8
9

reader = new BufferedReader(new FileReader(file));
//null means file end
while ((tempString = reader.readLine()) != null) {
    if(tempString !=null && tempString.indexOf('=')>0){
        sheet.addCell(new Label(KEY_COLUMN,++ROW, tempString.substring(0,tempString.indexOf('='))));
        sheet.addCell(new Label(ENGLISH_COLUMN,ROW, tempString.substring(tempString.indexOf('=')+1)));
    }
}
reader.close();

Вот так мы парсим файл *.properties в Java.

roman-kashitsyn, 03 Июля 2012

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

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

+78

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

String result = "";

for (Object obj : col) {
    if (obj instanceof String) {
        result += obj + SEPARATOR;
    } else {
        result += obj.toString() + SEPARATOR;
    }
}
					
if (result.length() >= 2) {
    result = result.substring(0, result.length() - SEPARATOR.length());
}

instanceof String доставил.
Кстати, к проекту подключены apache.commons.lang и guava, так что способов сделать join строк предостаточно.

roman-kashitsyn, 19 Июня 2012

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

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

+122

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

<table border='0' cellspacing='1' cellspacing='1' width=707>
  <thead>
    <a>
      <th width=44 align='center'>
        <IMG name='Image300' SRC='images/check.gif' height='22' width='44'>
      </th>
    </a>
    <a href=javascript:SortForm('Agent_Name');>
      <th  class='header'>Name</th>
    </a>
    <a href=javascript:SortForm('Description');>
      <th class='header'>Description</th>
    </a>
  </thead>

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

roman-kashitsyn, 28 Мая 2012

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

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

+76

1
2
3
4
5
6
7
8
9

$sizes = array(50,60,70,80,90,100,110,120,130,140,150,160,170,180,190,200);
foreach ($sizes as $v2)
{
    $file3 = $file1."-".$v2."x".$v2.".".$ext;
    if(file_exists($file3)) unlink($file3);
    $c = "convert $file2 -resize ".$v2."x".$v2." -quality 80 -strip $file3";
    $last_line = system($c, $retval);
    chmod($file3, 0777);
}

Мне одному кажется, что порождать пару десятков процессов из php скрипта при загрузке небольшой картинки немного... толсто?

roman-kashitsyn, 09 Мая 2012

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

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

+156

1
2
3
4
5
6
7
8
9

foreach ($items as $item_id)
{
    if ((int)$item_id<=0)
        die();
    // ...
    if (!$redis->sIsMember("views_today_stats_sadd:{$views_stats_namespace}",$key))
        $redis->sAdd("views_today_stats_sadd:{$views_stats_namespace}",$key);
    // ...
}

Пехепешник второго уровня: циклы уже осилил, а вот с множествами пока не разобрался.

roman-kashitsyn, 14 Апреля 2012

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

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

+75

1

if(batchControlImpl.startJobActivity(JobStatusType.STARTED));

К проекту подключили активированных активных индусов.

http://tinyurl.com/cvwefek

roman-kashitsyn, 09 Апреля 2012

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

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