- 1
- 2
- 3
if (!driver.findElement(By.id(DD_LAUNCH_ID)).equals(null)) {
pause(1000);
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+80
if (!driver.findElement(By.id(DD_LAUNCH_ID)).equals(null)) {
pause(1000);
}
Тогда уж почему не null.equals(...)?
+76
function returnUserId() {
if (document.getElementById('userstap') != null ) {
var obj = document.getElementById('userstap');
<% if ((sUserIdForPrint != null) && !sUserIdForPrint.equals("")) {%>
if (obj.options[obj.selectedIndex].value == 0) {
return "<%=sUserIdForPrint%>";
<% } %>
<% if (((sAccGrpUserIdForPrint != null) && !sAccGrpUserIdForPrint.equals("")) && ((sUserIdForPrint != null) && !sUserIdForPrint.equals(""))) {%>
} else if (obj.options[obj.selectedIndex].value == 1 ) {
return "<%=sAccGrpUserIdForPrint%>";
<% } else if ((sAccGrpUserIdForPrint != null) && !sAccGrpUserIdForPrint.equals("")){%>
if (obj.options[obj.selectedIndex].value == 1 ) {
return "<%=sAccGrpUserIdForPrint%>";
<% } %>
<% if (((sAccGrpUserIdForPrint != null) && !sAccGrpUserIdForPrint.equals("")) || ((sUserIdForPrint != null) && !sUserIdForPrint.equals(""))) {%>
} else if (obj.options[obj.selectedIndex].value == 2) {
return "<%=userId%>";
}
<% } else {%>
if (obj.options[obj.selectedIndex].value == 2) {
return "<%=userId%>";
}
<% } %>
} else {
return "<%=userId%>";
}
}
Любите ли вы JSP так, как люблю его я?
+79
// Number a, Number b
return new Double(a.doubleValue() - numericTolerance).compareTo(b.doubleValue()) < 0
В продолжение #16698, после рефакторинга. Видимо, диагноз: ООП головного мозга.
−153
eval {
my $from = $class->get_env_sender($message);
$smtp->mail($from)
|| croak("Email::Send::Gmail: error sending 'from' $from");
my @to = $class->get_env_recipients($message);
my @ok = $smtp->to( @to, { SkipBad => 1 } )
|| croak("Email::Send::Gmail: error sending 'to' @to");
if ( @to != @ok ) {
my %to;
@to{@to} = (1) x @to;
delete @to{@ok};
@bad = keys %to;
}
croak("Email::Send::Gmail: no valid recipients") if @bad == @to;
};
Думаю понятно, какой это модуль с CPAN. Убило незнание приоритетов, в результате чего "|| croak" всегда возвращало в @ok число вместо результата выполнения $smtp->to
−161
error_response("400 Bad request", "") unless (<> =~ /^GET\s+(.+?)\s+HTTP\/1\.1\r$/);
my $request_url = $1;
log_message "REQUEST $request_url";
get_bans() if ($request_url =~ /^\/bans\?auth=$authkey$/);
remove_ban($1) if ($request_url =~ /^\/bans\/(\d+\.\d+\.\d+\.\d+)\/remove\?auth=$authkey$/);
error_response("404 Not found", "");
REST API через inetd (на этом серваке нету веб сервера, и ставить его влом). Новая жизнь старых технологий ;)
+164
public function is_number($s){
$result = true;
for($i=0;$i<strlen($s); $i++){
switch($s[$i]){
case '0': break;
case '1': break;
case '2': break;
case '3': break;
case '4': break;
case '5': break;
case '6': break;
case '7': break;
case '8': break;
case '9': break;
default: $result = false; return;
}
}
return ($result) ? $s : int($s);
}
+132
Удалено по просьбе автора
−166
Дата = Формат(День(Запись.Период),"ЧЦ=2; ЧРГ=; ЧВН=") + "/" + Формат(Месяц(Запись.Период),"ЧЦ=2; ЧРГ=; ЧВН=") + "/" + Формат(Год(Запись.Период),"ЧГ=");
О как можно еще получить дату в формате: день/месяц/год.
Хотя тот же результат будет, если: Дата = Формат(Запись.Период, "ДФ=dd/MM/yyyy");
И этот бред писал специалист крупной внедренческой компании.
−143
$minfts = $maxfts = undef;
$c=$l=$x=$z=0;
while (($id, @a) = $sth->fetchrow_array) {
$l++;
next unless defined($t{$a[1]});
$c++;
$s{$id} = [$r, $id, @a];
$x++ if $s{$id}{rc} && !$s{$id}{rm};
$z++ if $s{$id}{rm};
$s{$id}{t} = $t{$s{$id}{tid}};
$s{$id}{bonuses} = 0;
push @{$sid{$s{$id}{sid}}}, $id;
### PATCH3 ###
$sid2pid{$s{$id}{sid}} = $s{$id}{id2};
### END PATH3 ###
$minfts = $s{$id}{sd} if !defined($minfts) || $s{$id}{sd}<$minfts;
$maxfts = $s{$id}{ed} if !defined($maxfts) || ($maxfts && $s{$id}{ed}>$maxfts) || !$s{$id}{ed};
}
$sth->finish;
$l-=$c;
Минимализм в название переменных
+127
<div class="video row">
<div class="col-md-4"></div>
<div class="col-md-8" class="gray-bg"></div>
<div class="container an">
<div class="row">
<div class="col-md-4" style="background-color:white"></div>
<div class="col-md-8 gray-bg"></div>
</div>
</div>
</div>
<div class="conn">
<div class="container">
<div class="row video-item">
<div class="col-md-4"></div>
<div class="col-md-8">
<div class="col-md-4"><img src="img/video.jpg"></div>
<div class="col-md-8"></div>
</div>
</div>
<div class="row video-item">
<div class="col-md-4"></div>
<div class="col-md-8">
<div class="col-md-4"><img src="img/video.jpg"></div>
<div class="col-md-8"></div>
</div>
</div>
<div class="row video-item">
<div class="col-md-4"></div>
<div class="col-md-8">
<div class="col-md-4"><img src="img/video.jpg"></div>
<div class="col-md-8"></div>
</div>
</div>
</div>
</div>
И стили:
.video div.col-md-4{
left:0;
}
.video div.col-md-8{
right:0;
background-color:#f2f2f3;
}
.conn {
top:65px;
z-index:1;
width:100%;
position:absolute;
}
.menu {
position:absolute;
z-index:1;
}
.an {
position:static !important;
height:100000px !important;
overflow:hidden !important;
}
.an div {
height:100%;
}
.video>div{
position:fixed;
height:100%;
min-height:100%;
max-height:100%;
}
.video {
position: fixed;
min-height: 100%;
height: auto;
width: 100%;
top: 0;
}
.video-item >div.col-md-8 > div.col-md-8 {
border:1px solid #e0dfdd;
border-right:0;
}
.video-item >div.col-md-8 > div.col-md-4 {
height:150px;
width:150px;
padding:0;
border:1px solid #e0dfdd;
}
.video-item >div.col-md-8 {
padding:0;
}
.video-item div,
.video-item {
height:150px;
}
.video .row .col-md-8 {
background-color:#F2F2F3;
}
Это пздц. дизайнер, из-за него пришлость так говнокодить, ебанная ширина на 100000....