Как изменить цвет/толщину моей краски на холсте в Android?

Question

Я не могу изменить цвет или толщину своей краски, когда я нажимаю кнопку imageButton, хотя я тоже запрограммировал ее. Я знаю, что я делаю что-то неправильно, и я думаю, что знаю, почему, но я не могу это исправить. Я чувствую, что не называю нужную кисть. Вот мой код класса Java:

import android.content.Context; 
import android.graphics.Bitmap; 
import android.graphics.Canvas; 
import android.graphics.Color; 
import android.graphics.Paint; 
import android.graphics.Path; 
import android.graphics.PorterDuff; 
import android.util.AttributeSet; 
import android.view.MotionEvent; 
import android.view.View; 

public class CanvasView extends View 
{ 
private Paint paint = new Paint(); 
private Path path = new Path(); 
private Canvas canvas = new Canvas(); 
private Paint canvasPaint = new Paint(); 
private Bitmap canvasBitmap; 

public CanvasView(Context context, AttributeSet attrs) 
{ 
    super(context, attrs); 

    paint.setAntiAlias(true); 
    paint.setStrokeWidth(5); 
    paint.setColor(Color.BLACK); 
    paint.setStyle(Paint.Style.STROKE); 
    paint.setStrokeJoin(Paint.Join.ROUND); 
    paint.setStrokeCap(Paint.Cap.ROUND); 
} 

@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); 
    canvas = new Canvas(canvasBitmap); 
} 

@Override 
protected void onDraw(Canvas drawCanvas) 
{ 
    drawCanvas.drawBitmap(canvasBitmap, 0, 0, canvasPaint); 
    drawCanvas.drawPath(path, paint); 
} 

@Override 
public boolean onTouchEvent(MotionEvent e) 
{ 
    // get the coords of the touch event 
    float eventX = e.getX(); 
    float eventY = e.getY(); 

    switch (e.getAction()) { 
     case MotionEvent.ACTION_DOWN: 
      // set a new starting point 
      path.moveTo(eventX, eventY); 
      return true; 
     case MotionEvent.ACTION_MOVE: 
      // connect the points 
      path.lineTo(eventX, eventY); 
      break; 
     default: 
      return false; 
    } 

    //makes you view repaint and call ondraw 
    invalidate(); 
    return true; 
} 

public void clearCanvas() 
{ 
    canvas.drawColor(Color.TRANSPARENT, PorterDuff.Mode.CLEAR); 
} 

Вот мой MainActivity код:

public class MainActivity extends Activity { 

private CanvasView canvasView; 
private Paint paint; 
private int orange; 
private int purple; 

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_main); 

    canvasView = (CanvasView) findViewById(R.id.canvasView); 
    paint = new Paint(); 
    orange = Color.rgb(255, 245, 238); 
    purple = Color.rgb(128, 0, 128); 

    // REGULAR BUTTONS: save, about, reset 
    Button saveB = (Button) findViewById(R.id.saveButton); 
    Button aboutB = (Button) findViewById(R.id.aboutButton); 
    Button resetB = (Button) findViewById(R.id.resetButton); 

    // IMAGE BUTTONS: red, blue, green, yellow, black, purple, orange, erase, brush thickness plus, brush thickness minus 
    ImageButton redIb = (ImageButton) findViewById(R.id.redButton); 
    ImageButton blueIb = (ImageButton) findViewById(R.id.blueButton); 
    ImageButton greenIb = (ImageButton) findViewById(R.id.greenButton); 
    ImageButton yellowIb = (ImageButton) findViewById(R.id.yellowButton); 
    ImageButton blackIb = (ImageButton) findViewById(R.id.blackButton); 
    ImageButton purpleIb = (ImageButton) findViewById(R.id.purpleButton); 
    ImageButton orangeIb = (ImageButton) findViewById(R.id.orangeButton); 
    ImageButton eraseIb = (ImageButton) findViewById(R.id.eraseButton); 
    ImageButton plusIb = (ImageButton) findViewById(R.id.plusButton); 
    ImageButton minusIb = (ImageButton) findViewById(R.id.minusButton); 

    minusIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setStrokeWidth(-5); 
     } 
    }); 

    plusIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setStrokeWidth(5); 
     } 
    }); 

    eraseIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setColor(Color.TRANSPARENT); 
     } 
    }); 

    orangeIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setColor(orange); 
     } 
    }); 

    purpleIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setColor(purple); 
     } 
    }); 

    blackIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setColor(Color.BLACK); 
     } 
    }); 

    yellowIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setColor(Color.YELLOW); 
     } 
    }); 

    greenIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setColor(Color.GREEN); 
     } 
    }); 

    blueIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setColor(Color.BLUE); 
     } 
    }); 

    redIb.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      paint.setColor(Color.RED); 
     } 
    }); 

    saveB.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 

     } 
    }); 

    aboutB.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      Intent intent = new Intent(getApplicationContext(), AboutActivity.class); 

      startActivity(intent); 
     } 
    }); 

    resetB.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      canvasView.clearCanvas(); 
      // canvasView.clearCanvas2(); 
     } 
    }); 
} 

Answer 1

Вы устанавливаете свои значения на всю различную краску, которая не имеет никакого отношения к вашему CanvasView. Вам нужно будет изменить Paint вашего CanvasView и называть его недействительным, чтобы внести какие-либо изменения.

реализовать что-то подобное в вашей CanvasView изменить ширину обводки:

public void setStrokeWidth(int strokeWidth) { 
    paint.setStrokeWidth(strokeWidth); 
    invalidate(); 
} 

и вызвать его в вашем onClick() с canvasView.setStrokeWidth(5);

То же самое касается цвета:

public void setColor(int color) { 
    paint.setColor(color); 
    invalidate(); 
} 

и вызов это в вашем onClick() с canvasView.setColor(Color.BLUE);