Pathにはどのようなデータが含まれているのでしょうか？

Question

＜疑問＞
Pathには「１つの線の情報」が入っていますか？「１つ目の線～最後に描いた線の複数の情報」が入っていますか？

https://dev.classmethod.jp/smartphone/android/android-touch-drawing-undo/
上記リンクのお絵描きアプリを改造していて疑問を持ちました
Kotlinで書き換えて Path を MutableList で管理をしているのですが
ペン色変更機能を追加したところUndo機能で問題が発生しました
（コチラの動作の問題は既にクリアしています、Pathに対する疑問のみです）

「Undo動作フロー」
1.CanvasをロックしてHolderに入れる
2.HolderのCanvasをクリア
3.１つ手前まで全ての線をList内の Path,Paint で順番に描く
4.HolderのCanvasをViewに戻す

3.の過去の線を描く手順で、最後に描く線の Paint が持つ Color の情報で全ての線が置き換わってしまったようです

MutableListに入れたPathの情報が「１つ目の線～最後に描いた線の複数の情報」を持っているのなら対応した最後のPaint情報で上書きされてる？
SurfaceViewのダブルバッファリングによる問題なのか？とも考えており、、、
疑問を抱えたまま先へ進んでしまったため質問させていただきました。

一応、現在のプログラムより抜粋

   // 1.MutableListに置き換えた
    private var mActionNumber : Int = 0
    private val mUndoStack : MutableList<Path> = mutableListOf()
    private var mRedoNumber : Int = 0
    private val mRedoStack : MutableList<Path> = mutableListOf()
    
    
   // 2.Paintも追加
    private val mRedoPaint : MutableList<Paint> = mutableListOf()
    private val mUndoPaint : MutableList<Paint> = mutableListOf()


   // 3.色にナンバー付けてリスト管理
    //////////////////////////////////////////////////////////
    private var mColorNumber :Int = 0
    private val mUndoColor : MutableList<Int> = mutableListOf()
    private val mRedoColor : MutableList<Int> = mutableListOf()
    //////////////////////////////////////////////////////////

   // 2.色変ボタン追加
    fun blackColor(){
        mPaint!!.color = Color.BLACK
        mColorNumber = 0    // 3.色にナンバー付けてリスト管理
    }
    fun redColor(){
        mPaint!!.color = Color.RED
        mColorNumber = 1    // 3.色にナンバー付けてリスト管理
    }


   // アンドゥがおかしい、、、
   // SurfaceView と Path の同期が取れない？？？？
    fun undo() {
        if (mActionNumber == 0) {
            return
        }
       // undoスタックからパスを取り出し、redoスタックに格納します。
        val lastUndoPath = mUndoStack.removeAt(mActionNumber-1)
        val lastUndoPaint = mUndoPaint.removeAt(mActionNumber-1)
        val lastUndoColor = mUndoColor.removeAt(mActionNumber-1)
        mActionNumber--

        mRedoStack.add(mRedoNumber,lastUndoPath)
        mRedoPaint.add(mRedoNumber,lastUndoPaint)
        mRedoColor.add(mRedoNumber,lastUndoColor)
        mRedoNumber++

        // ロックしてキャンバスを取得します。
        val canvas = mHolder!!.lockCanvas()

        // 取得したキャンバスをクリアします。
        canvas.drawColor(0, PorterDuff.Mode.CLEAR)

        // 描画状態を保持するBitmapをクリアします。
        clearLastDrawBitmap()

        // パスを描画します。
        var i:Int = 0
        for(i in 0..mActionNumber-1) {
            var path1 = mUndoStack.get(i)
            var paint1 = mUndoPaint.get(i)
            var color1 = mUndoColor.get(i)

           // 3.色にナンバー付けてリスト管理
            when(color1){    //ここで paint1 の color 情報を書き換える
                0 -> paint1.color = Color.BLACK
                1 -> paint1.color = Color.RED
             }
            canvas.drawPath(path1, paint1)
        }
        // ロックを外します。
        mHolder!!.unlockCanvasAndPost(canvas)
    }

Answer 1

"線" というのがちょっと曖昧な表現なので勘違いされているのかもしれません。お絵描きで書くような"線"は複数の連続する"直線"によって構成されるものなので、それを表現するためにPathが用いられます。

---

ご質問のコードを拝見する限りだと、問題の原因はPaintの実装にあると思います。

現状ではmUndoPaint/mRedoPaintに追加するPaintのインスタンスがすべて同じになってしまっていませんか？undo/redoなどを想定してペン色の変更機能を実装するなら、mUndoPaint/mRedoPaintに追加する時点でPaintのインスタンスを別に作るべきです。

mUndoPaint.add(mPaint)
mPaint = Paint(mPaint)

Comments

[HRSKT]

回答ありがとうございます

おっしゃる通りですね
Undo/Redoの動作では別の path1/Paint1 を作って対応しています
そこにMutableListに格納していた Path()/Paint() を入れていたのですが
Paintのcolor情報が繁栄されず？全て同じ色になっていました
格納されたPaintには色情報がないのかな？（新たな疑問ｗ）

SV/Path/Paintの疑問内容を納得できるようなプログラムを何個か書いてテストを試みてみます
ありがとうございました

[kakajika]

MutableList等に入れてPaintの色情報が失われるなどということはありません。回答に書いたとおりPaintのインスタンスが共有されていることが問題だと思いますので、まずはPaintのインスタンスが正しいタイミングで新しく作られているかを確認してみてください。

Answer 2

Path には複数の情報が保存出来たと思います. ドキュメントでご確認ください.

複数の情報が保存出来れば,

「１つの線の情報」が入っていますか？「１つ目の線～最後に描いた線の複数の情報」が入っていますか

は, 使い方次第です.

Comments

[HRSKT]

回答ありがとうございます

使い方次第なんですね、違う動作で作って試しながら感覚つかんでいこうと思います
SurfseViewのダブルバッファリングは別のスレッドで描画してるからタイミングにより結果が別なものになってたのかな？？？
PathもSurfaceViewも分かっておらず、、、
別なフロー試したり、develope細かく見て整理してみます
ありがとうございました