|
21 / 44 / 11
Регистрация: 08.02.2018
Сообщений: 443
|
|||||||||||||||||||||
Передача ID через ссылку30.03.2018, 14:46. Показов 1061. Ответов 8
Метки нет (Все метки)
Всем привет, подскажите пожалуйста.
К примеру я передаю для удаления страницы id=5. Как лучше будет передать id? В сессии , или в открытом ввиде? Правильно ли такое решение? При удалении проверяю передаваемый параметр, а также на имя в сессии.
Второй вопрос. я написал метод, который при передаче параметра , удаляет,обновляет,добавляет. Задача моя состоялось короткий код и чтобы много раз не писать одно и тоже. Только вот меня мучает одно, ПРАВИЛЬНО ЛИ Я ЭТО СДЕЛАЛ? Скрипт работает, вот только похож ли он на ООП? Вроде бы код укоротил, приятно читать, и не приходиться повторять один и тот же код. Вот скрипт это КАША или что , пожалуйста поправьте что да как?
Я подумал чтобы может вместо этой красоты , унаследовать ParamUser, и сделать такие методы как delete(),update что бы я мог выводить каждый метод отдельно.
[b]В общем скажите пожалуйста, является ли скрипт как бы объектным? Или эта процедурная каша?[/b
0
|
|||||||||||||||||||||
| 30.03.2018, 14:46 | |
|
Ответы с готовыми решениями:
8
Передача переменной через ссылку Передача значения переменной из одного файла в другой через ссылку Как через div вызвать ссылку? |
|
276 / 259 / 185
Регистрация: 15.08.2017
Сообщений: 1,483
|
|||||||
| 30.03.2018, 18:14 | |||||||
|
я не знаю, как лучше его передать, но поскольку обращение к переменной $_GET['del'] одно, то и результат она выдаст один, а вы пробуете передать два значения в одной переменной
0
|
|||||||
|
3899 / 3237 / 1353
Регистрация: 01.08.2012
Сообщений: 10,909
|
||||
| 30.03.2018, 18:28 | ||||
|
Передавайте id через форму методом POST. Как минимум нужно убрать все require (и вне класса, и в конструкторе). Один класс - один файл. И экземпляр Base передавать в конструктор, а не создавать внутри.
0
|
||||
|
21 / 44 / 11
Регистрация: 08.02.2018
Сообщений: 443
|
|||||
| 30.03.2018, 18:41 [ТС] | |||||
|
Добавлено через 43 секунды А сам процес ООП в этом скрипте есть?
0
|
|||||
|
3899 / 3237 / 1353
Регистрация: 01.08.2012
Сообщений: 10,909
|
|||
| 30.03.2018, 18:53 | |||
|
0
|
|||
|
276 / 259 / 185
Регистрация: 15.08.2017
Сообщений: 1,483
|
||
| 30.03.2018, 19:07 | ||
|
0
|
||
|
Особый статус
623 / 221 / 164
Регистрация: 18.11.2015
Сообщений: 1,086
|
|
| 30.03.2018, 22:23 | |
|
Если этот класс служит чем-то типа модели, то наверное есть смысл в наследовании класса Base. Вместо многочисленных require сделайте нормальную автозагрузку и наверное надо бы задуматься об исключениях (Exception) вместо простого exit
0
|
|
|
21 / 44 / 11
Регистрация: 08.02.2018
Сообщений: 443
|
|
| 31.03.2018, 09:26 [ТС] | |
|
FloppyDisc, Да согласен на счет исключений, подскажите пожалуйста, код хотябы пахнет ООП?
0
|
|
|
Особый статус
623 / 221 / 164
Регистрация: 18.11.2015
Сообщений: 1,086
|
||
| 31.03.2018, 21:36 | ||
|
Ну вроде и пахнет, я просто логики самого класса не понимаю чет. Я бы не стал передавать ГЕТ параметры в конструктор, они же суперглобальные как и те же сессии. Для более красивого использования глобальных переменных можно сделать обертки типа getQueryParams(), isGet(), isPost() ... , посмотрите в сторону фреймворков.
Опять же, я уже не помню когда использовал чистый require в файлах классов, трейтов и т.п. Ваш класс чем-то похож на "модель" - этакий класс связанный с БД, поэтому, не стоит изобретать велик и лучше глянуть, как уже говорили, в сторону ActiveRecord, DataMapper или QueryBuilder, вот замечательный пример последнего: https://github.com/usmanhalalit/pixie не раз использовал ни где не подводил, на его основе так же можно построить подобие mini-ActiveRecord при желании. Или хотя бы унаследуйте класс Base (UserParam extends Base), еще искренне не понимаю зачем метод ParamUser::operation(). Желательно чтобы в файле с классом не было ничего, кроме самого класса, никаких экземпляров не создавать, можно только namespace пуста строка и use. Вам нужно почитать правила оформления кода стандарты: PSR-1, PSR-2, например, область видимости нужно указывать для каждого свойства и каждая из них должна располагаться на новой строке. Или те же if, switch, foreach - фигурные скобки не переносятся на новую строку, как с классами, методами и функциями, а остаются на том же уровне с пробелом после скобок. Про свич отдельно, вообще не читабельно, фигурная скобка, case располагается правильно, но его тело так же должно располагаться с отступами break на том же уровне что и тело case.
0
|
||
| 31.03.2018, 21:36 | |
|
Помогаю со студенческими работами здесь
9
Передача данных в форму через ссылку Как методом post передать данные не через submit, а через ссылку? передача массив как ссылку в классе C#. Передача массива СОМ интерфейсу принимающему ссылку
Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
|||
|
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
|
[В процессе разработки] SDL3 для Web (WebAssembly): Сборка библиотек SDL3 и Box2D из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
|
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога
Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
|
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование
. \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json>
Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом.
# Check if. . .
|
|
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так:
https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347
Основана на STM32F303RBT6.
На борту пять. . .
|
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
|
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу,
и светлой Луне.
В мире
покоя нет
и люди
не могут жить в тишине.
А жить им немного лет.
|
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила»
«Время-Деньги»
«Деньги -Пуля»
|