search

eclipse-arrowhead / application-library-java-spring

Arrowhead Application Library | Java Spring-Boot
Eclipse Public License 2.0
1 stars 8 forks source link

implement toString() in all DTO and POJO #10

Closed BlackRose01 closed 4 years ago

BlackRose01 commented 4 years ago

Hello,

could you please implement the toString() method in all DTO and POJO classes. This will decrease debugging time and leads to a better code quality because we don't need to create extra methods or long nested implemenations to get the information from the objects. The GSON library would be a good choice for this purpose, as it can do this quickly and easily. For example:

@Override
public String toString() {
    return new Gson().toJson(this);
}

KR, BlackRose01

BlackRose01 commented 4 years ago

sorry but bump

borditamas commented 4 years ago

Hi, feel free to create a PR with your changes in core-java-spring repository. (All the DTO comes from the dto package of that repository).

It would be useful indeed, but for us it is not the highest priotity to deal with at the moment.

BlackRose01 commented 4 years ago

Hello,

I cloned this repository (master) and I cannot find the DTO package.

image

KR

borditamas commented 4 years ago

Hi,

as I wrote all DTO comes from core-java-spring repository.

BlackRose01 commented 4 years ago

Pull Request was created.