サブクラスの変数の値を変更しますか？

Question

私は2つのクラスを持っています。 1つは約24時間の時計で、2番目の時計は1番目の時計のサブクラスで、12時間の時計です。 新しい時刻を第2クラスに変換するには、最初のクラスの値を変更する必要があります。しかし、私はそれをすることはできません。 私はあなたはそれが設定されている場合hourを検証しなければならない

class AMPMClock extends Clock { 
    private boolean pm = false; 
    public void setHour(int hour) { 
    if (hour >= 12) { 
     hour -= 12; 
     pm = true; 
    } else { 
     pm = false; 
    } 
    super.setHour(hour); 
    } 
    public void setPM(boolean pm){ 
    this.pm = pm; 
    } 
} 

を書くことができる "H" 変数 具体的に...

class Clock{ 
    public int h, m , s; 
    public String a,b,c; 

    public void setHour(int hour){ 
     this.h = hour; 
    } 
    public void setMin(int min){ 
     this.m = min; 
    } 
    public void setSec(int sec){ 
     this.s = sec; 
    } 
    public void tick(){ 
     if(h != 23){ 
      if(m == 59 && s==59){ 
       m = 0; 
       s=0; 
       h++; 
      } 
      else if(m != 59 && s == 59){ 
       m++; 
       s=0; 
      } 
      else if (m != 59 && s != 59){ 
       s++; 
      } 
      else if(m == 59 && s!=59){ 
       s++; 
      } 
     } 

     else if(h == 23 && m == 59 && s !=59){ 
      s++; 
     } 
     else if(h == 23 && m!=59 && s == 59){ 
      s=0; 
      s++; 
      m++; 
     } 
     else if(h == 23 && m!=59 && s!=59){ 
      s++; 
     } 
     else if(h == 23 && m == 59 && s == 59){ 
      s = 0; 
      m =0; 
      h = 0; 
     } 
    } 
    public String toString(){ 
     a = ""; 
     b = ""; 
     c = ""; 

     if (h < 10) 
      a = "0"; 
     if (m <10) 
      b = "0"; 
     if (s <10) 
      c = "0"; 

     return a+h+":"+b+m+":"+c+s; 
    } 
} 

    class AMPMClock extends Clock{ 
    Clock clock2 = new Clock(); 
    public void setAMPM(boolean yes){ 
     if(yes == true){ 
      **clock2.h = clock2.h - 12**; 
     } 
    } 
} 

Answer 1

あなたAMPMClockどちらかだけClockを拡張したり、それを使用する必要があります。どちらの

class AMPMClock extends Clock { 
    Clock clock2 = new Clock(); 

を拡張：あなたは次のようにそれらの両方を組み合わせるためにしようとしている。しかし

class AMPMClock extends Clock { 

をパブリックaccesorメソッドが継承されている、ので、あなたはthisにgetter/setterメソッドを呼び出すことができます。例えば​​。

か、またはそれを使用しますが、拡張しない：

class AMPMClock { 
    Clock clock2 = new Clock(); 

をここでは、代わりにthis使用clock2の。例えばclock2.getHour()。

しかし、すべてのインスタンスメンバーを非公開にする必要があります（パブリックにしないでください）。それに対応するパブリックゲッター/セッターを作成します。

dasblinkenlightによって示唆されるように：

抽象クラスを持っている方が良いでしょう：

public abstract class BaseClock { 
    private int h, m, s; 
    //getter/setter 
} 

異なる2つの具象クラス：

public class TwelveHourClock extends BaseClock { } 

と

public class TwentyFourHourClock extends BaseClock { } 

Answer 2

について話しています。フィールドはすべてプライベートにする必要があります。時間はゲッターを&秒に設定してください。 AMPMClockはクロックであるため、AMPMClockでは2つのクロックは必要ありません。 AM/PMをyesに設定すると、混乱するだけです。

Answer 3

クラスとオブジェクトの違いについて混乱しているようです。

チェックアウト：http://www.codestyle.org/java/faq-Objects.shtml#classobjectdiff

Answer 4

戦術的な答えは、「this」でclick2を変更することです。戦略的で大きな問題は、実際に継承を適切に利用していないことです。