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

回答編集履歴

2

追記

2021/04/14 06:30

投稿

jeanbiego
jeanbiego

スコア3966

answer CHANGED
@@ -5,4 +5,29 @@
5
5
  ・最後にx,y座標を出力してますが、方向転換してるうちにxとy入れ替えてるのは最後戻さなくて良いんですか。
6
6
 
7
7
  改善案
8
- 方向転換時に、いまの座標でなく、移動差分を回転させたほうがわかりやすい気がします。
8
+ 方向転換時に、いまの座標でなく、移動差分を回転させたほうがわかりやすい気がします。
9
+
10
+ # 追記
11
+ ・進行方向directionをkey, 進むベクトルをvalueとした辞書を作ると、if文で場合分けしなくても`move_dct[direction]`とかで呼び出せます。
12
+ ・進むベクトルをnumpy配列にしとくと、現在座標とそのまま足し算できます。たとえば現在座標を`pos = np.array([x,y])`とすると、移動後の座標は`pos + move_dct[direction]`
13
+
14
+ たとえばこんな感じ(F,R,L,Bはその方向の一歩の長さ)
15
+ `move_dct = {"F":np.array([0,1])*F, "R":np.array([1,0])*R, "L":np.array([-1,0])*L, "B":np.array([0,-1])*B}`
16
+
17
+ 方向転換はちょっとスマートなのが思いつきませんでしたが、たとえば下記のように関数化してやって、move_dctを更新していくとか。
18
+
19
+ ```python3
20
+ def roted_dct(direction, move_dct):
21
+ if direction == "R":
22
+ for k in move_dct:
23
+ move_dct[k][0], move_dct[k][1] = move_dct[k][1], -move_dct[k][0]
24
+ elif direction == "L":
25
+ for k in move_dct:
26
+ move_dct[k][0], move_dct[k][1] = -move_dct[k][1], move_dct[k][0]
27
+ else: # B
28
+ for k in move_dct:
29
+ move_dct[k][0], move_dct[k][1] = -move_dct[k][0], -move_dct[k][1]
30
+ return move_dct
31
+ ```
32
+
33
+ こうしてやれば、for文の中身はif文一つで、移動か方向転換かの場合分けだけになります。

1

追記

2021/04/14 06:30

投稿

jeanbiego
jeanbiego

スコア3966

answer CHANGED
@@ -2,4 +2,7 @@
2
2
  ・移動するとき(a=="m"?)にLもRもx増える方向で良いんでしょうか。
3
3
  ・方向転換するとき(a=="t"?)にFとBで移動しているのでは良いんですか?
4
4
  ・方向転換するときにxとyを入れ替えていますが、LもRも同じ操作なのは直感的におかしいかと。
5
- ・最後にx,y座標を出力してますが、方向転換してるうちにxとy入れ替えてるのは最後戻さなくて良いんですか。
5
+ ・最後にx,y座標を出力してますが、方向転換してるうちにxとy入れ替えてるのは最後戻さなくて良いんですか。
6
+
7
+ 改善案
8
+ 方向転換時に、いまの座標でなく、移動差分を回転させたほうがわかりやすい気がします。