while loop in Java code for netbeans

Question

import java.util.ArrayList; 
import javax.swing.JOptionPane; 

public class MainApp { 

    public static void main(String[] args) { 

     ArrayList<Product> cart = new ArrayList<Product>(); 

     String s1 = JOptionPane 
       .showInputDialog("Please enter the quantity of the product you have selected"); 

     int r = Integer.parseInt(s1); 

     String s2 = JOptionPane 
       .showInputDialog("Please enter the name of the product"); 

     String s3 = JOptionPane 
       .showInputDialog("Please enter the price of the product"); 
     double d1 = Double.parseDouble(s3); 

     for (int i = 0; i < r; i++) { 

      Product p1 = new Product(); 
      p1.setName(s2); 
      p1.setPrice(d1); 

      cart.add(p1); 

     } 

     double total_price = 0; 
     for (int i = 0; i < cart.size(); i++) { 
      total_price = total_price + cart.get(i).getPrice(); 
     } 

     JOptionPane.showMessageDialog(null, cart.toString() 
       + "The total price of all the products in the cart is : " 
       + total_price + " KD", "Shopping Information", 
       JOptionPane.INFORMATION_MESSAGE); 

     cart.removeAll(cart); 

     JOptionPane.showMessageDialog(null, "Thank you for shopping with us! ", 
       "Prompt", JOptionPane.INFORMATION_MESSAGE); 

     String s4 = JOptionPane 
       .showInputDialog("Would you like to add items to your cart"); 

     while (s4 != "stop") { 

      String s5 = JOptionPane 
        .showInputDialog("Please enter the name of the product"); 

      String s6 = JOptionPane 
        .showInputDialog("Please enter the price of the product"); 
      double d2 = Double.parseDouble(s6); 

      Product p2 = new Product(); 

      p2.setName(s5); 
      p2.setPrice(d2); 

      cart.add(p2); 

      String s7 = JOptionPane 
        .showInputDialog("Would you like to add more items?"); 
      if (s7 == "No" || s7 == "no") 
       s4 = "stop"; 

     } 

     JOptionPane.showMessageDialog(null, cart.toString(), "Cart Details", 
       JOptionPane.INFORMATION_MESSAGE); 

    } 

} 

Все работает нормально до цикла while ... как-то я не могу заставить его работать на меня. Когда я вхожу «No» или «no» в конец, цикл продолжает работать и не останавливается, пока я не введу пространство для вопросов с строкой. Она просто дает мне следующую ошибку в конце:

Exception in thread "main" java.lang.NullPointerException 
    at sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:1008) 
    at java.lang.Double.parseDouble(Double.java:540) 
    at mainapp.MainApp.main(MainApp.java:66) 
Java Result: 1 
BUILD SUCCESSFUL (total time: 41 seconds) 

Answer 1

Почему бы не использовать метод String equals вместо == ?. Для работы == вам нужно будет вызвать метод intern() на s4, который изменит ссылку на точку на пул строк.

в то время как (! S4.equals ("стоп"))

Answer 2

Try:

while(!s4.equals("stop")){ 

Строка является объектом, а не примитивный тип, так что вы должны использовать .equals вместо ==.

Answer 3

Используйте метод equals. Строки не следует сравнивать с оператором == или! =.

Answer 4

while (!"stop".equalsIgnoreCase(s4)) 

Попробуйте это.

Пожалуйста, обратитесь это SO нить для большего понимания : Java String.equals versus ==

Answer 5

Я верю следующую строку производит ошибку исключения:

double d2=Double.parseDouble(s6); 

Чтобы уточнить это, скажите нам, что линия является линией 66 на вас основной метод

Это происходит потому, что значение s6 составляет null. постарайтесь убедиться, что это отсортировано первым.

Также, как и все остальные, используйте метод equals для проверки String равенства. < - (это не приведет к ошибке Exception, но делает вашу программу неправильной семантической).

Answer 6

Огромное спасибо всем, что рассказал мне о методе .equals. Я забыл про это. Я представил свою работу сегодня, подумав и все было работать, прежде чем я сделал это ...

package mainapp; 

import java.util.ArrayList; 
import javax.swing.JOptionPane; 


public class MainApp { 

    public static void main(String[] args) { 
ArrayList <Product> cart=new ArrayList <Product>(); 

    String s1=JOptionPane.showInputDialog("Please enter the quantity of the product you have selected"); 
    int r=Integer.parseInt(s1); 

    String s2=JOptionPane.showInputDialog("Please enter the name of the product"); 

    String s3=JOptionPane.showInputDialog("Please enter the price of the product"); 
    double d1=Double.parseDouble(s3); 

for(int i=0; i<r ;i++){ 

Product p1=new Product(); 
p1.setName(s2); 
p1.setPrice(d1); 

cart.add(p1); 

} 

double total_price=0; 
for(int i=0; i<cart.size();i++){ 
    total_price=total_price+cart.get(i).getPrice(); 
} 

JOptionPane.showMessageDialog(null, 
     cart.toString()+"The total price of all the products in the cart is : "+total_price +" KD" 
     ,"Shopping Information" 
     ,JOptionPane.INFORMATION_MESSAGE); 

cart.removeAll(cart); 

JOptionPane.showMessageDialog(null, 
     "Thank you for shopping with us! ", 
     "Prompt", 
     JOptionPane.INFORMATION_MESSAGE); 



int count=0; 
while(count>=0){ 

String s5=JOptionPane.showInputDialog("Please enter the name of the product"); 

String s6=JOptionPane.showInputDialog("Please enter the price of the product"); 

double d2=Double.parseDouble(s6); 


Product p2=new Product(); 

p2.setName(s5); 
p2.setPrice(d2); 

cart.add(p2); 
count++; 

String s8=JOptionPane.showInputDialog("Would you like to add more items to your cart?"); 
if(s8.contains("Yes")|| s8.contains("yes")){ 

} 

else{  
String s7=JOptionPane.showInputDialog("Please enter 'Stop' if you are done adding items to your cart"); 
if (s7.contains("stop")|| s7.contains("Stop")){ 
    count=-1; 
} 

} 
} 
JOptionPane.showMessageDialog(null, 
     cart.toString(), 
     "Cart Details", 
     JOptionPane.INFORMATION_MESSAGE); 

    } 
} 

Answer 7

Я попытался с помощью() метод .equals, он не работает для меня. Я думаю, что я не писал цикл, используя правильный формат; с «счетом» и всеми. «За» тоже стало катастрофой. Поэтому я написал это по-другому так, что это имело смысл для меня, и это закончилось тем, что работало ... спасибо всем тем, кто ответил!