- 1
- 2
- 3
function getAllStatuses() { # SHIT!
throw new Exception('OrdersAdmin::GetAllStatuses is shit');
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 8
+153
function getAllStatuses() { # SHIT!
throw new Exception('OrdersAdmin::GetAllStatuses is shit');
}
нашел в старом движке, видимо так и не вызывалось
+162
if(count($pacients)>-1) {
зачем так сложно писать if(true) ? да и зачем вообще..
+45
$parts = preg_replace('~[^0-9a-z-/]~', '', parse_url( $_SERVER['REQUEST_URI'] , PHP_URL_PATH ) );
$parts = explode('/', $parts );
$parts = array_values( array_filter( $parts, function( $var ){ return !empty( $var ); } ) );
функциональность пхп развращает
вместо array_filter с callback-ом можно было обработать строку trim($parts, '/');
если уж array_filter, то в данном случае прокатит и array_filter( $parts, 'strlen' );
это хотя бы не потребует обновления до версии 5.3
+162
$tmp_arr=array();
$id1_str='';
$root_str='';
foreach($id1 as $id){
$id1_str.=$id['id'].' , ';
if(!isset($tmp_arr[$id['root']])){
$root_str.=$id['root'].' , ';
$tmp_arr[$id['root']]=true;
}
}
это типа такой способ собрать distinct root ids в строку!
не забыть потом откусить хвост ' , '
нормальные герои не ищут легких путей
+159.5
function dbtime2unix($dt)
{
$year = substr($dt,0,4);
$month = substr($dt,5,2);
$day = substr($dt,8,2);
$hour = substr($dt,11,2);
$minute = substr($dt,14,2);
$sec = substr($dt,17,2);
return @mktime($hour,$minute,$sec,$month,$day,$year);
}
function timestamp2unix($dt)
{
$year = substr($dt,0,4);
$month = substr($dt,4,2);
$day = substr($dt,6,2);
$hour = substr($dt,8,2);
$minute = substr($dt,10,2);
$sec = substr($dt,12,2);
return @mktime($hour,$minute,$sec,$month,$day,$year);
}
...на случай если откажет strtotime!
+154.9
$time_portions = explode(' ', microtime());
$actual_time = $time_portions[1] . substr($time_portions[0], 1);
microtime(true) было бы слишком просто
+153.1
function thaw() {
$this->get_lock();
$vals = $this->that->ac_get_value($this->id, $this->name);
eval(sprintf(";%s",$vals));
}
vals, this, that.. еще не хватает комментов типа // eval something, return nothing
lock видимо отпускать и не надо...
+164.1
/**
function.php
*/
function result($selected, $sources, $date_from, $date_till, $step)
{
$ret = $header = array();
if (is_array($selected) and count($selected))
{
foreach ($selected as $item=>$value)
{
if ('on' == $value)
{
$header[$item] = $sources[$item]['name'];
if (isset($sources[$item]['db_host']))
{
// external db
class DB_EXT extends DB
{
.......
объявление класса внутри цикла внутри функции!!
рыдааййууу