Thread: How can I make a loop repeat?

import javax.swing.JOptionPane;
public class program7 {
 
    public static void main(String[] args) {
 
//ask user how many euros a dollar buys
String euroExchangeRateString = JOptionPane.showInputDialog
        (null, "Enter how many Euros one Dollar will buy");
double euroExchangeRate = Double.parseDouble(euroExchangeRateString);
 
//ask user how many pounds a dollar will buy
String poundExchangeRateString = JOptionPane.showInputDialog
        (null, "Enter how many Pounds one Dollar will buy");
double poundExchangeRate =  Double.parseDouble(poundExchangeRateString);
 
//ask user how many yen a dollar will buy
String yenExchangeRateString = JOptionPane.showInputDialog
        (null, "Enter how many Yen one Dollar will buy");
double yenExchangeRate = Double.parseDouble(yenExchangeRateString);
 
//ask user to enter the number of dollars they have
 
    String amountDollarsString = JOptionPane.showInputDialog
(null, "Enter the amount of foreign currency you want to buy in U.S. dollars");
double amountDollars = Double.parseDouble(amountDollarsString);
if (amountDollars <= 100){
amountDollars -= amountDollars * 0.10;
JOptionPane.showMessageDialog
        (null,"You can buy $" + amountDollars + " in foreign currency.");
}
else if (amountDollars > 100) {
amountDollars -= amountDollars * 0.05;
JOptionPane.showMessageDialog
        (null,"You can buy $" + amountDollars + " in foreign currency.");
    }
 
 
String reply = JOptionPane.showInputDialog 
    (null, "Enter 'E' to buy Euros, 'P' to buy Punds or 'Y' to buy Yen");
char firstChar = reply.charAt(0);
 
while (firstChar == 'E' || firstChar == 'e'){
double amountEuros = (amountDollars * euroExchangeRate);
JOptionPane.showMessageDialog
        (null, "You have purchased " + amountEuros + " Euros.");
}
if (firstChar == 'P' || firstChar == 'p') {
double amountPounds = (amountDollars * poundExchangeRate);
JOptionPane.showMessageDialog
        (null, "You have purchased " + amountPounds + " Pounds.");
}
else if (firstChar == 'Y' || firstChar == 'y') {
double amountYen = (amountDollars * yenExchangeRate);
JOptionPane.showMessageDialog
        (null, "You have purchased " + amountYen + " Yen.");
}
JOptionPane.showConfirmDialog
        (null, "Are there any more conversions you would like to perform?");
        }
    }

On the last part, the program is supposed to ask the user if there are anymore conversions to perform. I put in a confirm dialog box, but when the user clicks yes it does not go back to repeat the previous loop. How can I make it do that?

Please edit your code so that its format is readable including using standard and consistent indentation. Next, any code that you want to repeat must be enclosed in a loop, and I don't see you doing this. Enclose it all that needs to repeat in a while loop.

Okay I have enclosed the loop, but it still won't repeat when yes is selected. I have also attempted to make the program more readable with indentations

import javax.swing.JOptionPane;
public class program7 {
 
    public static void main(String[] args) {
 
//ask user how many euros a dollar buys
String euroExchangeRateString = JOptionPane.showInputDialog
        (null, "Enter how many Euros one Dollar will buy");
    double euroExchangeRate = Double.parseDouble(euroExchangeRateString);
 
//ask user how many pounds a dollar will buy
String poundExchangeRateString = JOptionPane.showInputDialog
        (null, "Enter how many Pounds one Dollar will buy");
    double poundExchangeRate =  Double.parseDouble(poundExchangeRateString);
 
//ask user how many yen a dollar will buy
String yenExchangeRateString = JOptionPane.showInputDialog
        (null, "Enter how many Yen one Dollar will buy");
    double yenExchangeRate = Double.parseDouble(yenExchangeRateString);
 
//ask user to enter the number of dollars they have
 
    String amountDollarsString = JOptionPane.showInputDialog
(null, "Enter the amount of foreign currency you want to buy in U.S. dollars");
double amountDollars = Double.parseDouble(amountDollarsString);
 
if (amountDollars <= 100){
    amountDollars -= amountDollars * 0.10;
    JOptionPane.showMessageDialog
        (null,"You can buy $" + amountDollars + " in foreign currency.");
}
else if (amountDollars > 100) {
    amountDollars -= amountDollars * 0.05;
    JOptionPane.showMessageDialog
        (null,"You can buy $" + amountDollars + " in foreign currency.");
    }
 
 
String reply = JOptionPane.showInputDialog 
    (null, "Enter 'E' to buy Euros, 'P' to buy Punds or 'Y' to buy Yen");
char firstChar = reply.charAt(0);
 
while (firstChar == 'E' || firstChar == 'e'){
    double amountEuros = (amountDollars * euroExchangeRate);
    JOptionPane.showConfirmDialog
        (null, "You have purchased " + amountEuros + 
        " Euros. Are there any other conversions you would like to perform?");
}
if (firstChar == 'P' || firstChar == 'p') {
    double amountPounds = (amountDollars * poundExchangeRate);  
    JOptionPane.showConfirmDialog
        (null, "You have purchased " + amountPounds + 
        " Pounds. Are there any other conversions you would like to perform");
}
else if (firstChar == 'Y' || firstChar == 'y') {
    double amountYen = (amountDollars * yenExchangeRate);
    JOptionPane.showConfirmDialog
        (null, "You have purchased " + amountYen + 
        " Yen. Are there any other conversions you would like to perform");
}
}
}

Okay, I have made some indentations at key points and just including the final loop. This is the part I want fixed.

String reply = JOptionPane.showInputDialog 
    (null, "Enter 'E' to buy Euros, 'P' to buy Punds or 'Y' to buy Yen");
 
    char firstChar = reply.charAt(0);
 
//begin loop
while (firstChar == 'E' || firstChar == 'e'){
 
    double amountEuros = (amountDollars * euroExchangeRate);
 
    JOptionPane.showConfirmDialog
        (null, "You have purchased " + amountEuros + 
        " Euros. Are there any other conversions you would like to perform?");
}
 
if (firstChar == 'P' || firstChar == 'p') {
 
    double amountPounds = (amountDollars * poundExchangeRate);  
 
    JOptionPane.showConfirmDialog
        (null, "You have purchased " + amountPounds + 
        " Pounds. Are there any other conversions you would like to perform");
}
 
else if (firstChar == 'Y' || firstChar == 'y') {
 
    double amountYen = (amountDollars * yenExchangeRate);
 
    JOptionPane.showConfirmDialog
        (null, "You have purchased " + amountYen + 
        " Yen. Are there any other conversions you would like to perform");
}
}
}

When Yes is selected, I want the program to go back to the beginning of the loop

Let me repost some of your code:

      String reply = JOptionPane.showInputDialog(null,
            "Enter 'E' to buy Euros, 'P' to buy Punds or 'Y' to buy Yen");
      char firstChar = reply.charAt(0);
 
      // begin loop
      while (firstChar == 'E' || firstChar == 'e') {
         double amountEuros = (amountDollars * euroExchangeRate);
         JOptionPane.showConfirmDialog(null, "You have purchased " + amountEuros
               + " Euros. Are there any other conversions you would like to perform?");
      }
 
      if (firstChar == 'P' || firstChar == 'p') {
         double amountPounds = (amountDollars * poundExchangeRate);
         JOptionPane.showConfirmDialog(null, "You have purchased " + amountPounds
               + " Pounds. Are there any other conversions you would like to perform");
      }
 
      else if (firstChar == 'Y' || firstChar == 'y') {
         double amountYen = (amountDollars * yenExchangeRate);
         JOptionPane.showConfirmDialog(null,  "You have purchased "+ amountYen
               + " Yen. Are there any other conversions you would like to perform");
      }

You need to put your code aside and instead think of the logic of your program, and even write some pseudo-code from this logic. Something like:

declare variable to check to see if user wants to continue looping.
check the result of variable above. While they want to continue looping, do loop:
  Find out what kind of converstion to make
  if conversion is foo
    do foo
  end if
  else if conversion is bar
    do bar
  end else if
  else if conversion is baz
    do baz
  end else if
 
  find out if the user wants to continue looping and put result in a variable.
end loop

It appears you might want to add to the first question " or 'X' to exit"

Then preceed the question with

char firstChar = "";
while ((firstChar != 'X') && ((firstChar != 'x'))) {

Remove the exiting declaration of firstChar since it is already declared.
Change the existing while to an "if".
Add a } at the end