0
これは完了コード、私は任意の修正を加える必要があるかどうか迷っは
これは、これは私の第二のクラスで私の最初のクラス
public class Account
{
private double balance; //STATE
private double interestRate; //STATE
private double rate;//STATE
public Account()
{
balance = 0;
interestRate = 0;
}
public Account(double amount, double interestRate)
{
balance = amount;
rate = interestRate;
}
public void deposit(double amount)
{
balance=balance+amount;
}
public void withdraw(double amount)
{
balance = balance - amount;
}
public void setInterest(double rate)
{
balance = balance + balance * rate;
//this.setInterst = setInterest;
//setInterest = InterestRate/12;
}
public double computeInterest(int n)
{
balance=Math.pow(balance*(1+rate),n/12);
return balance;
}
public double getsetInterest()
{
return rate;
}
public double getBalance()
{
return balance;
}
public void close()
{
balance =0;
}
}
である私はそれが修正を必要とするかどうかを疑問に思った私の完了コードがされ
public class TestAccountInterest
{
public static void main (String[] args)
{
Account acc1 = new Account(500, 0.1);//0.10);
Account acc2 = new Account(400, 0.2); //0.20);
/*************************************
ACC1 ACCOUNT BELOW
*************************************/
acc1.deposit(500);
acc1.withdraw(300);
acc1.computeInterest(12);
System.out.println(acc1.computeInterest(12));
/**************************************
ACC2 ACCOUNT BELOW
**************************************/
acc2.withdraw(200);
acc2.deposit(800);
acc2.computeInterest(24);
System.out.println(acc2.computeInterest(24));
}
}
私はコードをよりコンパクトにすることができるかどうか、誰かが私が完全に有効にコーディングした方法であるかどうか確認できます。このコードは、2番目のクラスのテストアカウントクラスを持つAccountsクラスについてのもので、最初のクラスの計算期間を12ヶ月とし、2番目のクラスの計算期間を約24ヶ月とします。 setIntrest()が興味を設定し、代わりにバランスを変更していない理由を私は理解していない以外
コードが正しく機能し改善するだけの場合は、[コードレビュー]（https://codereview.stackexchange.com/）のWebサイトでより適切な場合があります。 – AntonH
私が最初の12ヶ月間それを稼働させると問題はあるようですが、関心は普通に見えますが、24ヶ月間は非常に長い巻数になります。 – Omor
それは改善ではない、それは修正です。タイトルと質問を編集して、ヘルプを受け取るための詳細情報を提供してください。 – AntonH