search

scala-academy / castalia

API stub server
Other
8 stars 3 forks source link

Feature/#51 refactoring code style #52

Closed jeanmarc closed 8 years ago

jeanmarc commented 8 years ago

Ik heb een aantal pattern matching delen herschreven, maar verbetering is volgens mij minimaal. Codacy klaagt over risico van .head en .tail op een empty collection, maar de originele code had een foobar.isEmpty waarop gematched werd, en alleen als foobar.isEmpty == false werd er een foobar.head en/of foobar.tail gedaan.

Wel heb ik de overtollige return statements herschreven naar pattern match.

TimSoethout commented 8 years ago

@DieBauer, By default the Scala compiler optimises any calls that called in a tailwise fasion (also without the annotation). If you explicitly add @tailrec the compiler will fail if it can't, so it will notify you if you didn't write optimisable code.