ブール値を格納する

Question

与えられたクラスのブール値を変更する方法が問題になっています。再び発生すると最後に値が設定されます。

public void purchase(Sandwich s) { 
    boolean owned = s.owned; 

    //I tried also with accessor and mutator here but then changed to public 
    String type = s.getType(); 
    if (owned == false) { 
     if (money <= 0){ 
      System.out.println("Worker " + this.name + " can not buy " + type + " sandwich, cuz he doesn't have enough money"); 
     } else { 
      System.out.println("Worker " + this.name + " can buy " + type + " sandwich"); 
      this.money = money; 
      owned = true; 

      //this is the place where it is supposed to change value to true (sandwich was bought and has owner now 
      s.owned = owned; 
     } 
    } else if (owned == true) { 
     System.out.println("Worker " + this.name + " can not buy " + type + " sandwich cuz it was bought"); 
     System.out.println("Test"); 
    } 
} 

問題は、与えられたサンドイッチが過去に購入したものの、その所有値は毎回falseに設定されていることである：これは私のクラス

public class Sandwich { 
    private String type; 
    private double price; 
    private String ing; 
    public boolean owned; 

    Sandwich (String t, double p, boolean o){ 
     type = t; 
     price = p; 
     owned = o; 
    } 

    public boolean getO(){ 
     return this.owned; 
    } 

    public void setO(boolean o){ 
     this.owned = o; 
    } 

    public String getType(){ 
     return this.type; 
    } 
} 

、それがアクセスして変更することになっている場所であります私はこのコードを実行しようとします。私はサンドイッチのために所有の変更された値を記録する必要がありますので、次回には所有する状態を実行する== true。方法

Answer 1

に置き換えることができます。ワーカーとサンドイッチタイプの間に関係を作成する必要があります。

あなたができることは、購入したサンドイッチタイプのリストをワーカークラスに実装し、ワーカーがサンドイッチを購入するたびにそれを比較することです。

または、必要に応じて、タイプが既に購入されているかどうかを示すブール値を持つすべてのサンドイッチタイプのハッシュマップを持つことができます。

Answer 2

getルーチンとsetルーチンを作成し、使用しませんでした。私はこのコードを変更します。

public void purchase(Sandwich s){ 
      String type = s.getType(); 
      if (!(s.getO())){ 
       if (money <= 0){ 
        System.out.println("Worker " + this.name + " can not buy " + type + " sandwich, cuz he doesn't have eno 

ugh money"); 
       } else { 
        System.out.println("Worker " + this.name + " can buy " + type + " sandwich"); 
        this.money = money; 
        s.setO(true); 
       } 
      } else { 
       System.out.println("Worker " + this.name + " can not buy " + type + " sandwich cuz it was bought"); 
       System.out.println("Test"); 
      } 
    } 

Answer 3

だけ

boolean owned = s.owned; 

削除して、あなたが使用しているs.getO()を使用owned

例えば

if (owned == false){ 

は

if (!s.getO()){ 

ことそして、それを変更するためにsetterメソッドs.setO(true/false)を使用することができます。

owned = true; 
s.owned = owned; 

は、デザインに欠陥があるように思わ

s.setO(true);