За последние 24 часа нас посетили 53864 программиста и 1712 роботов. Сейчас ищут 913 программистов ...

Тривиальный вопрос по пользователям.

Тема в разделе "Подворотня", создана пользователем askanim, 19 июл 2016.

  1. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Ну и что твоя функция whereIs сгенерирует для такого вызова:
    PHP:
    1. ->where(['parent_category="'.$idArray['Category'].'"'])
    ?
    Вот тебе, пожалуйста, SQL-инъекция классическая (взял прямо твой код, только вытащил из объекта, и в ручную начальные переменные присвоил):
    http://phpfiddle.org/lite/code/ghex-28n4
     
  2. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    https://github.com/askanim/bike_001/blob/master/model
    А нет я внатуре тока ввёл их на инсёрт и апдейт запамятовал блин уже, надо завтра будет доделать эту тему спасибо
    --- Добавлено ---
    @mkramer есть такой момент вот посмотри

    https://github.com/askanim/bike_001/blob/master/model
    --- Добавлено ---
    я там не дописал короче, видно сделал только для insert, и update sql инъекции, а на остальные забыл говорю же сыровато ещё не спорю, есть такой момент, запамятовал, но до доделаю прямо завтра там то quote повставлять делов, думаю за пару часов сделаю и перепишу все уже существующую запросы...
     
  3. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    @mkramer Исправлено. :D

    Вот ссылки:
    это на дао
    https://github.com/LaravelForKettle/bike_001/blob/3e731b8f7b3bf3f956806d82f863a59f4dcea204/models

    это на модель

    https://github.com/LaravelForKettle/bike_001/blob/master/product

    И да вкурсе что где то бесполезно валяется строка

    PHP:
    1. $db = Db::getConnection();

    При релизе я всё почищу, сейчас не вижу необходимости потому что думаю что ещё этот файл будет подлежать редактированию, а то и вовсе совершенно переделаю :D
     
    #103 askanim, 21 июл 2016
    Последнее редактирование: 21 июл 2016
  4. Anhk

    Anhk Активный пользователь

    С нами с:
    13 дек 2015
    Сообщения:
    219
    Симпатии:
    11
    а не лучше ли использовать константы класса, вместо свойств? у тебя нет переопределений данных переменных, лишь конкатенация, а к константам доступ, вроде как, быстрее, если я ничего не путаю.
    --- Добавлено ---
    я не наезжаю
     
  5. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    @Anhk

    Суть в том что я хочу чтобы все методы данного класса использовались через $this->, ну идея была при создании такая, а она позволяет применять, наследование методов без вызова самого класса. Мне вообще статика особо не нравится... Прямо для меня статические классы это дикое извращение, я обычну пишу статику только в том случае если мне нужно создать одинокую функцию. В которой не происходит вызов, текущих свойств класса, а ты вкурсе что при использовании статики ты не можешь внутри статических методов использовать команды типа $this, а то есть какое либо наследование там проподает.... Вот как то так. Там можно использовать лишь статику. Можно конечно изощраться, и тыкать внутри везде self:: и использовать везде статические переменные. Но это не по мне, про скорость не слышал, но у меня сайт грузится довольно шустро и мой глаз не дёргается, значит норм.
    А вообще вот:

    https://habrahabr.ru/post/169301/

    Почитай мне нравится эта статья.
     
    #105 askanim, 21 июл 2016
    Последнее редактирование: 21 июл 2016
  6. Anhk

    Anhk Активный пользователь

    С нами с:
    13 дек 2015
    Сообщения:
    219
    Симпатии:
    11
    ну так методы итак будут вызываться через $this->,а при обращении к константе вызываешь её как self::constantName и всё.
    ну то такое, экономия на спичках, как говорят.
    --- Добавлено ---
    может ты статику и не любишь, но по факту - у тебя эти переменные статичны, они не изменяются никак.
     
  7. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    @Anhk Послушай, статика это почти тоже что и обычное функциональное программирование, с доп плюшками, там наследованием не пахнет.
     
  8. Anhk

    Anhk Активный пользователь

    С нами с:
    13 дек 2015
    Сообщения:
    219
    Симпатии:
    11
    есть же ещё parent::, например.
     
  9. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    @Anhk на почитай на досуге. И подумай, я не использую почти статику, только как отдельные не зависимые функции, и всё.
    https://habrahabr.ru/post/169301/
    --- Добавлено ---
    @Anhk Ты видн оне доконца понимаешь что такое наследование. Там много вещей, я даже ещё не все плюшки покушал его.
     
  10. Anhk

    Anhk Активный пользователь

    С нами с:
    13 дек 2015
    Сообщения:
    219
    Симпатии:
    11
    ну, дело твоё. я просто вижу, что по факту, у тебя есть неизменяемые в классе данные, которые находятся в переменных.
    и почему наследованием не пахнет я не понял.
    --- Добавлено ---
    да, понимаю.
     
  11. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    @Anhk читай статью там всё написано.
    --- Добавлено ---

    @Anhk нет ты не понимаешь.
     
  12. Anhk

    Anhk Активный пользователь

    С нами с:
    13 дек 2015
    Сообщения:
    219
    Симпатии:
    11
    и что же я не понимаю в наследовании? )
     
  13. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    @Anhk Давай сначала что такое статические данные?
     
  14. Anhk

    Anhk Активный пользователь

    С нами с:
    13 дек 2015
    Сообщения:
    219
    Симпатии:
    11
    если брать в контексте классов, то данные, для доступа к которым не требуется экземпляр класса. т.е. их можно вызывать непосредственно операторами self, parent, className
     
  15. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
  16. Anhk

    Anhk Активный пользователь

    С нами с:
    13 дек 2015
    Сообщения:
    219
    Симпатии:
    11
    ы. ладно, пустой разговор.
    а статья мне не нравится, у меня есть альтернативы решений автора статьи с использованием статики.
    впрочем, удачи с твоем проектом.
     
  17. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    @Anhk а вообще завязывай офтопить, создай новую тему.
     
  18. Anhk

    Anhk Активный пользователь

    С нами с:
    13 дек 2015
    Сообщения:
    219
    Симпатии:
    11
    лол, какой оффтоп? я тебе вопрос по твоему коду задал, а ты начал говорить мне чего я не знаю.
    бльо, ты странный очень.
     
  19. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    @Anhk Посмотри тему вообще.
     
  20. askanim

    askanim Старожил

    С нами с:
    7 апр 2016
    Сообщения:
    2.201
    Симпатии:
    166
    Адрес:
    GABRIEL
    @Anhk ох. Вот задай свои вопросы @Ganzal он ответит тебе на них чётко, у него кладясь в голове. У меня нет ещё знаний что бы обучать кого то, я сам ещё учусь, но тут я просто знаю что для чего нужно использовать. Я просто понимаю это, а ты нет, но объяснить я слов не нахожу даже просто, как объяснить. Что статика это не наследование.
     
  21. Anhk

    Anhk Активный пользователь

    С нами с:
    13 дек 2015
    Сообщения:
    219
    Симпатии:
    11
  22. Ganzal

    Ganzal Суперстар
    Команда форума Модератор

    С нами с:
    15 мар 2007
    Сообщения:
    9.893
    Симпатии:
    965
    @askanim ого! 30 жалоб! Не, давай-ка мы весь офтоп вырежем в подворотню, если уж очень хочешь почикать тред. Не будем ничего удалять. Ну чтоб твой пердак не забывал как ты лихо умеешь на людях обосраться и потом умоляешь поудалять свои же сообщения. И забанем тебя на недельку-другую. Ну чтоб пердак зажил и остыл, а ты бы придумал как впредь вести себя в общественном месте и профессиональном сообществе.

    Я тебя много раз просил не истерить. Я тебя много раз просил не писать сообщений, если не понимаешь своих собеседников. Но нет, ты развел говносрач на пять страниц. И тут ему и место. Возражения?
    --- Добавлено ---
    @askanim я решил не чикать тему а просто перенести её всю в подворотню. Потому что она на 99% из срача состоит. И будет лежать в подворотне, напоминая тебе о тех временах, когда ты умудрялся забыть подумать перед тем, как что-то сказать.

    Я задолбался закрывать 30 жалоб. Как ж ты не задолбался их размещать-то?
    --- Добавлено ---
    Товарищ @askanim разместил 30 жалоб на себя же, получил 1418 баллов на две недели за размещение 30 жалоб, и за то, что на него разместили 30 жалоб.
    1418 это столько дней длилась великая отечественная война 1941-1945гг. К кол-ву жалоб отношения не имеет.

    Форум PHP Программистов PHP.ru. У нас не только справедливые, но и эрудированные и остроумные модераторы. Вливайся в наше сообщество!
     
    mr.akv, Anhk и Junior) нравится это.
  23. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Целая неделя без веселухи от @askanim :(
     
  24. Junior)

    Junior) Новичок

    С нами с:
    13 июл 2016
    Сообщения:
    41
    Симпатии:
    5
    Адрес:
    Златоуст
    За то теперь есть время дописать свой код... ему по нему много замечаний сделали)
     
  25. Ganzal

    Ganzal Суперстар
    Команда форума Модератор

    С нами с:
    15 мар 2007
    Сообщения:
    9.893
    Симпатии:
    965
    две :cool:
    во-во. Во всём нужно искать позитивные стороны.