>>444220
foreach создаёт копию массива. Чтобы $value менялось, его нужно передавать по ссылке

foreach ($subject as $key => &$value) {

тогда в конце каждой итерации можно делать
$value = $father;

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

Поэтому правильно создать новый массив на основании старого

$newSubject = array();
foreach ($subject as $key => $value) {
$brother = explode(" ", $value);
$sister = array_reverse($brother);
$newSubject[] = implode(" ", $sister);
}

Или как ФП-бог:
$formatSubjects = function($value) {
$brother = explode(" ", $value);
$sister = array_reverse($brother);
return implode(" ", $sister);
}

$newSubjects = array_map($formatSubjects, $subject);

>>444139

> я добавлю метод подсчета количества сотрудников определенного типа.

наверно лучше не метод подсчета, а метод поиска (отобратьСотрудников) который найдет и вернет массив сотрудников (с которыми потом можно что-нибудь сделать

> я создам класс Кризис в котором буду находить сотрудников которых необходимо уволить, и с помощью нового метода в Депортаменте увольнять их

Логичнее назвать класс АнтикризисноеУправление или просто Руководсвто, но ты можешь сделать это и в функции если хочешь.

> это как проверить всех сотрудников в каждом департаменте?

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

> Есть идея создать класс который будет хранить в массиве каждого сотрудника

Этот класс уже есть — Департамент

>>444144

Можно походить. Сказать что нет опыта но быстро учишься и готов показать гитхаб и сделать тестовое задание.

>>444147

Циклом

>>444168

Молодец, решение правильное.

> почему не хочет компилировать задачку?

Если ты используешь кнопку indeone it, то ошибки не выводятся. Перезагрузи страницу чтобы их увидеть (я им отписал о баге, они потом его исправят).

>>444220

$subject[$key] = ....;

писать

$value = ...

бесполезно так как value содержит только копию значения и при его изменении массив не изменится.

Ну или ты можешь писать &$value но тут куча подвохов ии ты должен сначала прочесть и понять весь раздел про ссылки: http://php.net/manual/ru/language.references.php

Также прочитай предупреждения тут http://php.net/manual/ru/control-structures.foreach.php

Я бы не советовал лишний раз использовать ссылки.

>>444227

& имеет много подвохов, пример:

foreach ($a as &$value) {
echo "$value\n";
}

foreach ($b as &$value) {
echo "$value\n";
}

Видишь ли ты ошибку в этом коде? Для начинающего ее трудно увидеть. Если нет то тебе (и всем остальным анонам) лучше прочесть мануал про ссылки и не использовать их пока не разберешься.

>>444326

Ты еще с яваскриптом не работал наверно.

Ну и я надеюсь что ты меняешь язык на язык с типизацией вроде Явы или C#? Потому что в тех же Руби или Питоне все еще хуже, там даже тайп хинтов нет, а в Питоне еще и private. Я не представляю как там вообще программы больше чем на 500 строк писать можно.

Ну и справедливости ради, зачем ты сравниваешь строку с числом? Ты же сам создаешь себе проблему. Пиши in_array(..., array('1', '2')) и проблема внезапно исчезнет.

>>444336

Если ты сравниваешь значения разных типов то идет преобразование. Не сравнивая значения разных типов либо же используй строгое сравнение.

>>444362

Это надо массив. Циклом создаешь объекты и кладешь в массив. Ты предыдущие задачи решал?

Анон, есть такая хуйня:

class Base {
//базовый класс
//тут в конструкторе создаётся объект производного класса Config
}

class Config extends Base {
//производный класс Config
private $props = array(
'a' => 'A', //это свойство должно быть доступно только из класса TestA
'b' => 'B', //это свойство должно быть доступно из любого класса
'c' => 'C' //это свойство должно быть доступно из любого класса
);

public function __get($name)
{
//получаем приватное свойство из $props
}

}

class TestA extends Base {
print $this->config->get('a');
}

class TestB extends Base {
print $this->config->get('a');
}

Как сделать так, чтобы в TestA метод get вернул A, а в TestB метод не вернул ничего?

У меня вопрос, не знаю или сюда пишу, но всё же.

Есть библиотека ExtJS. Есть фреймворк Zend2.
Структура MVC приложения на ExtJS выглядит следующим образом:
-app
--controller
--model
--store
--view
+extjs
app.js
index.html

Далее. Структура Zend2 выглядит следующим образом примерно:
+config
+data
+module
+vendor
-public
--css
--fonts
--img
--js
--index.php
...
...

Так вот у меня вопрос. Скажите как мне совместить эти две структуры? Где должна находится библиотека extjs в структуре Zend и где вся папка app приложения на extjs.

>>445466
Теперь все сломалось. Почему?

\tpublic function calculateDepartmentTotalSalary() {
\t\t$totalSalary = 0;

\t\tforeach ($this->employees as $key => $Employee) {
\t\t\t$totalSalary += $Employee->salary; //Department.php line 30
\t\t}

\t\treturn $totalSalary;
\t}

================

class Employee {
\tpublic $name;

\tpublic $rang;
\tpublic $salary;
\tpublic $coffee;
\tpublic $document;
\tpublic $leader;

\tpublic function __construct($name, $salary, $coffee, $document, $rang = 1, $leader = false) {
\t\t$this->name = $name;
\t\t$this->salary = $salary;
\t\t$this->coffee = $coffee;
\t\t$this->document = $document;
\t\t$this->rang = $rang;
\t\t$this->leader = $leader;

\t\t$this->salary = $this->calculateSalary();
\t\t$this->coffee = $this->calculateCoffee();
\t\t$this->document = $this->calculateDocument();
\t}

>>444339

Наконец-то я нашел время все проверить.

Советы и замечания по https://github.com/NotBadCode/Students

> https://github.com/NotBadCode/Students/blob/master/list.php#L11

> $students = $mapper->getAllStudents( );

Ну не, давай лучше сделаем тут нормально, то есть выбираем только часть студентов, которые есть на текущей странице. Должны же мы научиться правильно делать постраничную навигацию. Для этого стоит добавить к функции параметры $offset и $limit. Незачем выбирать 100 студентов, чтобы вывести первых 20.

Список студентов надо вывести в виду таблицы, а не вертикально по строчкам. Сортировку надо сделать щелчком по заголовку, при этом колонка по которой идет сортировка должна быть помечена треугольничком или стрелочкой (указывающей направление сортировки). Включение сортировки сбрасывает номер страницы на первый.

Вот тут описаны HTML теги которыми можно сделать таблицу: http://htmlbook.ru/samhtml/tablitsy

Результаты поиска тоже должны просматриваться постранично и с сортировкой. Введенное в поиск слово должно выводиться в поле поиска. При выводе результатов поиска должен быть поясняющий текст вроде «Выводятся результаты поиска по слову «студент». Показать всех студентов.».

> `groupnum` int(5) NOT NULL,

Номера групп (как и номера телефонов, паспортов, домов) надо делать не числом, а строкой. Как ты будешь хранить номер вроде 00100? Также, потом могут появиться номера вроде 115M.

Числовой тип должен использоваться для обозначения количества чего-то или значения какой-то величины.

> https://github.com/NotBadCode/Students/blob/master/lib/pdo.php#L8

Не ставь ?> в конце файла. За ним можно забыть пробелы, они выведутся и сломают все функции работающие с заголовками (header, setcookie, session_start). Также, стандарт оформления кода (которого мы придерживаемся) PSR-2 недвусмысленно говорит:

http://www.php-fig.org/psr/psr-2/ru/

> Закрывающий тэг ?> НЕОБХОДИМО удалять из файлов, содержащих только PHP.

> https://github.com/NotBadCode/Students/blob/master/lib/pdo.php#L3

> require_once "/lib/config.php";

Конфиг правильнее подключать через require так как иначе если он уже подключался где-то, то тут он не подключится.

> class DataMapper

> lib/Datamapper.php

Название файла должно содержать буквы в том же регистре, что и имя класса. На linux регистр букв имеет значение, и когда ты захочешь подключить автозагрузик, он не найдет файл. На Windows регистр букв не имеет значения и если просто переименовать файл может быть git не увидит разницы. Есть 2 варианта решения проблемы:

1) http://stackoverflow.com/a/20907647
2) http://stackoverflow.com/a/17688308

Я бы советовал выставить эту настройку в false, после переименовать файл через git mv.

Аналогично надо сделать с profile.php. Заодно получишь опыта смены регистра имени файла на Windows.

Название DataMapper неудачное, надо было назвать ProfileMapper.

> https://github.com/NotBadCode/Students/blob/master/index.php#L44

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

> https://github.com/NotBadCode/Students/blob/master/templates/list.html#L33

> $students[$j×$_GET['num']]

Во-первых ты никогда не должен из шаблона обращаться к GET, POST, COOKIE и т.д. Шаблон должен использовать только переданные ему переменные, иначе он занимается не своим делом (его дело лишь выводить переданные ему значения).

Также, тут ошибка. Если num = 3, а в массиве 30 студентов, и цикл идет от 0 до 30 то мы выводим студентов с такими индексами: 0, 3, 6, 9, в общем явно не то что требуется.

> https://github.com/NotBadCode/Students/blob/master/templates/list.html#L25

> <?

Нельзя исплоьзовать короткий тег PHP (это написано в PSR-2). Можно только либо <?php либо <?=

> <br />

Слеш не ставится в конце HTML-тегов (так было в XHTML много лет назад).

> https://github.com/NotBadCode/Students/blob/master/templates/profile.html#L5

> ?if($error=="name"):?> style="border:2px solid red;"<?endif?>>

Не надо прописывать стили в атрибуте style. Надо добавлять CSS-класс и прописывать стили в нем (http://htmlbook.ru/samcss/klassy )

Вообще, я вижу ты HTML/CSS совсем плохо знаешь. Не хочешь порешать мои полезные задачки на эти темы? Ссылка стоит в ОП-посте.

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

> https://github.com/NotBadCode/Students/blob/master/templates/profile.html#L35

> ($student->getPlace()=='local')

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

$student->getPlace() == Profile::PLACE_LOCAL

Это делает код понятнее (понятно какие в общем есть варианты) и надежнее (защищает от опечаток). Профессиональные программисты использовали бы тут константу.

> https://github.com/NotBadCode/Students/blob/master/lib/Datamapper.php#L18

> return "error request";

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

https://gist.github.com/codedokode/65d43ca5ac95c762bc1a
http://php.net/manual/ru/language.exceptions.php

Также, как альтернатива, можно в случае передачи неправильного значения просто ставить вместо него правильное.

> https://github.com/NotBadCode/Students/blob/master/lib/Datamapper.php#L34

> public function isemailUsed($email, $code)

Чтобы проверить не занят ли email, не надо возвращать объект. Достаточно сделать запрос вида SELECT COUNT(...) и проверить что результат больше нуля.

> public function iscodeUsed($code)

То же самое

> bindValue

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

> $STH->setFetchMode(PDO::FETCH_CLASS, 'profile');

Вместо того чтобы глобально менять режим выбора, надо передавать эти параметры в fetch(). Не используй setFetchMode, это плохая идея.

> https://github.com/NotBadCode/Students/blob/master/lib/Datamapper.php#L87

> return $this->DBH->lastInsertId( );

Обычно маппер проставляет id в объект, то есть делает

$student->setId(...);

Это логично: после сохранения в базу у объекта появляется id.

>>444339

Советы и замечания по https://github.com/NotBadCode/Upload

Насчет проблемы с X-sendfile. Раз ты его установил, давай начнем с проверки, работает ли он вообще. Для этого положи картинку, например, picture.jpg и рядом файл например download.php и сделай чтобы он выдавал header для загрузки картинки, и проверь будет ли что загружаться. Если нет, то отладчиком (Ctrl + Shift + I в браузере) посмотри заголовки ответа на вкладке Network и напиши в тред.

Ну и установить модуль к Апачу мало, надо еще в httpd.conf его включить и перезапустить сервер. Включается модуль директивой LoadModule.

composer.phar и composer.lock не надо класть в репозиторий. Перемести их куда-нибудь, закоммить, добавь в gitignore, закоммить и потом верни назад — git должен их проигнорировать.

По базе данных.

> `path` text NOT NULL,

Вот это плохая идея. Дело в том, что мы не можем на колонку типа text добавить индекс, а значит MySQL не сможет использовать индекс для выборки отсортированных комментариев — вместо этого она будет вынуждена сначала выбрать комментарии в память, а потом отсортировать.

А так, индекс (fileid, path) позволил бы выбрать все комментарии для файла в уже отсортированном порядке. Ты конечно скорее всего не знаешь про это, но почитай чтобы хотя бы в общих чертах представлять когда и зачем нужны индексы:

http://ruhighload.com/post/%D0%A0%D0%B0%D0%B1%D0%BE%D1%82%D0%B0+%D1%81+%D0%B8%D0%BD%D0%B4%D0%B5%D0%BA%D1%81%D0%B0%D0%BC%D0%B8+%D0%B2+MySQL (очень полезная статья для начинающего)

Тут нужно использовать VARCHAR ограниченной длины (или CHAR как вариант).

Для связей между таблицами надо прописать внешние ключи: http://denis.in.ua/foreign-keys-in-mysql.htm

Для comments я бы советовал добавить поле parent_id. Оно избыточно, но в случае если ты где-то накосячишь с path, с его помощью ты сможешь восстановить порядок записей. Без него он будет потерян. Тут лучше подстраховаться.

Для работы с комментариями надо сделать отдельный маппер. Обычно делают на каждую таблицу свой маппер. Иначе класс начинает становиться слишком сложным.

> https://github.com/NotBadCode/Upload/blob/master/index.php#L5

> \Slim\Slim::registerAutoloader( );

При использовании композера это писать не надо, композер сам организует автозагрузку всех библиотек.

> https://github.com/NotBadCode/Upload/blob/master/index.php#L28

> $file = new File( );

Непонятно, зачем ты заранее создаешь пустой файл. Создавай там, где он понадобится.

> newFilesMapper'

Очень неудачное название. Почему new? Пиши просто fileMapper.

PDO тоже наверно стоит сделать отдельным синглтоном. А то где ты будешь брать PDO для CommentMapper?

> $numfiles = count($mapper->getAllFiles( )); //oi kak ne effektivnoooo!

Правильное замечание! Так писать нельзя. Если тебе нужно число файлов, стоит сделать отдельный метод в маппере. Но тут оно вообще не нужно, можно просто сделать не больше 10 попыток генерации кода например (если у тебя код 32 символьный, то там вероятность повтора практически нулевая).

> echo "Ошибка загрузки";

> die( );

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

> https://github.com/NotBadCode/Upload/blob/master/index.php#L79

> "files/$code.$name");

Определение места хранения надо сделать методом, чтобы можно было всегда вызвать его и получить путь к файлу

Также, нет ли тут ошибки? Ты используешь переданное имя файла для сохранения, а что если пользователь попробует загрузить файл 1.php и потом его вызвать? Он ведь выполнится. Вот твой сайт и взломан.

Также, при загрузке файлов надо бы проверять значение поля error, оно описано тут: https://php.net/manual/ru/features.file-upload.errors.php Не надо обрабатывать все коды ошибок, надо просто проверять есть ошибка или нет.

> upload.html

Шаблоны twig принято называть upload.twig или upload.html.twig (и кстати есть расширение для Sublime, если ты его используешь, которое дает подсветку таких файлов и сокращения).

> https://github.com/NotBadCode/Upload/blob/master/index.php#L94

> $list = $mapper->getAllFiles( );

Функцию надо переименовать. Она же не все файлы вернет.

> if (preg_match($RegExp, $file->getType())) {

Было бы удобнее сделать метод file->isImage()

> https://github.com/NotBadCode/Upload/blob/master/index.php#L127

> filename=\"" . basename($filename) . "\";"

Это не работает для русских букв, так как разные браузеры исплоьзуют разные кодировки для этого параметра. Единственный кроссбраузерный способ это ставить имя файла в конец URL, например download/100/задания.doc

> https://github.com/NotBadCode/Upload/blob/master/index.php#L175

> $app->render('/file.html', array(

После успешной обработки POST-формы не надо ничего выводить, надо делать редирект.

> https://github.com/NotBadCode/Upload/blob/master/lib/FileMapper.php#L23

> ORDER BY -id

Принято писать ORDER BY id DESC. Я подозреваю в случае -id это считается выражением и MySQL не сможет использовать индекс для выборки в отсортированном порядке.

Плохо что ты принимаешь и используешь path от пользователя. Ведь он может вписать тебе что угодно и замусорить базу. Я думаю, надо передавать id коммента, на который дается ответ.

Когда ты составляешь путь, ты должен дополнять число нулями. Иначе сортировка получается неправильной:

1.1
10.1
2.1.1
3.1

А надо так:

00001.00001
00010.00001
...

Это ограничивает число комментариев в ветке, но зато дает правильную сортировку. Процесс создания пути (и всю работу с ними) надо вынести в отдельные методы, чтобы легче было читать и править код.

> https://github.com/NotBadCode/Upload/blob/master/templates/file.html#L15

> style="width: 250px;"

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

> https://github.com/NotBadCode/Upload/blob/master/templates/file.html#L26

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

> '/>

В HTML не ставится слеш в конце тега

> comment.getPath()|length

Это же костыль, считать длину строки, надо сделать метод getDepth() у комментария

> https://github.com/NotBadCode/Upload/blob/master/templates/file.html#L45

> <input type='text' name='text' size='100' maxlength="200" />

Для текста комментария надо использовать textarea, а не input

В общем, я бы советовал тебе параллельно начинать изучать HTML/CSS, так как с версткой у тебя все плохо, а от php программистов обычно базовые знания верстки требуют. У меня в ОП-посте есть ссылка на хорошие задания по теме. Это изучать не так и долго, а тебе пригодится.

Было бы хорошо разобраться с этим языком и сделать нормальное оформление.

Ну и еще, у меня есть задания на SQL (в ОП посте), я бы тоже советовал их сделать, чтобы лучше понимать базы данных.

Ну и если тебе интересен JS, у меня и на него есть задания.

Без этих знаний задачу на тестирование (3-я большая задача) вряд ли получится сделать.

>>444461

Ругаться иди в другой тред. Анон задал правильный и хороший вопрос.

>>444506

> Предположим что php хранят такие небольшие значения в двух байтах

В 4 (=32 бита) на 32-битных системах и в 8 на 64-битных.

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

Но это нетрудно подобрать. злоумышленник может зарегистрировать много аккаунтов и сопоставить id и код. Ну и плюс, код может по разным причинам утечь. Фейсбук однажды отдавал код index.php вместо главной.

> используйте стандартные функции и будьте счастливы.

Здравая мысль. Жаль только в твоем посте нет конкретного алгоритма (я описал его выше).

>>444696

> div.graybox{

лучше писать .graybox — div тут явно лишний.

> \tfont-size: 14px;

Когда задаешь font задавай рядом line-height.

> .graybox div{

Во-первых, это плохая идея использовать тут div вместо класса. Что, если в тексте справа окажется див? К нему этот стиль тоже применится.

Во-вторых, тут не нужен лишний див. Просто поставь на graybox паддинг слева и все.

Желтый квадрат должен быть в 10px от левого края. А у тебя в 20px, видно же что расстояние больше чем надо.

>>445030

Советы

> <? session_start() ?>

используй либо <?php либо <?= и не используй <? так как он может быть отключен и не по стандарту

> <meta http-equiv="content-type" content="text/html; charset=utf-8" />

Слеш в конце тега в html не ставят, это было в XHTML

> <div id="form" align="left">

Тут надо исплоьзовать CSS. Не хочешь мои задачки на HTML/CSS из оп-поста порешать? Было бы полезно, там как раз позиционированию учат. Я вижу по использованию br, что ты CSS не знаешь.

> https://github.com/Si0n/winCalculator/blob/master/index2.php#L18

редирект надо делать через header("Location: ...") и die(), а не такие костыли. Почитай примеры тут: http://php.net/manual/ru/function.header.php

Также, ты зря смешиваешь php c html в этом файле. Это приводит к очень плохому и запутанному коду: https://github.com/Si0n/winCalculator/blob/master/index2.php#L18

Почитай про шаблоны тут: http://www.phpinfo.su/articles/practice/shablony_v_php.html

Классы надо класть в отдельные файлы, причем имя файла должно совпадать с именем класса, и ничего кроме класса там не должно быть.

При ошибке заполнения формы лучше не сбрасывать все значения полей, а подставлять их в форму. Чтобы это сделать, надо обрабатывать форму по такому универсальному алгоритму:

------------

$values = значения по умолчанию (пустые);
$errors = пустой массив;

Если (форма отправлена через POST) {
Переносим значения полей из POST в $values;

Если (все заполнено верно) {
Делаем требуемое действие (например рассчет боя);
Редиректим куда-нибудь или выводим результат;
Выходим;
}

Кладем ошибки в $errors;
}

Выводим форму($values, $errors);

-------------

Сессия не нужна если ты сразу же выводишь результат после расчета. Если редиректишь, то нужна. В ней тогда можно сохранять введенные значения.

>>445030

тут https://github.com/Si0n/winCalculator/blob/master/css/style.css#L2 что-то дофига стилей в одном блоке.

>>445081

Ок, теперь все верно, но не забывай ставить line-height рядом с font-size.

>>445093

input:cheked + label

Это применит стиль к label который идет за активной кнопкой.

надо писать label input:checked

То есть именно так писать не надо. Если ты хочешь применить стиль ко всем радиокнопкам везде, то надо писать

input[type=radio]:checked

А если только в конкретном месте то

.some-class input:checked

>>445102

Лучше сделать if/else или switch или хотя бы проверку через массив разрешенных значений.

И не надо GET пихать прямо в вызов, помести значение в переменную и обращайся через нее

$obj->$method()

> Так я в меньшей опасности?

Все равно ненадежно. завтра ты добавишь метод в класс и забудешь что его можно вызвать снаружи. Ну и ты зря изобретаешь велосипеды, есть фреймворки и CMS.

>>445193

У нас есть задание на фреймворк Yii (в Оп-посте), но оно сложное.

>>445108

Не беда, я дам подсказки.

> как сделать разные границы для кнопок, не используя при этом id

Есть такие варианты:

http://htmlbook.ru/css/first-child
http://htmlbook.ru/css/last-child (заметь что этот поддерживается меньшим числом браузеров чем first-child так как добавлен позже)

Изучи какие вообще есть псевдоклассы:

http://vremenno.net/html-css/css-pseudo-classes/
http://htmlbook.ru/css/cat/pseudoclass

И псевдоэлементы:

http://htmlbook.ru/css/cat/pseudoelement

Также, почитай интересную статью:

http://habrahabr.ru/company/paysto/blog/251933/

> Тень какая-то не такая, сколько я с ней не шаманил.

Это действительно непросто, новичку на глаз определить параметры тени. Но я подскажу, как это делать. Для начала, почитай описание box-shadow и убедись что запомнил все параметры тени. Теперь давай посмотрим на картинку.

Для начала определяем цвет и прозрачность тени. Цвет, очевидно серый или черный, и тень очень бледная, то есть скорее всего у нее стоит прозрачность около 0.05 - 0.15 (черный цвет при такой прозрачности становится бледно-серым).

Далее, попробуем определить смещение тени. Мы видим что больше всего тени сверху, немного еле-еле видно с боков и нет снизу. Значит, тень смещена вниз. Если бы она была смещена вбок, то с одного боку было бы больше тени чем с другого. Если бы не была смещена, то тени было бы поровну со всех сторон. А тут тень смещена вниз настолько, что нижний кусок тени полностью не виден (ну то есть источник света сверху над кнопкой).

Сравнив размер тени слева и сверху, мы определим на сколько она смещена. На глаз — где-то 5-6 пикселей.

Далее, остались последние параметры тени — радиус размытия + растяжение. Их можно определить как расстояние от плотной части тени до того места, где она исчезает. Так как тень не смещена вбок то это расстояние от левого края кнопки до места исчезновения тени. на глаз, пикселей 5-8. Растяжение тут по моему либо нулевое либо 1-2 пикселя, так как тень размывается, а не тянется равномерной плотностью.

Ну и если тебе придется верстать с фотошопного макета, там все проще — параметры тени можно посмотреть в эффектах слоя. Но фотошопная тень отличается от браузерной по плотности, так что скорее всего придется все равно на глаз подгонять.

> И почему-то кнопки не отжимаются, хотя я указал атрибут name у всех.

так надо одинаковый name дать.

При наведении мыши надо бы сделать чтобы курсор менял форму на руку с пальцем.

Линии надо сделать более мягким цветом, не черными.

Шрифт, который используется — Arial.

> input{

>\tdisplay: none;

это плохой селектор. Ведь если это добавить на страницу, на ней исчезнут все поля ввода. Сделай, чтобы стили влияли только на переключалку.

>>445243

> класс-хелпер для парсинга коротких строк вида "9×ме1" в нужные в моей реализации ассоциативные массивы.

Зачем вводить лишнюю сущность — массив, когда у тебя есть классы? Логичнее наверно создавать сразу объекты. А так ты в массиво-ориентированное программирование скатываешься.

Также, мне кажется поля в массиве лучше было назвать по-английски, а не по-русски. А вместо «аналитик» хранить имя класса.

> if(!is_string($single)) {

>\t\t\t throw new WrongDataException("string");

Это удобнее делать через assert:

assert(is_string($single));

Если у тебя включено преобразование ошибок в исключения то провалившийся ассерт завершит всю программу.

Названия переменных конечно плохие. Незачем писать arr, можно просто использовать множественное число. То есть $strings, $descriptions, $employees, как-то так.

Класс InitDataParser спроектирован неудачно. Во-первых, тут используется статическое поле, это значит мы не можем создать 2 парсера с разными настройками, фактически у тебя настройки это что-то вроде глобальной переменной. Во-вторых, странно что строки передаются прямо в конструктор, ведь мы не можем задать настройки парсера до передачи строк. Обычно используется такой паттерн:

$factory = new EmployyeeFactory( );
$factory->addWorkerTypes(...);
$workers = $factory->createWorkers('2ме3');
$moreWorkers = $factory->createWorkers('2ме3');
$boss = $factory->createBoss('ме2');

Здесь как ты видишь мы можем использовать один парсер несколько раз и можем задать настройки до парсинга.

Поля private $depInit, private $depBoss это вообще что-то лишнее по моему.

> if(!in_array(self::$typesDispatcher[$workType], array_keys($this->depInit))) {

Проверить наличие ключа можно более простым isset($array[$key]) или боле правильным array_key_exists (isset воспримет null как отсутствие элемента).

> We should somehow get over array_key duplications.\t

если честно, очень странный код, еще и с рандомом (это чтобы отладку усложнить?). Если ты хотел для одинаковых типов складывать количество, то можно просто писать

$this->depInit[$jobTitle]['количество'] += $count;

В общем, на мой взгляд реализация класса InitParser переусложненная без какой-либо выгоды.

> return $this->salary + $this->salary ($this->rank / 100);

Формула ведь не соответствует задаче:

> Сотрудник второго ранга получает на 25% больше, чем первого, а сотрудник 3-го ранга - на 50% больше, чем первого.

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

> if(is_numeric($num)) {

>\t\t\t$this->workDone = (int)$num;

>\t\t} else {

>\t\t\tthrow new WrongDataException("int");

>\t\t}

Это лучше писать как

if (!is_numeric(..)) {
...
}

$this->workDone = ...;

Тогда явно виден основной код и дополнительная ветка для проверки. А у тебя проверка типа смешана с логикой функции. Но короче по моему assert использовать.

> public function getBossSalary()

Странный метод. Получается у каждого сотрудника есть 2 метода, getSalary b getBossSalary и мы должны угадать какой надо вызвать?

Если ты хотел вынести бизнес-логику расчета зарплаты с учетом статуса босса и ранга из класса рабочего, было логично сделать объект, допустим, класса WorkerConsumptionAndProductionStrategy (согласен, длинно, можно короче написать как SalaryAndCoffeeCalculator) с методами вроде

calculateSalary($baseSalary, $rank, $isBoss)
calculateCoffeeConsumption(..)

По моему, это паттерн «стратегия». Он выносит логику расчета из класса сотрудника и позволяет например менять глобально надбавки в зависимости от инфляции, курса доллара и прочих внешних факторов.

> class WorkersCreator

Очень странный и непонятный класс. Судя по названию, он создает сотрудников, но кода в нем подозрительно много для этой цели.

> if(!is_array($inDepInit)) {

Для array есть тайп-хинт: http://php.net/manual/ru/language.oop5.typehinting.php

> $tempRank = array_values($depInit)[$i]["ранг"];

Что за странные конструкции? Зачем тут array_values?

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

Это все можно сделать намного проще и понятнее.

> class Department extends WorkersCreator

Это по моему неправильно. Наследник это улучшенная или измененная версия родителя. Например, Самолет может наследоваться от Транспорта. Но с какой статти Департамент наследуется от СоздателяРабочих? департамент не создает рабочих, он их нанимает.

> public function fillTotals()

Нет никакого смысла хранить totalSalary. Ее всегда можно посчитать заново, и нам не надо писать сложный код для пересчета totalSalary если мы например уволиил или наняли работника, понизили ранг, и тд (у тебя он впрочем и не написан).

> $this->statData["Департамент"] = self::$depName;

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

$dep->fillTotals( );
$dep->initStatData( );
$data = $dep->getStatData( );

А можно сделать

$data = $dep->getStatData( *);

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

Также, на мой взгляд эти statData не нужны, это ненужное усложнение. Достаточно методов вроде getTotalSalary().

> public static function addNewWorkerType($newWorker)

тут тоже логики не вижу. Почему мы добавляем новые профессии через Департамент? И почему тут передается массив непонятного формата из 4 полей с ничего не говорящими названиями 0, 1, 2 и 3?

> public static function changeWorkerCond($newConds)

Какой-то странный метод который непонятно чем занимется. Если он вызывает
parent::changeSalary($victim, $newVal); то не проще ли сразу ее и вызвыать?

> class Department extends WorkersCreator

> function __construct($inDepInit, $inBoss, $depName)

Это тоже какое-то усложнение. Если подумать, то как связаны Департамент и Сотрудники? В Департамент можно добавлять Сотрудников и можно их увольнять. Но у тебя в Департамент передаются какие-то строки вместо объектов-сотрудников. зачем? И почему Департамент занимается не своим делом - создает сотрудников? Он их по идее только нанимает.

Мне кажется, должны быть методы вроде добавитьСотрудника(сотрудник)/уволитьСотрудника(сотрудник) где сотрудник это объект. Так будет и проще и понятнее, и максимально универсально: ты можешь сам создать какого хочешь сотрудника и добавить в Департамент.

> $crisis = ["аналитик", "зарплата", 700];

> Department::changeWorkerCond($crisis);

Опять ты все запутываешь. Почему не Department::changeWorkerCond("аналитик", "зарплата", 700)? А еще лучше, сделать нормально примерно так:

аналитики = департамент->найтиВсех(аналитиков);
(для каждого аналитика) {
аналитик->задатьБазовуюЗарплату(N);
}

Ну посмотри, насколько проще и универсальнее получается. Без непонятных массивов, статических методов и прочих антипаттернов. И мы можем отбирать аналитиков по любым условиям, например пропустить тех у кого фамилия на «А» начинается, с помощью одного if.

> У меня все строится на диспетчеризации.

Я честно говоря не слышал про такой паттерн.

В общем, по моему код переусложнен, и требует серьезного рефакторинга. Предлагаю упростить схему на картинке. Сделать, чтобы Department занимался только своим делом, а именно, хранением, приемом, увольнением, отбором сотрудников и расчетом общих показателей. Чтобы у сотрудника была одна функция расчет зарплаты, а не 2. Убрать все лишние многомерные массивы. Мне кажется, так будет гораздо проще и код будет меньше по объему.

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

> Я изначально понимал, что ООП нужно для создания универсального кода, с максимально простым расширением функционала.

Боюсь, пока что это не очень ООП, а просто код в котором встречаются классы.

> я почему-то решил делать это через статические свойства и методы

Это неправильно. С ними ты не можешь например в одном департаменте повысить зарплату, а в другом нет.

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

Сами таблички получились аккуратные, норм.

>>445302

Не понял вопрос, приведи кусочек кода или поясни, что ты имеешь в виду под словами «ловить вложенность родителя».

>>445327

bash + популярные команды (cat, grep, sed, tail, stat, tar, du, touch, chmod, head, less, sort, uniq и подобные). Чтобы решить задачу вроде «дан лог сервера, посчитать сколько раз там встречается имя каждого домена» или «найти все файлы где встречается слово eval». Написать простейший скрипт выгрузки измененных файлов на удаленный сервер.

Управление сервисами: запуск, останов, перезапуск, reload. Как прописать и убрать сервис из автозапуска.

Настройка Апач/PHP: где лежат конфиги, как применить изменения

Файловая система: основные стандартные директории, система прав

Пользователи: концепции, добавление/удаление/изменение пользователей и групп, sudo, su, как дать себе sudo-права.

ssh, scp, использование ключей.

Пакетный менеджер: как искать, устанавливать пакеты

> "уметь поднять сервер"

Установить и настроить Апач, PHP, mysql.

> А то я тут скачал курс по системному администрированию, сижу слушаю про какие-то шины, драйвера и прочие увлекательные вещи.

Это по моему что-то не то.

Тут есть список полезных команд: http://ss64.com/bash/ — хотя для начинающего может он великоват, но было бы неплохо хотя знать про их наличие. Я большинство знаю например.

>>445397

Про куки есть:

https://ru.wikipedia.org/wiki/HTTP_cookie
http://citforum.ru/internet/html/cookie.shtml

Авторизация:

- если ненадолго то создаешь сессию, и если пароль/логин правильный, записываешь в нее loggedUser = id пользователя. Сессия после 20 минут неактивности умирает. Проверяешь залогиненность наличием сессии с переменной loggedUser.
- если надо надолго то дополнительно ставишь куки с логином/хешем (пароль + соль) и когда сессия умирает, пересоздаешь ее из кук.

>>445449

О, да это же неплохой код. Почти решает задачу. Но нужно его еще улучшить.

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

Исходя из этого давай допилим функцию. Вот тут вот:

> if ($point == $target) {

>\techo "путь пройден за $time\n";

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

Сделай чтобы в случае достижения цели функция возвращала пройденный путь.

Далее, вот тут:

> foreach($paths[$point] as $name => $value) {

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

Сделаешь? Если что-то непонятно, напиши что, я могу еще подсказок дать.

>>445460

> Не регулярками же парсить?

Конечно нет.

Парсить удобно либо XML-парсером (DOMDocument, SimpleXMLDom) либо аналогичными библиотеками (вроде phpQuery). В случае XML парсера надо еще знание языка для поиска узлов в дереве XPAth:

https://ru.wikipedia.org/wiki/XPath
http://zvon.org/xxl/XPathTutorial/General_rus/examples.html

В случае с яваскриптом лол, опечтался и написал язваскрипт можно либо попробовать раскодировать содержимое onmouseover либо использовать эмулятор браузера наподобие phantomJS, либо расширение php v8.

>>445511

> Скажите как мне совместить эти две структуры? Где должна находится библиотека extjs в структуре Zend и где вся папка app приложения на extjs.

Я думаю JS код должен быть внутри JS-папки, например public/js

>>445545

Через CSS, есть минимум 2 способа:

- с исплоьзованием якорей и псевдоселектора :target (профит: выбранный таб есть в адресной строке и не теряется при перезагрузке страницы)
- с помощью скрытого input type=radio (профит: адресная строка не загрязняется, можно переключать табы стрелками если выделить)

Советую изучить оба способа. Чтение:

http://habrahabr.ru/post/223583/
http://habrahabr.ru/post/174029/
http://habrahabr.ru/sandbox/40690/
http://habrahabr.ru/post/167151/

>>445551

Почему он тупой? Он начинающий и задал нормальный вопрос. Сам ты тупой.

> Там же есть кнопочка Edit code

Это все равно что списать решение. знаний не дает.

>>445600

Перегруппировать полученный массив кодом на php в массив нужного тебе вида. Делается с помощью циклов и if.

>>445719

А что плохого? Там как я понимаю, талантливый творческий коллектив и можно много чему научиться. Ну и строчка в резюме весомая получается.

>>445744

Может быть платят ниже рынка за возможность научиться, работать с именитыми заказчиками и получить строчку в резюме?

> Я видел код чуваков оттуда, весьма посредственно.

Там в первую очередь интересно дизайнерам, программисты там обслуживающий персонал.

>>445751

Они добавляются же:

> ["a"]=>

> array(3) {

> [0]=>

> object(Test)#3 (1) {

>>445791

Это не я. Я обычно к написанию не придираюсь если вижу что человек на «вы» с английским. Но конечно лучше бы слова в google translate хотя бы смотреть.