Tkinter绘图应用中PIL ImageGrab.crop()报ValueError问题排查

问题根源 #

报错提示Coordinate 'right' is less than 'left'，核心原因是你给crop()传入的坐标顺序完全错误。PIL的Image.crop()方法要求传入的元组格式为(left, upper, right, lower)，即左边界x坐标、上边界y坐标、右边界x坐标、下边界y坐标，且必须满足right > left、lower > upper。

你当前代码中：

ImageGrab.grab().crop((cx1, cy1, cx, cy)).save(filename)

cx1 = cx + self.canvas.winfo_width()，显然cx1是画布的右边界，cx是左边界，但你把右边界放在了left的参数位置，左边界放在了right的参数位置，直接导致right（cx）小于left（cx1），触发ValueError。

修正方案 #

将crop的坐标顺序改为正确的(cx, cy, cx1, cy1)即可：

ImageGrab.grab().crop((cx, cy, cx1, cy1)).save(filename)

额外优化建议 #

• 文件名后缀重复问题：你当前强制给文件名添加.jpeg，如果用户在保存对话框已输入后缀（比如test.png），最终会变成test.png.jpeg，可以添加判断逻辑：

  if filename and not filename.endswith(('.jpeg', '.jpg', '.png')):
      filename += '.jpeg'
  

• Canvas超出窗口问题：你设置的Canvas尺寸width=1105, height=630远大于窗口的800x520，会导致画布被截断，建议调整Canvas尺寸适配窗口，或者将窗口缩放设置改为root.resizable(True, True)（原代码的100,100不符合规范）。
• 颜色选择取消处理：若用户在颜色选择对话框点击取消，colorchooser.askcolor()会返回(None, None)，此时访问color[1]会报错，建议添加判断：

  def canvas_color(self):
      color = colorchooser.askcolor()
      if color[1] is not None:
          self.canvas.configure(background=color[1])
          self.eraser_color = color[1]
          # 此处无需强制将画笔颜色改为画布颜色，可根据需求保留或删除
          # self.pen_color = color[1]
  

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