Question

我是编程新手，我尝试编写一个从列表中获取最短字符串的函数，但每次运行它时，visual studio都会显示错误＆＃34;抛出异常：read访问违规＆＃34;。错误在哪里？

#include <iostream>
#include <string>
using namespace std;

const string &shortest_string(initializer_list<string> strings) {
    string *shortest_one = nullptr;
    for (string string : strings) {
        if (shortest_one == nullptr) shortest_one = &string;
        else {
            if (string.size() < shortest_one->size()) shortest_one = &string;
        }
    }
    return *shortest_one;
}

int main() {
    cout << shortest_string({ "hello" , "my", "name", "is", "dan" }) << endl;
    return 0;
}

Answer 1

if (shortest_one = nullptr)不是比较操作。这是分配，将shortest_one设置为nullptr。此操作的计算结果为0，因此if表达式等同于if (0)或if (false)。

然后在else块中，您使用shortest_one->size()但shortest_one为空...

尝试改为使用if (shortest_one == nullptr)。

Answer 2

您创建了名称与类型名称匹配的变量（字符串变量，字符串类型？）。此外，还有一个问题是您返回指向具有本地生活范围的对象的指针。那是UB。使用迭代器，您的函数将如下工作：

const string shortest_string(initializer_list<string> strings) {
    if(!strings.size()) return string();
    auto shortest_one = strings.begin();
    for (auto it = shortest_one+1; it < strings.end(); it++   ) 
    {
        shortest_one = (it->size()< shortest_one->size()) ? it : shortest_one;
    }

    return *shortest_one;
}

C ++为什么我的程序会抛出异常？

2 个答案: