如何在设置字符串值时优化函数调用？ [关闭]

Posted 2023-02-22

【中文标题】如何在设置字符串值时优化函数调用？ [关闭]

【英文标题】：How can I optimize function calls while setting the string values? [closed]

【发布时间】：2019-08-20 09:34:26

【问题描述】：

在下面的代码 sn-p 中，我从 ReadFile() 函数调用 SetParams() 和 Execute() 多次。 我可以通过单个调用优化每个 SetParams() 和 Execute() 吗？

bool SubscriptionRead::ReadFile()

    IVerification* pReader = new FileReader();
    std::wstring oemPathPublicKey(oemFolderPath)
        , oemPathSessionKey(oemFolderPath)
        , oemPathUserChoices(oemFolderPath);
    oemPathPublicKey.append(PUBLIC_KEY_FILE);
    oemPathSessionKey.append(SESSION_KEY_FILE);
    oemPathUserChoices.append(USERCHOICES_FILE);
    pReader->SetParams((wchar_t*)oemPathPublicKey.c_str(), L"file");
    pReader->Execute();
    pReader->SetParams((wchar_t*)oemPathSessionKey.c_str(), L"file");
    pReader->Execute();
    pReader->SetParams((wchar_t*)oemPathUserChoices.c_str(), L"file");
    pReader->Execute();
    return True;


void FileReader::SetParams(wchar_t* wszParams, wchar_t* wszParamType)

    m_wszParamType = wszParamType;
    m_wszParams = wszParams;


bool FileReader::Execute()

    if (wcscmp(m_wszParamType, L"registry") == 0)
    
        function1();
    
    else
    
        function2();
    
    return true;

【问题讨论】：

也许可以解释为什么你认为你不能，因为不清楚是什么问题

是什么阻碍了您创建另一个在单个调用中执行此操作的函数？

将需要的参数直接传递给Execute()，而不是使用其他函数来设置它们。如果您想将多个 Execute() 调用合并为一个，请更改参数，使其接受多对参数，或接受数组参数。还可以考虑使用标准库类型（如std::wstring，或者，如果传递一组，std::vector<std::wstring>）而不是使用wchar_t 的数组。

老实说，我认为这很好。如果您还有更多要添加的内容，请担心。

【参考方案1】：

如果您的问题是在不同的行中调用具有不同参数的函数，您可以使用std::ref 如下迭代引用包装器的initializer_list 到对象（即std::wstring s )，这减少了一些打字：

#include <functional>        // std::ref
#include <initializer_list>  // std::initializer_list

bool SubscriptionRead::ReadFile()

    IVerification* pReader = new FileReader();  
    // .... other code
    for(auto strKey: std::ref(oemPathPublicKey), std::ref(oemPathSessionKey), std::ref(oemPathSessionKey))
    
        pReader->SetParams((wchar_t*)strKey.c_str(), L"file");
        pReader->Execute(); // does it needed to executed for every path? if no: outside the loop!
    
    return True;

---

还要注意，在现代 C++ 中，您有 smart pointers。因此，请在适当的时候使用它们，并避免手动分配内存。