2016-08-17 4 views
0

カスタムブラシでキャンバスドローイングアプリケーションを実装したいと思います。&アンドゥ/リドゥ操作。まず最初に、私のコードは、カスタムブラシ(アンドゥ/リドゥ操作を含む)を使わずに完全に動作します。この回答によるとHow to make custom brush for canvas in android?私は単純な画像スパイクをビットマップ描画に使用しました。 アンドロイドキャンバスでのアンドゥ/リドゥ操作のカスタムブラシ

enter image description here

今問題がある

  1. アンドゥ、操作が作品、カスタムブラシ塗料は何度も繰り返しタッチポイントを移動していない時はいつでもやり直し。

    Q:元に戻す/やり直しを行うにはどうすればいいですか?

  2. カスタムブラシストーカーは、滑らかではありません。今、彼らは粗くて人工的に見えます。

    Q.カスタムブラシストロークで絵を滑らかに自然にする方法は?

enter image description here

あなたのブラシはそれが何を意味するのか、100%確実ではない「滑らかに」に見えるが、あなたは引き締める必要があるだろうにするために、ここで

public class DrawingView extends View { 

    private Context ctx; 

    private ArrayList<Path> paths = new ArrayList<Path>(); 
    private ArrayList<Path> undonePaths = new ArrayList<Path>(); 

    private Map<Path, Float> brushMap = new HashMap<Path, Float>(); 
    private Map<Path, List<Vector2>> customBrushMap = new HashMap<Path, List<Vector2>>(); 

    private Bitmap mBitmapBrush; 
    private Vector2 mBitmapBrushDimensions; 
    private List<Vector2> mPositions = new ArrayList<Vector2>(100); 
    private boolean isCustomBrush = false; 

    private int selectedColor; 
    private float brushSize, lastBrushSize; 

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

    private Path drawPath; 
    private Paint drawPaint, canvasPaint; 
    private int paintColor = 0xFF660000, paintAlpha = 255; 
    private Canvas drawCanvas; 
    private Bitmap canvasBitmap; 

    private static final class Vector2 { 
     public Vector2(float x, float y) { 
      this.x = x; 
      this.y = y; 
     } 

     public final float x; 
     public final float y; 
    } 

    public DrawingView(Context context, AttributeSet attrs) { 
     super(context, attrs); 
     ctx = context; 
     setupDrawing(); 
    } 

    private void setupDrawing() { 
     brushSize = getResources().getInteger(R.integer.small_size); 
     lastBrushSize = brushSize; 

     drawPath = new Path(); 
     drawPaint = new Paint(); 
     drawPaint.setColor(paintColor); 
     drawPaint.setAntiAlias(true); 
     drawPaint.setDither(true); 
     drawPaint.setStrokeWidth(brushSize); 
     drawPaint.setStyle(Paint.Style.STROKE); 
     drawPaint.setStrokeJoin(Paint.Join.ROUND); 
     drawPaint.setStrokeCap(Paint.Cap.ROUND); 
     canvasPaint = new Paint(Paint.DITHER_FLAG); 
    } 

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

    private void touch_start(float x, float y) { 
     undonePaths.clear(); 
     drawPath.reset(); 
     drawPath.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) { 
      drawPath.quadTo(mX, mY, (x + mX)/2, (y + mY)/2); 
      mX = x; 
      mY = y; 
     } 
     customBrushMap.put(drawPath, mPositions); 
    } 

    private void touch_up() { 
     drawPath.lineTo(mX, mY); 
     drawCanvas.drawPath(drawPath, drawPaint); 
     paths.add(drawPath); 
     brushMap.put(drawPath, brushSize); 
     drawPath = new Path(); 
     drawPath.reset(); 
     invalidate(); 
    } 

    public void onClickUndo() { 
     if (paths.size() > 0) { 
      undonePaths.add(paths.remove(paths.size() - 1)); 
      invalidate(); 
     } 
    } 

    public void onClickRedo() { 
     if (undonePaths.size() > 0) { 
      paths.add(undonePaths.remove(undonePaths.size() - 1)); 
      invalidate(); 
     } 
    } 

    @Override 
    public boolean onTouchEvent(MotionEvent event) { 

     //detect user touch 
     float x = event.getX(); 
     float y = event.getY(); 

     switch (event.getAction() & MotionEvent.ACTION_MASK) { 

      case MotionEvent.ACTION_DOWN: 
       touch_start(x, y); 
       invalidate(); 
       break; 

      case MotionEvent.ACTION_MOVE: 
       touch_move(x, y); 
       if (isCustomBrush) { 
       mPositions.add(new Vector2(x - mBitmapBrushDimensions.x/2, y - mBitmapBrushDimensions.y/2)); 
       } 
       touch_move(x, y); 
       invalidate(); 
       break; 

      case MotionEvent.ACTION_POINTER_DOWN: 
       invalidate(); 
       break; 

      case MotionEvent.ACTION_UP: 
       touch_up(); 
       invalidate(); 
       break; 
     } 

     return true; 
    } 

    @Override 
    protected void onDraw(Canvas canvas) { 
     canvas.save(); 
     for (Path p : paths) { 

     drawPaint.setColor(colorsMap.get(p)); 
     drawPaint.setShader(shaderMap.get(p)); 
     drawPaint.setStrokeWidth(brushMap.get(p)); 
     drawPaint.setAlpha(opacityMap.get(p)); 

     if (isCustomBrush) { 
      if (customBrushMap.get(p) != null) { 
       for (Vector2 pos : customBrushMap.get(p)) { 
        Paint paint = new Paint(); 
        ColorFilter filter = new PorterDuffColorFilter(selectedColor, PorterDuff.Mode.SRC_IN); 
        paint.setColorFilter(filter); 
        canvas.drawBitmap(mBitmapBrush, pos.x, pos.y, paint); 
       } 
      } 
     } else { 
      canvas.drawPath(p, drawPaint); 
      drawPaint.setColor(selectedColor); 
      drawPaint.setStrokeWidth(brushSize); 
      canvas.drawPath(drawPath, drawPaint); 
     } 
    } 
     canvas.restore(); 
    } 

    public void setCustomBrush(Activity activity, String customBrush) { 
     isCustomBrush = true; 
     invalidate(); 
     int patternID = getResources().getIdentifier(customBrush, "drawable", "com.androidapp.drawingstutorial"); 
     mBitmapBrush = BitmapFactory.decodeResource(getResources(), patternID); 
     mBitmapBrushDimensions = new Vector2(mBitmapBrush.getWidth(), mBitmapBrush.getHeight()); 
    } 
} 

答えて

0

を私のサンプルコードをチェックカスタムブラシに使用している画像の点。それらをよりコンパクトにすると、エッジが滑らかに見えるようになります。

元に戻すやり直しについては、touchUpメソッドでは、customBrushMapにbrushMapだけを追加することはありません。したがって、オンドローでは、そのマップは常に空であるため描画するものはありません。

+0

onDraw()内のfor(Vector2 pos:customBrushMap.get(p)){ canvas.drawBitmap(mBitmapBrush、pos.x、pos.y、null); }ループはパスを再度上書きし、パスの描画を2回行います。どのようにパスを1回だけ描​​くことができますか?ループがなければ、verter2リストからxとyの座標を得ることはできません。 –

+0

私が見る唯一のことはあなたのelseではdrawPathを2回呼び出すことです。 – Ben

+0

私が話している問題を確認するためにコードを一度実行してください。 –

関連する問題