doループは、身体が最初に正常に動作しますが、それはそれは私が代わりに名前を入力することはできません必要がありますように、それがループするとき、それは最初の二つの文を印刷し、それを入力するように直進しながら、ループは私のDOで文

Question

をスキップしながらピンと私はそれを入力するものは、残りの部分をスキップし、私は別のトランザクションが欲しい場合は私に尋ねる。

私は部分的に提出された配列オブジェクトを持っています。オブジェクト内の変数は、名前、ピン、口座番号、残高です。新しいオブジェクトを追加して残高を設定すると、新しい残高が入力されるため、前のオブジェクトの残高も同様に変更されます。私はそれが静的として宣言されてバランス変数とは何かがあると思うが、私はそれを静的にしないでください、私は "静的メソッドを静的メソッドをCustomerRecord型からwithdraw（double）にすることはできません"。 （解決済み）ありがとう。

public class BankCustomers 
{ 
public static void main(String[] args) 

    //------------------------------------------------ 
    //Part4: Find a customer record from anotherArray 
    //to do transaction(s) and update the record's balance 

    char repeat; // User control to repeat or quit 
    Scanner keyboard = new Scanner(System.in); //creating the scanner 

    String aName; 
    int aPin; 
    double aWithdraw; 
    double aDeposit; 

    do{ 


     //Read customer information before search 
     System.out.println(); 
     System.out.println("Lets make a transaction"); 
     System.out.println("Enter customer full name"); 
     aName = keyboard.nextLine(); 

     System.out.println("Enter Pin"); 
     aPin = keyboard.nextInt(); 


     //Search an Array for equal aName and aPin 
     for (int i = 0; i < index; i++) 
     { 
      CustomerRecord cRecord = anotherArray[i]; 
       if((cRecord.getName().equalsIgnoreCase(aName)) && (cRecord.getPin()   ==(aPin))) 
       { 
        System.out.println(cRecord); 

        System.out.println("Enter Withdraw Amount"); 
        aWithdraw = keyboard.nextDouble(); 
        CustomerRecord.withdraw(aWithdraw); 

        System.out.println("Enter Deposite Amount"); 
        aDeposit = keyboard.nextDouble(); 
        CustomerRecord.deposit(aDeposit); 
        System.out.println(cRecord);    
       }        
     } 


     System.out.println("\nAnother Transaction? (y for yes)"); 
     repeat = keyboard.next().charAt(0); 

    }while(repeat == 'y' || repeat == 'Y'); 

     //Print the records on screen 
    for (int i = 0; i < index; i++) 
     System.out.print(anotherArray[i]); 

    }       

Answer 1

あなたの問題は二つある：

まず、balance istatic、それはクラスに関連付けられていることを意味する、そのインスタンス（オブジェクト）のいずれか、または他の用語では、すべての間で共有されていませんそのクラスのインスタンス

第2に、あなたが指摘したように、静的メソッドwithdrawはbalanceにアクセスしています。静的に関する考慮事項も同様に適用されます。メソッドはクラスに関連付けられており、非静的なメンバーにはアクセスできません。第二の問題を解決withdraw宣言から静的を削除（および同様balanceから静的を削除するには

。withdrawが今クラス自体が、それのインスタンスに関連付けられていないbecuase Thyisは、あなたのコードCustomerRecord.withdraw()がコンパイルされないようになります。だから、将来の投稿のためdeposit

Answer 2

あなたがこれらを呼び出すしようとしているとして、あなたがそれらを静的になりたくない

private static double balance 

と

public static void deposit(double aDeposit) 

と

public static void withdraw(double aWithdraw) 

から静的を削除します作成したオブジェクトから直接メソッドを取得し、したがって、それらはそれぞれCustomerRecordに固有のものになります。あなたのコード内で次に

、これらの行に変更：によって

CustomerRecord.deposit(aDeposit); 
CustomerRecord.withdraw(aDeposit); 

を：今、行った変更は、各CustomerRecordbalance変数にしていない単一balanceの変数に適用されます

cRecord.deposit(aDeposit); 
cRecord.withdraw(aDeposit); 

（ユニーク静的だったのと同じように）。

Answer 3

public class CustomerRecord implements Serializable 
{ 
    private String name; 
    private int pin; 
    private int account; 
    private double balance; 
} 

public void deposit(double aDeposit) 
{ 
    balance = balance + aDeposit; 
} 

public void withdraw(double aWithdraw) 
{ 
    if (balance >= aWithdraw) balance = balance - aWithdraw; 
    else System.out.println("Withdraw cannot be negativeegative"); 
} 

1. あなたが述べたように、balance非静的にします。これでバグが修正されます。
2. だけでなく非静的CustomerRecord#deposit()と#withdraw()方法を確認します。 ＃1によってコンパイルエラーが修正されます。

Answer 4

あなたは問題があなたは物事が静的作られたということであることを正しいです。それらのすべてが特定の顧客レコードにではなく、同時に全ての顧客レコードに適用されるものであるためbalance、deposit、withdrawはいずれも静的であるべきではありません。

コードを静的にしないとコードが機能しないという事実は、withdrawメソッドを呼び出す方法に関係します：CustomerRecord.withdraw(aWithdraw)。あなたの注意がそれに引き寄せられたので、何が間違っているのを見ることができますか？ CustomerRecordのためのあなたのインスタンス変数で

Answer 5

あなたはbalanceは静的変数として宣言されています。

これは、いつでもあなたはそれがすべてのインスタンスに変更されることクラスの1つのインスタンスでbalanceを変更することを意味します。たとえば、預金と引き出しの方法。

これらの2つのメソッドを動作させるためにバランスをとる必要があると仮定していますが、静的宣言を3つすべて取り除くだけです。次に、あなただけの静的クラスではなく、クラスのインスタンスを使用する

CustomerRecord.withdraw(); 
CustomerRecord.deposit(); 

からすべてのコールを変更する必要があります。だから、

// Whatever values you want here, you seem to have 4 already declared so you can use those 
CustomerRecord c = new CustomerRecord("", 0, 0, 0); 

c.withdraw(); 
c.deposit();