Создание простой программы «Корзина»

Question

Я пытаюсь создать программу корзины покупок, используя класс ArrayList. Моя проблема в том, что я не могу понять, как выйти из цикла while, когда пользователь совершает покупки. Элементы вводятся как цены. Ниже мой код до сих пор:

import java.util.Scanner; 
import java.util.ArrayList; 
** 
* 
*/ 
public static void shoppingCart() 
{ 
    Scanner inputReader = new Scanner(System.in); 
    Scanner itemReader = new Scanner (System.in); 
    System.out.print("Would you like to input items? (y/n)"); 
    String input = inputReader.next(); 
    ArrayList<Double> items = new ArrayList<Double>(); 
    while (!input.equals("y") && !input.equals("n")) 
    { 
     System.out.print("Sorry, we need a (y/n): "); 
     input = inputReader.next(); 
    } 

    while (input.equals("y")) 
    { 
     while (!items.equals("-1")) 
     { 
      System.out.print("Please enter an item price, or -1 to exit: $"); 
      items.add(itemReader.nextDouble()); 
     } 
    } 
} 

Answer 1

Вам просто нужно, чтобы получить дополнительный вход со стороны пользователя:

while (input.equals("y")) 
{ 
    while (!items.equals("-1")) 
    { 
     System.out.print("Please enter an item price, or -1 to exit: $"); 
     items.add(itemReader.nextDouble()); 
    } 

    // get the user's input here and set the input variable with it. 

} 

Я хотел бы использовать только один сканер, привязанный к System.in. Я не вижу причин использовать два.

Answer 2

Прямо сейчас у вас есть путь больше циклов, чем вам действительно нужно. попробуйте что-то вроде этого. , когда вы запускаете такую ​​программу, когда пользователь должен ввести значение для выхода из программы, это то, на чем должен основываться ваш цикл while, таким образом он будет оставаться в цикле до тех пор, пока не будет предложена команда закрытия.

while(!input.equals("-1")){ 
    //do your stuff here.. 
    if(!input.equals("y") || !input.equals("n")){ 
     //Try again... 
     //input check 
    }else{ 
     //do your stuff here... 
     //input check 
    } 
} 
if(input.equals("-1"){ 
    //Exit... 
} 

Answer 3

while (input.equals("y")) 
{ 
    while (!items.get(items.size()-1).equals("-1")) 
    { 
     System.out.print("Please enter an item price, or -1 to exit: $"); 
     items.add(itemReader.nextDouble()); 
    } 
} 

в

if (input.equals("y")) 
{ 
    while (!items.equals("-1")) 
    { 
     System.out.print("Please enter an item price, or -1 to exit: $"); 
     items.add(itemReader.nextDouble()); 
    } 
} 

Если это сделать.

У вас есть проверка «y» только для проверки того, следует ли вам вводить петли и принимать входные данные. Но когда вы заканчиваете ввод входных данных, вы не хотите зацикливаться и запускать это снова. Оператор if - это все, что вам нужно, с циклом внутри него.

Answer 4

У вас есть несколько проблем в этой части кода:

while (input.equals("y")) //input never changes in the loop so the loop will never end 
    { 
     while (!items.equals("-1")) //you're comparing ArrayList to a string. You need to 
     //compare list item (such as items.get(items.length()-1) to double (i.e. price) 
     { 
      System.out.print("Please enter an item price, or -1 to exit: $"); 
      items.add(itemReader.nextDouble());// there is no way to check what user 
      //entered because you add it to collection right away 
     } 
    } 

То, что вы действительно хотите что-то вроде этого:

if (input.eqals("y")) { 
    while (true) { 
    System.out.print("Please enter an item price, or -1 to exit: $"); 
    double price = itemReader.nextDouble(); 
    if (price < 0) { 
    break; 
    } 
    items.add(price); 
    } 
}