C++ Почему моя программа бросает исключение?

Question

Я новичок в программировании, и я пытаюсь закодировать функцию, которая получает кратчайшую строку из списка, но каждый раз, когда я ее запускаю, визуальная студия показывает ошибку «Исключение брошено: нарушение прав доступа на чтение». Где ошибка?

#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

Вы указали переменную с именем, которое соответствует типу имени (строковая переменная, тип строки?). Кроме того, есть проблема, что вы возвращаете указатель на объект с локальным охватом жизни. Это 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; 
} 

Answer 2

if (shortest_one = nullptr) не является операцией сравнения. Назначение , которое устанавливает shortest_one в nullptr. Эта операция оценивается как 0, поэтому выражение if эквивалентно if (0), или if (false).

Тогда в else блоке, вы используете shortest_one->size() но shortest_one равно нулю ...

Попробуйте использовать if (shortest_one == nullptr) вместо этого.