回答編集履歴
5
間違ってる気がしたので削除
answer
CHANGED
@@ -9,33 +9,3 @@
|
|
9
9
|
- StringはCharSequenseインタフェースを実装しているので,わざわざ変換しなくともそのままsetTextの引数として渡せる
|
10
10
|
- 入れ替え作業の際,入れ替え対象に0が選ばれるとtaihiに0が入るため,「入れ替えが起きたのにループを抜ける」という事態になる
|
11
11
|
- 配列0番目以外はfor(i = 1)により再度入れ替えの対象となり,望む順番にならない.
|
12
|
-
- 入れ替えの部分が冗長すぎる.条件の数値が違うだけでやってることは全部同じなのでまとめられる.
|
13
|
-
```java
|
14
|
-
for (int j = 0; j < 6; j++) {
|
15
|
-
List<Integer> target = null;
|
16
|
-
switch(code[j]){
|
17
|
-
default:
|
18
|
-
continue;
|
19
|
-
case 0:
|
20
|
-
target = Arrays.asList(new Integer[]{2});
|
21
|
-
break;
|
22
|
-
case 3:
|
23
|
-
target = Arrays.asList(new Integer[]{5});
|
24
|
-
break;
|
25
|
-
case 4:
|
26
|
-
target = Arrays.asList(new Integer[]{6});
|
27
|
-
break;
|
28
|
-
case 6:
|
29
|
-
target = Arrays.asList(new Integer[]{3, 5});
|
30
|
-
break;
|
31
|
-
}
|
32
|
-
for (int i = j + 1; i < 6; i++) {
|
33
|
-
if(target.contains(code[i])){
|
34
|
-
taihi = code[j];
|
35
|
-
code[j] = code[i]; //前後入れ替え
|
36
|
-
code[i] = taihi;
|
37
|
-
}
|
38
|
-
}
|
39
|
-
}
|
40
|
-
|
41
|
-
```
|
4
並べ替え
answer
CHANGED
@@ -9,3 +9,33 @@
|
|
9
9
|
- StringはCharSequenseインタフェースを実装しているので,わざわざ変換しなくともそのままsetTextの引数として渡せる
|
10
10
|
- 入れ替え作業の際,入れ替え対象に0が選ばれるとtaihiに0が入るため,「入れ替えが起きたのにループを抜ける」という事態になる
|
11
11
|
- 配列0番目以外はfor(i = 1)により再度入れ替えの対象となり,望む順番にならない.
|
12
|
+
- 入れ替えの部分が冗長すぎる.条件の数値が違うだけでやってることは全部同じなのでまとめられる.
|
13
|
+
```java
|
14
|
+
for (int j = 0; j < 6; j++) {
|
15
|
+
List<Integer> target = null;
|
16
|
+
switch(code[j]){
|
17
|
+
default:
|
18
|
+
continue;
|
19
|
+
case 0:
|
20
|
+
target = Arrays.asList(new Integer[]{2});
|
21
|
+
break;
|
22
|
+
case 3:
|
23
|
+
target = Arrays.asList(new Integer[]{5});
|
24
|
+
break;
|
25
|
+
case 4:
|
26
|
+
target = Arrays.asList(new Integer[]{6});
|
27
|
+
break;
|
28
|
+
case 6:
|
29
|
+
target = Arrays.asList(new Integer[]{3, 5});
|
30
|
+
break;
|
31
|
+
}
|
32
|
+
for (int i = j + 1; i < 6; i++) {
|
33
|
+
if(target.contains(code[i])){
|
34
|
+
taihi = code[j];
|
35
|
+
code[j] = code[i]; //前後入れ替え
|
36
|
+
code[i] = taihi;
|
37
|
+
}
|
38
|
+
}
|
39
|
+
}
|
40
|
+
|
41
|
+
```
|
3
質問の回答となる原因をようやく発見
answer
CHANGED
@@ -1,3 +1,8 @@
|
|
1
|
+
MainActivityのonCreateの途中でstartActivityForResultを起動してSubActivityを起動しているわけですが,
|
2
|
+
この時点で処理が中断するわけではなく,後に続く処理は普通に続行します.
|
3
|
+
すなわち,「値が変わらない」のではなく,「値が変わる前に処理を行っている」というのが現在の状態となります.
|
4
|
+
---
|
5
|
+
|
1
6
|
質問とは関係ないですが,気になった点をいくつか
|
2
7
|
- わざわざこの乱数生成を別Activityに起こす必要はないのでは?同じActivity内で別メソッドとして入れれば十分
|
3
8
|
- SubActivityにおいて,TextViewがnullのままになっているように見える.そのままsetTextしたら例外が発生するはずでは?
|
2
追加2
answer
CHANGED
@@ -2,4 +2,5 @@
|
|
2
2
|
- わざわざこの乱数生成を別Activityに起こす必要はないのでは?同じActivity内で別メソッドとして入れれば十分
|
3
3
|
- SubActivityにおいて,TextViewがnullのままになっているように見える.そのままsetTextしたら例外が発生するはずでは?
|
4
4
|
- StringはCharSequenseインタフェースを実装しているので,わざわざ変換しなくともそのままsetTextの引数として渡せる
|
5
|
-
- 入れ替え作業の際,入れ替え対象に0が選ばれるとtaihiに0が入るため,「入れ替えが起きたのにループを抜ける」という事態になる
|
5
|
+
- 入れ替え作業の際,入れ替え対象に0が選ばれるとtaihiに0が入るため,「入れ替えが起きたのにループを抜ける」という事態になる
|
6
|
+
- 配列0番目以外はfor(i = 1)により再度入れ替えの対象となり,望む順番にならない.
|
1
気になった点追加
answer
CHANGED
@@ -1,4 +1,5 @@
|
|
1
1
|
質問とは関係ないですが,気になった点をいくつか
|
2
2
|
- わざわざこの乱数生成を別Activityに起こす必要はないのでは?同じActivity内で別メソッドとして入れれば十分
|
3
3
|
- SubActivityにおいて,TextViewがnullのままになっているように見える.そのままsetTextしたら例外が発生するはずでは?
|
4
|
-
- StringはCharSequenseインタフェースを実装しているので,わざわざ変換しなくともそのままsetTextの引数として渡せる
|
4
|
+
- StringはCharSequenseインタフェースを実装しているので,わざわざ変換しなくともそのままsetTextの引数として渡せる
|
5
|
+
- 入れ替え作業の際,入れ替え対象に0が選ばれるとtaihiに0が入るため,「入れ替えが起きたのにループを抜ける」という事態になる
|