Можете прокомментировать такой набор функций php?

Question

[dk-web]

Познавая мир веб-программирования на php - изобретаю очередной велосипед для себя.
Делаю небольшой функционал для обработчика форм...

Выложил здесь:
ideone.com/oRRaAy

Буду признателен за комментарии.
Вопросы пока следующие...

UPDATE cities2 SET `id`='7', `city`='Иваново' WHERE `id` = 7
Запрос выполнен успешно

Не может быть успешным, потому что id=7 не существует...

Все остальное работает... но хотелось бы немного упростить.
На ООП пока не умею программировать...

П.с. перехожу с mysql на mysqli, но на сервере этот код в принципе рабочий

Answer 1

[Дмитрий]

Не может быть успешным, потому что id=7 не существует...

ну и что? Этот запрос не вызывает ошибку, потому что он 1) синтаксически верен; 2) не противоречит принципам mysql

Comments

[dk-web]

спасибо! Дмитрий

Answer 2

[Дмитрий Ковальский]

Запрос вполне верен. То что не нашлось полей, которым проставить новые значения - ничего не значит. Не поверите, но бывают и запросы на удаление, которые ничего не удаляют.
UPD : В коде по ссылке у вас грехов побольше. Конкатенирование sql-запросов, мелькающие элементы разметки и echo(видимо месиво работы с базой и работы с клиентским представлением).

Comments

[dk-web]

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

[Дмитрий Ковальский]

dk-web: склеивание(конкатенация) sql-запроса - верный путь к sql-injection. Если не в курсе-гуглите, это опасная уязвимость. В эхо и принтф ничего плохого нет. Плохо перемешивать html,php,sql код, всему свое место. Вам нужна функция которая вернет набор данных по запросу или результат оного с кодом ошибки. А клиентское представление, на основе данного результата работы функции, будет отрисовывать разметку

[dk-web]

Дмитрий Ковальский: спасибо большое за внятный комментарий - логику понял. Попробую реализовать.
а вот с конкатенацией не понятно... какая альтернатива?
у меня в одном проекте есть форма поиска по 40 критериям...

естественно я делаю такой сложный запрос - там WHERE и до 20 AND и IN может быть...
как тогда правильно такое реализовывать?

На запихивать же в массив 24000 на 40 показателей... смысл тогда БД?
буду признателен за комментарий, скорее всего не понял основной посыл

[Дмитрий Ковальский]

dk-web: ну тут можно целые простыни писать об архитектуре. Лично я не использую PHP но это не важно. Можно использовать хранимые процедуры - там делается проверка типов входных данных и склейка не проходит. Внутрь можно засунуть сложный запрос, а из кода php будет несколько строчек с вызовом. В идеале ваш подход с универсальными скриптами - не лучшая идея. По хорошему на каждую сущность должен быть отдельный набор хранимок по вставке,удалению(Иногда помимо вставки нужно сделать сопутствующие какие-то запросы и все оборачивать в транзакцию). Вы проверяете на инкрементность по названию поля? а если у него другое название? или там нет автоинкремента? Ключом может быть все что угодно. Текстовая константа, GUID, непоследовательные цифровые значения(к примеру 4 значные цифры отвечающие некоторым принципам последовательности(1011,1012,а потом 1101,1102))

[dk-web]

Дмитрий Ковальский: ну у меня пока не так все сложно) все же под свои конкретные вопросы адаптирую... в любом случае спасибо за хорошие советы! постараюсь следовать

[dk-web]

Дмитрий Ковальский: Дмитрий, приветствую! в общем худо-бедно старался следовать Вашим советам..
приветствую! вы весьма помогли решиться попробовать эти PDO и конкате..... и в принципе я готов поделиться тем, к чему пришел... буду очень признателен, если найдете возможность посмотреть...
dkweb.coolpage.biz/data-control/data-control.php

исходники выложил здесь
https://drive.google.com/file/d/0BxRag0FeNedKbHoyU...

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

мне важно понять насколько колхозен мой колхоз, но еще пару недель о таком я и мечтать не мог..
п.с. я специально не задействовал потенциал jquery table.. там уж слишком мореная документация( сейчас буду, скорее всего разбираться, но для моих нужд пока и такого варианта с головой...

функции я постарался написать используя PDO (как смог) - здесь все настаивали на необходимости перехода...
jquery раньше мог только .html(data) и alert...

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

умолкаю... хорошего дня!

[Дмитрий Ковальский]

dk-web: Кривая ссылка похоже. Открыть не смог. Повесьте UPD в содержании, вдруг еще кто-то посмотрит и порекомендует что-то.

[dk-web]

Дмитрий Ковальский: сорри... и правда..
сутки все переписывал в итоге...
но сделал даже лучше, чем планировал.
https://drive.google.com/file/d/0BxRag0FeNedKQ0xPQ...
выложил тут и тут - хостинги медленные... плюс я начудил с подключениями в pdo...(((

User 'u848107404_dk' has exceeded the 'max_user_connections' resource (current value: 3)'
через 20 секунд на сайте..
dkweb.pe.hu/data-control/data-control.php
dkweb.coolpage.biz
если глянете буду признателен - надо двигаться дальше и уже писать статистику по таблицам.
п.с. я конечно взорвался от документации к этому плагину... вроде бы все и просто, но... не хватает силенок правильно объекты перебирать и формировать -
entermix - во всю использовал)))) - как вы и завещали;)

Answer 3

[entermix]

Используйте PDO для работы с БД

Comments

[dk-web]

entermix если PDO это ООП, то я написал, что пока его не осилил...

[entermix]

dk-web: www.phpfaq.ru/pdo

[dk-web]

entermix: entermix - посмотрел еще раз - ну хорошо... .что кроме некоего упорядочивания я получу? то, что это вроде как модно?

[entermix]

dk-web: Почему стоит пользоваться PDO для работы с базой д...

Answer 4

[Oleg]

если вы хотите проверить "изменил ли запрос что то " - то можете просто проверять, affected_rows.

Правда если вы два раза выполните один и тот же апдейт как у вас в примере - то второй раз он сообщит что изменено 0 строк.. потому что нечего менять.

Comments

[dk-web]

Почему два раза?... спасибо за комментарий Oleg

[Oleg]

допусстим в таблице:
cities2:
id city
7, Москва


делаете запрос:
UPDATE cities2 SET `id`='7', `city`='Иваново' WHERE `id` = 7

данные меняются, всё ок, изменена 1 строка

делаете повторно запрос:
UPDATE cities2 SET `id`='7', `city`='Иваново' WHERE `id` = 7

данные не меняются (хотя в остальном всё ок) - измено 0 строк

я про подобный сценарий

Т.е. вопрос - а надо ли вам точно знать что строки обязательно изменились ? :)