PHP — Говнокод.ру

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

+49

1
2
3
4

If($z["timeout"]==1 || $z['timeout']==3 || $z['timeout']==4 || $z['timeout']==5 || $z['timeout']==7 || $z['timeout']==10) { 
}  else {
     $z['timeout'] = 3;
}

'<>' и 'or' ? неее, не слышали!

RickMan, 17 Января 2013

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

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

+42

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

function test($method)
{
  $picfile = 'pic1.png';
  $bgfile = 'output.png';
  $background = 'pic2.png';
  $foreground = 'pic3.png';

  if ($method == 'Imagick') {
    $img = new Imagick($picfile);

    $mask = new Imagick($background);
    $img->compositeImage($mask, imagick::COMPOSITE_COPYOPACITY, 0, 0);
    $mask->destroy();

    $overlay = new Imagick($foreground);
    $img->compositeImage($overlay, imagick::COMPOSITE_OVER, 0, 0);
    $overlay->destroy();

    $img->setImageFormat('png');
    file_put_contents($bgfile, $img->getImageBlob()); // $img->writeImage($bgfile) работает медленнее
    $img->destroy();
  } else if ($method == 'Wand') {
    $img = NewMagickWand();
    MagickReadImage($img, $picfile);

    $mask = NewMagickWand();
    MagickReadImage($mask, $background);
    MagickCompositeImage($img, $mask, MW_CopyOpacityCompositeOp, 0, 0);
    DestroyMagickWand($mask);

    $overlay = NewMagickWand();
    MagickReadImage($overlay, $foreground);
    MagickCompositeImage($img, $overlay, MW_OverlayCompositeOp , 0, 0);
    DestroyMagickWand($overlay);

    MagickSetImageFormat($img, 'png');
    file_put_contents($bgfile, MagickGetImagesBlob($img)); // ditto
    DestroyMagickWand($img);
  } else {
    $cmdline = 'convert -compose copy-opacity ' . $picfile . ' ' . $background . ' -composite';
    $cmdline .= ' -compose src-over ' . $foreground . ' -composite ' . $bgfile;
    exec($cmdline);
  }
}

$methods = array('Imagick', 'Wand', 'Command line');
foreach ($methods as $m) {
  $start_time = microtime(true);
  for ($i = 0; $i < 4; $i++) {
    test($m);
  }
  $elapsed_time = microtime(true) - $start_time;
  echo 'Method: ' . $m . '; elapsed ' . strval($elapsed_time) . PHP_EOL;
}

Результаты выполнения на локальной машине:
Method: Imagick; elapsed 0.45...
Method: Wand; elapsed 0.82...
Method: Command line; elapsed 0.87...
Результаты выполнения на VPS:
Method: Imagick; elapsed 79.64...
Method: Wand; elapsed 151.64...
Method: Command line; elapsed 46.49...

Что-то тут не так...

inkanus-gray, 17 Января 2013

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

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

+54

1
2
3
4
5
6
7
8

// sys
if ($flag==1) {
        $rr = implode("</B>, <B>",$nks1)."</B> и <B>".implode("</B>, <B>",$nks2);
} elseif ($flag==2) {
    $rr = implode("</B>, <B>",$nks1)."</B> и <B>".implode("</B>, <B>",$nks2)."";
} else {
    $rr = implode("</B>, <B>",$nks1)."</B> и <B>".implode("</B>, <B>",$nks2)."";
}

Это нашел в движке одной онлайн игры... Три проверки, но 1 и то же действие... Логика, мать её...

RickMan, 16 Января 2013

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

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

+55

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

<?php

// ... много строк до этого

echo "<table width=100% border=0>";
if ($row = mysql_fetch_array($result))
	for ($i = 0; $i < mysql_num_fields($result); $i++):	
		if (mysql_field_name($result, $i) == "ID" || (mysql_field_name($result, $i) == "info") ||
				mysql_field_name($result, $i) == "state" || (mysql_field_name($result, $i) == "date_st") ||
				mysql_field_name($result, $i) == "date" || (mysql_field_name($result, $i) == "email") ||
				mysql_field_name($result, $i) == "flag" || (mysql_field_name($result, $i) == "limit_time") ||
				mysql_field_name($result, $i) == "cr_time" || mysql_field_name($result, $i) == "cr_money" ||
				mysql_field_name($result, $i) == "flag" || (mysql_field_name($result, $i) == "info") ||
				mysql_field_name($result, $i) == "attention" || mysql_field_name($result, $i) == "location" ||
				mysql_field_name($result, $i) == "mailflag" || (mysql_field_name($result, $i) == "radio") ||
				mysql_field_name($result, $i) == "radio30" || (mysql_field_name($result, $i) == "date_num") ||
				mysql_field_name($result, $i) == "number" || mysql_field_name($result, $i) == "login_i"
				|| mysql_field_name($result, $i) == "is_juridical")
		{
			if (mysql_field_name($result, $i) == "state")
			{
				echo "<select name=state style=\"background-color:red;color:white\"><option value=OPEN";
				if ($row[$i] == "OPEN")
					echo " selected ";
				echo ">" . trans("OPEN") . "</option>\n";
				echo "<option value=CLOSE";
				if ($row[$i] == "CLOSE")
					echo " selected ";
				echo ">" . trans("CLOSE") . "</option></select\n";
			}
			elseif (mysql_field_name($result, $i) == "location")
			{
				genLeased($row[$i]);
			}
			elseif (mysql_field_name($result, $i) == "radio30" || mysql_field_name($result, $i) == "radio" || 
				 mysql_field_name($result, $i) == "mailflag" )
			
			{
			}
			elseif (mysql_field_name($result, $i) == "date_st")
			{
				$ar = explode("-", $row[$i]);
				echo "<b id=red>" . $ar[2] . "." . $ar[1] . "." . $ar[0] . "</b></td></tr>\n";
			}
			elseif (mysql_field_name($result, $i) == "date_num")
			{
				$ar = explode("-", $row[$i]);
				$dnum = $ar[2] . "." . $ar[1] . "." . $ar[0];
				
				echo "<b>" . $dnum . "</b></td></tr><tr><td><a href=\"contracts/index.php?user_id=" . $row['ID'] . "\">контракти</a></td></tr><tr><td colspan=2><hr noshade></td></tr>\n";
			}

			elseif (mysql_field_name($result, $i) == "date")
			{
				$ar = explode("-", $row[$i]);
				$row[$i] = $ar[2] . "." . $ar[1] . "." . $ar[0];
				echo "<input type=text name=date value=\"" . $row[$i] . "\"</td></tr>\n";
			}
			elseif (mysql_field_name($result, $i) == "info")
			{
				echo "<textarea  rows=3 name=" . mysql_field_name($result, $i) . " >" . $row[$i] . "</textarea></td></tr>\n";
			}
			elseif (mysql_field_name($result, $i) == "attention")
			{
				echo "<textarea id=warn rows=3 name=" . mysql_field_name($result, $i) . " >" . $row[$i] . "</textarea></td></tr>\n";
			}
// ... и так обрабатываем далее все поля

Разработчик сначала запросил результат с помощью mysql_fetch_array,
потом ищет название каждого поля с помощью mysql_field_name, функцией trans()
читает файлик со словарем и переводит название каждого поле.

Писалось не индусами, вариант "для количества" не прокатывает.
Поэтому я не могу даже предположить, что сподвигло автора на создание сего творения.
Файл содержит около 1400 строчек.

rsvasilyev, 16 Января 2013

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

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

+66

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

// генрация паролей
function generatePassword($l = 8) {
	function make_seed() {
		list($usec, $sec) = explode(' ', microtime());
		return (float) $sec + ((float) $usec * 100000);
	}
	$le = split('-', '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-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-7-2-1-8-6-3-8-7-2-1-6-0-9-g-s-a-d-f-k-a-k-j-w-q-h-2-7-8-1-6-3-g-d-h-m-H-U-F-G-K-J-H-A-S-G-J-G');
	$password = '';
	while (strlen($password)<$l) {
		srand(make_seed());
		$rnd = rand(0, count($le)-1);
		$password.=$le[$rnd];
	}
	return $password;
}

Крайне хардкорный генератор случайных паролей заданной длины. Особенно гениальна строчка №7. Всем говнокоду за мой счет!

SunnyMagadan, 15 Января 2013

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

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

+51

1
2
3
4

function striptags($variables_for_function=array()) {
  if (!isset($variables_for_function[1])) {$variables_for_function[1]='';}
  return strip_tags($variables_for_function[0], $variables_for_function[1]);
 }

Только сегодня и только для вас, безопасный strip_tags. Теперь с заSHITой от говнокодеров.
PHP Innovations.

SunnyMagadan, 15 Января 2013

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

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

+34

1

if (date('dmY', $lmtime) === date('dmY')) {

Нужно было узнать, не сегодняшний ли день в отметке $lmtime.

7ion, 15 Января 2013

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

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

+55

1
2
3

$result["rg"][count($result["rg"])-1]["status"][] = Array();
$c_res = count($result["rg"][count($result["rg"])-1]["status"])-1;
$result["rg"][count($result["rg"])-1]["status"][$c_res] = $arr_source_data[$i];

Им за число строк платят, что ли?

clauclauclau, 14 Января 2013

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

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

+52

1
2
3
4

$new_addresses = $_POST['friendE1']."\n".$_POST['friendE2']."\n".$_POST['friendE3']."\n".$_POST['friendE4']."\n".$_POST['friendE5']
    ."\n".$_POST['friendE6']."\n".$_POST['friendE7']."\n".$_POST['friendE8']."\n".$_POST['friendE9']."\n".$_POST['friendE10'];

$list = explode( "\n", $new_addresses);

Отакое счастье в очередном проекте :-)

tsybulskyserg, 11 Января 2013

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

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

+62

1

$data['date'] = date('Y'.'-'.'m'.'-'.'d'.' '.'G'.':'.'m'.':'.'s');

nicksevenfold, 11 Января 2013

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

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

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

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

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

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

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

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

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

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

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

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