Работа со ссылочными переменными в C ++

У этого кода много проблем, кроме как в изображении (!!!)

Наверное, вы пытаетесь найти элемент в позиции pos-1 в списке или что-то в этом роде. Основная проблема, связанная с вашим вопросом, состоит в том, что вы сначала назначаете val по значению, тогда у вас нет ссылки на return. Вы должны сразу вернуть значение n2->value, которое должно быть ссылкой на unsigned int, например:

const unsigned int &list::operator[](unsigned int pos) const
{
    node *n1 = ???, *n2 = ???;

    for (unsigned int k = 0; k < _size; k++)
    {
        if (k == pos)
            return n2->value;

        n1 = n2->next;
        n2 = n1;
    }

    return ???;
}

Остаются проблемы, например

• почему вам нужен два node* а не один (поиск позиции pos-1 напрямую)

• как инициализировать n1, n2 (как-то указывая на заголовок вашего списка, очевидно, new node() не должен работать)

• что вернуть, если входной аргумент pos находится вне допустимого диапазона (возможно, верните ссылку на какую-либо статическую переменную, которую вы можете обнаружить или выбросить исключение)

Для этих проблем вам понадобится больше контекста с вашей стороны.

Я собираюсь угадать.

Foo& doStuff()
{
    // blah blah
    Foo val;
    // ...
    return val;
    // val is no longer valid end of scope. Returning invalid reference.
}

Либо передайте в результате экземпляр Foo на doStuff, либо создайте new Foo в куче и верните в качестве указателя.

Так,

void doStuff(Foo& val)
{
    // blah blah
    // ...
    val = x;
}

или

Foo* doStuff()
{
    // blah blah
    Foo* val = new Foo;  // dont forget to delete
    // ...
    return val;
}

Конечно, вы можете вернуться по значению:

Foo doStuff()
{
    // blah blah
    Foo val;
    // ...
    return val;
}

В зависимости от того, насколько тяжелен Foo. Конечно, так как в этом случае Foo - это просто небольшой int, вы должны просто вернуться по значению. Для некоторых случаев возврата по значению для больших/нетривиальных типов создается временная копия (в тех случаях, когда отсутствует копирование с помощью RVO или NRVO); в этих случаях вам может потребоваться избегать возврата больших типов объектов по значению.

Справочные переменные действительны только в том случае, если объект, к которому относится ссылка, существует в памяти. Передача ссылок на переменную вне области видимости считается неопределенным поведением. Это ошибка в вашем коде. Пожалуйста, исправьте это.

const unsigned int& list::operator[] (unsigned int pos)const
{
const unsigned int val = 0;

return val; //this is a local variable, whose scope ends here, a reference to this should not be returned
}

Это предупреждение компилятора для вашего кода.

warning: reference to local variable ‘val returned [enabled by default]

Пожалуйста, прислушайтесь к предупреждениям компилятора (особенно c/c++ !!), в вашем случае просто используя pass by value, было бы достаточно.

Редактировать:

В случае, если возвращаемая переменная используется как ссылочный тип, и ее нельзя избежать, вы можете продлить жизнь вашей локальной переменной на протяжении всего существования программы, сделав ее статической.

const unsigned int& list::operator[] (unsigned int pos)const
{
static const unsigned int val = 0;

return val; 
}

Th переменная val теперь статическая локальная переменная, жизнь которой во всей программе, поэтому обход вокруг ссылок на эту переменную должен быть в порядке, но не рекомендуется программировать, так как пропуск по значению будет достаточным для нужд вашего приложения.