PHP & mySQL написание и доработка скриптов (Консультации)

Почти угадал, есть не ошибки а "странности"

- is_resource можно было просто убрать (данные места в любом случае возвращают либо ресурс либо false что для конструкций if вполне достаточно)

- Например функция chr вовсе тут не нужна, в данном случае проще было просто использовать "\t" и "\n" (chr надо использовать когда параметром служит переменная, иначе он просто заменяется типичными строками)

Это хоть и не ошибки, но я ожидал что это сразу бросится в глаза.

так это уже претензии к индивидуальному стилю программирования :)

Это не стиль, это неразумное использование в первую очередь своего времени (ну и процессорных тактов, но на этом уровне скриптов это не важно)

в таком случае без разницы как использовать:

if (isset($_GET["id"])) {}

или

if (@$_GET["id"]) {}

но в то же время есть понятие правильного подхода и неправильного.

В данном задании имеем функцию: mysql_connect

Её возможные результаты: Returns a MySQL link identifier on success, or FALSE on failure.

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

Проверка успешности подключения в данном случае наиболее грамотно реализуется проверкой результата функции на соответствие требуемому типу: is_resource,

или на несоответствие булевому типу: !is_bool

А принудительное приведение типов - это от лукавого. Ввиду того, что ПХП позволяет свободную конвертацию, результаты могут быть иногда непредсказуемыми.

В качестве примера приведу аналогичную функцию, возвращающую разнотипные результаты, но с небольшим отличием - вместо ресурса возвращается целое число:

	if ($pos = strpos("abcdef", "abc")) {
		echo("found");
	} else {
		echo("not found");
	}

Так что у вас прослеживается явное неразумное НЕиспользование того, что должно быть использовано. ;)

дубль............

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

Это аналогичная но не эквивалентная, тут чуть чуть другая ситуация. (хотя очень похожая)

ситуация отличается единственным:

ноль интерпретируется как false

только код в вашей версии рискует потерять работоспособность, если результат функции mysql_connect будет расширен. скажем, потребуется возвращать код ошибки.

ПС. Самое сложное признать свою неправоту :))

у многих хостеров и, есди не ошибаюсь, в конфиге по умолчанию нотисы вообще отключены и начинающие девелоперы могут не знають об их существовании. В таком случае можно обойтись и без лигатуры. но суть примера не изменится - принудительное приведение типов там, где его делать не следует.

😡😡😡😡😡

ТС справился успешно с поставленной задачей по обработке скриптов за что ему респект большой!

Советую всем!