如果鼠标按下后马上弹起，则不应绘制任何元素

[0x738A]

[0x738A]

复现方法为：双击启动程序，随便选个窗口后鼠标左键单击，此时会抓取一张图并钉在桌面上。选中箭头工具并绘制一个箭头后，点击钉住窗口的任意位置，程序崩溃。
操作系统：win 10 19044

[0x738A]

我调试了一下，BUG复现的过程为：双击启动程序，随便选个窗口后鼠标左键单击，此时会抓取一张图并钉在桌面上。选中箭头工具并绘制一个箭头后，点击钉住窗口的任意位置，程序崩溃。

崩溃原因为：单击钉住的窗口时，History::createShape会创建一个新的ShapeArrow对象并添加到vector里面去，此时尚未调用ShapeArrow::makeArrow()方法。当触发WM_MOUSEMOVE消息时，由WinPin::onMouseDrag调用ShapeArrow::mouseDrag方法，并进一步调用ShapeArrow::makeArrow()方法，初始化ShapeArrow的资源。由于鼠标仅触发WM_LBUTTONDOWN、WM_LBUTTONUP，未触发WM_MOUSEMOVE；此时ShapeArrow对象已创建且添加至History::shapes成员变量，但未调用ShapeArrow::makeArrow()，ShapeArrow::path变量为nullptr，触发WM_PAINT消息立即导致崩溃

[xland]

感谢反馈，稍晚点我看看，今天有点忙

[xland]

帮忙看看最新版本解决了此问题没有~

[0x738A]

仅从操作的角度来看，是没问题的，但是从代码层面看，问题还是存在。可以在ShapeArrow::paint()这个方法里面检查一下path这个成员变量是否为空。还有一个优化建议是：可以在WM_LBUTTONUP消息里面，把无效的形状从WinPin::history这个unique_ptr里面删掉节约内存。比如：当鼠标左键按下再弹起，中间不触发WM_MOUSEMOVE，此时的形状其实是无效的，无法显示出来