Added expression to check values(must be > 0), and added joptionpane