Странные ошибки доступа к структуре

Question

Странные ошибки доступа к структуре

0

В основном у меня есть этот код:

#include <iostream>
using namespace std;

    struct foo{
        string *question;
        //other elements
    };

    int main(){
        foo *q;
        foo *q2;
        q->question = new string("This is a question"); 
        //q2->question = new string("another question");
    }

и когда я раскомментирую q2->question = new string("another question"); это ошибки, и я понятия не имею, почему.

Обновление: ошибка представляет собой сообщение Windows, в котором говорится, что программа перестала работать, и она печатает Process exited with return value 3221225477

Gibby 27 дек. 2013, в 22:04

Источник

0

Какие ошибки вы получаете?
0x499602D2 27 дек. 2013, в 20:29
0

Процесс завершен с возвращаемым значением 3221225477
Gibby 27 дек. 2013, в 20:30
4

Вы не инициализируете эти указатели!
Mat 27 дек. 2013, в 20:30
0

Я бы посоветовал вам повысить уровень предупреждения: coliru.stacked-crooked.com/a/19708e563bc6b00a . Стыдно, ничего не сказано о пропавшем заголовке.
chris 27 дек. 2013, в 20:31
0

Почему тогда он компилируется с одной строкой, а не с обеими?
Gibby 27 дек. 2013, в 20:32
0

Запустите вашу программу через отладчик .
Lightness Races in Orbit 27 дек. 2013, в 20:32
0

@chris: в C ++ 11 iostream почти наверняка содержит string .
Lightness Races in Orbit 27 дек. 2013, в 20:33
0

@Gibby Gibby, он компилируется с обоими, но вылетает в вашем случае.
chris 27 дек. 2013, в 20:34
0

Спасибо, это было глупо с моей стороны, это не позволит мне принять ответ, но на него дан ответ.
Gibby 27 дек. 2013, в 20:34
1

@LightnessRacesinOrbit, а? Клянусь, в последнее время некоторые реализации перестали включать, в том числе <string> в <iostream> .
chris 27 дек. 2013, в 20:35

Показать ещё 8 комментариев

Теги:

c++

struct

3 ответа

1

foo *q;
foo *q2;

Вы получаете Undefined Behavior при использовании косвенности на этих указателях, потому что эти указатели не были инициализированы (т.е. они не указывают на действительный адрес).

Вам нужно указать, что они указывают на объект:

foo* q  = new foo();
foo* q2 = new foo();

// don't forget to delete

или просто использовать распределение стека:

foo q;
foo q2;

q.question = "This is a question";

0x499602D2 27 дек. 2013, в 19:27

1

Вы не выделили память для q и q2.

замещать

foo *q;
foo *q2;

с

foo *q = new foo();
foo *q2 = new foo();

В конце удалите все созданные объекты:

delete q->question;
delete q;
delete q2->question;
delete q2;

Joritan 27 дек. 2013, в 18:46

Ещё вопросы

Процесс завершен с возвращаемым значением 3221225477
Вы не инициализируете эти указатели!
Я бы посоветовал вам повысить уровень предупреждения: coliru.stacked-crooked.com/a/19708e563bc6b00a . Стыдно, ничего не сказано о пропавшем заголовке.
Почему тогда он компилируется с одной строкой, а не с обеими?
Запустите вашу программу через отладчик .
@chris: в C ++ 11 iostream почти наверняка содержит string .
@Gibby Gibby, он компилируется с обоими, но вылетает в вашем случае.
Спасибо, это было глупо с моей стороны, это не позволит мне принять ответ, но на него дан ответ.
@LightnessRacesinOrbit, а? Клянусь, в последнее время некоторые реализации перестали включать, в том числе <string> в <iostream> .

Alexander Kondratskiy · Accepted Answer · 2013-12-27T17-58-00.000Z

Ваши q и q2 указатели не инициализированы. Вы пытаетесь получить доступ к памяти, которую вы не выделили. В краткосрочной перспективе исправить:

int main(){
    foo *q = new foo;
    foo *q2 = new foo;
    q->question = new string("This is a question"); 
    q2->question = new string("another question");

    // don't forget to release the memory you allocate!
    delete q->question;
    delete q2->question;
    delete q;
    delete q2;

}

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

int main(){
    string q1 = "This is a question";
    string q2 = "another question";
}