Блог

Я думаю, что это может быть обычная операция. Так что, возможно, это внутри API, но я не могу его найти. Также меня интересует эффективное функциональное / простое решение, если нет.

Учитывая последовательность кортежей ("a" -> 1, "b" ->2, "c" -> 3) , я хочу превратить это в карту. Это простое использование TraversableOnce.toMap . Но я хочу потерпеть неудачу в этой конструкции, если результирующая карта «будет содержать противоречие», то есть разные значения, присвоенные одному и тому же ключу. Как в последовательности ("a" -> 1, "a" -> 2) . Но дубликаты должны быть разрешены.

В настоящее время у меня есть этот (очень важный) код:

 def buildMap[A,B](in: TraversableOnce[(A,B)]): Option[Map[A,B]] = {
  val map = new HashMap[A,B]
  val it = in.toIterator
  var fail = false
  while(it.hasNext){
    val next = it.next()
    val old = map.put(next._1, next._2)
    fail = old.isDefined amp;amp; old.get != next._2
  }

  if(fail) None else Some(map.toMap)
}
  

Побочный вопрос

Действительно ли необходим final toMap ? Я получаю ошибку типа при ее пропуске, но я думаю, что это должно сработать. Реализация toMap создает новую карту, которой я хочу избежать.

Как всегда при работе с Seq[A] оптимальным решением, производительность зависит от конкретного типа коллекции. Общим, но не очень эффективным решением было бы свернуть Option[Map[A,B]] :

 def optMap[A,B](in: Iterable[(A,B)]): Option[Map[A,B]] = 
  in.iterator.foldLeft(Option(Map[A,B]())) {
    case (Some(m),e @ (k,v)) if m.getOrElse(k, v) == v => Some(m   e)
    case _ => None
  }
  

Если вы ограничитесь использованием List[A,B] s, оптимизированная версия будет:

 @tailrec
def rmap[A,B](in: List[(A,B)], out: Map[A,B] = Map[A,B]()): Option[Map[A,B]] = in match {
  case (e @ (k,v)) :: tail if out.getOrElse(k,v) == v =>
    rmap(tail, out   e)
  case Nil =>
    Some(out)
  case _ => None
}
  

Кроме того, менее идиоматичная версия, использующая изменяемые карты, может быть реализована следующим образом:

 def mmap[A,B](in: Iterable[(A,B)]): Option[Map[A,B]] = {
  val dest = collection.mutable.Map[A,B]()

  for (e @ (k,v) <- in) {
    if (dest.getOrElse(k, v) != v) return None
    dest  = e
  }

  Some(dest.toMap)
}
  

Вот решение, которое не приводит к сбоям (если создание всей карты, а затем удаление ее допустимо):

 def uniqueMap[A,B](s: Seq[(A,B)]) = {
  val m = s.toMap
  if (m.size == s.length) Some(s) else None
}
  

Вот изменяемое решение без сбоев (отключается при обнаружении ошибки):

 def uniqueMap[A,B](s: Seq[(A,B)]) = {
  val h = new collection.mutable.HashMap[A,B]
  val i = s.iterator.takeWhile(x => !(h contains x._1)).foreach(h  = _)
  if (h.size == s.length) Some(h) else None
}
  

И вот неизменяемое безотказное решение:

 def uniqueMap[A,B](s: Seq[(A,B)]) = {
  def mapUniquely(i: Iterator[(A,B)], m: Map[A,B]): Option[Map[A,B]] = {
    if (i.hasNext) {
      val j = i.next
      if (m contains j._1) None
      else mapUniquely(i, m   j)
    }
    else Some(m)
  }
  mapUniquely(s.iterator, Map[A,B]())
}
  

Редактировать: и вот решение, использующее put для ускорения (надеюсь):

 def uniqueMap[A,B](s: Seq[(A,B)]) = {
  val h = new collection.mutable.HashMap[A,B]
  val okay = s.iterator.forall(x => {
    val y = (h put (x._1,x._2))
    y.isEmpty || y.get == x._2 
  })
  if (okay) Some(h) else None
}
  

Редактировать: теперь протестировано, и это работает в ~ 2 раза быстрее при вводе (возвращает true), чем Мориц или мое простое решение.

Scala 2.9 уже близко, так почему бы не воспользоваться методом комбинаций (вдохновленным ответом Морица):

 def optMap[A,B](in: List[(A,B)]) = {
  if (in.combinations(2).exists { 
    case List((a,b),(c,d)) => a == c amp;amp; b != d
    case _ => false 
  }) None else Some(in.toMap)
}

scala> val in = List(1->1,2->3,3->4,4->5,2->3)
in: List[(Int, Int)] = List((1,1), (2,3), (3,4), (4,5), (2,3))

scala> optMap(in)
res29: Option[scala.collection.immutable.Map[Int,Int]] = Some(Map(1 -> 1, 2 -> 3, 3 -> 4, 4 -> 5))

scala> val in = List(1->1,2->3,3->4,4->5,2->3,1->2)
in: List[(Int, Int)] = List((1,1), (2,3), (3,4), (4,5), (2,3), (1,2))

scala> optMap(in)
res30: Option[scala.collection.immutable.Map[Int,Int]] = None
  

Вы также можете использовать gourpBy следующим образом:

   val pList = List(1 -> "a", 1 -> "b", 2 -> "c", 3 -> "d")

  def optMap[A,B](in: Iterable[(A,B)]): Option[Map[A,B]] = {
    Option(in.groupBy(_._1).map{case(_, list) => if(list.size > 1) return None else list.head})
  }

  println(optMap(pList))
  

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