How to concatenate a list to another list in scala

Question

Here i use to concatenate 2 Lists (with same type) using this :+

and the code is testResult :+ studentsearch

the length of the list get value zero in the console

Controller

@RequestMapping(value = Array("parent/ViewReports.html"))
 def viewReportPage(test: Test): String = {    

    var userAccount: UserAccount = secService.getLoggedUserAccount
    var z = userAccount.getId().toLong
    var studentsearch: java.util.List[StudentTest] = Nil
    var studentTestResult: java.util.List[StudentTest] =Nil
    var testResult: java.util.List[StudentTest] =Nil      

    var searchstudent = parentStudentService.findParentStudentByParent(z)

    for(srchStd <- searchstudent){    
      var sid= srchStd.getStudent().getId()  
      studentsearch = studentTestService.findAllStudentTestByStudent(sid)
    }

    testResult :+ studentsearch 
    println("Length:" +testResult.length)  
    studentTestResult = ListBuffer(testResult: _*)    .
    results = studentTestResult

    "redirect:/parent/result.html"
}

the concatenate code is wrong ?

please share your answers

Chris Martin · Answer 1 · 2013-12-11T16:51:20.373

4

You should look at the API docs for the :+ methods you're calling.

I'm not sure which Scala collection type that java.util.List is being implicitly converted to, but I think the :+ method is the same everywhere: http://www.scala-lang.org/api/current/#scala.collection.Seq

java.util.List is implicitly converted to scala.collection.mutable.Buffer.

A copy of this sequence with an element appended.

In other words, :+ doesn't modify the collection, so the statement TestResult :+ studentsearch has no effect.

You could call the Java collection's method directly: TestResult addAll studentsearch.

Although, may I ask, why are you using java.util.List instead of a Scala collection?

I should also note, of course, that in Scala just like in Java, starting a variable name with a capital letter is frowned upon heavily. (testResult, not TestResult). It's even more important in Scala because variables and types share the same namespace, unlike in Java.

edited Dec 11 '13 at 16:51

answered Dec 11 '13 at 07:12

Chris Martin

30,334
10
78
137

why i use java.util.list because it's a java -scala interoperable project – Prasanth A R Dec 11 '13 at 07:32
i change :+ to addAll the error on browser Request processing failed; nested exception is java.lang.UnsupportedOperationException – Prasanth A R Dec 11 '13 at 07:35
can you try appendAll – Vishal John Dec 11 '13 at 07:44
@John It's my mistake i didn't get – Prasanth A R Dec 11 '13 at 07:46
1

java.util.List implicitly gets converted to scala.Buffer -[Link](http://docs.scala-lang.org/overviews/collections/conversions-between-java-and-scala-collections.html). So refer to methods of Buffer - [Doc](http://www.scala-lang.org/api/current/index.html#scala.collection.mutable.Buffer) – Vishal John Dec 11 '13 at 07:51
Prasanth: From the code you've shown, the `testResult` structure doesn't have to interop with any Java structures. – Chris Martin Dec 11 '13 at 16:54

score 3 · Accepted Answer · answered Dec 11 '13 at 09:09

Your code uses scala constructs in an interesting way. Which might not lead to correct results. The for comprehension in

for(srchStd <- searchstudent){    
  var sid= srchStd.getStudent().getId()  
  studentsearch = studentTestService.findAllStudentTestByStudent(sid)
}

reassigns studentsearch in each iteration to a new value, so in the end it is set to the value of the last iteration. If you want to accumulate the results something like

import scala.collection.JavaConversions._

val searchstudent = parentStudentService.findParentStudentByParent(z)

val testResult = searchstudent.flatMap{ srchStd =>
  val sid = srchStd.getStudent().getId()  
  studentTestService.findAllStudentTestByStudent(sid)
}

will result in all tests collected in testResult.

The import of scala.collection.JavaConversions._ helps converting between scala and java collections. More information can be found here: http://www.scala-lang.org/api/current/index.html#scala.collection.JavaConversions$.

score 2 · Answer 3 · answered Dec 11 '13 at 08:39

You may want to look at the JavaConversions object within the Scala standard library, which allows you to migrate between Scala types and Java collections. Which from the documentation supports the following conversions:

scala.collection.Iterable <=> java.lang.Iterable
scala.collection.Iterable <=> java.util.Collection
scala.collection.Iterator <=> java.util.{ Iterator, Enumeration }
scala.collection.mutable.Buffer <=> java.util.List
scala.collection.mutable.Set <=> java.util.Set
scala.collection.mutable.Map <=> java.util.{ Map, Dictionary }
scala.collection.mutable.ConcurrentMap (deprecated since 2.10) <=> java.util.concurrent.ConcurrentMap
scala.collection.concurrent.Map <=> java.util.concurrent.ConcurrentMap

You should probably also consider using native Scala collections (List, Seq etc...) in situations where the collections are being created/populated natively within Scala and not being generated through an outcall to an existing lump of Java code.

Hope this helps

score 2 · Answer 4 · answered Dec 11 '13 at 12:43

2

For completeness, the operator used to concatenate a list to another list is ++=. Although as the accepted answer notes, it is better to use flatMap in this case. (You can also use for..yield.)

answered Dec 11 '13 at 12:43

Robin Green

32,079
16
104
187

How to concatenate a list to another list in scala

4 Answers4