//code 1 if (!condition) { // code 2 }

phpmd во время анализа вывалил - The method assignLocations uses an else expression. Else is never necessary and you can simplify the code to work without else. сейчас else использовать вообще не тру?

да оно почти всего не тру

потому и матерится

//code 1 if (!condition) { // code 2 }

не всегда так уместно

потому и матерится

вот смотрю

в одном месте реально стремный стрем

мож у него там if ($foo) { $bar=1; } else { $bar = 2; }

$bar = 1; if (!$foo) { $bar = 2; }

$bar = 1; if (!$foo) { $bar = 2; }

так а я о чем? я вродь о том же

if ($part === '..') { array_pop($absolutePath); } else { $absolutePath[] = $part; }

на такой штуке ругается

а на предыдущей строке что?

foreach ($this->path as $part) { if ($part === '..') { array_pop($absolutePath); } else { $absolutePath[] = $part; } }

прост цикл

ну а в этом методе, логично: public function assignLocations(array $locations = null) { if (empty($locations)) { $this->locations = new ArrayCollection(); } else { $newLocations = new ArrayCollection($locations); foreach ($this->locations as $location) { if (!$newLocations->contains($location)) { $this->locations->removeElement($location); } } foreach ($newLocations as $location) { if (!$this->locations->contains($location)) { $this->locations->add($location); } } } }

можно переделать в : if (!$locations) { $this->locations = new ArrayCollection(); return; } $newLocations = new ArrayCollection($locations); //...

$newLocations не всегда обновляется

в его коде

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

можете подсказать где любят?

вакансии

для js есть клевый чатик с вакансиями https://t.me/javascript_jobs а вот для php не знаю такого

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

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

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

на пхп редко пишут ботов

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

https://t.me/botoid

на пхп редко пишут ботов

чорд) а я как раз в качестве развлекухи вокруг своей херотени бота развернул) ничё так, старается)

чорд) а я как раз в качестве развлекухи вокруг своей херотени бота развернул) ничё так, старается)

"редко", а не "никогда")

ну и я все равно не верю в продолжительную работу php

можно, конечно, подсунуть костылей, но зачем?

каких костылей?)

какой процент что uniqid может совпасть?

ребуты раз в час и тд

ребуты раз в час и тд

сутками работает, ничё не ребутаю) явно чёт не так делаю

какой процент что uniqid может совпасть?

uuid юзай

сутками работает, ничё не ребутаю) явно чёт не так делаю

ну хз, у меня консьюмер кролика через 6-8 часов мог начать делать непредсказуемые штуки

какой процент что uniqid может совпасть?

посчитай, и еще зависит от того берешь ли ты "больше энтропии"

какой процент что uniqid может совпасть?

Бывает. Вообще ты не должен это т процент учитывать. Потому-что процент тем выше, чем больше ты из генеришь

в комментах к доке пишут типа опасненько

хотя хз, свечку не держал

в комментах к доке пишут типа опасненько

хуевый рандом это всегда опасно)

хуевый рандом это всегда опасно)

русская рулетка с макарычем, да)

Но где то был интересный факт, что две проги не могли установится на виндовс хр потому-что совпадал uuid

вон в либке ramsey uuid был как-то описан кейс когда много коллизий генерится

ну и я все равно не верю в продолжительную работу php

Правильно. Раз ты не умеешь, значит это нереально

Правильно. Раз ты не умеешь, значит это нереально

вроде как профилирование памяти в php подвезли.... а значит уже не так уж и плохо

Правильно. Раз ты не умеешь, значит это нереально

а я разве говорю "айнане, не пишите ботов на пхп"?)

но я честно ни разу еще не пробовал

я говорю, что я бы не писал но это моя личная имха, основанная на моих личных синяках

я говорю, что я бы не писал но это моя личная имха, основанная на моих личных синяках

а на чем бы ты писал?)

вроде как профилирование памяти в php подвезли.... а значит уже не так уж и плохо

Ну с xhprof было норм утечки смотреть. А для 7 хз что делать

а на чем бы ты писал?)

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

я не говорю про то, что течет память

но поведение было очень странным

а я разве говорю "айнане, не пишите ботов на пхп"?)

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

Ну с xhprof было норм утечки смотреть. А для 7 хз что делать

xdebug с версии 2.6 вроде как достаточно

Хм... Тоже не пробовал

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

хорошо когда есть выбор) я б на элексире попробовал бы)

хорошо когда есть выбор) я б на элексире попробовал бы)

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

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

не питон норм че

не питон норм че

ну сейчас свой проект я бы на нем замаялся писать) мне просто чатик показал мир нормальной типизации и мне стало грустненько :D

такой вопрос - в деве обновили один пакет, композер лок изменился, я в локальной ветке обновил другой пакет и тоже изменил композер лок, при мерже все заресолвил, но вопрос по поводу "content-hash" - что в таком случае там должно быть? или еще раз композер инстал сделать, и он все разрулит?

нефиг делать апдейт пакета на сервере

ERROR: S client not available

на серваке все впушь в репо

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

потом на сервере делай только install

никто не собирался апдейт делать на серваке - причем здесь это вообще? лок нужен для инсталла

я думаю да, композер инстал сделай просто у себя и все

такой вопрос - в деве обновили один пакет, композер лок изменился, я в локальной ветке обновил другой пакет и тоже изменил композер лок, при мерже все заресолвил, но вопрос по поводу "content-hash" - что в таком случае там должно быть? или еще раз композер инстал сделать, и он все разрулит?

если ты уверен, что все зарезолвил верно, чтобы пофиксить хеш, есть спец команда composer update --lock

если ты уверен, что все зарезолвил верно, чтобы пофиксить хеш, есть спец команда composer update --lock

спасибо, полезная инфа)

но вообще это не совсем круто - composer.lock руками мерджить. Правильно, это брать один composer.lock, мерджить composer.json а потом через composer update заставлять композер самому докинуть других зависимостей в composer.lock Да, это сложнее, но это истинный путь!

но вообще это не совсем круто - composer.lock руками мерджить. Правильно, это брать один composer.lock, мерджить composer.json а потом через composer update заставлять композер самому докинуть других зависимостей в composer.lock Да, это сложнее, но это истинный путь!

хрень в том что composer.json может даже не измениться ;)

мержить все равно надо

но вообще это не совсем круто - composer.lock руками мерджить. Правильно, это брать один composer.lock, мерджить composer.json а потом через composer update заставлять композер самому докинуть других зависимостей в composer.lock Да, это сложнее, но это истинный путь!

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

мержить все равно надо

Ну так, если ты обновил зависимости не обновляя json, то во первых у тебя какие-то странные проблемы (зачем так делать? У тебя все версии прописаны как *? Тогда ты сам себе злой буратино) Во вторых, что тебе мешает, обновить эти же пакеты на машине, где ты мерджишь изменения (composer update specific/package ) ? composer.lock руками мерджить можно только если ты знаешь composer кунгфу

Ну так, если ты обновил зависимости не обновляя json, то во первых у тебя какие-то странные проблемы (зачем так делать? У тебя все версии прописаны как *? Тогда ты сам себе злой буратино) Во вторых, что тебе мешает, обновить эти же пакеты на машине, где ты мерджишь изменения (composer update specific/package ) ? composer.lock руками мерджить можно только если ты знаешь composer кунгфу

ну а как иначе, у тебя на компе одна версия composer.lock, в репозитории уже другая. Что делать то? по любому надо либо мержить, либо откатывать. а на счет версий, у многих пакетов годами версия dev стоит, ну или даже 4.1.*

ну а как иначе, у тебя на компе одна версия composer.lock, в репозитории уже другая. Что делать то? по любому надо либо мержить, либо откатывать. а на счет версий, у многих пакетов годами версия dev стоит, ну или даже 4.1.*

я уже ответил. composer update specific/package

я уже ответил. composer update specific/package

чем это поможет для уже измененного omposer.lock ?

например, ты не будешь мерджить файл вручную. У тебя твоя версия, смотришь какие пакеты поменялось в удаленной версии. composer update specific/package1 specific/package2 etc решает проблему твоего мерджа. Profit

например, ты не будешь мерджить файл вручную. У тебя твоя версия, смотришь какие пакеты поменялось в удаленной версии. composer update specific/package1 specific/package2 etc решает проблему твоего мерджа. Profit

ну а как ты их увидишь если, в composer.json ничего не поменялось? хеши сравнивать? не проще тогда откатить свои изменения, сделать пулл. а потом накатить composer update ?

ну так смотреть в composer.lock тебе не запрещается - посмотри какие пакеты там поменялись, их и выполняй. Если ты поменяешь местами шаги в моем алгоитме (на удаленный composer.lock накатишь обновление своих пакетов) то "сумма не меняется от перемены мест слагаемых" - это как ты хочешь. А вот "composer update " делать не стоит - это дерьмо обновит ВООБЩЕ все пакеты... тебе нужна такая регрессия?

ну так смотреть в composer.lock тебе не запрещается - посмотри какие пакеты там поменялись, их и выполняй. Если ты поменяешь местами шаги в моем алгоитме (на удаленный composer.lock накатишь обновление своих пакетов) то "сумма не меняется от перемены мест слагаемых" - это как ты хочешь. А вот "composer update " делать не стоит - это дерьмо обновит ВООБЩЕ все пакеты... тебе нужна такая регрессия?

еще ни разу не сталкивался с проблемами про composer update/ Даже хз какие они могут быть?

почему это регрессия?

ну, например та библиотека, что ты обновишь с версии 3.2.1 до версии 3.2.2 повиксила баг, который твой код уже активно использует 3 года в проде. И у тебя код стал работать по другому. Да, либа пофиксила баг, но проблемы появились у тебя. Даже если говорить про гигантов symfony они и то, иногда чет подламывают в патчах (слава богу редко) а если твоя библиотека myPizdatiyProvider/PerfectLib то тут вообще рандом

ну, например та библиотека, что ты обновишь с версии 3.2.1 до версии 3.2.2 повиксила баг, который твой код уже активно использует 3 года в проде. И у тебя код стал работать по другому. Да, либа пофиксила баг, но проблемы появились у тебя. Даже если говорить про гигантов symfony они и то, иногда чет подламывают в патчах (слава богу редко) а если твоя библиотека myPizdatiyProvider/PerfectLib то тут вообще рандом

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

оч малая вероятность. я тебя понял

оч малая вероятность. я тебя понял

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

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

Я недавно с таким столкнулся. Коллега сделал composer update и запушил новый lock файл. У него локально стоит php 7.1. Потом через неделю мне надо было добавить новую зависимость. Композер начал ругаться на то, что doctrine\inflector требует 7.1, а проект работает на версии 7.0. Полазив в локе, увидел что она была обновлена. Написал коллеге, спросил за composer update - вот и причина:) Я думаю, Борис имеет в виду, что надо не сразу все обновлять, а постепенно по каждой либе. Ну и желательно, чтобы были тесты, хотя не факт, что они и помогут)

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

слежку можно автоматизировать

Я недавно с таким столкнулся. Коллега сделал composer update и запушил новый lock файл. У него локально стоит php 7.1. Потом через неделю мне надо было добавить новую зависимость. Композер начал ругаться на то, что doctrine\inflector требует 7.1, а проект работает на версии 7.0. Полазив в локе, увидел что она была обновлена. Написал коллеге, спросил за composer update - вот и причина:) Я думаю, Борис имеет в виду, что надо не сразу все обновлять, а постепенно по каждой либе. Ну и желательно, чтобы были тесты, хотя не факт, что они и помогут)

а версия php у тебя не прописана в composer.json ?

слежку можно автоматизировать

(с) ЦРУ

слежку можно автоматизировать

каким образом, типа оповещения?