- 1
- 2
- 3
- 4
- 5
$isLoginned = true;
if (strlen(strval($isLoginned)) == 4) {
} else {
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+163.2
$isLoginned = true;
if (strlen(strval($isLoginned)) == 4) {
} else {
}
В одном из опенсурсных форумов в исходниках видел
+91.8
...
if DM.LastCOID <> -1 then
begin
if Pos(IntToStr(DM.LastCOID), Label1.Caption) <= 0 then
begin
Label1.Caption := 'Телефон: ' + '(' + IntToStr(DM.LastCOID) + ')';
DataSet['CHANNELOUTERID'] := IntToStr(DM.LastCOID);
end;
end;
...
LastCOID - номер внешней линии, который отображается в лейбле Label1 и заносится в поле базы. Как и чем думал автор вставляя второе условие? Самое главное, для чего? В итоге, софтина иногда пропускала сохранение LastCOID в БД, тем самым портя статистику и настроение клиента.
−117.2
my $cache_sql="SELECT `content`, `insertTime` FROM _cache WHERE `confid`='$$conf{confid}'";
...
$cache_sql.=" AND `location`=".$self->{dbh}->quote($url);
...
$cache_sql.=" AND `cgiparams`=".$self->{dbh}->quote($cgi_hash);
...
$cache_sql.=" AND `callparams`=".$self->{dbh}->quote($callp_hash);
...
$cache_sql.=" AND `userid`=".int($$params{userid});
...
$cache_sql.=" AND `groupid`=".int($$params{groupid});
my $replace_sql=$cache_sql;
$replace_sql=~s/SELECT.*?FROM\s+_cache\s+WHERE/REPLACE INTO _cache SET/s;
perl+sql
−101
my $iref=$self->{ditems}->_get_item({itemid=>$weather_itemid, set=>'Weather', fields=>['MinT', 'MaxT', 'Precip', 'Cloud', 'twater'], prefix=>$$p{prefix}});
if ($$iref{$$p{prefix}.'MinT'}<0 && $$iref{$$p{prefix}.'MaxT'}<=0) {
my $tmp=$$iref{$$p{prefix}.'MaxT'};
$$iref{$$p{prefix}.'MaxT'}=$$iref{$$p{prefix}.'MinT'};
$$iref{$$p{prefix}.'MaxT'}=$tmp;
};
#warn $$iref{$$p{prefix}.'MinT'} if !$$iref{$$p{prefix}.'MinT'}>0;
$$iref{$$p{prefix}.'T'}=int( ($$iref{$$p{prefix}.'MinT'}+$$iref{$$p{prefix}.'MaxT'})/2 );
$$iref{$$p{prefix}.'T'}='+'.$$iref{$$p{prefix}.'T'} if $$iref{$$p{prefix}.'T'}>0;
#$$iref{$$p{prefix}.'T'}=~s/\-/\&\#150\;/gs;
$$iref{$$p{prefix}.'MinT'}='+'.$$iref{$$p{prefix}.'MinT'} if ($$iref{$$p{prefix}.'MinT'}>0);
$$iref{$$p{prefix}.'MaxT'}='+'.$$iref{$$p{prefix}.'MaxT'} if ($$iref{$$p{prefix}.'MaxT'}>0);
$$iref{$$p{prefix}.'twater'}='+'.$$iref{$$p{prefix}.'twater'} if ($$iref{$$p{prefix}.'twater'}>0);
$$iref{$$p{prefix}.'time'}=$$p{daystart}+$$p{hour}*3600;
+161
if ($_GET['prot'] && !ereg("^[_a-zA-Z0-9]+$",$_GET['prot'])) $_GET['prot'] = "";
if ($_GET['id'] && !ereg("^[_a-zA-Z0-9]+$",$_GET['id'])) $_GET['id'] = "";
разбирал один скрипт и нашел там такую проверку
это особенно весело смотрится, если учесть, что в этих переменных должно быть число
+155
public function throwExceptions($flag = null)
{
if ($flag !== null) {
$this->_throwExceptions = (bool) $flag;
return $this;
}
return $this->_throwExceptions;
}
Зенд
+34
void dump_indexes(map<int,stride_slices> picks, ofstream &f);
{
f<<"# full size: "<<picks.size()<<endl;
for(map<int, stride_slices>::iterator
p=picks.begin(); p!=picks.end(); ++p)
{
f<<"## "
<<p->first<<"'st stride slices count: "
<<p->second.size()<<endl;
for(stride_slices::iterator
pp=p->second.begin(); pp!=p->second.end(); ++pp)
{
f<<"### "
<< p->first<<"'st stride, "
<<pp->first<<"'st slice size: "
<<pp->second.size()<<endl;
for(max_slice::iterator
ppp=pp->second.begin(); ppp!=pp->second.end(); ++ppp)
{
f<<"#### "
<<p->first<<"'st stride, "
<<pp->first<<"'st slice, "
<<ppp->first<<"'st chunk size:"
<<ppp->second.size()<<endl;
for(max_chunk::iterator
pppp=ppp->second.begin(); pppp!=ppp->second.end(); ++pppp)
f<<" "
<<p->first<<" "
<<pp->first<<" "
<<ppp->first<<" "
<<pppp->first<<" : "
<<pppp->second<<endl;
}
}
}
}
а как нормально просмотреть кучу вложенных контейнеров?
+144.9
$data=funny_parser($data);
//модуль недобавления слов вконец
$andtr=array('Андатра','Выпь','Пеликан','Пучеглаз','Хвостозуб','Декроль','Устрица','Челюстёлог','Рукоед','Йыждивение','Оторопь','Осётр','Кингура','Большой лось','Пученоска','Вомбат','Плоскогубка','');
$data=$data.' <span style="color:#93FF4A;font-size:15px">'.$andtr[rand(0,count($andtr)-1) ].'</span>';
кусок кода из upyachka.ru =)
этим всё сказано
+155
if ($id=$trips->create($user['id'], strip_tags($_POST['name']), $_POST['country'], strip_tags($_POST['city']), strip_tags($_POST['place']), $depart, $arrive, strip_tags($_POST['text']), $type)) {
+159.3
for ($x=5;$x<135;$x+=25){
$cislo=mt_rand(0,9);
$string.=$cislo;
}