Java question. Help needed.

• Refugees

  23 May 07, 19:03

  import javax.swing.JOptionPane;
  
  public class Revision_Practical {
  public static void main(String[] args) {
  int years = getYears();
  double salary = getSalary();
  String output = "";
  }
  
  public static int getYears() {
  int years;
  do {
  String input = JOptionPane.showInputDialog(null,
  "Please enter the number of years worked");
  years = Integer.parseInt(input);
  if (years <= 0) {
  JOptionPane.showMessageDialog(null,
  "Years worked must be more than 0. Please re-enter");
  } else {
  JOptionPane.showMessageDialog(null,
  "The number of years worked is: " +
  years);
  
  }
  
  } while (years <= 0);
  return years;
  } //end getYears
  
  public static double getSalary() {
  double salary;
  do {
  String input1 = JOptionPane.showInputDialog(null,
  "Please enter the salary");
  salary = Double.parseDouble(input1);
  if (salary <= 0) {
  JOptionPane.showMessageDialog(null,
  "Salary must be more than 0. Please re-enter");
  } else {
  JOptionPane.showMessageDialog(null, "The salary is: " + salary);
  }
  } while (salary <= 0);
  return salary;
  } //end getSalary
  
  public static int checkIncrement(int years, double salary) {
  int increase = 0;
  if (years < 10) {
  
  if (salary < 1000) {
  increase = 100;
  } else if (salary < 2000) {
  increase = 200;
  } else {
  increase = 300;
  }
  } //end top if
  
  if (years > 10) {
  if (salary < 1000) {
  increase = 200;
  } else if (salary < 2000) {
  increase = 300;
  } else {
  increase = 400;
  }
  } //end top if
  return increase;
  } //end checkIncrement
  } //end class
  
  This is the first part of the question which i manage to do but what i don't understand is this.
  
  d) Modify the main() method as follows:
  • Invoke getYears() method to get the number of years worked.
  
  • Invoke getSalary() method to get the salary.
  
  • Invoke checkIncrement() method to determine the increment amount.
  
  • Declare a String variable called output to concatenate the required information (Years Worked, Salary and Increment)
  
  Can anyone help?
  
• ndmmxiaomayi

  23 May 07, 19:07

  I don't see your main() method anywhere.
  
  Anyway, my Java fail... but roughly it means to:
  
  Write a main() method which will output the number of years, salary and increment.
• turbo_drift

  23 May 07, 19:09

  No main(), how to do like this nia...
  
  Put main() at the top most and try..
  
  Btw, my java just pass only..
  
• Refugees

  23 May 07, 19:15

  so the last part i must write a method out again?
  how do i concatenate the first 3 method tgt using the string output?
  
  
• ndmmxiaomayi

  23 May 07, 19:21

  Use the + operator...
• Refugees

  23 May 07, 19:27

  import javax.swing.JOptionPane;
  
  public class Revision_Practical {
  public static void main(String[] args) {
  int years = getYears();
  double salary = getSalary();
  int increase = checkIncrement(years, salary);
  }
  
  public static int getYears() {
  int years;
  do {
  String input = JOptionPane.showInputDialog(null,
  "Please enter the number of years worked");
  years = Integer.parseInt(input);
  if (years <= 0) {
  JOptionPane.showMessageDialog(null,
  "Years worked must be more than 0. Please re-enter");
  } else {
  JOptionPane.showMessageDialog(null,
  "The number of years worked is: " +
  years);
  
  }
  
  } while (years <= 0);
  return years;
  } //end getYears
  
  public static double getSalary() {
  double salary;
  do {
  String input1 = JOptionPane.showInputDialog(null,
  "Please enter the salary");
  salary = Double.parseDouble(input1);
  if (salary <= 0) {
  JOptionPane.showMessageDialog(null,
  "Salary must be more than 0. Please re-enter");
  } else {
  JOptionPane.showMessageDialog(null, "The salary is: " + salary);
  }
  } while (salary <= 0);
  return salary;
  } //end getSalary
  
  public static int checkIncrement(int years, double salary) {
  int increase = 0;
  if (years < 10) {
  
  if (salary < 1000) {
  increase = 100;
  } else if (salary < 2000) {
  increase = 200;
  } else {
  increase = 300;
  }
  } //end top if
  
  if (years > 10) {
  if (salary < 1000) {
  increase = 200;
  } else if (salary < 2000) {
  increase = 300;
  } else {
  increase = 400;
  }
  } //end top if
  
  String output = "";
  if (increase < 100) {
  output = "fair";
  } else if (increase < 200) {
  output = "ok";
  } else if (increase < 300) {
  output = "good";
  } else {
  output = "excellent";
  }
  
  JOptionPane.showMessageDialog(null,
  "Years Worked: " + years + "\n" + "Salary: " +
  salary + "\n" + "Increment: " + increase +
  "\n" + "\n" + "Increment amount " +
  output);
  
  return increase;
  
  } //end checkIncrement
  
  }
  
  This is the new coding. But the value for increase & increment doesn't appear. Where got error? Being doing since 1 nw still cannot finish. sigh
• turbo_drift

  23 May 07, 19:28

  Originally posted by Refugees:

  so the last part i must write a method out again?
  how do i concatenate the first 3 method tgt using the string output?
  
  

  I assume ur last part is this..
  
  

  Originally posted by Refugees:

  if (years > 10) {
  if (salary < 1000) {
  increase = 200;
  } else if (salary < 2000) {
  increase = 300;
  } else {
  increase = 400;
  }
  } //end top if
  return increase;
  } //end checkIncrement
  } //end class
  

  Based on what i've learned.. I think u may try it out (using the below one to replace the last part..
  
  

  Originally posted by turbo_drift:
  
  if (years > 10 + salary < 1000)
  increase = 200;
  else if (salary < 2000)
  increase = 300;
  else
  increase = 400;
  
  return increase;

  Some part of the java that i've learned, has returned to my teacher liaox.. If wrong, don't blame me hor..
• ndmmxiaomayi

  23 May 07, 19:40

  Try commenting out this part of the codes first:
  
  

  if (years > 10) {
  if (salary < 1000) {
  increase = 200;
  } else if (salary < 2000) {
  increase = 300;
  } else {
  increase = 400;
  }
  } //end top if
  
  String output = "";
  if (increase < 100) {
  output = "fair";
  } else if (increase < 200) {
  output = "ok";
  } else if (increase < 300) {
  output = "good";
  } else {
  output = "excellent";
  }

  Then print out the results.
  
  If the results are correct, it's your subsequent that is wrong. If you still get the same thing, then I don't know liao.