Павел
Какие то узкие места пометить комментом - это хорошо, но обмазываться везде - нот гуд.
Павел
Как раз вчера такое было, помог именно коммент, так как объяснил не ЧТО написано в коде, а именно ЗАЧЕМ это написано.
Павел
Так как тестов на это не было)
The Ant
хз, как пример могу посоветовать посмотреть код фреймов, где почти каждый метод описан
The Ant
просто наводишь мышкой и читаешь )
Null
Может кто объяснить за юзер чекеры? Там есть пре и пост методы. Но я смотрю юзер и там и там уже типа войден.
Павел
хз, как пример могу посоветовать посмотреть код фреймов, где почти каждый метод описан
Ну описать метод и параметры - это хорошо, это помогает. И сам код не захламляет сильно.
The Ant
наверняка же пользуешься подсказками? а теперь посмотри на подсказки в твоем коде. Чуэшь разницу? 😄
Павел
The Ant
Ну описать метод и параметры - это хорошо, это помогает. И сам код не захламляет сильно.
аннотации к классу тоже можно писать, зачем он нужен и почему
Павел
аннотации к классу тоже можно писать, зачем он нужен и почему
Это помогает, я не спорю. Но обмазываться комментариями везде - не есть хорошо.
The Ant
а что плохого?
Павел
Есть дока к методу, а есть код внутри метода
Павел
а что плохого?
Плохого, что у тебя код наполовину в комментах
The Ant
хороший код, это когда тебе не надо читать код внутри метода
Павел
В тривиальных зачастую
Павел
А смысл почти - 0
The Ant
прочитал коммент к нему и посмотрел что зашло что вышло. и в тесты лазить не надо
Zloy_Программист
Доброго времени всем все отлично сайт запустил проксировал nginx запросы для nodejs на порт 3030
Павел
Я не понимаю, что ты хочешь доказать
Zloy_Программист
У меня есть вопрос. Я так понял что после правок страниц на vue надо пересобирать проект
Павел
10 юзкейсов и 10 выводов - в докблоке - изи
The Ant
Я не понимаю, что ты хочешь доказать
что тесты это исключительно средство контроля, а не какая-то там документация. Это бред вообще )
Zloy_Программист
yarn build запускал, он прошел по всем страницам но изменения не применились
The Ant
Для меня очевидно, что люди, которые утверждают что тесты это хорошая документация к коду, сами не умеют хорошо документировать свой код
Павел
что тесты это исключительно средство контроля, а не какая-то там документация. Это бред вообще )
пусть будет так, мне смысл тебя переубеждать, ты уже устоявшийся мурвей, а не какой то мурашоношь))
The Ant
а уж что помогает понять логику, так это вообще за гранью. Понимать логику помогает экспертиза в предметной области
The Ant
но никак не тесты )
Юра
В расте вон тесты прям рядом с кодом пишут. Удобно
Юра
Идее может их скрыть если не нужно
Юра
А тесты не особо помогают понять логику. Это как понять как работает машина постукивая по двигателю )
Zloy_Программист
Zloy_Программист
yes
Sergey
образ после билда пересобирали?
Zloy_Программист
Zloy_Программист
Zloy_Программист
пожалуйста
Sergey
docker-compose up -d --build если там docker-compose, конечно
Zloy_Программист
docker-compose up -d --build если там docker-compose, конечно
нет не запускается пишет не обнаружен
Zloy_Программист
Zloy_Программист
но команда не работает
Vlad
что консоль говорит
Zloy_Программист
not found
Sergey
учитывая, что тут не хватает базовых знаний о докере, я бы подошел с другой стороны. Как вы запускаете проект?
Sergey
какой командой?
Ivan
Попробуй docker-compose без дефиса, просто docker compose
Zloy_Программист
yarn start той которая описана
Zloy_Программист
в докерфайле
Vlad
not found
комопоз стоит то?
Zloy_Программист
Sergey
yarn start той которая описана
Это внутри контейнера. А сами контейнеры как поднимаете? Должна быть команда которая начинается с docker: "docker run", "docker-compose", "docker compose"
Zloy_Программист
есть команда, которая описана в docker-compose.yml
Zloy_Программист
ей и поднял
Павел
У меня есть вопрос. Я так понял что после правок страниц на vue надо пересобирать проект
Поидее там есть дев режим, который поднимает свой сервер, который следит за изменениями и сам перебилживает
Sergey
есть команда, которая описана в docker-compose.yml
т.е. вы заходите в docker-compose.yml, смотрите какая там команда описана в файле, открываете терминал и в нем её вводите. Я все правильно понимаю?
Zloy_Программист
Поидее там есть дев режим, который поднимает свой сервер, который следит за изменениями и сам перебилживает
по идее есть и я его запускал только он вообще открыл все страницы пустыми
Zloy_Программист
т.е. вы заходите в docker-compose.yml, смотрите какая там команда описана в файле, открываете терминал и в нем её вводите. Я все правильно понимаю?
только одну)) Изначально сайт не запускался после ребута сервака я залез туда и увидел что там указан порт 3030 , перепроверил, действительно для nodejs проксировалось все на этот порт потом запустил yarn start и все заработало
Zloy_Программист
посмотрел доступные команды yarn start yarn build yarn l
Zloy_Программист
мне надо просто вывести изменения на страницах vue больше ничего
Zloy_Программист
телефон поменялся у конторы
Zloy_Программист
yarn dev поднимал дев режим
Andrei
Коллеги, вопрос на засыпку. кто в симфе разолвит такие плейсхолдеры: '%env(resolve:DATABASE_URL)%' из ямл-конфига. В slim пытаюсь dbal прикрутить, а оно туда без подстановки значения напрямки прямо это и отдает. Поставил симфонийский yaml - парсер. Может еще какой компонент надо зацепить?
Andrei
подключаю вот так:
Миша
В sonataAdmin есть фильтры с множественным выбором? Например по полю которое связано с основной сущностью связью ManyToMany.
Денис
Товарищи, подскажите, пожалуйста, как в этой ситуации лучше поступить? Есть миграция, которая добавляет поле а таблицу базы данных. Поле обязательное и его нужно сразу заполнить данными, которые формируются в утилитном классе. Следовательно, это придется прямо в миграции подключать репозиторий, проходить по всем записям, заполнять их, и сразу после этого навешивать на это поле уникальный индекс. Но делать все это в миграции как-то не очень хочется, вроде там коду не место. С другой стороны, где, если не там? Никто не сталкивался с такими ситуациями?
Денис
Не вариант. Там около 20к записей и они отличаются на разных окружениях. А миграция должна везде отработать
Денис
Зачем репозиторий нужен?
Нужно вытащить все записи из таблицы, чтобы заполнить новое поле
Денис
Вытащи через select и заполни
Можно и чистым селектом, конечно, но вопрос не совсем про это бы. Я не знаю, приемлемо ли размещать такую логику в транзакции. Хотя где еще я не представляю
Денис
В миграции максимум connection можно передавать, никаких сервисов
А сервис нужен. В нем логика формирования данных для этого поля
Nikolay
А сервис нужен. В нем логика формирования данных для этого поля
Поле вначале можно сделать необязательным, если оно добавляется сейчас. Обычно три этапа: 1. Миграция схемы 2. Миграция данных 3. Сделать обязательным поле
Nikolay
Оно и так будет в начале не обязательным. Это не отменяет того, что в миграции нужно его заполнять
Так в чем проблема, заполняй через sql, потом делай обязательным поле