логическое значение не изменилось

Question

логическое значение не изменилось

0

Я пытаюсь использовать логическое значение, чтобы сохранить состояние, в котором я находится. В моем заголовочном файле я заявляю:

bool *modified;

В моем конструкторе класса я инициализирую состояние на false:

bool initState = false;
modified = &initState;

Затем у меня есть кнопка, которая изменяет состояние на true:

bool change = true;
modified = &change;

У меня также есть кнопка, чтобы увидеть состояние:

if(!*modified){
    // doing something 
} else{
   // do something else 
}

Проблема в том, что если я действительно изменил состояние, * * измененный будет по-прежнему верным. Я не понимаю, в чем проблема. У кого-нибудь есть идеи, где это.

leam 10 сен. 2014, в 11:10

Источник

3

почему вы используете логический указатель, когда обычный логический будет делать? Ваш фрагмент кода не является полным, чтобы объяснить вашу ошибку
EdChum 10 сен. 2014, в 08:15
1

Являются ли change и initState локальные переменные, случайно?
Andreas Fester 10 сен. 2014, в 08:15
0

Не очень хорошая идея иметь определение modified в заголовке, потому что вы рискуете получить несколько определений, если их содержит более одного исходного файла. Лучше использовать extern для объявления в заголовочном файле и определения один раз в исходном файле.
Andy Brown 10 сен. 2014, в 08:16
0

Я не уверен, почему вы используете указатель для этого. Но, похоже, вы устанавливаете адрес своего указателя на локальные переменные, которые выйдут из области видимости. Как только они выходят из области видимости, их значения могут меняться случайным образом. Так что я думаю, что у вас есть UB (неопределенное поведение).
Galik 10 сен. 2014, в 08:19
1

Если бы у вас был простой логический тип, то присвоение значения локального логического значения сохраняло бы состояние, которое вы усложняли, используя указатель
EdChum 10 сен. 2014, в 08:27
1

Теперь я это понимаю. Сейчас я переключаюсь на обычный бул.
leam 10 сен. 2014, в 08:30

Показать ещё 4 комментария

Теги:

c++

qt

3 ответа

0

Попробуй это.

if (!(*modified))
else


bool change = true;
*modified = change;

bool initState = false;
*modified = initState;

Lee 10 сен. 2014, в 07:10

0

попробуйте этот код bool * modified;

bool initState = false;
modified = initState;   false

bool change = true;  
modified = change; true

if ( *modified != initState)
{
do something
}
else 
{
do something
}

Mohamed Ayadi 10 сен. 2014, в 06:34

Ещё вопросы

почему вы используете логический указатель, когда обычный логический будет делать? Ваш фрагмент кода не является полным, чтобы объяснить вашу ошибку
Являются ли change и initState локальные переменные, случайно?
Не очень хорошая идея иметь определение modified в заголовке, потому что вы рискуете получить несколько определений, если их содержит более одного исходного файла. Лучше использовать extern для объявления в заголовочном файле и определения один раз в исходном файле.
Я не уверен, почему вы используете указатель для этого. Но, похоже, вы устанавливаете адрес своего указателя на локальные переменные, которые выйдут из области видимости. Как только они выходят из области видимости, их значения могут меняться случайным образом. Так что я думаю, что у вас есть UB (неопределенное поведение).
Если бы у вас был простой логический тип, то присвоение значения локального логического значения сохраняло бы состояние, которое вы усложняли, используя указатель
Теперь я это понимаю. Сейчас я переключаюсь на обычный бул.

TartanLlama · Accepted Answer · 2014-09-10T06-30-00.000Z

Вы изменяете значение modified, которое является bool*, и, возможно, указывать на данные, выделенные стеком, которые будут уничтожены при возврате функции. Вы хотите изменить значение, на которое указывает modified.

bool initState = false;
*modified = initState;

bool change = true;
*modified = change;

Предполагается, что modified указатель фактически размещен где-то.

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