do_more icon indicating copy to clipboard operation
do_more copied to clipboard

Try to use Equatable for the user models

Open AYM1607 opened this issue 6 years ago • 1 comments

The models use named constructors and the code necessary to make it work with Equatable is too verbose and unnecessary. This is because the call to the super constructor (Equatable) complains when accessing non static members.

sticking to overriding hashCode and the == operator for now.

AYM1607 avatar Feb 25 '19 01:02 AYM1607

overriding the operators seemed like a good idea. However, some models have attributes of non primitive types, this causes the equality check to fail even with the overrode operators.

Would be helpful to find out if equatable handles this situations well.

If it does, the best option is to move to it even if the code is a little more verbose.

AYM1607 avatar Feb 25 '19 21:02 AYM1607