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

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

+149

1

return str_replace("\t", '', str_replace("\n", '', str_replace('"', '\"', get_partial($partial, $data))));

Это вместо простого
return str_replace(array("\n","\t"), '', get_partial($partial, $data));

programmer, 18 Июля 2011

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

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

+160

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

function generateMessage($template, $tokens){
      foreach($tokens as $key => $value){
        $search[] = $key;
        $replace[] = $value;
      }
      return str_replace($search, $replace, $template);
    }

$template = "Здравствуйте, %contact%! К нам поступил запрос на восстановление пароля на сайте...";
$token['%contact%'] = 'Иван';
generateMessage($template, $token);

вместо:
sprintf("Здравствуйте, %s! К нам поступил запрос на восстановление пароля на сайте...", 'Иван');

нашёл в коде на работе

morozovsk, 17 Июля 2011

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

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

+147

1
2
3
4
5
6
7

if (strrpos(' ' . $lsText, $v)) {
//
}
вместо 
if (strpos($lsText, $v) !== false) {
//
}

нашёл у себя на сайте vovkontakte.ru :) как же давно я его писал...

morozovsk, 17 Июля 2011

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

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

+165

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

<?php
$str = $data;
$length = 10;
 
function strEx($str, $length){
$str = explode(" ", $str);
$nstr = array();
for($t=0;$t<count($str);$t++){
$strl = strlen(implode($nstr));
$strr = strlen(implode($nstr)." ".$str[$t]);
if($strl<$length && $strr<$length){
array_push($nstr, " ".$str[$t]);
}else{
return trim(implode($nstr));
}
}
}
echo strEx($str, $length ); 
echo ($_POST["data"]);

Жду ваших предложений

?>

На фрилансе оставили такой заказ (пунктуация и синтаксис сохранены):

Нужен скрипт обрезания строки доделать . Сейчас почемуто не работает.
Сам текст передается программой в переменную $data

И собственно приведён код скрипта в неизменном виде)

glutaminefee, 17 Июля 2011

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

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

+145

01
02
03
04
05
06
07
08
09
10
11

скажите плиз - что не правильно в этом цикле на php?
 
for($tags == true;) 
{ 
$tags = explode(",",$countnews['tags']); 
$num_tag++; 
echo $tags[$num_tag]; 

} 

это у меня для вывода меток как на ответах

href=/question/60981758/

nislova, 17 Июля 2011

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

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

+165

1
2
3
4
5
6
7
8
9

day = new Date(); 
hour = day.getHours(); 
if (hour>=0 && hour<9) greeting = "Ночной NON-STOP"; 
else { if (hour>=10 && hour<12) greeting = "По вашим заявкам"; 
else { if (hour>=12 && hour<13) greeting = "Базар о интересном"; 
else { if (hour>=13 && hour>=17) greeting = "Дневной NON-STOP";
else { if (hour>=17 && hour<=20) greeting = "Расслабляющая музыка";
else { if (hour>=20 && hour<=24) greeting = "Музыка 90-х"; }}}}  
document.write(greeting);

Автор кода явно не знал, что в js можно использовать else if. Выражение hour>=12 && hour<13 можно было упростить, написав hour == 12. И фигурную скобку в конце закрыть забыл.
(Отсюда: intobservatory.ru/forum/index.php/topic,4141.0.html)

high-five, 16 Июля 2011

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

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

+153

1
2
3
4
5
6

if ($myrow9['fupload'] =='' or empty($myrow9['fupload']))
                {
$avachat = "../file/avatars/net-avatara.jpg";
                }  else {
$avachat = $myrow9['fupload'];
                }

Говно высшей пробы....

invision70, 16 Июля 2011

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

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

+164

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

if (check($user))			
{
				$names='(';
				$values="values(";
				foreach ($user as $key => $value )
				{
					$names=$names.$key.',';
					$values=$values."'".$value."',";
				}
				$names[strlen($names)-1]=')';
				$values[strlen($values)-1]=')';
			}
		$sql="insert into users ".$names.' '.$values;

друг говорит, что это говнокод, но это же не правда ?

syntheticcharms, 16 Июля 2011

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

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

+162

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

/*Проверка авторизации*/
if ($action == 'login') {
 if (($entlogin && $entpass) <> '') {
  $per = "SELECT * FROM ".$sql_dop."users";
  if(!mysql_query($per)) {
   $page = 'error';
   $id = '2,1,1';
  }
  $result = mysql_query($per);
  $qwe = 0;
  while ($row = mysql_fetch_row($result)) {
   if ($entlogin == $row[1] && md5($entpass) == $row[2]) {
    $qwe = 1;
	$date = date("d m Y H:i:s");
	$mmsec = microtime(msec);
	$kod = md5($date.$row[0].$row[1].$row[2].$mmsec);

	$does_into_sessions = "INSERT INTO ".$sql_dop."sessions(user,kod) VALUES ('$row[0]','$kod')";
	if(!mysql_query($does_into_sessions)) {
	error(3,1,1);
	}				
	setcookie('session_code', $kod);
	$ubr = 1;
	if ($row[10] == 2) {
		$urb = 2;
	}
	if ($row[10] == 3) {
		$urb = 2;
	}
	setcookie('userbar', $urb);
	$session_code = $kod;
	$page = '';
   }
  }
 } else {
   $page = 'error';
   $id = '5,0,0';

 }
 if ($qwe == 0) {	
	echo "<strong>Ошибка авторизации...</strong>";
	echo "<meta http-equiv='refresh' content='1; url=index.php'>";
	exit();
 }
}

Авторизация от старого программиста

Amadey, 16 Июля 2011

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

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