回答編集履歴

2

指摘箇所の誤りを削除、参考URLを記載

2018/07/05 10:22

投稿

shiron46
shiron46

スコア111

test CHANGED
@@ -50,8 +50,8 @@
50
50
 
51
51
  ```
52
52
 
53
+  
53
54
 
55
+ ### 参考URL
54
56
 
55
- - **ナップザックの残容量**
56
-
57
- 合計重量やナップザックの容量が考慮できていないように見受けられます。
57
+ [pieceofnostalgy.blogspot.com/2013/12/01.html](http://pieceofnostalgy.blogspot.com/2013/12/01.html)

1

処理の中で怪しそうな箇所についての記述を追加

2018/07/05 10:22

投稿

shiron46
shiron46

スコア111

test CHANGED
@@ -19,3 +19,39 @@
19
19
  ```
20
20
 
21
21
  dpは2次元配列ですが、1次元分しかインデックスを指定していないためです。
22
+
23
+
24
+
25
+ ---
26
+
27
+ 以下は見直し方が良さそうな箇所です。
28
+
29
+ - **dpメソッド内のループ条件**
30
+
31
+ 特に下記のコードは問題があります。
32
+
33
+ -- ループに入ると無条件でreturnしてしまいます。
34
+
35
+  -- 配列dp の2次元目は大きさW(15)ですが、N(20)までループさせようとしています。
36
+
37
+
38
+
39
+ ```Java
40
+
41
+ for(int j=0;j<N;j++) {
42
+
43
+ bootlearn[j]=dp[i][j];
44
+
45
+ System.out.println();
46
+
47
+ return dp[0][0];
48
+
49
+ }
50
+
51
+ ```
52
+
53
+
54
+
55
+ - **ナップザックの残容量**
56
+
57
+ 合計重量やナップザックの容量が考慮できていないように見受けられます。