- 1
- 2
- 3
- 4
- 5
try{
self::throwingCoin();
}catch(Exception $e){
throw $e;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 9
−16
try{
self::throwingCoin();
}catch(Exception $e){
throw $e;
}
Два презерватива.
−23
public function isOverflowed(){
if(is_null($this->isOverflow)){
$this->isOverflow = ($this->getMemberCount() > $this->getMaxMembers());
}
return $this->isOverflow === true;
}
Придадим значимости результату.
−23
/**
* Конвертация формата времени в удобочитаемый вид
* @param mixed $data
* @param mixed $year
* @param mixed $time
* @param mixed $second
*/
public static function data_convert($data, $year, $time, $second){
$res = "";
$part = explode(" " , $data);
$ymd = explode ("-", $part[0]);
if(count($ymd)==1){
$ymd = explode (".", $part[0]);
$ymd[2]="20".$ymd[2];
$ymd = array_reverse($ymd);
}
$hms = explode (":", $part[1]);
if ($year == 1) {$res .= $ymd[2]; $res .= '.'.$ymd[1]; $res .= '.'.$ymd[0];}
if ($time == 1) {$res .= ' '.$hms[0]; $res .= ':'.$hms[1]; if ($second == 1) $res .= ':'.$hms[2];}
return $res;
}
На вход из БД могут заходить два вида дат: (mysql)
- TIMESTAMP "2016-11-13 05:29:38"
- VARCHAR "13.11.16 05:29"
На выходе получаем единый формат "16.11.2016 05:29"
Хранить дату в varchar ? Это модно ))
Гурманов прошу обратить внимание на строку 19. Как там органично не указаны {} в секундах
−27
$fields = array_keys($data);
$n = count($data);
for($i=0; $i < $n; $i++){
if($data[$fields[$i]]['type']==1){ ?>
<div class="incConfBox">
...
Для тренировки мозгов читающих коллег.
+10
if( ! empty($user->getClanId()) > 0 && ...
Обрефракторились :)
+61
if(!$del){
throw new Exception("При добавлении .......... возникла ошибка, обратитесь к администрации");
return false;
}
return true;
+65
$needDied = $this->battle_data['type'] == 104 ? true : false;
+145
switch ($_GET['otdel']) {
case null:
echo "Название отдела №1";
$_GET['otdel'] = 1;
break;
case 1:
echo "Название отдела №1";
break;
case 11:
// и так далее
Отдел по умолчанию.
Вариантов написать толково - тьма. Попробуем?
+177
if (!($user_info['room'] == 900002) || $user_info['room'] != 900002) {
header("Location: main.php");
die();
}
Разгребаю чужой код и нарыл.
Чтобы условие выглядело посолидней ... или я чего-нить не понимаю.