Esenin1987 Сообщение 23/11/2010 10:54 Копия темы
Стилистика кода: лаконичность или развернутость? Интересует также и другой вопрос. К примеру, существует функция, которая отдает выборку по запросу из БД. 
1. private function _getData ($sql) {  
     return  (false !== ($resultSet = $db->query($sql)) && $resultSet->num_rows) ? $resultSet->fetch_assoc(): false;   
    }
2. private function _getData($sql) {
       
      $resultSet = $db->query($sql);
      if (!$resultSet) {
        return false;
      }

     if (!$resultSet->num_rows) {
      return false;
     }
     
     $data = $resultSet->fetch_assoc();
     
     return $data; 
   }

Для себя отметил некоторые преимущества первого и втрого подхода:
1. удобство кодирования, сокращение количества строк кода
2. простота чтения кода, возможность дебагить код в любом месте.

Может что нибудь еще?

hurumba Сообщение 23/11/2010 12:30 Копия темы
1)

if (!$resultSet){ 
return false; 
} 
if (!$resultSet->num_rows){ 
return false; 
}
/* (было) не понятна необходимость такой конструкции */

if (!$resultSet || !$resultSet->num_rows){
return false;
}
/* (стало, вариант 1) так быстрее, короче и понятнее */

if (empty($resultSet->num_rows)){
return false;
}
/* (стало, вариант 2) но мне по душе вот такой вариант */

hurumba Сообщение 23/11/2010 12:30 Копия темы
2)

$data = $resultSet->fetch_assoc(); 
return $data; 
/* (было) явно лишняя переменная */

return $resultSet->fetch_assoc();
/* (стало) и вот так гораздно лучше смотриться */

hurumba Сообщение 23/11/2010 12:31 Копия темы
3) В итоге, второй вариант упрощается в три раза — с 9-ти строчек до 3-х:

private function _getData($sql){         
$resultSet = $db->query($sql); 
if (empty($resultSet->num_rows)) return false; 
return $resultSet->fetch_assoc(); 
}

abbat Сообщение 23/11/2010 17:58 Копия темы
Код гораздо чаще читается, чем пишется.