Часто встречающиеся ошибки

Вид для печати

23.12.2009, 15:28
Apay

Часто встречающиеся ошибки
Огромная просьба в этой теме не мусорить, аккуратно и точно указывать на часто встречающиеся ошибки и правильные варианты.

Очень часто встречается ошибка при выборке данных из базы, она есть в учебнике некоего Попова.
Для выборки из базы люди используют цикл do-while вместо while, например
```
$result = mysql_query( 'SELECT * FROM table WHERE enabled=1' );

$row = mysql_fetch_array( $result );

do{

  echo '<p>Data: '.$row['data'].'</p>';

}while( $row = mysql_fetch_array( $result ) );
```
в корне неверно, надо
```
$result = mysql_query( 'SELECT * FROM table WHERE enabled=1' );

while( $row = mysql_fetch_array( $result ) ){

  echo '<p>Data: '.$row['data'].'</p>';

  }
```
У первого варианта проверка условия идет после выполнения цикла, у второго - перед. То есть если sql запрос вернет ноль строк, то в первом варианте цикл все равно выполнится, чего в принципе быть не должно.
24.01.2010, 20:55
Apay
Заголовки
--
Многие начинающие допускают следующую ошибку в применении функций, работающих с заголовками (такиих как setcookie, header): функия вызывается после того как был осуществлен вывод.

Выводом является результат работы echo, print, любой текст (включая пробелы, перевод строки, заголовки BOM в utf-8) вне <?php и ?>.

приведу пару примеров неверных и верных решений (способов верного решения очень много, я приведу только по одному)

пример 1
неверно
файл index.php
```
<?php

include( 'header.php' );

...

setcookie( $name, $value );

...

?>
```
файл header.php
```
<!DOCTYPE ...> 

<html...

...

<?php

...

?> 
```
решение
```
<?php

ob_start();

include( 'header.php' );

...

setcookie( $name, $value );

header( ... );

...

ob_end_flush();

?>
```
пример 2 (обработчик формы)
неверно
```
<!DOCTYPE ...> 

<html...

...

<?php

if( empty($_POST['name']) ){

  echo 'Вы не ввели имя, заполните форму заново<br /><a href="form.html">Перейти к форме</a>';  

  header( 'Refresh: 10; url=form.html' );

  }

...

?> 

...

</html>
```
решение
```
<?php

$output = '';

if( empty($_POST['name']) ){

  $output = 'Вы не ввели имя, заполните форму заново<br /><a href="form.html">Перейти к форме</a>';  

  header( 'Refresh: 10; url=form.html' );

  }

 ...

?><!DOCTYPE ...> 

<html...

...

<?php echo $output; ?>

...

</html>
```
22.03.2010, 09:31
ZhukV
Часто втречаются ошибки при выводе кареток (перевод на другую строку)
Если указать каретку (\r\n) в одинарных кавычках, действия не будет никакого!!! Нужно обезательн в двойных!!!
ПРИМЕР:
```
$text='ZhukV\r\n';//НЕВЕРНО

$text="ZhukV\r\n";//ВЕРНО
```
Также чато есть, когда хотят включить в текстувую переменную элемент массива. В таком случае нужно эелемнет включить либо в фигур скобки либо через сцепление текста.
ПРИМЕР
```
$array['ZhukV']='ZhukV';

$text="text -- $array['ZhukV']";НЕВЕРНО

$text="text -- ".$array['ZhukV'];ВЕРНО

$text="text -- {$array['ZhukV']}";ВЕРНО
```
Также заметил, что очень часто встречаются ошибки при инклюд файла. Лучше сначала проверить его наявность, а аж потом подключать:
```
$inc_file='inc/zhukv.php';

if(is_file($inc_file))include($inc_file);

else echo 'not_file';
```
26.03.2010, 16:39
Zippovich

Еще про перевод строки.
Перевод строки в разных ОС разный:

1. Windows, MacOS:
\r\n

2. Unix, MacOS X:
\n
16.04.2010, 15:35
ZhukV
Очень часто приходится проверять данные. Вот к примеру оброботчика форм, на истинность веденных даных.
Шаря по инету, и по своей практике, заметил в этом одну КОНЧЕНУЮ ошибку начинающего програмера.
Вот к примеру проверка поля формы (метод передачи $_GET) с именем name
```
if(isset($_GET['name'])){

  ....

}
```
Либо при выборке с базы
```
$sql="SELECT * FROM table";

$ires=mysql_query($ires);

$ires_result=mysql_fetch_array($ires);

if(isset($ires_result)){

  ....

}
```
И вот в этом есть некая ошибка начинающего програмера, а особенно в функции isset()
Ошибка в том, что функция проверит, есть ли переменная в ядре PHP. Проверяет только переменную, а не ее значение.. С этого можна сказать, что если переменная имеет нулевое значение, isset() все равно вернет true

Значение переменной лучше всего проверять при помощи empty()
Вернет true если переменная значение не имеет, в противном случае false
```
if(!empty($_GET['name'])){

  ....//если имеется переменная и имеет значение кроме нуля

}
```
16.04.2010, 15:56
alexdrob
Цитата:

Сообщение от ZhukV

Вот к примеру проверка поля формы

С другой стороны ошибка бывает как раз наоборот с использованием empty
Например передают значение 0 в какой либо форме или в списке select есть значение 0 его передать надо
что то типа
```
<select>

    <option valeu="0"></option>

	<option valeu="1"></option>

	...

</select>
```
Если выбрать первое в списке то empty не пропустит переменную...
24.09.2010, 14:45
Mulder
Часто встречающаяся ошибка: средствами пхп производить разбор и вычисления данных из баз данных. Золотое правило: операции над данными, принадлежащие бд, нужно производить самой бд. Бд организует и хранит данные специальным образом, только она знает, как наиболее оптимально и быстро произвести запрошенные операции. В идеале вы должны использовать только функции парсинга результатов работы бд - mysql_fetch_*().
Распространенные ошибки.
Подсчет числа строк:
неверно:
```
$result = mysql_query("SELECT * FROM `table`");

$count = mysql_num_rows($result);
```
верно:
```
$result = mysql_query("SELECT COUNT(*) AS `count` FROM `table`");

$data = mysql_fetch_assoc($result);

$count = $data['count'];
```
Подсчет числа символов в поле (данные типа varchar или text):
неверно:
```
$result = mysql_query("SELECT `text` FROM `table` WHERE `id` = 1"); 

$data = mysql_fetch_assoc($result); 

$count = strlen($data['text']); 
```
верно:
```
$result = mysql_query("SELECT CHAR_LENGTH(`text`) FROM `table` WHERE `id` = 1");

$data = mysql_fetch_assoc($result);

$count = $data[0];
```
Определение разницы даты (текущей и датой из таблицы):
неверно:
```
$result = mysql_query("SELECT `date` FROM `table` WHERE `id` = 1");

$data = mysql_fetch_assoc($result);

$date = $data['date'];

// функции php по определению разницы дат

...
```
верно:
```
$result = mysql_query("SELECT DATEDIFF(NOW(), `date`) FROM `table` WHERE `id` = 1");

$data = mysql_fetch_assoc($result);

$date = $data[0];
```
и т.д.
11.01.2011, 10:48
Apay
Цитата:

Сообщение от ZhukV

Либо при выборке с базы

При выборке с базы при проверке использовать:
```
$result = mysql_query( ... );

if( $result === false ){

  // ERROR!!!

  }

else{

  // do something

  }
```
т.к. mysql_query при ошибке почти всегда возвращает false
07.04.2011, 13:44
zBool
Очень распространенная ошибка в разработке разных видов каптч.
```
<?php

if ($_POST['captcha'] != $_SESSION['captcha'])

 echo 'не правильно';

else

 echo ' правильно';

?>
```
Обходится легко, отключаете Cookies, из исх кода(если опера) удаляете <input type="text" name="captcha" ..... />
и отправляете запрос, тогда $_POST['captcha'] равна NULL и $_SESSION['captcha'] равна NULL, получается — условие выполнено, "катпча введена верно.".
решение:
```
<?php

if ($_POST['captcha'] != $_SESSION['captcha'] || !isset($_POST['captcha']) )

 echo 'не правильно';

else

 echo ' правильно';

?>
```
07.04.2011, 14:45
Mulder
Цитата:

Сообщение от zBool

Очень распространенная ошибка в разработке разных видов каптч.

Вообще-то, при сравнении возможно неинициализированных данных пишут так:
```
if (isset($_POST['a']) && $_POST['a'] == $b)
```
27.06.2011, 04:41
zabmix
```
if (isset($_POST['a']) && $_POST['a'] == $b)
```
а так куда будет правильнее:
```
if (!empty($_POST['a']) && $_POST['a'] == $b)
```
27.06.2011, 08:47
Mulder

Цитата:

Сообщение от zabmix

а так куда будет правильнее:

Не правильнее, а зависит от ситуации. Если $_POST['a'] = 0, то результат вашего выражения всегда ЛОЖЬ, тогда, как на самом деле результат должен зависеть от вычисления второй части сравнения.
03.07.2011, 08:09
Nitrex
Поднимаю старый пост, а разве после обновления или редиректа через header
```
exit();
```
ставить не нужно? А то совсем "конфиденциальную информацию" получить можно.
04.10.2011, 22:08
Arks

нужно ставить exit после редиректа.. полностью согласен!
это всегда актиально, например по умолчанию скрипты вызывает пару методов "вызванный метод", "метод генерирующий шаблон"
например в ajax-запросах генерировать шаблон не нужно.. поэтому стоит писать после отдачи нужного контента echo'м - exit. Если этого не сделать скрипт все отдаст клиенту echo'м, потом например перенаправит пользователя(браузер пользователя забудет про текущий запрос и сделает новый), но потом начнет генерировать "в пустоту" шаблон в http-сокет который уже занят другим запросом пользователя. Разумеется http решает эту ситуацию, так что в новом запросе браузер уже не получает "старый" контент, но это все равно как-то идеологически неправильно с точки зрения сервера.
02.04.2012, 23:41
Arks
ошибки c
```
t

true || array('result' => true) 

true || array('result' => false)

true || array()
```
просто бесят...