why this considered bad and how we fix it ?

We decided to add a feature to the Product class by adding the product's weight information. We decided to make it public, so any subclass can change it easily. public class Product private int id; private double price; private String name; public double weight; public Product (int id, double price, string name, int orderId, string orderstatus) l this. id - id; this, price - price; this, name = name; this. orderId = orderid; this. orderstatus = "created" this. quantity +