Scala: List[Tuple3] to Map[String,String]

Question

I've got a query result of List[(Int,String,Double)] that I need to convert to a Map[String,String] (for display in an html select list)

My hacked solution is:

val prices = (dao.getPricing flatMap {
  case(id, label, fee) =>
    Map(id.toString -> (label+" $"+fee))
  }).toMap

there must be a better way to achieve the same...

Jean-Philippe Pellet · Accepted Answer · 2012-05-28 13:29:52Z

9

How about this?

val prices: Map[String, String] =
  dao.getPricing.map {
    case (id, label, fee) => (id.toString -> (label + " $" + fee))
  }(collection.breakOut)

The method collection.breakOut provides a CanBuildFrom instance that ensures that even if you're mapping from a List, a Map is reconstructed, thanks to the type annotation, and avoids the creation of an intermediary collection.

answered May 28, 2012 at 13:29

Jean-Philippe Pellet

60.1k23 gold badges178 silver badges241 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

virtualeyes Over a year ago

+1. odd, what on earth is collection.breakout? ;-) I like that Map has been removed from the map{} block, but now it's been added to the return type ;-(, compiler inference rocks, no like type type type, that's for Java people ;-)

Jean-Philippe Pellet Over a year ago

@virtualeyes Either you write toMap and you have to create an intermediary collection, or you use breakOut with the type annotation.

virtualeyes Over a year ago

@Jean-Phillipe ah ok, so my approach is not only poor, but inefficient as well ;-) No big deal here, pricing consists of a handful of records, any which way does the trick, was just looking for a cleaner solution than what I had hacked together

Johnny Everson · Accepted Answer · 2012-11-13 03:23:25Z

8

A little more concise:

val prices =
  dao.getPricing.map { case (id, label, fee) => ( id.toString, label+" $"+fee)} toMap

shorter alternative:

val prices =
  dao.getPricing.map { p => ( p._1.toString, p._2+" $"+p._3)} toMap

edited Nov 13, 2012 at 3:23

answered May 28, 2012 at 13:35

Johnny Everson

8,6597 gold badges42 silver badges78 bronze badges

7 Comments

virtualeyes Over a year ago

there you go, nice one, +1, plus a prize, concision is king

Matt R Over a year ago

+1 for first alternative, -1 to the second ... giving things names is at lot more readable than p._1 etc. (Although virtualeyes might consider making the dao return a Pricing case class rather than using a triple, in which case the second option would be fine).

Johnny Everson Over a year ago

I agree, I prefer it, too. Just added for reference.

virtualeyes Over a year ago

@MattR you're right, using ScalaQuery, could just as well return a case class instance, but thought I'd explore tuples (since vast majority of query results are cast to case class instances, quite convenient)

virtualeyes Over a year ago

@MattR I switched to case class instance, the cleanest approach, and minimal extra overhead compared to returning a tuple subset

|

Collectives™ on Stack Overflow

Scala: List[Tuple3] to Map[String,String]

2 Answers 2

3 Comments

7 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

7 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related