Android图片多路径自由裁剪问题：每个路径重复添加首个绘制点

我看了你的代码和遇到的问题，核心是每个裁剪路径都会重复添加第一个绘制点，导致最终裁剪效果异常。问题主要出在触摸事件的点处理逻辑和裁剪时的路径构建逻辑上，下面一步步给你修正：

一、修正触摸事件中的点添加逻辑 #

你的代码在ACTION_UP时手动把mfirstpoint添加到points2里，这直接导致点列表多了重复的起点；另外ACTION_DOWN时没重置bfirstpoint，也会让新路径的点处理混乱。修改后的触摸事件代码如下：

public boolean onTouch(View view, MotionEvent event) {
    switch (event.getAction()) {
        case MotionEvent.ACTION_DOWN:
            points.clear();
            points2 = new ArrayList<Point>();
            bfirstpoint = false; // 重置标记，开始新路径
            break;
        case MotionEvent.ACTION_MOVE:
            if (flgPathDraw && zooming) {
                Point point = new Point();
                point.x = (int) event.getX();
                point.y = (int) event.getY();
                zoomPos.x = event.getX();
                zoomPos.y = event.getY();

                if (!bfirstpoint) {
                    // 处理第一个点，直接添加并记录起点
                    points.add(point);
                    points2.add(point);
                    mfirstpoint = point;
                    bfirstpoint = true;
                } else {
                    // 非第一个点，直接添加即可
                    points.add(point);
                    points2.add(point);
                }
                invalidate();
            }
            break;
        case MotionEvent.ACTION_UP:
            mlastpoint = new Point((int) event.getX(), (int) event.getY());
            if (flgPathDraw) {
                // 移除手动添加起点的代码，避免重复
                if (points.size() > 12) {
                    zooming = false;
                    addpaths.add(path);
                    invalidate();
                }
            }
            if (points2 != null && !points2.isEmpty()) {
                pointlists.add(points2);
            }
            break;
    }
    return true;
}

修改说明： #

1. 把触摸事件拆分为三个分支，逻辑更清晰；
2. ACTION_DOWN时重置bfirstpoint为false，确保新路径的第一个点处理正确；
3. 删掉ACTION_UP中points2.add(mfirstpoint)这行，避免重复添加起点；
4. 简化第一个点的判断逻辑，只在首次添加时记录起点，不再做多余的分支判断。

二、修正裁剪时的路径构建逻辑 #

你之前的裁剪代码直接循环所有点调用lineTo，没有先定位到起点，也没闭合路径，这会导致绘制异常。修改后的裁剪代码如下：

// 裁剪路径的绘制逻辑
for (int j = 0; j < CutPhotoView.pointlists.size(); j++) {
    ArrayList<Point> currentPoints = CutPhotoView.pointlists.get(j);
    if (currentPoints.size() < 2) continue; // 跳过无效的短路径

    path.reset(); // 重置路径，避免和上一个路径混淆
    // 先移动到当前路径的第一个点
    path.moveTo(currentPoints.get(0).x, currentPoints.get(0).y);
    // 从第二个点开始绘制路径
    for (int i = 1; i < currentPoints.size(); i++) {
        path.lineTo(currentPoints.get(i).x, currentPoints.get(i).y);
    }
    path.close(); // 自动闭合路径，连接最后一个点和第一个点
    canvas.drawPath(path, paint);
}

修改说明： #

1. 处理每个路径前调用path.reset()，避免路径残留；
2. 先用path.moveTo()定位到路径起点，而不是直接lineTo；
3. 循环从索引1开始，跳过第一个点，避免重复绘制；
4. 调用path.close()自动闭合路径，不需要手动添加起点到点列表。

三、额外小建议 #

• 如果comparepoint是用来判断最后一个点是否接近起点（实现点击闭合路径），可以保留这个逻辑，但不要在点添加阶段重复插入起点；
• 可以在ACTION_UP时判断最后一个点和起点的距离，如果足够近，直接调用path.close()让路径更规整。

这样修改后，应该就能解决路径重复添加第一个点的问题，裁剪效果也会和预期一致啦！

内容的提问来源于stack exchange，提问作者Raji Raji