How do I design my POJO to enable the correct JSON output ?

[tangara goh]

Hi expert,

I am running into another problem.

Guess my Java fundamental is really bad such that I don't know how to handle this case.

Basically, I have a Post endpoint where the @RequestBody is Order order

Now, this Order will have contains a petid

So, I have a constructor as follows:

Order(Integer id, Pet pet, Date ordertime, OrderStatus status, boolean complete){// id belongs to Order class
blah blah blah
}

Now, I create a order where I do the getter and setter

[code=java]
public Order create(Order order){
Order newOrder = new Order();
Pet pet = new Pet();
newOrder.setId(order.getd);
blah blah blah
newOrder.setId(Pet.getPetId); // using static in 
return newOrder
}

My apologies I coulnt't get the image out cos the photo is real bad.

When I test out using postman, it will give me :

"quantity" : null // I didn't ask for this so why it will appear ? but quanity is part of the fields in Order class.
"ordertimestamp : "2020-11-17T11:13343"
"status" : "approved"
"complete": false
"id" : null // ? why it will appear null ?

This is what I entered into the body of Postman:

{
 "id" : 11,
 "petid: 3,
 "orderTimestamp" :"2020-11-17T11:13343"
 "status" : approved",
 "complete": false,
 "id" : 2
}

My question is how do i get the petid to show up in postman and why would id - which is order id will appear null ?


[/code]

Any tips, hints are greatly appreciated.  

[Ron McLeod]

tangara goh wrote:How do I design my POJO to enable the correct JSON output ?

What does the POJO look like now?

[tangara goh]

Ron McLeod wrote:

tangara goh wrote:How do I design my POJO to enable the correct JSON output ?

What does the POJO look like now?

Hi Ron,

public class Pet{

private int petid; // i am thinking of changing this to static but I don't think this is a good idea

private Set<Order> orders = new LinkedHashSet<>(); // but I am using List<Order> in my OrderController - is this contradicting?

private User user;

private String petImage;

public Pet(int petId, Order order, String name, Status status){ 
this.orders = orders;
this.name = name;
this.status = status;
}


getter and setter ...

public class Order

private Integer orderId;
private Integer qty;
private Date orderTimestamp;
prviate OrderStatus status;
private boolean complete;

private Pet pet;

Order(Integer orderid, Pet pet, Date orderTimestamp, OrderStatus status, boolean complete){
this.OrderId = orderid;
this.pet = pet;
this.orderTimestamp = orderTimestamp;
this.status = status;
this.complete = complete;

Please let me know what to do in order to let the petId show up.

Is using statis at petId a good idea or ?

Thanking you in advance for your help/hints.

[Dave Tolls]

Why do you feel making petId static is a reasonable change?
What does petId represent, and how does making it static help with that representation?

For the main question, where is that createOrder method used?
Why are you (I think) duplicating the Order, and why are you setting the Order id twice (lines 11 and 13 in your posted code)?

[tangara goh]

Dave Tolls wrote:Why do you feel making petId static is a reasonable change?
What does petId represent, and how does making it static help with that representation?

For the main question, where is that createOrder method used?
Why are you (I think) duplicating the Order, and why are you setting the Order id twice (lines 11 and 13 in your posted code)?

Hi Dave,

the createOrder method will be used in Order Controller.

Sorry the line 13 should be setPetId, and not OrderId.  

Now, I have changed the constructor in Pet to use int PetId instead of Pet pet.

I hope to confirm with the experts here whether it is ok to use int petId  or it is advisable not to?

However, I have tried for a few hours using Pet pet inside the Order constructor I just couldn't get the petId ...

Furthermore, this petId will be used in another Controller endpoints so if I don't convert it into static or global variable, will there be a null pointer error problem ?

I am not sure how to test multiple end points in Postman though.

Hope I can have some guidance on the above.

Thanks.