まったくわかりません

質問

Kakeibo クラスに対し、さらに日付、収入、支出に最初から値を入力するコンストラクタ（引
数：year, month, date, syunyu, shisyutsu の順）をオーバーロードするプログラムで、下記のプログラムを「収入及び支出が0 以上か判断し、満たしていなければエラーメッセージを出力し、日付、収入、支出に値を入力しないようにする。満たしていれば引数の値を代入する。」と「main クラス内でKakeibo クラスの変数kakeibo2 を追加し、課題2 で追加したコンストラクタにより日付と収入、支出を入力し、その内容をテスト１で作成したshowData を用いて表示する。さらにKakeibo クラスの変数kakeibo3 を作成し、無効な日付を入力し、showData を用いて表示する。」ということを入れて作成したのですが、
　入力した金額は無効です。　　　　　　　　入力した金額は無効です
　0 年0 月0 日 ： 収入0 円, 支出0 円　→。2010 年12 月1 日 ： 収入3000円, 支出500円
になります。なぜこのようなことが起こるのですか？

class Kakeibo
{
private int year;
private int month;
private int date;
private int syunyu;
private int shisyutsu;

public Kakeibo()
{
year = 0;
month = 0;
date = 0;
syunyu = 0;
shisyutsu = 0;
}
public Kakeibo(int y, int m, int d, int sy, int sh)
{
this();
if(syunyu>=0 && shisyutsu>=0)
year = y;
month = m;
date = d;
syunyu = sy;
shisyutsu = sh;
　　System.out.print(year+"年"+month+"月"+date+"日:収入"+syunyu+"円、支
　　　　　　　　　　　　　　　　　　出"+shisyutsu+"円\n");
System.out.println("入力した金額は無効です。");
}
public void showDate()
{
System.out.print(year+"年"+month+"月"+date+"日:収入"+syunyu+"円、支
　　　　　　　　　　　　　　　　　　出"+shisyutsu+"円\n");
}
}

class K
{
public static void main(String args[])
{
Kakeibo kakeibo2 = new Kakeibo();
kakeibo2.showDate();

Kakeibo kakeibo3 = new Kakeibo(2010, 12, 1, 3000, 500);
kakeibo3.showDate();
}
}

回答No.2

yamada_g · ベストアンサー

class Kakeibo {
private int year;
private int month;
private int date;
private int syunyu;
private int shisyutsu;

public Kakeibo() {
year = 0;
month = 0;
date = 0;
syunyu = 0;
shisyutsu = 0;
}

public Kakeibo(int y, int m, int d, int sy, int sh) {
this();
//if (syunyu >= 0 && shisyutsu >= 0)
//チェックするのはコンストラクタの引数なのでは・・
//あと { がないと if文の直後の1文しか実行されませんよ・・ この場合なら year = y; だけ
if (sy >= 0 && sh >= 0) {
year = y;
month = m;
date = d;
syunyu = sy;
shisyutsu = sh;
} else { //無条件でエラーメッセージを出力してましたけど・・
//System.out.print(year + "年" + month + "月" + date + "日:収入" + syunyu
// + "円、支出" + shisyutsu + "円\n");
System.out.println("入力した金額は無効です。");
}
}

public void showDate() {
System.out.print(year + "年" + month + "月" + date + "日:収入" + syunyu
+ "円、支出" + shisyutsu + "円\n");
}
}

public class A {
public static void main(String args[]) {
//Kakeibo kakeibo2 = new Kakeibo();
//課題2 で追加したコンストラクタにより って書いてありますけど・・
Kakeibo kakeibo2 = new Kakeibo(2010, 12, 1, 3000, 500);
kakeibo2.showDate();

//Kakeibo kakeibo3 = new Kakeibo(2010, 12, 1, 3000, 500);
// 無効な金額じゃないのかって気がしますけど・・
Kakeibo kakeibo3 = new Kakeibo(2010, 12, 41, 3000, 500);
kakeibo3.showDate();
//なので金額も
kakeibo3 = new Kakeibo(2010, 12, 41, 3000, -1);
kakeibo3.showDate();
}
}

回答No.1

ketuago65

ところどころ文章があれでわかりづらいですがこういうことですかね？

if(syunyu>=0 && shisyutsu>=0) { //ここの括弧
year = y;
month = m;
date = d;
syunyu = sy;
shisyutsu = sh;
　　System.out.print(year+"年"+month+"月"+date+"日:収入"+syunyu+"円、支
　　　　　　　　　　　　　　　　　　出"+shisyutsu+"円\n");
} //　ここの括弧

System.out.println("入力した金額は無効です。");

上にしるしをつけた部分が足りません。
ifに括弧がついてないのでif文以降の１行目にしか条件がつきません。
つまり条件に関係なくエラーメッセージが出るようになっています。

あと、this()で上のコンストラクタを呼んでしまっているのでどんな値を渡しても
全部0になってしまうと思うのですが・・・

if文の条件もshishutsuとsyunyuではなくshとsyじゃないと意味がないです。