#include "QCustomGraphicsEffect.h" QCustomGraphicsEffect::QCustomGraphicsEffect(QObject* parent) :QGraphicsEffect(parent) { } void QCustomGraphicsEffect::draw(QPainter* painter) { QImage image; image = sourcePixmap().toImage(); image = image.convertToFormat( QImage::Format_Grayscale8); for (int i = 0; i < image.byteCount(); i++) { image.bits()[i] = image.bits()[i] < 100 ? 0 : 255; painter->drawPixmap(0, 0, QPixmap::fromImage(image)); } } 帮我重构一下这段代码

好的，下面是重构后的代码：

QCustomGraphicsEffect::QCustomGraphicsEffect(QObject* parent) : QGraphicsEffect(parent) {}

QRectF QCustomGraphicsEffect::boundingRectFor(const QRectF& rect) const
{
return rect;
}

void QCustomGraphicsEffect::draw(QPainter* painter)
{
if (!sourcePixmap().isNull()) {
QImage image = sourcePixmap().toImage().convertToFormat(QImage::Format_Grayscale8);

const int threshold = 100;
const uchar black = 0;
const uchar white = 255;

for (int i = 0; i < image.byteCount(); i++) {
image.bits()[i] = image.bits()[i] < threshold ? black : white;
}

painter->drawPixmap(QPointF(0.0, 0.0), QPixmap::fromImage(image));
}
}

这里主要对原有代码进行了以下几个改进：

1. 添加了 boundingRectFor 函数。这个函数是 QGraphicsEffect 类的虚函数，它返回该效果应用后的边界矩形。在这个例子中，我们可以直接返回传入的矩形 rect。

2. 添加了一些常量。这里我们将变量 threshold、black 和 white 定义为常量，以提高代码可读性。

3. 重构了绘制函数。在这个新版本的代码中，我们首先检查 sourcePixmap() 是否为空，如果不为空，则将其转换为 QImage 对象，并将其转换为灰度图像格式。然后使用一个 for 循环遍历图像的每个像素，并将像素值小于 threshold 的像素点置为黑色（值为0），其他像素点置为白色（值为255）。最后，我们使用 painter 对象绘制经过处理后的图像，将其作为 QGraphicsEffect 的输出。

这个新版本的代码更加简洁，易于理解，同时也提高了代码的可读性和可维护性。