Будет покрасивше

А мне кажется, что нет. Но это вкусовщина.

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

Так, второй заход: val artRegex: Regex = raw"^([A-Z])[A-Z]+\s(\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(_._1) .map(x => { val sum = x._2.map(_._2).sum val cat = x._1 (cat, sum) }) .withDefaultValue(0) lstOfCat.toStream .map(cat => s"($cat : ${catMap(cat)})") .mkString(" - ") }

Так. нужен код ревью от любителей скалировать. примитивная задача с кодварс есть массив строк вида "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

Особо не вчитывался, пишу то, за что взгляд сходу зацепился. 1. Инстанс регекспа - это сразу unapplier, его можно использовать в паттерн матчах. 2. Вместо исключения лучше Either, если уж про ФП говорим 3. Сложные функции из map лучше перенести в отдельную локальную функцию.

1. не понял 2. Ща попробую, спасибо 3. Ща попробую, спасибо

val MyRegex = """(a+)""".r "aaaa" match { case MyRegex(as) => as case _ => "Some other string" }

@omgszer пиши на котлине 🤓

что-то не то получается lstOfArt.toStream.map(art => art match { case artRegex(cat, qty) => (cat, qty.toInt) case _ => throw new Exception(s"wrong art - $art") })

Во, выглядит збс. Очень большое спасибо

Во, выглядит збс. Очень большое спасибо

Надо было доге сразу пинговать

Проверь на всякий случай, что работает. Я не проверял. Обрати ещё внимание на фукнцию collect и на тип PartialFunction, очень интересная штука, которая во многих случаях может пригодиться.

я правильно понимаю что здесь PartialFunction в том, что мы не все кейсы указываем и это валидно?

val ArtRegex: Regex = raw"([A-Z])[A-Z]+ (\d+)".r object IsInt { def unapply(arg: String): Option[Int] = Try(arg.toInt).toOption } final case class Art(name: String, qty: Int) def stockSummary(lstOfArt: Array[String], lstOfCat: Array[String]): String = { val catMap = lstOfArt .collect { case ArtRegex(name, IsInt(int)) => Art(name, int) } .groupBy(_.name) .map { case (name, group) => (name, group.map(_.qty).sum) } .withDefaultValue(0) lstOfCat .map(cat => s"($cat : ${catMap(cat)})") .mkString(" - ") }

ещё вопрос. Как заставить это работать? case class Art(name: String, qty: Int) { def unapply(arg: String): Option[Int] = Try(arg.toInt).toOption }

Хмм... а чего именно хочешь добиться? Потому что тут у тебя деконструктор обьявлен у инстанса Art и именно для Int.

Ну наприме я хочу создать некий тип у которого есть конструктор вида Art(string, int) и патерн для деконстракта из стринги в Option int

Бля, Visual studio на маке - какой-то пздц

Бля, Visual studio на маке - какой-то пздц

Там какой-то треш с чтением из консоли

Какой треш?

@DogeShibu ешё вопрос. вот в такой конструкции, если lstOfCat массив - map сделает ещё один массив? lstOfCat .map(cat => s"($cat : ${catMap(cat)})") .mkString(" - ")

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

а недостаточно ли .toStream сделать?

toStream тоже не самая быстрая штука, если что.

а эт не жаба стримы разве?

Нет, это ленивый связвный список

нуууу......... стримы то есть?)))