Scala client composition with Traits vs implementing an abstract class

Question

I have read that with Scala, it is generally advised to use Traits instead of Abstract classes to extend a base class.

Is the following a good design pattern and layout? Is this how Traits were intended to replace Abstract?

• client class (with def function1)
• trait1 class (overrides function1)
• trait2 class (overrides function1)
• specificClient1 extends client with trait1
• specificClient2 extends client with trait2

Answer 1

I don't know what your source is for the claim that you should prefer traits over abstract classes in Scala, but there are several reasons not to:

1. Traits complicate Java compatibility. If you have a trait with a companion object, calling methods on the companion object from Java requires bizarre MyType$.MODULE$.myMethod syntax. This isn't the case for abstract classes with companion objects, which are implemented on the JVM as a single class with static and instance methods. Implementing a Scala trait with concrete methods in Java is even more unpleasant.
2. Adding a method with an implementation to a trait breaks binary compatibility in a way that adding concrete methods to a class doesn't.
3. Traits result in more bytecode and some additional overhead related to the use of forwarder methods.
4. Traits are more powerful, which is bad—in general you want to use the least powerful abstraction that gets the job done. If you don't need the kind of multiple inheritance they support (and very often you don't), it's better not to have access to it.

The last reason is by far the most important in my view. At least a couple of the other issues might get fixed in future versions of Scala, but it will remain the case that defaulting to classes will constrain your programs in ways that are (at least arguably) consistent with good design. If you decide you actually really do want the power provided by traits, they'll still be there, but that'll be a decision you make, not something you just slip into.

So no, in the absence of other information, I'd suggest using an abstract class (ideally a sealed one) and two concrete classes that provide implementations.