CORESH
// Выбираю товары
$ps = $pdo->prepare('SELECT * FROM Goods LIMIT :startCount, :endCount');
$ps->bindValue(':startCount', intval($startCount), PDO::PARAM_INT);
$ps->bindValue(':endCount', intval($num), PDO::PARAM_INT);
$ps->execute();
CORESH
//Выбираю характеристики товара и заполняю обьект
$row['features'] = Good::GetGoodFeatures($row['id']);
$goods[] = new Good($row['id'], $row);
Anonymous
Что за класс Good
Anonymous
Что за статический метод GetGoodFeatures
Anonymous
И характеристики машины напиши
Anonymous
Локальной
Anonymous
Как-то много пасты
Bat
//Выбираю характеристики товара и заполняю обьект
$row['features'] = Good::GetGoodFeatures($row['id']);
$goods[] = new Good($row['id'], $row);
какая разница что за клас good, если он явно берет характеристики в цикле по одному товару
Anonymous
какая разница что за клас good, если он явно берет характеристики в цикле по одному товару
И? Это не понизит производительность до 30 секунд, по словам автора
CORESH
GetGoodFeatures из другой таблицы выбирает все характеристики товара и возвращает массив
Bat
на один хит у него около 20ти sql запросов
Anonymous
И что?
CORESH
подскажите как оптимизировтаь
The Ant
сморя какие запросы
Anonymous
GetGoodFeatures из другой таблицы выбирает все характеристики товара и возвращает массив
Залей на гист, пожалуйста класс свой со всеми методами
The Ant
на один хит у него около 20ти sql запросов
вообще мускул в состоянии на средненькой машинке обработать тысяч 5-10 легких запросов (это без множественных джойнов)
The Ant
в секунду разумеется
CORESH
https://gist.github.com/Glebsky/ba1953d3913ee3d78b4b1d4fdbb7136d
Anonymous
оверхедище
Anonymous
У тебя одно создание класса выжирает все, что можно
CORESH
тогда как организовать?
The Ant
жесть. бро возьми ларавель чтоль )
The Ant
просто закопай проект до его рождения. заархивируй и в облако на память ;D
The Ant
потом поржошь
CORESH
)
Ilija
CORESH
свой)
CORESH
аж стыдно стало, код свой кому еще показывать
The Ant
)
смори. пагинатору не нужно знать никаких запросов вообще. ему скармливаешь тока скоко штук на страницу, количество кнопок и тотал каунт, он должен выдавать список тебе кнопачек
Anonymous
аж стыдно стало, код свой кому еще показывать
Скайп есть? Попробую тебе помочь, отпишись в личку
CORESH
Скайп есть? Попробую тебе помочь, отпишись в личку
скайп есть, но сейчас к сожалению не могу общаться
Anonymous
скайп есть, но сейчас к сожалению не могу общаться
Как появится время, отпишись в личку, помогу тебе с ооп разобраться
Katulos
Ссылку на Котерова в скайп пришлешь штоле?
Anonymous
@ExileeD приветствую, не подскажете, в вашем сервисе, фейсбук через раз работает, это я что-то не так делаю или все же он глючит
CORESH
Как появится время, отпишись в личку, помогу тебе с ооп разобраться
хорошо, спасибо. Обязательно напишу
Katulos
А надо
Anonymous
А надо
Не думаю, просто покажу ему на примере его кода, как нужно организовывать класс правильно
Артур
всем привет))
Артур
кто работал с dotplant ?))
Артур
создал категорию а она не доступна по ЧПУ(((
Артур
http://docs.dotplant.ru/ru/dev/category-route.html в доках написано но не пойму)
Артур
что под каждую категорию создавать файл ?)
SarcasmIO
https://gist.github.com/Glebsky/ba1953d3913ee3d78b4b1d4fdbb7136d
Зачем ты в конструкторе возвращаешь this
Используй такую штуку как Primitive Obsession
Передавай либо объект полностью либо частично
бесполезные ветки if elseif можно заменить с помощью полиморфизма
if elseif в условии успешно избавляются благодаря стратегии
у тебя конструктор разве что кофе не умеет готовить выдели общие сетапы логики и вынеси их в приватные методы
в методы объекта более чем ~3-4 примитивов следует передавать объект as is либо же класс конфига(new Object(new Confid($array));
в общем там все настолько плохо, что проще написать заново, чем переделывать.
каждый класс god object,каждый метод априори статика, пагинатор с закосом под синглтон, общая логика не унифицирована, information expert плачет, закон деметры рыдает в сторонке
SarcasmIO
главное при всем при этом не воспринимай критику в штыки, это обычная практика, ты априори молодец что пробуешь и пишешь, со временем придет.
я бы посоветовал посмотреть какие-то примеры пагинации в адекватных фреймворках
и не боятся объектов,интерфейсов,трейтов.
CORESH
главное при всем при этом не воспринимай критику в штыки, это обычная практика, ты априори молодец что пробуешь и пишешь, со временем придет.
я бы посоветовал посмотреть какие-то примеры пагинации в адекватных фреймворках
и не боятся объектов,интерфейсов,трейтов.
спасибо, та полюбому буду стараться, до тех пор пока не получится
Wekar
Wekar
Wekar
Wekar
Почему ошибка выходит?
Wekar
Всем привет
Ihor
привет
Ihor
https://paste.fedoraproject.org/
Ihor
пользуйся эти м)
Ihor
а то глаза болят, бордовый на тёмном фоне )
Anonymous
Через сервис, например выше
Ihor
не на объекте вызвал )
Wekar
надо регаться?
Sergey
Vladimir
Че пасаны уже знаете как через мельдаун взламывать сайты?
Vladimir
Ну или защититься
Жора
Anonymous
главное при всем при этом не воспринимай критику в штыки, это обычная практика, ты априори молодец что пробуешь и пишешь, со временем придет.
я бы посоветовал посмотреть какие-то примеры пагинации в адекватных фреймворках
и не боятся объектов,интерфейсов,трейтов.
во во в тему сказал. когда тебя критикуют часто мотивация слетает
Anonymous
ну в плане кодинга имею ввиду
Vladimir
А у меня наоборот
Vladimir
Я понимаю что много ещё чему надо научиться и стараюсь прислушаться к совету более шарящего профессионала
Wekar
https://paiza.io/projects/XF7I-taKkdH1eGYPNfim0g
Wekar
вот в чем ошибка?
Anonymous
Ты его в начале открыл и тут же закрыл