回答編集履歴

2

タイミングについて修正

2019/12/02 00:33

投稿

kakajika
kakajika

スコア3131

test CHANGED
@@ -16,8 +16,8 @@
16
16
 
17
17
  ```kotlin
18
18
 
19
+ mUndoPaint.add(mPaint)
20
+
19
21
  mPaint = Paint(mPaint)
20
22
 
21
- mUndoPaint.add(mPaint)
22
-
23
23
  ```

1

タイミングについて修正

2019/12/02 00:33

投稿

kakajika
kakajika

スコア3131

test CHANGED
@@ -10,7 +10,7 @@
10
10
 
11
11
 
12
12
 
13
- 現状ではmUndoPaint/mRedoPaintに追加するPaintのインスタンスがすべて同じになってしまっていませんか?undo/redoなどを想定してペン色の変更機能を実装するなら、色を変更する時点でPaintのインスタンスを別に作るべきです。
13
+ 現状ではmUndoPaint/mRedoPaintに追加するPaintのインスタンスがすべて同じになってしまっていませんか?undo/redoなどを想定してペン色の変更機能を実装するなら、mUndoPaint/mRedoPaintに追加する時点でPaintのインスタンスを別に作るべきです。
14
14
 
15
15
 
16
16
 
@@ -18,6 +18,6 @@
18
18
 
19
19
  mPaint = Paint(mPaint)
20
20
 
21
- mPaint.color = ...
21
+ mUndoPaint.add(mPaint)
22
22
 
23
23
  ```