I’d also say that the validation method should be in the Trade entity constructor itself and not in a different struct (invalid state should be illegal).