rxue / dictionary

Practice in Java EE
1 stars 0 forks source link

Analysis & Refactor on the simplification of the jpaentity package #42

Open rxue opened 1 year ago

rxue commented 1 year ago

atm separate ItemValue seems not simplify the logic but make the code structure a bit more convoluted => the use of @Embedded paried with @Embeddable seems not a good use case inside this jpaentity package, or at least the ItemValue has no need to be inside the jpaentity

Another problem is of the ItemValue is the naming: after some time away from this project, I am not able to catch up with the meaning and use of ItemValue immediately, meaning the naming is not intuitive enough