クラスメソッドについて

質問

Kakeibo クラスにあるsyojikin（初期値0）をprivate フィールドとし、その代わりにクラスメソッドshowSyoji（戻り値、引数無し）を用意して、下記のプログラムを作成したのですがクラスメソッドのshowSyojiがうまく回りません。これはもしかしてクラス変数も必要なのですか（privateもつけて）？また、このプログラムはどのようなプログラムになるのですか？

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

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

public Kakeibo(int y, int m, int d, int sy, int sh)
{
this();
if(sy>=0 && sh>=0){
year = y;
month = m;
date = d;
syunyu = sy;
shisyutsu = sh;
}
else{
System.out.println("入力した金額は無効です。");

}
}

public static void showSyoji()
{
System.out.println("所持金："+syojikin+"円");
}

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

class KＮＯ５
{
public static void main(String args[])
{
Kakeibo.showSyoji();

//Kakeibo kakeibo1 = new Kakeibo();
//kakeibo1.showData();

Kakeibo kakeibo2 = new Kakeibo(2010, 12, 1, 3000, 500);
kakeibo2.showData();

Kakeibo kakeibo3 = new Kakeibo(2010, 12, 1, -5000, 300);
kakeibo3.showData();

Kakeibo kakeibo4 = new Kakeibo(2010, 11, 7, 0, 1000);
kakeibo4.showData();

Kakeibo kakeibo5 = new Kakeibo(2010, 11, 18, 100, 800);
kakeibo5.showData();

Kakeibo.showSyoji();
}
}

回答No.3

yamada_g · ベストアンサー

class Kakeibo {
private int year;
private int month;
private int date;
private int syunyu;
private int shisyutsu;
// private int syojikin;
private static int syojikin = 0;

private Kakeibo() {
year = 0;
month = 0;
date = 0;
syunyu = 0;
shisyutsu = 0;
// syojikin = 0;
}

public Kakeibo(int y, int m, int d, int sy, int sh) {
this();
if (sy >= 0 && sh >= 0) {
year = y;
month = m;
date = d;
syunyu = sy;
shisyutsu = sh;
syojikin = syojikin + sy - sh; //add
} else {
System.out.println("入力した金額は無効です。");

}
}

public static void showSyoji() {
System.out.println("所持金：" + syojikin + "円");
}

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

class KＮＯ５ {
public static void main(String args[]) {
Kakeibo.showSyoji();

// Kakeibo kakeibo1 = new Kakeibo();
// kakeibo1.showData();

Kakeibo kakeibo2 = new Kakeibo(2010, 12, 1, 3000, 500);
kakeibo2.showData();

Kakeibo kakeibo3 = new Kakeibo(2010, 12, 1, -5000, 300);
kakeibo3.showData();

Kakeibo kakeibo4 = new Kakeibo(2010, 11, 7, 0, 1000);
kakeibo4.showData();

Kakeibo kakeibo5 = new Kakeibo(2010, 11, 18, 100, 800);
kakeibo5.showData();

Kakeibo.showSyoji();
}
}

回答No.2

Tacosan

「クラスメソッドのshowSyojiがうまく回りません」とはどういう意味でしょうか?
また, 「これはもしかして」以降はあなたが何をどうしたいのかによるので, これでは答えることは不可能です.

回答No.1

yamada_g

複数インスタンスでひとつの値を使いたいならクラス変数（static）にしないとダメです。
mainメソッドでインスタンス化するたびに所持金は0になりますよ。
っていうかkakeibo2のsyojikinと、kakeibo3のsyojikinは全く別物です。

こういう課題なのでしょうが、設計がおかしいんじゃないかと思います。

なんの返事もないし、ちゃんと読んでるのかもわからんのに真面目に回答している自分が不思議だ・・w