Question

这可能是一个非常愚蠢的问题，但是如何优化这些代码以使其更高效（更快，更少内存消耗）？我制作了这段代码来帮助我排序一些文本文件。它从第一个文件中读取每个字符串，然后搜索第二个文件，直到找到所有相关的字符串，在第三个文件中，它会写入一些匹配的字符串。这是代码：

ifstream h("SecondFile.txt");
ifstream h2("FirstFile.txt");
ifstream uh("MatchedStrings.txt");
ofstream g("sorted.txt");    
int main()
    {
        string x, y, z;
        cout << "Sorting..." << endl;;
        while (!h.eof()){
            h >> x;
            while (!h2.eof() || (y == x)){
                h2 >> y;
                uh >> z;
                if (y == x){
                    g << z << endl;
                    break;
                    h2.clear();
                    h2.seekg(0);
                    uh.clear();
                    uh.seekg(0);
                }
            }
            if (h2.eof() && (y != x)){
                g << "none" << endl;
                h2.clear();
                h2.seekg(0);
                uh.clear();
                uh.seekg(0);
            }
        }
        cout << "Finished!";
    }

我已将代码更改为：

#include <iostream>
#include <fstream>
#include <string>

using namespace std;
ifstream h("SecondFile.txt");
ifstream h2("FirstFile.txt");
ifstream uh("MatchedStrings.txt");
ofstream g("sorted.txt");

int main()
{
    string x;
    bool write_none = true;
    int i = 0,l=0;
    string check[] = {""};
    string unhashed_checked[] = { "" };
    string sorted_array[] = { "" };
    cout << "Sorting..." << endl;
    //Get to memory
    while (!h2.eof())
    {
        h2 >> check[i];
        uh >> unhashed_checked[i];
        i++;
    }

    while (!h.eof()){
        h >> x;
        write_none = true;
        for (int t = 0; t <= i;t++)
        {
            if (x == check[t])
            {
                break;
                write_none = false;
                sorted_array[l] = unhashed_checked[i];
                l++;
            }
        }
        if (write_none)
        {
            sorted_array[l] = "none";
            l++;
        }
    }
    for (int k = 0; k <= l; k++)
    {
        g << sorted_array[k]<<endl;
    }
    cout << "Finished!";
}

但是我在运行程序时遇到了这个异常：

Unhandled exception at 0x01068FF6 in ConsoleApplication1.exe: 0xC0000005: Access violation writing location 0xCCCCCCCC

Answer 1

在字符串向量中加载h2，并通过将每个字符串与向量的内容进行比较，循环遍历h。

由于您的测试是对称的，因此您可以选择std::set作为两个文件中最小的一个。这样，您可以节省内存和时间，特别是如果其中一个文件比另一个文件大得多。如果比较花费了大量时间，使用集合（callback google.search.ImageSearch.RawCompletion('1', null, 403, 'This API is no longer available.', 200)）而不是向量也会有所帮助。

Answer 2

假设文件中的字符串数分别为 n 和 m 。

你现在的方式，复杂性是Θ（n m）。而且，复杂性常量是文件操作的常量，它们非常慢。

相反，您应该只将其中一个文件读入std::unordered_*容器，然后比较容器之间的密钥。这应该将运行时间缩短到预期的Θ（n + m）。

作为旁注，您可能需要查看more modern ways to read strings into containers（使用例如std::istream_iterator）。

优化.txt文件中的字符串搜索

2 个答案: