Fix: Add file type check for highlight action to match remove highlig…

[LiHua000]

…ht behavior

Log: as title

Bug: https://pms.uniontech.com/bug-view-343677.html

[deepin-ci-robot]

deepin pr auto review

我来分析一下这段代码的改进建议：

1. 代码逻辑改进：

• 代码中有两处相同的文件类型检查 sheet->fileType() == Dr::FileType::PDF || sheet->fileType() == Dr::FileType::DOCX，可以考虑将其提取为一个布尔变量，提高代码可读性和维护性。
• 建议将文件类型检查放在函数开始处，避免重复判断。

2. 性能优化：

• 每次调用 sheet->fileType() 都会进行一次文件类型查询，可以将其结果缓存起来。

3. 代码结构改进：

• 建议将高亮相关的代码块封装成一个独立的方法，提高代码的可读性和可维护性。

4. 安全性考虑：

• m_pColorWidgetAction 的创建和使用都在同一个 if 块中，但在其他地方可能会访问这个指针，需要确保在其他地方使用前进行空指针检查。

改进后的代码建议：

void BrowserMenu::initActions(DocSheet *sheet, int index, SheetMenuType_e type, ...)
{
    // 缓存文件类型检查结果
    const bool isPdfOrDocx = (sheet->fileType() == Dr::FileType::PDF || 
                             sheet->fileType() == Dr::FileType::DOCX);
    
    // ... 其他代码 ...
    
    this->addSeparator();
    
    if (isPdfOrDocx) {
        setupHighlightAction(type);
        qCDebug(appLog) << "BrowserMenu::initActions() - Adding PDF/DOCX highlight actions";
    }
}

void BrowserMenu::setupHighlightAction(SheetMenuType_e type)
{
    QAction *highAct = createAction(tr("Highlight"), 
        type != DOC_MENU_SELECT_TEXT ? "ChangeAnnotationColor" : "AddTextHighlight");
    
    if (type == DOC_MENU_ANNO_HIGHLIGHT) {
        qCDebug(appLog) << "BrowserMenu::setupHighlightAction() - Disabling highlight action for annotation highlight";
        highAct->setDisabled(true);
    }

    m_pColorWidgetAction = new ColorWidgetAction(this);
    connect(m_pColorWidgetAction, SIGNAL(sigBtnGroupClicked()), 
            this, SLOT(onSetHighLight()));
    this->addAction(m_pColorWidgetAction);
}

这样的改进带来以下好处：

1. 提高了代码的可读性和可维护性
2. 减少了重复的文件类型检查
3. 通过封装提高了代码的模块化程度
4. 缓存文件类型结果提高了性能
5. 更清晰的代码结构便于后续维护和扩展

建议在类的头文件中添加 setupHighlightAction 的声明，并确保在其他使用 m_pColorWidgetAction 的地方进行适当的空指针检查。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: LiHua000, lzwind

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[LiHua000]

/merge