JavaでShoe Inventoryプログラムを実装する際に問題が発生する

Question

私は靴用のインベントリプログラムを実装しようとしています。私がしようとしているのは、各靴にid（productId）と靴の在庫量（ammountToPick）を与えることです。私は、プログラムを実行しているユーザーに靴IDを入力して、在庫に残っている靴の種類（ammountToPick）を取得するようにしたいと考えています。私のプログラムの私の現在の問題は何も返されておらず、単に「無効な製品ID」を印刷し続けているということです。私は以下の私のコードを提供してきた

：最初の行で

public class Product { 
    private String productId = ""; 
    private int ammountToPick = 0; 
    private int ammountToRestock = 0; 

    public Product (String productId, int ammountToPick){ 

    this.productId = productId; 
    this.ammountToPick = ammountToPick; 

    } 

    public String getProductId() { 
    return productId; 
    } 

    public int getAmmountToPick() { 
    return ammountToPick; 
    } 

} 

public class Shoes extends Product{ 
    public Shoes(String productId, int ammountToPick){ 
    super(productId, ammountToPick); 

    } 


} 

import java.util.Scanner; 
public class Inventory 
{ 
    private static String productId = ""; 
    private int ammountToPick = 0; 
    private int ammountToRestock = 0; 
    public static final int MAX_ITEMS = 999999; 
    private static Product product [] = new Shoes[MAX_ITEMS]; 

    public static void main (String args[]){ 

    buildInventory(); 
    getInventory(); 
} 

public static void buildInventory(){ 

product[1] = new Shoes("shoe101", 19); 
product[2] = new Shoes("shoe200", 1); 
product[3] = new Shoes("shoe420", 9); 
} 

public static void getInventory() { 
    Scanner input = new Scanner(System.in); 
    System.out.println("Enter the product id of the product you would like to pick: "); 
    String userinput = input.nextLine(); 
    if(userinput.equals(productId)) { 
    System.out.println("there are" + product[1].getAmmountToPick() +"left"); 
    } 
    else { 

    System.out.println("invalid product id "); 
    } 
} 
} 

Answer 1

if(userinput.equals(productId)) { 
    System.out.println("there are" + product[1].getAmmountToPick() +"left"); 
    } 
else { 

、あなたの問題は、商品コードは、クラスの先頭に空の文字列に設定されていることです。実際のproductIdを入力せずにEnterキーを押すだけで、実際に動作するはずです。ユーザー入力は ""です。ユーザ入力は、あなたの配列に

の項目のいずれかの商品コードに一致した場合でも、この作品にあなたが望む方法を作るためにあなたのProductID変数を取り除くとチェックあなたはProductどちら

userinput = ...; //this part is fine 
for (Product p : products) { 
    if (userinput.equals(p.getProductId())) { 
    System.out.println('there are ' + p.getAmmountToPick() + " left"); 
    } 
} 

Answer 2

を行う必要があるとInventoryクラスにはメンバ変数productIdがあります。 userInputをクラスのprivate staticproductIdと比較しています。これは空の文字列です。

ProductのproductIdに一致するものを探して在庫配列を繰り返す必要があると思います。ここで

が強化ループなし新しい方法でそれを行うための一つの方法です：

boolean found = false; 
for (final Product aProduct: product) { 
    if(userInput.equals(aProduct.getProductId())) { 
     System.out.println("there are" + aProduct.getAmmountToPick() +"left"); 
     found = true; 
     break; 
    } 
} 
if(!found) { 
    System.out.println("invalid product id "); 
} 

私はその意図が明確であるproductsにproductメンバ変数の名前を変更したいです。それはコードをより明確にするでしょう。

Inventory、それを使用している方法では、productIdは必要ありません。削除する必要があります。