- 1
our @hilight_critical_files = ( 'celebrities.txt', 'obscene.txt', 'lekarstva.txt' );
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 19
−165
our @hilight_critical_files = ( 'celebrities.txt', 'obscene.txt', 'lekarstva.txt' );
Глобальная переменная, неверно написанное английское слово, захардкоженные значения, транслит, транслит вперемешку с английскими словами<s>, и вообще perl</s>.
И всё это в одной короткой строке.
+160
this.isWait && result.push(campLinkCid);
result.push(this.isWait ? camp.bid : campLinkCid);
Нужно больше логики.
−153
sub asd {
print "WUT\n";
return { 1 => 2 };
}
while( my ( $k, $v ) = each( %{ asd() } ) ) {
}
http://ideone.com/EvoMvV
−151
if( my $row = &dbselectrow( $sql ) )
{
if( $row -> { 'ofid' } == 9782 ) # moscow
{
if( $row -> { 'jt' } =~ /Начальник /i )
{
&subscribe_to_management( $row -> { 'email' } );
}
}
}
Автоматическая подписка сотрудника руководящей должности на список рассылки для руководителей.
−166
l.handler is null and
s.serialn = l.serialn and
n.serialn = s.serialn and
k.kid = s.id and
oc.id = k.cid and
o.id = oc.orderid and
c.id = d.country and
d.id = (
case
when ( o.qr_dealer is not null ) then
o.qr_dealer
when ( s.pid = any( %s::int[] ) ) then
( case when (
n.expires <= ( now() + '90 days'::interval )
) then (
select
c.dealer
from
table_name_hidden_1 c
where
s.serialn = any( c.serials ) and
c.dealer is not null and
c.status = any( %s::int[] )
limit 1
) else null end )
else
(
select
d2.id
from
table_name_hidden_2 d2
where
d2.id = any( array[ s.dealer, o.for_partner ] ) and
d2.active = true and
(
(
d2.role = any( %s::int[] ) and
(
(
exists((
select
1
from
table_name_hidden_3
where
table_name_hidden_3.dealer = d2.id
limit 1
))
) or (
o.for_partner is not null and
d2.id = o.for_partner and
exists((
select
1
from
table_name_hidden_2 d3
where
d3.id = s.dealer and
d3.role = any( %s::int[] )
))
)
)
) or (
d2.role = any( %s::int[] ) and
(
(
o.for_partner is null
) or (
o.for_partner = s.dealer
)
)
)
)
limit 1
)
end
)
Зато одним запросом.
−158
sub get_number_from_possible_b64
{
my $s = shift;
my $rv = $s;
if( $s =~ /^\d+$/ )
{
$rv = int( $s );
} elsif( $s =~ /=$/ )
{
$rv = decode_base64( $rv );
}
return $rv;
}
Где-то есть что-то, что либо цифорка, либо строка, оканчивающаяся на знак равенства.
−139
>> # HTML/4.01 says that line breaks are represented as "CR LF" pairs (i.e., `%0D%0A')
>> $content =~ s/(?<!%0D)%0A/%0D%0A/g if defined($content);
HTTP::Request::Common 6.04, строка 86
http://cpansearch.perl.org/src/GAAS/HTTP-Message-6.04/lib/HTTP/Request/Common.pm
Оно просто берёт и изменяет передаваемый контент. Любой. В том числе просто бинарные данные.
−119
return { nocache => 1,
data => %{ &template_process() } -> { 'data' } };
Конечно же template_process возвращает HashRef сам по себе.
−145
sub is_home_license
{
my $self = shift;
my $rv = $self -> product() -> is_home_product();
unless( $rv )
{
unless( $self -> product() -> is_nl_product() )
{
if( ( int( $self -> licdata() -> { '1' } ) > 0 ) and ( int( $self -> licdata() -> { '1' } ) < 6 ) ) # 1 .. 5
{
$rv = 1;
}
unless( $self -> licdata() -> { '1' } )
{
if( $self -> licdata() -> { '8' } == 1 )
{
$rv = 1;
}
}
}
}
return $rv;
}
−149
sub is_demo_license
{
my $self = shift;
my $rv = 0;
if( ( $self -> dealer() -> id() == 116 )
and
( $self -> comments() =~ /^DEMO\s+STID/ ) )
{
$rv = 1;
}
return $rv;
}