- Поисковые системы
- Практика оптимизации
- Трафик для сайтов
- Монетизация сайтов
- Сайтостроение
- Социальный Маркетинг
- Общение профессионалов
- Биржа и продажа
- Финансовые объявления
- Работа на постоянной основе
- Сайты - покупка, продажа
- Соцсети: страницы, группы, приложения
- Сайты без доменов
- Трафик, тизерная и баннерная реклама
- Продажа, оценка, регистрация доменов
- Ссылки - обмен, покупка, продажа
- Программы и скрипты
- Размещение статей
- Инфопродукты
- Прочие цифровые товары
- Работа и услуги для вебмастера
- Оптимизация, продвижение и аудит
- Ведение рекламных кампаний
- Услуги в области SMM
- Программирование
- Администрирование серверов и сайтов
- Прокси, ВПН, анонимайзеры, IP
- Платное обучение, вебинары
- Регистрация в каталогах
- Копирайтинг, переводы
- Дизайн
- Usability: консультации и аудит
- Изготовление сайтов
- Наполнение сайтов
- Прочие услуги
- Не про работу
Авторизуйтесь или зарегистрируйтесь, чтобы оставить комментарий
Почти угадал, есть не ошибки а "странности"
- 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
А принудительное приведение типов - это от лукавого. Ввиду того, что ПХП позволяет свободную конвертацию, результаты могут быть иногда непредсказуемыми.
В качестве примера приведу аналогичную функцию, возвращающую разнотипные результаты, но с небольшим отличием - вместо ресурса возвращается целое число:
Так что у вас прослеживается явное неразумное НЕиспользование того, что должно быть использовано. ;)
дубль............
в таком случае без разницы
В этом случае разница есть, тут явная проверка против подавления ошибки (что само по себе грубейшая ошибка)
В качестве примера приведу аналогичную функцию
Это аналогичная но не эквивалентная, тут чуть чуть другая ситуация. (хотя очень похожая)
тут чуть чуть другая ситуация
ситуация отличается единственным:
ноль интерпретируется как false
только код в вашей версии рискует потерять работоспособность, если результат функции mysql_connect будет расширен. скажем, потребуется возвращать код ошибки.
ПС. Самое сложное признать свою неправоту :))
тут явная проверка против подавления ошибки (что само по себе грубейшая ошибка)
у многих хостеров и, есди не ошибаюсь, в конфиге по умолчанию нотисы вообще отключены и начинающие девелоперы могут не знають об их существовании. В таком случае можно обойтись и без лигатуры. но суть примера не изменится - принудительное приведение типов там, где его делать не следует.
😡😡😡😡😡
ТС справился успешно с поставленной задачей по обработке скриптов за что ему респект большой!
Советую всем!