1

このコードがすべてのオブジェクトを描画しない理由を誰かが説明してくれますか?

public class A extends View {
private Paint paint = new Paint();
private Path path = new Path();
ArrayList<Pair<Path, Paint>> paths = new ArrayList<Pair<Path, Paint>>();


public A(Context context) {
    super(context);
}

@Override
protected void onDraw(Canvas canvas) {
    for (Pair<Path, Paint> p : paths) {
        canvas.drawPath(p.first, p.second);
    }
    canvas.drawPath(path, paint);
}



@Override
public boolean onTouchEvent(MotionEvent event) {
    float eventX = event.getX();
    float eventY = event.getY();
    paint.setAntiAlias(true);
    paint.setStyle(Paint.Style.STROKE);
    paint.setStrokeJoin(Paint.Join.ROUND);
    paint.setStrokeWidth(3f);

    switch (event.getAction()) {
        case MotionEvent.ACTION_DOWN:
            int color = Color.rgb(new Random().nextInt(255),
                                new Random().nextInt(255), 
                                new Random().nextInt(255));
            paint.setColor(color);
            path.reset(); //new stroke, get old one erased
            int historySize = event.getHistorySize();
            for (int i = 0; i > historySize; i++) {
                path.moveTo(eventX, eventY);
            }
             path.moveTo(eventX, eventY);

            return true;
        case MotionEvent.ACTION_MOVE:
            path.lineTo(eventX, eventY);
            return true;
        case MotionEvent.ACTION_UP:
            path.lineTo(eventX, eventY);
            // End of stroke, add this to the collection
            paths.add(new Pair<Path, Paint>(path, paint));
            break;
        default:
            break;
    }
    // Schedules a repaint.
    invalidate();
    return true;
}

}

私は onTouchEvent ですべてのストロークをキャッチしており、ペア 1 に格納されているさまざまなパス/ペイント オブジェクトを作成しています。悲しいことに、OnDraw でそれらをすべて描画しようとすると失敗します..正しい答えが見つからないトピックを読みました。ビットマップを作成して作業し、画面に描画することをお勧めする人もいますが、この解決策は避けたいと思います。

ご協力いただきありがとうございます ！

4

3 に答える 3

1

問題は、常に同じPathandPaintオブジェクトを使用することです。Path新しいものを作成する必要があり、起動PaintするたびにMotionEvent.ACTION_DOWN

于 2013-03-10T11:39:35.390 に答える
0

スイッチ状態のpath.resetを削除します。ドローは機能します。ただし、前回の描画でも色は変わります。前のドローをクリアしているパスをリセットします。

毎回違うペア（パスとペイント）を使って描きます。

キャンバスを使用して描画する場合は、以下のコードで問題ありません。また、ユーザーが自分の好きな色を選択できるように、カラーピッカーを追加します。

  public class MyView extends View {

    private static final float MINP = 0.25f;
    private static final float MAXP = 0.75f;

    private Bitmap  mBitmap;
    private Canvas  mCanvas;
    private Path    mPath;
    private Paint   mBitmapPaint;

    public MyView(Context c) {
        super(c);

        mPath = new Path();
        mBitmapPaint = new Paint(Paint.DITHER_FLAG);
    }

    @Override
    protected void onSizeChanged(int w, int h, int oldw, int oldh) {
        super.onSizeChanged(w, h, oldw, oldh);
        mBitmap = Bitmap.createBitmap(w, h, Bitmap.Config.ARGB_8888);
        mCanvas = new Canvas(mBitmap);
    }

    @Override
    protected void onDraw(Canvas canvas) {
        canvas.drawColor(0xFFAAAAAA);

        canvas.drawBitmap(mBitmap, 0, 0, mBitmapPaint);

        canvas.drawPath(mPath, mPaint);
    }

    private float mX, mY;
    private static final float TOUCH_TOLERANCE = 4;

    private void touch_start(float x, float y) {
        mPath.reset();
        mPath.moveTo(x, y);
        mX = x;
        mY = y;
    }
    private void touch_move(float x, float y) {
        float dx = Math.abs(x - mX);
        float dy = Math.abs(y - mY);
        if (dx >= TOUCH_TOLERANCE || dy >= TOUCH_TOLERANCE) {
            mPath.quadTo(mX, mY, (x + mX)/2, (y + mY)/2);
            mX = x;
            mY = y;
        }
    }
    private void touch_up() {
        mPath.lineTo(mX, mY);
        // commit the path to our offscreen
        mCanvas.drawPath(mPath, mPaint);
        mPaint.setXfermode(new PorterDuffXfermode(PorterDuff.Mode.SCREEN));
        // kill this so we don't double draw
        mPath.reset();
    }

    @Override
    public boolean onTouchEvent(MotionEvent event) {
        float x = event.getX();
        float y = event.getY();

        switch (event.getAction()) {
            case MotionEvent.ACTION_DOWN:
                touch_start(x, y);
                invalidate();
                break;
            case MotionEvent.ACTION_MOVE:
                touch_move(x, y);
                invalidate();
                break;
            case MotionEvent.ACTION_UP:
                touch_up();
                invalidate();
                break;
        }
        return true;
    }
}
于 2013-03-10T11:29:21.330 に答える