teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

5

間違ってる気がしたので削除

2015/08/23 14:24

投稿

swordone
swordone

スコア20675

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

並べ替え

2015/08/23 14:24

投稿

swordone
swordone

スコア20675

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

質問の回答となる原因をようやく発見

2015/08/23 14:16

投稿

swordone
swordone

スコア20675

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

2015/08/22 03:10

投稿

swordone
swordone

スコア20675

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

気になった点追加

2015/08/22 02:11

投稿

swordone
swordone

スコア20675

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が入るため,「入れ替えが起きたのにループを抜ける」という事態になる