case MotionEvent.ACTION_MOVE: if (event.getPointerCount() == 1) { float x = event.getX(0); float y = event.getY(0); float dx = x - mPrevX; float dy = y - mPrevY; mRotationDegree += Math.atan2(dy, dx) * 180 / Math.PI; setRotation(mRotationDegree/20); mPrevX = x; mPrevY = y;优化这段代码

这段代码的主要功能是根据手指滑动的方向旋转一个视图，我的优化建议如下：

1. 使用局部变量来缓存 getX 和 getY 的返回值，避免多次调用这些方法；
2. 使用 switch-case 语句代替 if-else 语句，提高代码的可读性；
3. 将常量提取到类成员变量中，避免多次创建和销毁对象；
4. 将角度转换为弧度，减少计算量；
5. 将除以 20 的操作改为乘以 0.05，提高运算效率。

修改后的代码如下：

case MotionEvent.ACTION_MOVE:
    switch (event.getPointerCount()) {
        case 1:
            float x = event.getX(0);
            float y = event.getY(0);
            float dx = x - mPrevX;
            float dy = y - mPrevY;
            double angle = Math.atan2(dy, dx);
            mRotationDegree += angle * 180 / Math.PI;
            setRotation(mRotationDegree * 0.05f);
            mPrevX = x;
            mPrevY = y;
            break;
    }
    break;

相关问题

package com.blog.demo11; import android.annotation.SuppressLint; import android.os.Bundle; import android.view.MotionEvent; import android.view.View; import android.widget.ImageView; import androidx.annotation.Nullable; import com.blog.BaseActivity; import com.blog.R; import static android.view.MotionEvent.INVALID_POINTER_ID; /** * 可拖拽效果类。 */ public class ViewDragActivity extends BaseActivity { private ImageView backgroundImage; private ImageView image1; private ImageView image2; @SuppressLint("MissingInflatedId") @Override protected void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_main); backgroundImage = findViewById(R.id.background_image); image1 = findViewById(R.id.image1); image2 = findViewById(R.id.image2); // Set touch listeners for each image image1.setOnTouchListener(new MultiTouchListener()); image2.setOnTouchListener(new MultiTouchListener()); } private class MultiTouchListener implements View.OnTouchListener { private float lastTouchX, lastTouchY; private int activePointerId = INVALID_POINTER_ID; @Override public boolean onTouch(View v, MotionEvent event) { switch (event.getActionMasked()) { case MotionEvent.ACTION_DOWN: { final int pointerIndex = event.getActionIndex(); final float x = event.getX(pointerIndex); final float y = event.getY(pointerIndex); lastTouchX = x; lastTouchY = y; activePointerId = event.getPointerId(pointerIndex); break; } case MotionEvent.ACTION_MOVE: { final int pointerIndex = event.findPointerIndex(activePointerId); final float x = event.getX(pointerIndex); final float y = event.getY(pointerIndex); final float dx = x - lastTouchX; final float dy = y - lastTouchY; v.setX(v.getX() + dx); v.setY(v.getY() + dy); lastTouchX = x; lastTouchY = y; break; } case MotionEvent.ACTION_UP: case MotionEvent.ACTION_CANCEL: { activePointerId = INVALID_POINTER_ID; break; } case MotionEvent.ACTION_POINTER_UP: { final int pointerIndex = event.getActionIndex(); final int pointerId = event.getPointerId(pointerIndex); if (pointerId == activePointerId) { final int newPointerIndex = pointerIndex == 0 ? 1 : 0; lastTouchX = event.getX(newPointerIndex); lastTouchY = event.getY(newPointerIndex); activePointerId = event.getPointerId(newPointerIndex); } break; } } return true; } } }

这是一段 Android 应用程序代码，实现了一个可拖拽效果类 ViewDragActivity。在该类中，通过监听 onTouch 事件来实现多指触摸拖拽图片的效果，其中 MultiTouchListener 类实现了 View.OnTouchListener 接口，重写了 onTouch 方法，通过获取 MotionEvent 的不同事件来实现图片的移动。具体实现过程可以参考代码中的注释。

请优化代码，以下代码无法释放焦点 ，不能变换拖动目标 public boolean onTouchEvent(MotionEvent event) { switch (event.getAction() & MotionEvent.ACTION_MASK) { case MotionEvent.ACTION_DOWN: // 手指压下屏幕 mode = MODE.DRAG; // 寻找被点击的图片 CustomBitmap clickedBitmap = null; for (CustomBitmap bitmap : _bitmaps) { float[] values = new float[9]; bitmap.matrix.getValues(values); float globalX = values[Matrix.MTRANS_X]; float globalY = values[Matrix.MTRANS_Y]; float width = values[Matrix.MSCALE_X] * bitmap.getBitmap().getWidth(); float height = values[Matrix.MSCALE_Y] * bitmap.getBitmap().getWidth(); Rect rect = new Rect((int) globalX, (int) globalY, (int) (globalX + width), (int) (globalY + height)); if (rect.contains((int) event.getX(), (int) event.getY())) { clickedBitmap = bitmap; break; } } // 切换操作对象 if (clickedBitmap != null) { _bitmaps.remove(clickedBitmap); _bitmaps.add(clickedBitmap); currentMatrix.set(clickedBitmap.matrix);// 记录ImageView当前的移动位置 clickedBitmap.matrix.set(currentMatrix); clickedBitmap.startPoint.set(event.getX(), event.getY()); _curCustomBitmap = clickedBitmap; } postInvalidate(); break; case MotionEvent.ACTION_POINTER_DOWN: // 当屏幕上还有触点（手指），再有一个手指压下屏幕 mode = MODE.ZOOM; _curCustomBitmap.oldRotation = rotation(event); _curCustomBitmap.startDis = distance(event); if (_curCustomBitmap.startDis > 10f) { _curCustomBitmap.midPoint = mid(event); currentMatrix.set(_curCustomBitmap.matrix);// 记录ImageView当前的缩放倍数 } break; case MotionEvent.ACTION_MOVE: // 手指在屏幕移动，该事件会不断地触发 if (mode == MODE.DRAG) { float dx = event.getX() - _curCustomBitmap.startPoint.x;// 得到在x轴的移动距离 float dy = event.getY() - _curCustomBitmap.startPoint.y;// 得到在y轴的移动距离 _curCustomBitmap.matrix.set(currentMatrix);// 在没有进行移动之前的位置基础上进行移动 _curCustomBitmap.matrix.postTranslate(dx, dy); } else if (mode == MODE.ZOOM) {// 缩放与旋转 float endDis = distance(event);// 结束距离 _curCustomBitmap.rotation = rotation(event) - _curCustomBitmap.oldRotation; if (endDis > 10f) { float scale = endDis / _curCustomBitmap.startDis;// 得到缩放倍数 _curCustomBitmap.matrix.set(currentMatrix); _curCustomBitmap.matrix.postScale(scale, scale, _curCustomBitmap.midPoint.x, _curCustomBitmap.midPoint.y); _curCustomBitmap.matrix.postRotate(_curCustomBitmap.rotation, _curCustomBitmap.midPoint.x, _curCustomBitmap.midPoint.y); } } break; case MotionEvent.ACTION_UP: // 手指离开屏 break; case MotionEvent.ACTION_POINTER_UP: // 有手指离开屏幕,但屏幕还有触点（手指） mode = MODE.NONE; break; } invalidate(); return true; }

以下是对代码的优化建议：

1. 将对位图的操作封装成一个方法，以提高可读性和可维护性。

2. 在ACTION_DOWN事件中，可以使用MotionEvent的findPointerIndex()和getX()、getY()方法来获取点击位置，避免使用循环查找位图。

3. 在ACTION_MOVE事件中，可以使用Matrix的postTranslate()和postScale()方法来进行位图的移动和缩放，避免重复创建Matrix对象。

4. 在ACTION_POINTER_DOWN事件中，可以使用MotionEvent的getX()、getY()方法来获取缩放中心点的坐标，避免重复计算。

5. 在ACTION_POINTER_DOWN事件中，可以将记录位图旋转角度和缩放倍数的代码提取出来，避免与缩放和旋转的代码耦合。

6. 在ACTION_POINTER_UP事件中，可以将模式设置为MODE.NONE，避免在其他事件中误操作。

7. 在代码中添加注释，以提高可读性和可维护性。

下面是优化后的代码：

public boolean onTouchEvent(MotionEvent event) {
    switch (event.getAction() & MotionEvent.ACTION_MASK) {
        case MotionEvent.ACTION_DOWN:
            // 手指压下屏幕
            mode = MODE.DRAG;
            // 查找被点击的图片
            int index = event.getActionIndex();
            float x = event.getX(index);
            float y = event.getY(index);
            CustomBitmap clickedBitmap = findClickedBitmap(x, y);
            if (clickedBitmap != null) {
                // 切换操作对象
                _bitmaps.remove(clickedBitmap);
                _bitmaps.add(clickedBitmap);
                // 记录ImageView当前的移动位置
                currentMatrix.set(clickedBitmap.matrix);
                clickedBitmap.matrix.set(currentMatrix);
                clickedBitmap.startPoint.set(x, y);
                _curCustomBitmap = clickedBitmap;
            }
            postInvalidate();
            break;
        case MotionEvent.ACTION_POINTER_DOWN:
            // 当屏幕上还有触点（手指），再有一个手指压下屏幕
            mode = MODE.ZOOM;
            // 记录位图的旋转角度和缩放倍数
            _curCustomBitmap.oldRotation = rotation(event);
            _curCustomBitmap.startDis = distance(event);
            if (_curCustomBitmap.startDis > 10f) {
                // 获取缩放中心点的坐标
                float x1 = event.getX(0);
                float y1 = event.getY(0);
                float x2 = event.getX(1);
                float y2 = event.getY(1);
                _curCustomBitmap.midPoint.set((x1 + x2) / 2, (y1 + y2) / 2);
                // 记录ImageView当前的缩放倍数
                currentMatrix.set(_curCustomBitmap.matrix);
            }
            break;
        case MotionEvent.ACTION_MOVE:
            // 手指在屏幕移动，该事件会不断地触发
            if (mode == MODE.DRAG) {
                // 移动图片
                float dx = event.getX() - _curCustomBitmap.startPoint.x;
                float dy = event.getY() - _curCustomBitmap.startPoint.y;
                _curCustomBitmap.matrix.set(currentMatrix);
                _curCustomBitmap.matrix.postTranslate(dx, dy);
            } else if (mode == MODE.ZOOM) {
                // 缩放和旋转图片
                float endDis = distance(event);
                float rotation = rotation(event) - _curCustomBitmap.oldRotation;
                if (endDis > 10f) {
                    float scale = endDis / _curCustomBitmap.startDis;
                    _curCustomBitmap.matrix.set(currentMatrix);
                    _curCustomBitmap.matrix.postScale(scale, scale, _curCustomBitmap.midPoint.x, _curCustomBitmap.midPoint.y);
                    _curCustomBitmap.matrix.postRotate(rotation, _curCustomBitmap.midPoint.x, _curCustomBitmap.midPoint.y);
                }
            }
            break;
        case MotionEvent.ACTION_POINTER_UP:
            // 有手指离开屏幕,但屏幕还有触点（手指）
            mode = MODE.NONE;
            break;
    }
    invalidate();
    return true;
}

// 查找被点击的图片
private CustomBitmap findClickedBitmap(float x, float y) {
    for (CustomBitmap bitmap : _bitmaps) {
        float[] values = new float[9];
        bitmap.matrix.getValues(values);
        float globalX = values[Matrix.MTRANS_X];
        float globalY = values[Matrix.MTRANS_Y];
        float width = values[Matrix.MSCALE_X] * bitmap.getBitmap().getWidth();
        float height = values[Matrix.MSCALE_Y] * bitmap.getBitmap().getHeight();
        RectF rect = new RectF(globalX, globalY, globalX + width, globalY + height);
        if (rect.contains(x, y)) {
            return bitmap;
        }
    }
    return null;
}

// 计算两点之间的距离
private float distance(MotionEvent event) {
    float x1 = event.getX(0);
    float y1 = event.getY(0);
    float x2 = event.getX(1);
    float y2 = event.getY(1);
    return (float) Math.sqrt((x1 - x2) * (x1 - x2) + (y1 - y2) * (y1 - y2));
}

// 计算两点之间的旋转角度
private float rotation(MotionEvent event) {
    float x1 = event.getX(0);
    float y1 = event.getY(0);
    float x2 = event.getX(1);
    float y2 = event.getY(1);
    double radians = Math.atan2(y2 - y1, x2 - x1);
    return (float) Math.toDegrees(radians);
}