да, но тут получается, что порядок бизнес операций определяется за пределами бизнес-слоя

Тебе сколько лет?

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

Коллеги, а у нас хоть один дебаггер умеет в инлайн функции вставать?

Так. нужен код ревью от любителей скалировать. примитивная задача с кодварс есть массив строк вида "ABCD 20", где первая буква - категория, дальше неизвестное кол-во рандомных букв и цифра - кол-во товара в категории. Дан массив таких товаров и дан массив категорий. Надо найти кол-во товаров по данным категориям. Пример товары = Array("ABART 20", "CDXEF 50", "BKWRK 25", "BTSQZ 89", "DRTYM 60") категории = Array("A", "B", "C", "W") результат (A : 20) - (B : 114) - (C : 50) - (W : 0) мой говнокод val artRegex: Regex = raw"([A-Z])[A-Z]+ (\d+)".r def stockSummary(lstOfArt: Array[String], lstOfCat: Array[String]): String = { val catMap = lstOfArt.toStream.map(art => artRegex.findFirstMatchIn(art) match { case Some(matches) => (matches.group(1), matches.group(2).toInt) case None => throw new Exception(s"wrong art - $art") }) .groupBy(x => x._1) .map(x => { val sum = x._2.map(x => x._2).sum val cat = x._1 (cat, sum) }) .withDefaultValue(0) lstOfCat.toStream .map(cat => { val sum = catMap(cat) s"($cat : $sum)" }) .mkString(" - ") } он рабочий, но мне нужно знать как его улучшить, где тут подвохи, может я тут неоптимально чот делаю. И ещё фишки как упростить, укоротить и всё такое прочее. Не факт что я их заюзаю мне их просто надо знать, потому что я не шарю суммоню всех известных скалистов @vshapenko @DogeShibu @grishace @fvnever

Настолько не понравился walmart? :)

Тут сложно. Пока только обещания

Так. нужен код ревью от любителей скалировать. примитивная задача с кодварс есть массив строк вида "ABCD 20", где первая буква - категория, дальше неизвестное кол-во рандомных букв и цифра - кол-во товара в категории. Дан массив таких товаров и дан массив категорий. Надо найти кол-во товаров по данным категориям. Пример товары = Array("ABART 20", "CDXEF 50", "BKWRK 25", "BTSQZ 89", "DRTYM 60") категории = Array("A", "B", "C", "W") результат (A : 20) - (B : 114) - (C : 50) - (W : 0) мой говнокод val artRegex: Regex = raw"([A-Z])[A-Z]+ (\d+)".r def stockSummary(lstOfArt: Array[String], lstOfCat: Array[String]): String = { val catMap = lstOfArt.toStream.map(art => artRegex.findFirstMatchIn(art) match { case Some(matches) => (matches.group(1), matches.group(2).toInt) case None => throw new Exception(s"wrong art - $art") }) .groupBy(x => x._1) .map(x => { val sum = x._2.map(x => x._2).sum val cat = x._1 (cat, sum) }) .withDefaultValue(0) lstOfCat.toStream .map(cat => { val sum = catMap(cat) s"($cat : $sum)" }) .mkString(" - ") } он рабочий, но мне нужно знать как его улучшить, где тут подвохи, может я тут неоптимально чот делаю. И ещё фишки как упростить, укоротить и всё такое прочее. Не факт что я их заюзаю мне их просто надо знать, потому что я не шарю суммоню всех известных скалистов @vshapenko @DogeShibu @grishace @fvnever

Кстати, в скале не option, а Either чаще находит применение

Мне не нравится findFirstMatchIn: получается, ты пропускаешь невалидные описания типа ABAB 34ХЕР

val sum = x._2.map(x => x._2).sum И вот ради этого ты забросил F#? :)

настолько всё плохо

На самом деле не очень плохо, но ты его довольно дефективненько используешь :)

Мне не нравится findFirstMatchIn: получается, ты пропускаешь невалидные описания типа ABAB 34ХЕР

я тут подумал, я их не пропускаю, а кидаю экспешун же, ну типа нехер мне тут невалидные описания вставлять

Не кидаешь

ааа, ну эт надо регес поправить

Далее, x => x._1 можно сократить до _._1 :)