回答編集履歴
1
コメントに対して回答の追加
answer
CHANGED
|
@@ -46,4 +46,85 @@
|
|
|
46
46
|
「totalHolder」はstatic なHashMap<String, Profit>なはずです。
|
|
47
47
|
> totalHolder.get(itemName).salesAmount[i]; //合計額
|
|
48
48
|
|
|
49
|
-
↑こんなんしてもコンパイルエラーになりますよ？
|
|
49
|
+
↑こんなんしてもコンパイルエラーになりますよ？
|
|
50
|
+
|
|
51
|
+
■以下追加：
|
|
52
|
+
|
|
53
|
+
```
|
|
54
|
+
for(Entry<String, Profit> entry : totalHolder.entrySet()) {
|
|
55
|
+
//System.out.println(entry.getKey() + " ," + entry.getValue().count + " ," + entry.getValue().salesAmount);
|
|
56
|
+
arr = totalHolder.values().toArray(new Profit[i]);
|
|
57
|
+
//System.out.println(arr[i]);
|
|
58
|
+
i += 1;
|
|
59
|
+
}
|
|
60
|
+
```
|
|
61
|
+
まずここ。
|
|
62
|
+
**無意味なループ**です。
|
|
63
|
+
ここでやりたいことは
|
|
64
|
+
|
|
65
|
+
**・「Profit[] arr = null;」で宣言したProfitインスタンスの配列にHashMap<String, Profit>の中身を詰め込む。**
|
|
66
|
+
|
|
67
|
+
ことですね。
|
|
68
|
+
|
|
69
|
+
```
|
|
70
|
+
arr = totalHolder.values().toArray(new Profit[]);
|
|
71
|
+
|
|
72
|
+
```
|
|
73
|
+
この1行で完了します。
|
|
74
|
+
|
|
75
|
+
また、
|
|
76
|
+
```
|
|
77
|
+
/**
|
|
78
|
+
* 売上金額を大きい順に並べ替えるメソッド
|
|
79
|
+
*/
|
|
80
|
+
sortAmount(arr);
|
|
81
|
+
```
|
|
82
|
+
ここではメソッドにProfit配列を渡して処理してもらってますね？
|
|
83
|
+
|
|
84
|
+
これと同じように、出力するメソッドも
|
|
85
|
+
```
|
|
86
|
+
/**
|
|
87
|
+
* 売上金額の上位３位を出力するメソッド
|
|
88
|
+
*/
|
|
89
|
+
public static void outputBest3(Profit[] arr){
|
|
90
|
+
|
|
91
|
+
```
|
|
92
|
+
のように定義してやって、このメソッドにソートが終わった配列を渡してあげれば完了ではありませんか？
|
|
93
|
+
```
|
|
94
|
+
// outputBest3の実行
|
|
95
|
+
outputBest3(arr);
|
|
96
|
+
|
|
97
|
+
```
|
|
98
|
+
|
|
99
|
+
まとめ：
|
|
100
|
+
こんな書き方になるはずです。
|
|
101
|
+
```
|
|
102
|
+
if(sold > 0){
|
|
103
|
+
// 1.新しい月の売上が出てきたので、今までの分を出力
|
|
104
|
+
|
|
105
|
+
// 2.処理対象の月のヘッダを出力する
|
|
106
|
+
bwA.write(date.substring(0, 6) + "月売上高BEST３");
|
|
107
|
+
bwA.newLine();
|
|
108
|
+
// 3.Profitインスタンスの配列にHashMap<String, Profit>の中身を詰め込む
|
|
109
|
+
arr = totalHolder.values().toArray(new Profit[]);
|
|
110
|
+
// 4.上で取得したProfit配列を売上金額が大きい順に並べ替える
|
|
111
|
+
sortAmount(arr);
|
|
112
|
+
// 5.売上金額の上位３位を出力する
|
|
113
|
+
outputBest3(arr);
|
|
114
|
+
|
|
115
|
+
date = itemA[0]; //売上日を再セット
|
|
116
|
+
}
|
|
117
|
+
|
|
118
|
+
```
|
|
119
|
+
|
|
120
|
+
**「何を」「どうする」**というのがしっかり切り分けられないとプログラムになりません。(この場合の主語は「プログラムが」)
|
|
121
|
+
このことが明確になってないとプログラムはおかしくなります。
|
|
122
|
+
たとえば質問のソースで
|
|
123
|
+
```
|
|
124
|
+
// 商品マスタ一覧を入力するメソッド
|
|
125
|
+
ProductMasterList.productList();
|
|
126
|
+
```
|
|
127
|
+
こんなのがありましたが、この1行でやってることはなんですか？
|
|
128
|
+
**「何を」「どうする」**にあてはめると、**「商品マスタ一覧を」「取得する」**ではありませんか？
|
|
129
|
+
入力じゃないですよね？
|
|
130
|
+
ささいなことかもしれませんが、コメントに書いていることと実際の処理内容が食い違っていれば勘違いや混乱のもとになります。
|