«phpClub» — архив тем ("тредов"), посвящённых изучению PHP и веб-технологий.
Аноним 2018/10/22 10:14:23  №1282626
>>1282462

Нужно быть менее категоричным. А то складывается ощущение, что ты вчера изучил JS и теперь без него обойтись не можешь. То, что ты предлагаешь - просто перенести генерацию ссылок на JS. Необходимость генерации никуда не исчезает. Более того, если ты прочтешь описание ниже, то увидишь более интересный подход к решению, который позволит и обеспечить обновление без перезагрузки, и работу с отключенным JS (например, поисковые боты), и (почти) не писать JS код.

Также, в твоем коде нет индикатора загрузки и обработки ошибок, советую почитать мой урок про аякс: https://github.com/codedokode/pasta/blob/master/js/ajax.md

Также, в твоем решении не будет работать история в браузере, нельзя отправить ссылку на страницу с определенной сортировкой, нельзя добавить ее в закладки. Таблица будет недоступна для индексирования поисковиками.

>>1282306

Списка нет, но можно на гитхабе поискать по student list.

> Я в twig нашел как засунуть в href текущий url {{ app.request.query.all }}

Это для фреймворка Симфони, а не для Твига. В Симфони в шаблон передается глобальная переменная app, которая содержит много всяких полей. Это немного нарушает разделение ответственности в MVC, если view может так спокойно залезть в параметры запроса (хотя и сокращает код).

Я вижу, что у многих вызвает вопросы генерация ссылок, давай разберемся. В протоколе HTTP у нас нет состояния: сервер не "помнит", какие параметры пагинации или поиска ты задавал раньше. Потому ты их должен указывать при генерации ссылки. Да, ссылки будут получаться длинные.

Вот вариант решения: сделать функцию, которая будет принимать на вход текущее значение параметров (сортировка, поиск итд), название колонки и выдавать ссылку для нее:

function generateSortLink($viewParams, $column): string

Это может быть функция, метод, статических метод (паттерн Utility Class), функция-расширение twig. Она определяет по названию колонки и текущей выбранной колонке направление сортировки, собирает параметры и генерирует ссылку, например, с помощью http_build_query.

Что такое $viewParams? Это текущие параметры вывода таблицы, их конечно можно сделать массивом, а можно сделать классом с комментариями:

class TableFilter
{
public $searchPhrase;
public $sortBy;
public $sortDir;
public $page
}

Заметим, что класс получился универсальный и подойдет не только к таблице студентов, но и вообще к любой таблице с фильтром и сортировкой.

Аналогично, для пагинации можно написать похожую функцию:

function generatePagerLink(TableFilter $viewParams, int $page): string

При желании, можно как-то объединить эти функции в одну универсальную, например, такую:

function generateTableLink(TableFilter $currentParams, array $replace): string

Не забудь, что при смене вида сортировки или при поиске мы сбрасываем номер страницы.

Теперь затронем тему аякса и JS. В страницу можно встраивать код на JS, который может сам отправлять запросы на сервер аяксом и вставлять результат на страницу без ее перезагрузки. Если ты разбираешься или интересуешься JS и хочешь сделать обновление таблицы без перезагрузки, то есть довольно интересная библиотека pjax. Она перехватывает клики по ссылкам, запрашивает указанную в ссылке страницу аяксом и вставляет ее в указанную область страницы. Таким образом ты можешь сделать страницу, работающую без JS, а затем в соответствие с принципом progressive enhancement, для пользователей с JS добавить возможность работы без перезагрузки страницы. И это все - почти без написания JS кода.

pjax умеет отправлять формы, потому ты можешь заставить форму поиска работать без перезагрузки страницы при наличии JS.

При этом будет корректно работать кнопка "назад" и обновляться URL в браузере.

Если что-то еще непонятно - задавай вопросы. Это обучающий тред.
Аноним 2018/10/21 13:33:55  №1282306
>>1281608 (OP)
Подскажите пожалуйста, мб не туда смотрю, вроде был список сделанных списков студентов.
Я уже спрашивал, но нифига не понял как сделать много гет-параметров, хотел бы подсмотреть как другие люди решали мой вопрос.

Допустим я искал студента по фамилии Иванов и сейчас у меня в адресной строке: index.php?search=Иванов
И я хотел бы чтоб кнопка сортировки (например по баллам) тогда имела внутри ссылку на index.php?search=Иванов&sort_by=grades, а если ничего не искал то index.php?sort_by=grades.

Я в twig нашел как засунуть в href текущий url {{ app.request.query.all }}, но как туда добавлять что-то вообще не понимаю, и не гуглится нихрена, и документацию twig почитал нихрена не вижу, я может вообще не с той стороны подхожу?
Аноним 2018/09/25 15:32:04  №1270133
>>1270092

Нужно выводить ссылку с учетом текущего типа сортировки. То есть если выбрана сортировка -name, то надо выводить в заголовке +name вместо -name.
Ответы: >>1270146
Аноним 2018/09/25 14:27:36  №1270081
>>1270046

Сортировка легко делается без JS: делаем заголовки колонок ссылками вида ...&sort=-name
Ответы: >>1270092
https://github.com/mlmn/vector.loc Аноним 2018/05/01 10:30:17  №1182534
1365085375083.jpg (321, 1680x1050)
1050x1680
1364843503614.jpg (1221, 1600x1200)
1200x1600
1403859616399.jpg (419, 2560x1600)
1600x2560
1355431579513.jpg (661, 985x739)
739x985
>>1176476
1. Вывод шаблонов в конструкторе и деструкторе убрал, сделал обычные методы.

2.
>А зачем нужно OrgInfo? Нельзя ли в организации сделать методы для вычислений этих чисел? Просто у меня ощущение, что этот класс заточен только под вывод таблицы, и тогда код getOrgInfo() желательно вынести из компании куда-нибудь, хотя бы в ту же OrgInfo. Или вообще ликвидировать этот метод.
Класс OrgInfo - выпилил, перенес вычисления общих и средних показателей по организации в отдельные методы. Теперь поясню почему я так держался за это с самого начала. Изначально очень удобно и просто было высчитывать всё это за 1 цикл, тупо прокручивая все департаменты в нем 1 раз, и складывая всю инфу в 1 выходной объект, поэтому в итоге это и выросло в такую вот шизу как отдельный класс для хранения этой информации на выходе этого метода. Мне и сейчас решение с вынесением всех вычислений в разные методы не кажется хорошим, просто потому что, во первых они почти все повторяют друг друга и у них отличается только переменная на входе, выходе и название соответственно, а во вторых всё это вместо одного компактного цикла использует каждый раз по циклу на метод и опять же перебирает снова и снова департаменты. В общем я сделал как рекомендовано, но остались сомнения с точки зрения оптимизации.

3. Тайп хинты расставил везде где нашел возможным.

4. try/catch выпилил, самому не нравилась идея оборачивать каждый код с exeptionon'ом внутри в эту конструкцию. А если я например чужой класс ипользую - мне нужно идти смотреть получается нет ли там внутри выброса исключений и на методы внутри которых есть рисовать try/catch каждый раз? Звучит тупо, так что только рад что это всё не нужно.

5.
>> public function promoteStuff(array $promotionList) {
>Я не думаю, что это надо делать в департаменте. Тут от департамента ничего не требуется и работников можно повышать напрямую.
Ок, сделал напрямую повышение, зачем лишние действия совершать.

6.
>По моему было бы гораздо удачнее сделать так:
>if ($employeeSelector->matches($employee)) ...
>Это позволило бы поместить логику отбора в класс-фильтра, где ей и место. Ну и еще есть такой вариант с коллбеком:
Перенес из департамента метод фильтрации сотрудников в сам класс EmployeeSelector, сделал там метод match который принимает на вход сотрудника и выдает булево значение - подходит сотрудник под текущие параметры или нет.
> new EmployeeSelector('Engineer', [1, 2, 3], [true, false]);
>лучше сделать возможность указать "не важно" для ранга. null, например, передать.
При этом сам способ фильтрации почти не изменился, разве что добавил возможность передавать в конструктор null как вариант "не важно"

7.
> return round($pageCost, 3);
>Не надо делать тут round, так как это нужно только для вывода в таблицу и должно быть там, где делается вывод, а не тут. Тут должен быть метод, который возвращает точное значение.
Ок, округление теперь во вьюхах

8.
> $vectorSecondAC = clone $vectorVanilla;
>В клонировании у тебя допущена ошибка. Этот метод не создает глубокую копию компании. он делает клон компании, но в него помещает ссылки на те же самые департаменты с теми же самыми работниками - они по умолчанию не клонируются, а просто копируются ссылки. Изучи магический метод __clone().
Переделал, теперь при клонировании организации должны клонироваться еще и объекты департаменты, а не просто ссылки на них, и при клонировании департамента соответственно объекты сотрудников

9.
> $this->organisation->setTitle("после антикризисных мер #1");
>Это неправильно, ради решения задачи про антикризисные меры добавлять в Компанию поле-комментарий. Это не нужно компании, это нужно только в антикризисных мерах, и это надо делать где-то в другом месте.
А мне это казалось клевым - мы изменили объект - мы отметил куда-то о том что он теперь другой и не нужно ниоткуда выцеплять это из других классов, комментарий идет сразу с объектом.
У нас ведь класс компании и служит не для каких-то там бизнесс-процессов компании, а для по сути отчетов, вот и это туда же.
Убрал, но как в другом месте это выводить или впиливать - не понял.

10.
>Сортировку лучше делать в одной функции, так:
>Также, есть еще такой трюк, вычисляем "вес" и сравниваем его:
Выбрал вариант с вычислением веса, так как он мне понятнее.

11.
>По view: в идеале, надо использовать htmlspecialchars при вставке текста в HTML. Иначе может быть уязвимость XS
Ок, обернул, только ведь это имеет смысл если мы выводим информацию, на которую как-то могут влиять сами юзеры, а так то кто в мою вьюху что подсунет. И вообще сделал обертку которая тупо сокращает длину названия функции, что бы было компактнее и удобнее её исользовать в перемешку с html

С простыми правками надеюсь что пока что всё, а не будет еще больше замечаний над которыми я просижу опять неделю.
Теперь опять к самому главному.
12.
>> abstract protected function setDefaults();
>Это не очень хорошо, так как непонятно, что эта функция должна делать. Это никак не описано и никак не проверяется. Мне непонятно, что в ней надо написать. По моему так лучше сделать функции getStartingSalary(), getStartingCoffee() и тд, с которыми все проще и понятнее.
>> В общем тут у меня проблема и не понимание как из этого слабого места выкрутиться. [про абстрактные методы]
>Сделать абс. методы вроде getStartingSalary():float { return 500; } для каждого значения.
Я вообще не понимаю что тут от меня нужно.

Теперь немного истории. Сначала я так и сделал, что в классе Employee просто сидели подобные абстрактные методы, и просто переопределялись для каждого нового класса сотрудника - это было удобно на этапе решения базовой задачи.

Но потом когда я начал решать антикризисные меры, там вы первых написано:
>Пришло время проверить, соответствует ли твой код принципам ООП? Гибок ли он и легко ли поддается изменениям?
В во вторых в одном методе там есть условие:
>Увеличить в целях стимуляции умственной деятельности базовую ставку аналитика с 800 до 1100 тугриков, а количество выпиваемого им кофе с 50 до 75 литров.
И тут становится понятно, что просто ретернить захардкоженные значения уже не работает. Ведь их придется менять, а что бы менять - их нужно где-то хранить (в полях и храним, а значит всё предельно удачно теперь встает на свои места и зменяемов с помощью опять же сеттеров и геттеры не просто хардкод а берут то, что в текущий момент в объекте установлено. Я опять же легко могу кому-угодно что угодно поменять, а как всё это провернуть с методами вроде этого:
>getStartingSalary():float { return 500; }
- я просто НЕ ПОНИМАЮ. Если у меня снова будут в сотруднике такие методы - то как я смогу вот эти вот 800 заменить на 1100 в коде?


13. Так же добавил свистелку в виде методов setTimePoint() и benchMark() - прошу оценить реализацию. Саму идею я уже где-то видел, но вот как обычно называется это под капотом и как раскидано по коду я не ковырял пока.

14.
>Кстати, у нас еще есть задача про Гостиницу. Не хочешь отточить навыки ООП?
Да хочу, лучше было сразу просто условие скинуть.


В итоге 2 главных проблемы так и не решил - OrgInfo ращбил на кучу копипастных методов, которые пусть и удобные, но само их существование мне не нравится и они дублируются и лишь загромождают код, и вообще ничего не понял как нужно сделать абстрактные методы в сотрудниках по типу: getStartingSalary():float { return 500; }

Ответы: >>1196649
https://github.com/mlmn/vector.loc/ Аноним 2018/04/19 07:23:05  №1176476
>>1173663

> public function __construct() {
> include 'views/header.php';

Это плохая идея. Конструктор для инициализации объекта, а не для вывода шаблонов.

> public function __destruct() {

Эта идея еще хуже. PHP - язык со сборкой мусора и ты не можешь предсказать, когда будет удален объект и вызван деструктор. Деструктор используется только для освобождения каких-то ресурсов, а лучше по возможности вообще не использовать.

> $orgInfo = $org->getOrgInfo();
> include 'views/reportBody.php';
А зачем нужно OrgInfo? Нельзя ли в организации сделать методы для вычислений этих чисел? Просто у меня ощущение, что этот класс заточен только под вывод таблицы, и тогда код getOrgInfo() желательно вынести из компании куда-нибудь, хотя бы в ту же OrgInfo. Или вообще ликвидировать этот метод.

Поля orgName и orgTitle точно не нужны, так как для них есть методы у компании.

> public function getClass() {
Тут нужен тайп-хинт на возвращаемое значение.

> $this->addEmployee($employee);
> } catch (Exception $e) {
> Dbg::exceptionEcho($e);
Это не нужно делать. try/catch не нужен. PHP сам выводит непойманные исключения, если поменять настройки в php.ini.

> public function promoteStuff(array $promotionList) {
Я не думаю, что это надо делать в департаменте. Тут от департамента ничего не требуется и работников можно повышать напрямую.

> $classMatch = (get_class($employee) == $employeeSelector->getClass());
> $rangMatch = (in_array($employee->getRang(), $employeeSelector->getRang()));
> $leaderMatch = (in_array($employee->isLeader(), $employeeSelector->getLeader()));

По моему было бы гораздо удачнее сделать так:

if ($employeeSelector->matches($employee)) ...

Это позволило бы поместить логику отбора в класс-фильтра, где ей и место. Ну и еще есть такой вариант с коллбеком:

function find...(callable $filter) {
...
if ($filter($employee)) {
$list[] = $employee;
}
...


> public function countDepPageCost(): float {
> return round($pageCost, 3);
Не надо делать тут round, так как это нужно только для вывода в таблицу и должно быть там, где делается вывод, а не тут. Тут должен быть метод, который возвращает точное значение.

> abstract protected function setDefaults();
Это не очень хорошо, так как непонятно, что эта функция должна делать. Это никак не описано и никак не проверяется. Мне непонятно, что в ней надо написать. По моему так лучше сделать функции getStartingSalary(), getStartingCoffee() и тд, с которыми все проще и понятнее.

> public function setLeader($leader) {
Нужны тайп-хинты.

> public function upRang() {
нет проверки на выход за пределы допустимых значений.

> $vectorSecondAC = clone $vectorVanilla;

В клонировании у теябя допущена ошибка. Этот метод не создает глубокую копию компании. он делает клон компании, но в него помещает ссылки на те же самые департаменты с теми же самыми работниками - они по умолчанию не клонируются, а просто копируются ссылки. Изучи магический метод __clone().

> $this->organisation->setTitle("после антикризисных мер #1");
Это неправильно, ради решения задачи про антикризисные меры добавлять в Компанию поле-комментарий. Это не нужно компании, это нужно только в антикризисных мерах, и это надо делать где-то в другом месте.

> new EmployeeSelector('Engineer', [1, 2, 3], [true, false]);
лучше сделать возможность указать "не важно" для ранга. null, например, передать.

Или так:

$selector = new Selector;
$selector->setClass(...);

Сортировку лучше делать в одной функции, так:

$aLeader = $a->isLeader() ? 1 : 0;
$bLeader = $b->isLeader() ? 1 : 0;

if ($aLeader != $bLeader) {
return ...;
}

// иначе, сравниваем ранг

Также, есть еще такой трюк, вычисляем "вес" и сравниваем его:

$aWeight = ( $a->isLeader() ? 100 : 0) + $a->getRank();
$bWeight = ...;

// сравниваем вес

> $managersOfSertainRangs
$managersByRank

> $thisRangToPromote = count($currentRangManagers);
> $neededToPromote = ceil(0.5 * ($thisRangToPromote));
Это лучше записать в одно выражение без промежуточной переменной.

По view: в идеале, надо использовать htmlspecialchars при вставке текста в HTML. Иначе может быть уязвимость XSS: https://github.com/codedokode/pasta/blob/master/security/xss.md

По вопросам:

-------

> Только вот у меня вопрос возникает, а для кого пишутся эти исключения?
Тут есть такой ответ:

Выброс исключения - это способ функции заявить о невозможности выполнения задачи. То есть это такой способ передачи информации out-of-band (не через return). Если исключение планируется ловить, то для этого используется кастомный класс.

То есть функция либо возвращает результат (если он есть), либо выбрасывает исключение. НЕ выбросила - значит, все ок.

> Для самого разработчика, что бы на этапе тестирования увидеть ошибки?
Да.

> Или для пользователя?
Только если чтобы сообщить о них разработчику. Пользователь ведь не разбирается в коде.

Но ты не забывай, что бывает разный пользователь. Есть пользователь программы, а есть пользователь функции - другой разработчик, который ее использует в своем коде.

> Останавливать программу после срабатывания такого вот исключения или нет?
Да. если она неправильная, то ее надо останавливать. Чем раньше ошибка найдена, тем дешевле ее исправить. Читай

- https://habrahabr.ru/post/218325/
- https://www.martinfowler.com/ieeeSoftware/failFast.pdf

> Если например пытаешься добавить 2 одинаковых департамента, то второй просто не добавляется, но без остановки всё как бы и хорошо работает. Обошли дублирование и всё ок, выполнению программы это не мешает.
В программе явно есть ошибка, ты ее скрываешь, в итоге она вылезет где-то на более позднем этапе, например, в виде неправильных чисел. И ее найти будет гораздо труднее. Особенно когда эта программа станет огромной и в ней будут спрятаны сотни разных ошибок, которые сделали предыдущие разработчики, а исправлять придется тебе.

> Первый серьезный ступор был тут, и он по сути никуда не делся. Я просто написал класс OrgInfo, но оставил все поля в нем public, так как просто бомбануло при попытке написать в этот класс 24 геттера/сеттера и в том месте где я их заполняю
Ок, но другой вариант был не делать этот класс, а сделать геттеры в Company. Это позволяет получать отдельные цифры, не вычисляя все.

Еще один вариант - сделать только геттеры, без сеттеров, такого вида:

$info = new OrgInfo($company);
echo $info->getAvgSalary();

зачем тебе там сеттеры?

> Ну опять же кроме геттеров/сеттеров пустышек. На них тоже нужно выставлять тайпхинт по паравилу хорошего тона, даже если ты просто возвращаешь что-то без изменения/вычисления?
Желательно. Для полей ведь ты не указал типы значений, как понять, что там хранится? Плюс, это защищает от ошибок, например, попытки записать некорректное значение. Ты бы мог еще больше проверок сделать, например, запретить ставить неправильный ранг.

> Еще не понятно как быть в случае как с моей функцией getTopAnalyst() - либо она возвращает объект Analyst, либо null (по логике отсутствие объекта это как раз оно), и сюда ничего нельзя поставить на выходной тип, будут ошибки.

Есть тип ?Analyst со знаком вопроса. Гугли. Например https://wiki.php.net/rfc/nullable_types

> Но не стал склеивать методы demoteLeader() и promoteLeader() в один большой метод замены, просто заприватил их и оставил для лучшей читабельности. завел публичный swapLeader(), который просто вызывает их в нужной последовательности, логика в этом есть имхо.

Ну ок, тогда, получается, ты за возможность "обезглавить" департамент. Такое тоже возможно, почему бы и нет. Просто если оставить только swapLeader, то можно запретить делать департамент без босса или с несколькими боссами. А с promote/demote это возможно.

> В общем тут у меня проблема и не понимание как из этого слабого места выкрутиться. [про абстрактные методы]

Сделать абс. методы вроде getStartingSalary():float { return 500; } для каждого значения.

> И просто вынес это в отдельное поле да еще и выставляющееся в конструкторе. Не понял если честно что в этом плохого.
Ненужное дублирование данных, по моему. Можно подумать, что это какой-то отдельный список, не совпадающий с тем, что в компании. Ничего страшного в том, чтобы сделать getDepartments(), нету.

> Может быть стоит антикризис сделать вообще статическим классом?
Не думаю. Тогда ты должен будешь хранить компанию в статическом поле и нельзя работать с несколькими компаниями.

> А просто передавать ему объект, а он его будет возвращать в измененном виде аки обычная процедурная функция?
Вообще, можно.Но лучше бы обычный метод, а не статический. Так как в теории у "антикризисного менеджера" могут быть настройки (сколько процентов увольнять), зависимости, а для них нужны поля и $this.

Кстати, у нас еще есть задача про Гостиницу. Не хочешь отточить навыки ООП?
Ответы: >>1176480 >>1182534
Аноним 2018/03/17 04:00:02  №1157100
>>1157075

Не надо так категорично. Никто не говорит, что сортировка через JS - плохо. В Википедии любые таблицы сортируются через JS и это прекрасно работает - но там они выводятся не постранично, а целиком.

Не стоит говорить, что "X - плохо". Лучше говорить, вариант X имеет такие достоинства/недостатки, а Y - такие и в данной ситуации лучше подходит X.

Я так-то не против сортировки через JS, но хорошо бы сохранить при этом все удоства - возможность скопировать и переслать ссылку, например.


Аноним 2018/03/17 02:52:22  №1157090
>>1157073
>>1157075
Не совсем. Страница загружается с сервера новая и табличку сортирует по нужным значениям именно сервер, а не скрипт. Скрипт выдирает из адресной строки параметры sort и desc, (я напейсал соответствующую функцию), ищет среди ссылок в заголовке с параметром name, совпадающим с параметром sort и в зависимости от значения параметра desc проставляет ей класс "asc" или "desc", которые в css через псевдокласс after приписывают после ссылки соответствующую стрелочку. Порядок сортировки desc скрипт тоже меняет в свойстве href заголовка напрямую, в зависимости от его предыдущего значения, которое он опять-таки выдирает из адресной строки. Если щелкаешь по другому заголовку, происходит то же самое, только предварительно этот же скрипт убирает стрелочку с предыдущего и выставляет ему desc в свойстве href обратно в false. Так что после загрузки стрелочки не пропадают, бо информация о том, где их ставить скрипт тащит из адресной строки.
У меня проблема в том, что в шаблоне я заголовки таблицы изначально жестко прописал и они загружаются as is и теперь приходится так изъёбываться. Да и модификация переключателя страниц реально бредовая получилось, я только теперь это понял. Он и так генерируется на сервере, и модифицировать его как-либо нужно было там. Просто делал это задание именно сейчас, почти всю ночь, у нас уже 6 час утра и голова не варила совершенно. Алсо, я всю неделю до этого почти 2 недели хуярил learn.javascript.ru, прошел его почти полностью и решил большую часть задач. Соответственно, жабаскриптом у меня забита голова и меня на нём явно подклинило. Сам знаю, что черезжопно все получилось, так что теперь займусь переделкой.

В любом случае, прошу не судить строго, мне еще до этих ваших собеседований как до Китая раком. Это из-за сайта Кантора всё в башке смешалось, наверное.
Ответы: >>1157093 >>1160480
Аноним 2018/03/17 01:50:01  №1157073
>>1157060

Я не очень понял, что ты сделал. В моем понимании, заголовки таблицы - это ссылки вида

list.php?sort=+name
list.php?sort=-name
list.php?sort=+points
list.php?sort=-points

И при клике по ним загружается новая страница. И я не очень понимаю, как тут яваскрипт помогает пририсовывать стрелочки. Ведь их проще вставить в шаблон в момент генерации страницы на сервере.

То же самое касается пагинации - модификация ссылок яваскриптом выглядит как костыль. Тогда логичнее уж их целиком генерировать с нуля яваскриптом, а не дописывать что-то к существующим ссылкам.

Или ты сделал сортировку данных при клике яваскриптом внутри страницы? Это вряд ли будет корректно работать, так как при смене вида сортировки данные на странице меняются.

То есть я плохо понимаю, что именно ты сделал.

> Но, например, добавлять по клику на заголовке столбца стрелочку порядка сортировки рядом с ним, на JS объективно удобнее,

Так страница после этого (клика по заголовку) сразу перезагружается и твоя стрелочка пропадает. Или нет?
Ответы: >>1157075 >>1157090
Аноним 2018/03/17 01:06:26  №1157053
>>1157043
В задаче не было обязательного условия использовать бутстрап, а очень хотелось поскорее попробовать свои силы именно собственно в кодинге, так что я по-быстрому напейсал свою таблицу стилей попроще (сделал фон поприятнее, навигационную панельку, кнопочки посимпатичнее и переключатель страниц стилизовал, благо CSS как раз знаю достаточно неплохо, бо не пожлобился на платные уроки у htmlacademy). Думаю, как только окончательно доведу до ума двигло "сайта", натянуть на него другие стили не очень уж и большая проблема. JS я использовал для того, чтобы при сортировке по кликам на заголовках рядом с ними появлялись стрелочки вверх-вниз (в зависимости от порядка сортировки) и модификации ссылок на переключателе страниц. Например, при первой загрузке таблицы без сортировки ссылка на запрос второй страницы таблички у меня имеет вид main.php?from=10 (я их по 10 вывожу). А при клике на заголовок для сортировки ссылка на вторую страничку приобретала вид вроде main.php?from=10&sort=grade&desc=true, чтобы порядок сортировки не прерывался при переключении страниц. Я это делал именно жабаскриптом. Защиту от произвольных данных в адресной строке тоже предусмотрел, если вбить несуществующий параметр, просто ничего не произойдёт. Может быть это дико кривой костыль, но работает нормально.