@phpclubru

Страница 154 из 956
Eugene
22.03.2017
09:51:50
Сань, скучно же будет

Alexandr
22.03.2017
09:52:19
Нужны нам тут вакансии или пускай в форум пишут? Да - не помешает – 22 ??????? 69% Нет - не нужны, мешают – 9 ??? 28% Сейчас напишу свое развернутое мнение ? – 1 ▫️ 3% ? 32 people voted so far.

Это не повод не поголосовать ?

Eugene
22.03.2017
09:53:00
Ну сунем, чо уж:)

Google
Серёга
22.03.2017
09:53:21
да, е помешают. отменить и запретить всегда можно успеть

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

открытость

Alexandr
22.03.2017
09:55:20
Меня бесит что несогласовано кидают большую простыню.. как и с кодом собственно.. лучше ссылку или hh или http://phpclub.ru/talk/forums/9/

Серёга
22.03.2017
09:55:44
это типа вайп своего рода

Pavel
22.03.2017
10:00:16
Я особо форум не читаю, а здесь все в одном месте собрано, мне удобно

хотя простыни могли бы и поменьше быть ;)

Yoskaldyr
22.03.2017
10:02:44
Простыни не айс, а внешнюю ссылку я врядли буду открывать. Хотя очень часто самое важное из ваканчии - это на чем и что писать придется и сколько платить будут. А если заинтересовало то дальше можно и по ссылке

dypa
22.03.2017
10:41:35
Это не повод не поголосовать ?
сами вакансии не мешают, мешает их оформление.

Grigori
22.03.2017
11:17:59
при этом народ стесняется писать сколько будут платить

стесняется потому что не знает сколько предлагать

Google
RunOver
22.03.2017
11:21:36
Всем утра хорошего. Мы все продолжаем искать мощных PHP разработчиков в IQ option. Ищем не одного. Вилка от 120 и до 200+ https://spb.hh.ru/vacancy/17392377

RunOver
22.03.2017
12:21:08
Ну нет =)

Max
22.03.2017
12:34:50
мне давали на собесе mvc, правда рудиментарный, пара контроллеров, вьюха, роутер, да конфиг, такое как нефиг делать пишется

Dmitry
22.03.2017
12:51:07
Не, там на собеседовании микросервис писать придется ;)

Grigori
22.03.2017
12:51:48
и задеплоить в прод, проверить владение методикой

енот
22.03.2017
12:52:20
без паролей и ключей

Eugene
22.03.2017
13:00:53
с паролями каждый может

енот
22.03.2017
13:01:09
не только лишь все могут с паролями

хотите, пароль от моей vds дам?

пароль есть, а доступа нет

Alexandr
22.03.2017
13:41:31
Видео с HolyJS https://habrahabr.ru/company/jugru/blog/324364/-dostup-k-luchshim-dokladam-holyjs-2

Pavel
22.03.2017
14:41:02
$className = __NAMESPACE__ . '\\' . ucfirst($tableName); if (class_exists($className)) { $query = new $className; if (!$query instanceof QueryInterface) { throw new Exception\QueryNotFound; } }Как вы относитесь к такому коду в приложении?

Имхо тут все плохо.

Adel
22.03.2017
14:42:02
Если ты не делаешь либу какую-нибудь, а свое приложение и тебе надо new $className - значит чтото идет не так

Pavel
22.03.2017
14:42:36
Да да я про то же. Выглядит так что программист сам не очень уверен какие файлы у него в исходниках лежат.

dypa
22.03.2017
14:43:46
$tableName - ORM?

Pavel
22.03.2017
14:44:49
Ну это по сути название сущности, объект создается для запроса к амазоновской DynamoDB

Adel
22.03.2017
14:48:54
ну если чел пытается сделать чтото типа ормки, то это как раз и есть библиотека

Max
22.03.2017
14:49:01
а в чем проблема, обычная проверка, есть ли сущность для обработки таблицы в БД. Криво только, что сущность через конструктор не заполняется.

Google
Adel
22.03.2017
14:49:02
и тут правила другие

Макс, а что если это не сущность? :)

ай

сорри :)

Max
22.03.2017
14:50:03
если это не сущность, то вся архитектура кривая скорее всего

Adel
22.03.2017
14:50:12
он же написал что это и есть сущность

Pavel
22.03.2017
14:56:31
а в чем проблема, обычная проверка, есть ли сущность для обработки таблицы в БД. Криво только, что сущность через конструктор не заполняется.
Проблема в том, что сущность либо есть, либо ее нету. И разработчик должен об этом знать, а не надеятся что она появится волшебным образом.

И потом если кто-то добавит этот класс Query в исходники, то поведение системы изменится, что также совсем не очевидно на diff pull request

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

Ну то есть отдаленно это может и выглядит как ORM но на деле это на 90% не так.

Dmitry
22.03.2017
15:00:28
а ни у кого не бывало, что пхпшный курл отваливался на ресолве хоста?

Adel
22.03.2017
15:00:58
на машине не подключенной к инету?

Dmitry
22.03.2017
15:01:04
причем стабильно, т.е. в какой-то момент перестает ресолвить хост

блин ну ясное дело, что хост ресолвится на этой машине, Adel :)

Max
22.03.2017
15:01:24
Так проверку и делают, чтобы знать. Забыли сущность к новой таблице сделать - получайте Exception автоматом. Ты же не думаешь, что разраб должен на свою память везде надеяться.

Pavel
22.03.2017
15:02:26
И так будет эксепшен Class not found, зачем еще поверх свой костыль писать?

Max
22.03.2017
15:03:01
свой можно отловить и кодом снабдить, потом этот код в доку прописать

Pavel
22.03.2017
15:03:24
Вот непонятно совершенно зачем

Ошибка - она и в африке ошибка. Если она есть ее надо исправить.

Max
22.03.2017
15:03:48
везде так делаю, свои exception классы вещь полезная

Google
Max
22.03.2017
15:03:55
и коды те же

Pavel
22.03.2017
15:04:08
Для этого существует механизм автолоадеров и прочая низкоуровневая лабуда, а то так можно на каждый чих проверять что класс действительно существует и правильно загрузился и экстендит правильные классы.

Max
22.03.2017
15:04:09
проще потом от клиентов ошибки разбирать, сразу ясно что к чему

Dmitry
22.03.2017
15:04:41
А в чем смысл проверять "есть ли сущность для таблицы"... Это как, программсит написал код работы с данными в таблице, а сущность - забыл? ;)))

Max
22.03.2017
15:04:44
автолоадер не поможет, если у тебя нет класса для таблицы

таблицу может кто угодно потом добавить, она с кодом никак не связана

все правильно там проверяется, такие вещи всегда проверять надо

Pavel
22.03.2017
15:05:32
Ну вообще-то еще как связана. Таблица и код это взаимосвязанные вещи.

Admin
ERROR: S client not available

Dmitry
22.03.2017
15:05:58
или, типа... нужно сделать вывод данных из таблицы заказов... таблицу - добавил, а весь код вывода - забыл написать, закрыл задачу ;) Не, ну может и такое бывает ;)))

Max
22.03.2017
15:06:13
если класса для таблицы нет, то очевидно что не связана

Dmitry
22.03.2017
15:06:59
Если класса для таблицы нет - значит класс для таблицы не нужен. Или вообще или в данный момент.

Pavel
22.03.2017
15:07:11
А если класс положили в другое место? А если таблицу переименовали? А если в ней структуру поменяли? Это все тоже надо проверить и выкинуть эксепшены соответствующие? =)

Max
22.03.2017
15:07:31
это все проверяется отсутствием класса как раз

структура отлавливается на момент записи/чтения и снабжается своим exception классом

Dmitry
22.03.2017
15:08:33
Это все проверяется на стадии комита или сборки отдельными скриптами. К коду ПО это отношения не имеет.

Pavel
22.03.2017
15:09:08
Max
22.03.2017
15:09:25
вот так и выходит потом софт с горой не пойми каких ошибок

структура таблицы

Dmitry
22.03.2017
15:10:57
О, да, ошибка "Уважаемый пользователь, мы забыли создать класс для таблицы, извините", конечно, очень помогут ;)

Google
Pavel
22.03.2017
15:11:32
структура отлавливается на момент записи/чтения и снабжается своим exception классом
Ну по сути это делает фреймворк обычно. А на уровне приложения никто никогда не проверяет, а не изменилась ли вдруг сущность. Мы полагаемся на нижний слой и что он правильно работает. Иначе пострадает абстракция.

Max
22.03.2017
15:11:41
кодом снабжается, по коду потом тебе же проще найти будет, когда клиенты кривые скрины с филиалов в другом городе шлют

Dmitry
22.03.2017
15:11:59
О логах с трейсами слышал? ;)

Прислали ошибку, поднял логи, увидел class not found, профит

Max
22.03.2017
15:12:56
где ты интересно логи поднимешь, если твой софт в другом городе, и доступа туда физически нет

скрин с сотового - максимум что пришлют

Dmitry
22.03.2017
15:13:55
Что значит "Доступа нет"? Десктопный софт без доступа в интернет? ;)

Pavel
22.03.2017
15:13:57
Ну значит надо просить прислать логи.

Dmitry
22.03.2017
15:14:15
Но вообще, если у вас ошибки уровня class not found... пожалуй, пора что-то менять в разработке... или кого-то

Max
22.03.2017
15:14:31
ага, попробуй убеди клиентов, которые часто в линуксах не шарят, найти и распечатать логи ?

Pavel
22.03.2017
15:14:44
Хотя в общем я начинаю понимать к чему ведет Max. Это для случаев когда все очень хреново с менеджентом и разработчики вообще без понятия как и в каких условиях используется их софт.

Max
22.03.2017
15:15:30
так обычно и бывает, поэтому на все вешаются свои Exception классы и/или коды ошибок

Dmitry
22.03.2017
15:15:38
да и сами разработчики непойми как ;) Да ;) Забыли сущность к таблице ;)

Pavel
22.03.2017
15:15:49
О, да, ошибка "Уважаемый пользователь, мы забыли создать класс для таблицы, извините", конечно, очень помогут ;)
Еще прикол в том что там дальше идет вот такой код: if (!isset($query)) { switch ($tableName) {

Dmitry
22.03.2017
15:15:57
Интересно, как они таблицу то создают ;)

Max
22.03.2017
15:16:18
это норм как раз, если несколько отделов из разных городов работают над софтом, там часто что-то забывают

Pavel
22.03.2017
15:16:22
То есть типа, если нашли класс то используем его, а если не нашли... Ну ладно, давай попробуем ченить другое сделать, вдруг прокатит.

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

Dmitry
22.03.2017
15:17:26
Наладьте минимальное тестирование и не занимайтесь херней

Max
22.03.2017
15:17:31
если не нашли, то exception с кодом, а клиент пошел делать скрин с сотовым

Dmitry
22.03.2017
15:18:07
Сущность они забыли создать, угу ;)

Страница 154 из 956