Why underscore private `_domainEvents` in AggregateRoot? about white-label HOT 3 OPEN

Hey,

I believe that this is only a stylistic choice. In general, it's a common JS convention and even though it's not needed in TS lot's of developers still use it for readability.

from white-label.

Hey,

I believe that this is only a stylistic choice. In general, it's a common JS convention and even though it's not needed in TS lot's of developers still use it for readability.

Thanks for getting back to me @adriana-olszak :)

I understand what you are saying, I was more curious because in The SOLID book, Khalil specifically says to not do that in TypeScript:

Rule: Don’t use unnecessary member prefixes
Sometimes, developers prepend their private member variables with an underscore to signify that it’s private.
Example: _firstName.
In JavaScript, there’s no concept of private access modifiers, so the convention is to signal that it’s private this way.
In TypeScript, we do have private access modifiers.
Page 167

In typed languages like Java and TypeScript, usage of the underscore convention is discour-
aged because we have access to scopemodifiers like private and protected. We should use
the language constructs, given their availability, and look to conventions when a common
task cannot be done with the language.
Page 252

I was curious if there was a particular reason, or just a mistake here :)

from white-label.

I think I understand now. We use _somePropOrMethod in an abstract class so that we can use somePropOrMethod in a concrete class.

from white-label.