обычно писатели таких ифов очень страстно защищают их, типа читаемость, или "ты что нас тут всех потом еще и на хаскель заставишь перейти?"

Отвечай, что да - заставишь. Всегда так делаю, исполняю.

обычно писатели таких ифов очень страстно защищают их, типа читаемость, или "ты что нас тут всех потом еще и на хаскель заставишь перейти?"

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

Я так и отвечаю =)

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

ну так просто покажи им как можно без такого

ну иквалз на самом деле не критично

скаластайл настрой там или что то такое и бей линейкой по рукам, ну или чем там

тут же вопрос не стиля

если народ иквалз использует , то хорошо

защита от всяких нуллов должна быть по краю приложения а не внутри здесь косяк архитектурный

У нас есть tech debt burndown day раз в месяц. Приходится к нему готовиться, создавать карточки на каждый такой кодсмелл, приводя примеры, ссылки приделывать и еще объяснять как этого избежать и как автоматически это все поменять. Потом это рассматривается командой и кто-нибудь имплементит

У нас есть tech debt burndown day раз в месяц. Приходится к нему готовиться, создавать карточки на каждый такой кодсмелл, приводя примеры, ссылки приделывать и еще объяснять как этого избежать и как автоматически это все поменять. Потом это рассматривается командой и кто-нибудь имплементит

имплементит не тот кто втащил код с душком?)

У нас есть tech debt burndown day раз в месяц. Приходится к нему готовиться, создавать карточки на каждый такой кодсмелл, приводя примеры, ссылки приделывать и еще объяснять как этого избежать и как автоматически это все поменять. Потом это рассматривается командой и кто-нибудь имплементит

или не имплементит =)

имплементит не тот кто втащил код с душком?)

Нет, потому что one team, bro.

или не имплементит =)

Это как официальная задача, поэтому имплементит. Но нужно добиться приоритеризации

так по семейному... нагадил младший, а убирать старшему

У нас есть tech debt burndown day раз в месяц. Приходится к нему готовиться, создавать карточки на каждый такой кодсмелл, приводя примеры, ссылки приделывать и еще объяснять как этого избежать и как автоматически это все поменять. Потом это рассматривается командой и кто-нибудь имплементит

vam norod ne nuzhen ?

vam norod ne nuzhen ?

Конечно, нужен :)

но у них погода плохая =(

А можно попробовать с помощью scalafix автоматом это все переделывать

но у них погода плохая =(

Лучше, чем в питере :)

Вам скажу страшную вещь, но в питере уже довольно долго солнечно

А можно попробовать с помощью scalafix автоматом это все переделывать

Я кстати завела такую задачу, жду следующего ревью, чтобы уже было сделано. Но у нас мавен, а с мавеном всё не так

Вам скажу страшную вещь, но в питере уже довольно долго солнечно

Весной всегда так

Ну ты можешь Мавеном таску запускать )

Или сделать прекоммит хук

но у них погода плохая =(

я думал в сша

А можно попробовать с помощью scalafix автоматом это все переделывать

Новое правило писать?

Новое правило писать?

Угу

Или сделать прекоммит хук

Не чистое решение, хотя локально у меня так и есть

Можно взять список у поляка и фиксить им)

Не чистое решение, хотя локально у меня так и есть

Чем оно не чистое?)

Чем оно не чистое?)

Устанавливать пре коммит хук как предлагаешь всем?

Эт другой вопрос) но вообще можно и так

Или сделать прекоммит хук

Нет. Я бы с ума сошёл, если бы коммитился не тот код, что я написал

Нет. Я бы с ума сошёл, если бы коммитился не тот код, что я написал

Там другая логика, если б он нашёл чтот , то он бы тебе не дал закомитить

Эт другой вопрос) но вообще можно и так

Для моей команды это тот же самый вопрос, поэтому сказала что не чистое решение

Ну ок, добавь вызов в мавен плагине. Пусть падает на сборке если чтот нашёл )))

Внезапно у мене не получилось сделать это быстро и поэтому я создала задачу. Пусть заплатят за свой мавен :)

Я в каком-то выпуске скалалаза очень жаловалась на мавен и стайлчекеры

Да фиг знает, по мне и мавен норм

Напиши свои плагин да и всё

Я даже сказала, что напишу свой плагин лол

Вот, молодец)

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

Даже у scala native есть maven плагин)

Нет. Я бы с ума сошёл, если бы коммитился не тот код, что я написал

делаешь ты такой пуш в мастер, а тебе приходит уведомление, что ты скалачек не прошёл, всем приходит об этом уведомление

при этом ключи скалачека на твой бранч и на мастер разные

Е бой) эт будет круто

делаешь ты такой пуш в мастер, а тебе приходит уведомление, что ты скалачек не прошёл, всем приходит об этом уведомление

Пуш в мастер?

Форс пуш в мастер

Чтоб красивее звучало

Форс пуш в мастер

И засквошить все предыдущие коммиты.

В один

Е бой

Шаришь

И назвать его initial commit )

я думал в сша

Так он про Сиэттл

Я даже сказала, что напишу свой плагин лол

я тут недавно откопал https://github.com/sbtrun-maven-plugin/sbtrun-maven-plugin . Нужно было положить в проект помку, потому что ci ничего другого не умеет, а только мавен. В помке указал, чтобы на mvn package делал sbt assembly , ну и прочего такого. И все, живу с sbt

Пуш в мастер?

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

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

я тут недавно откопал https://github.com/sbtrun-maven-plugin/sbtrun-maven-plugin . Нужно было положить в проект помку, потому что ci ничего другого не умеет, а только мавен. В помке указал, чтобы на mvn package делал sbt assembly , ну и прочего такого. И все, живу с sbt

Прикольно. Я только думаю, что это что-то религиозное, что не позволяет юзать сбт. Но я обязательно посмотрю, может сгодится нам :)

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

+1

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

Чеки в гик кладешь и все

Оверкоммит конфиг вообще влет

Чеки в гик кладешь и все

блин, ну у меня не скала была , есть checkpatch.pl пускаешь его у себя на патч со всеми строгими флагами, тебе говорят, зис патч из файн, а когда ты делаешь пуш в девел тебе приходит, еее ты накосячил в чём-то и это ВСЕМ приходит, и непонятно почему

кто знает, как твой тёмный гений реализует проверку коммитов

ERROR: S client not available

Pl это poland?

А чем вас сонар не устроил со скала плагином?

Прикольно. Я только думаю, что это что-то религиозное, что не позволяет юзать сбт. Но я обязательно посмотрю, может сгодится нам :)

ну да, если религиозное, то не поможет, ага. А иначе для меня это пока самый простой путь)

pl - это перл вроде

Pl это poland?

pl это perl

А чем вас сонар не устроил со скала плагином?

Его можно настраивать вечность)

Зато когда настроишь...)

Выйдет новая версия и придётся все заново делать

Можно не обновляться

кому не слабо распарсить слова в код?)

https://scastie.scala-lang.org/T3UHsUZZRK60qurE5IYyXw

Вроде бы дошло, теперь есть другой вопрос, связаны ли trait и object?

связаны

Хотя ещё есть вопрос, почему мы можем вернуть Example (trait)? Мне не понятен момент с видимостью, она только на имена распространяется?

Вроде бы дошло, теперь есть другой вопрос, связаны ли trait и object?

Что такое Example(trait)?

Вообще рекомендую сначала какую-нибудь книжку почитать, красную там или Scala for impatient

Я читаю

Programming Scala

Видимо там делается ExampleImpl.asInstanceOf[Example] в apply

Хотя ещё есть вопрос, почему мы можем вернуть Example (trait)? Мне не понятен момент с видимостью, она только на имена распространяется?

да

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

аля билдер в скала стиле

https://scastie.scala-lang.org/T3UHsUZZRK60qurE5IYyXw

Могу также заметить, что лично я конкретно в этом случае не стал бы определять специальный класс Impl, просто написал бы new Example{...} в реализации apply

Это упрощённый пример из книги (из серии "можно и так"), хотел разобраться как это устроено

Это упрощённый пример из книги (из серии "можно и так"), хотел разобраться как это устроено

Всё правильно. Иногда такие вот классы/трейты очень даже нужны

Если посмотреть даже на библиотеки функционального программирования, там тоже очень часто реализации всё более мощных инстансов реализуют как цепочки наследования таких приватных классов / трейтов

вопрос тут назрел, пришёл на новый проект , запустил идеевский аналайзер , нашёл кучу всяких !Option(...).isEmpty в if ах и кучу подобных ошибок ( СОТНИ ИХ ), как дипломотичнее их вмержить в проект ? вообще пользуетесь ли вы идеевским аналайзером?

а что именно говорит идеевский аналайзер о !Option(...).isEmpty? И на что менять будешь? x != null?

Можно без isEmpty сделать match

а что именно говорит идеевский аналайзер о !Option(...).isEmpty? И на что менять будешь? x != null?

nonEmpty

Или засунуть все в for

почему не isDefined?

почему не isDefined?

isDefined тоже есть