Обнуление указателя

[blood_shadow]

Доброго времени суток!
Возник вопрос, не могу дойти до истины. Вызывается ф-ция addNewRecord() она создает объект Day который является диалоговым окном(имеет 3 кнопки ">>" "Ok" "Cancel") результат нажатия пользователя возращается как

Код:

 i = Day->exec(); 

в самом последнем еlse вызывается обработка нажатия "Cancel", так вот вопрос почему строка

Код:

Day = 0;

следующая после удаления объекта вызывает крах программы?

Код:

void Analysis_of_accidents::addNewRecord()
{
    Day = new Day_and_person(SizeForDialogs);
    // Construct structs for Data
    DataRepository = new QStringList;

    int i;
    i = Day->exec();

    if (i) {
        if (i==1) //Ok
        {
            *DataRepository << Day->returnDay_and_personList();

            while (DataRepository->size() < (ColumnCount - 1)) *DataRepository << *emptyString;

            delete Day;
            Day = 0;

            emit StringListIsReady();
        }
        else // >>
        {
            *DataRepository << Day->returnDay_and_personList();

            addNewRecordEmployee();
        }
    }
    else { // Cancel
        delete DataRepository;
        delete Day;
        Day = 0;
        return;
    }

    return;

}

[DOOMer]

а зачем вы пытаетесь присовить нулевое значение уже удаленному указателю?

[Пантер]

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

[lit-uriy]

>>а зачем вы пытаетесь присовить нулевое значение уже удаленному указателю?
Указатель НЕ удалён. Удалён объект, на который указывал указатель. А обнуление указателя - хорошая практика.

[blood_shadow]

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

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

[DOOMer]

а можно тогда поинтересоваться в чем состоит ужастность кода?

Ну вот хотя бы в этом, имхо

Код

C++ (Qt)
if (i) {
        if (i==1) //Ok

Может лучше было бы:

Код

C++ (Qt)
if ( i == QDialog::Accepted )
{
//  if select ОК
}
else
{
// if select Cancel
}

[Пантер]

+1
А еще всякие *DataRepository << *emptyString; не очень красиво.

А дело где-то дальше по коду.

[blood_shadow]

а можно тогда поинтересоваться в чем состоит ужастность кода?

Ну вот хотя бы в этом, имхо

ну вот отредаченая версия:

Код:

        if (i==QDialog::Accepted)
        {
            *DataRepository << Conditions->returnConditionsList();

            while (DataRepository->size() < (ColumnCount - 1)) *DataRepository << *emptyString;

            delete Conditions;
            Conditions = 0;
            delete Day;
            delete Employer;

            emit StringListIsReady();
        }
        else if (i == Day_and_person::Next)
        {
            *DataRepository << Conditions->returnConditionsList();

            addNewRecordPeoples();
        }
        else {
            delete DataRepository;
            delete Conditions;
            Conditions = 0;
            delete Day;
            delete Employer;
            return;
        }

У меня три кнопки поэтому я определил перечисление Day_and_person::Next, которое равно 3, вопрос можно как-то в базовом классе QDialog определить перечисление типа QDialog::Next?

[DOOMer]

Код

C++ (Qt)
    else {
        delete DataRepository;
        delete Conditions;
        Conditions = 0;
        delete Day;
        delete Employer;
        return;
    }

А этот elsre откуда? Покажите уж полный код этой вашей падающей  функции.

IPD  а все, увидел.

[blood_shadow]

+1
А еще всякие *DataRepository << *emptyString; не очень красиво.

А дело где-то дальше по коду.

согласен указатели выглядят в этих выражениях не красиво, а как тогда поступить если мне нужно передать этот DataRepository типа QStringList дальше по программе? или лучше передавать в качестве параметра ф-ций?

[Пантер]

Программа большая? Просто по кусочкам ничего толком не понятно, кроме того, что с программированием у тебя не очень. (Извини, но это так). Если есть желание, пришли мне код на мыло, а я по свободе поковыряю его в поисках ошибки, да и постараюсь поправить код до нормального состояния.

[blood_shadow]

А этот elsre откуда? Покажите уж полный код этой вашей падающей  функции.

IPD  а все, увидел.

вот полная ф-ция

Код:

void Analysis_of_accidents::addNewRecord()
{
    Day = new Day_and_person(SizeForDialogs);
    // Construct structs for Data
    DataRepository = new QStringList;

    int i;
    i = Day->exec();

    if (i==QDialog::Accepted) //Ok
        {
            *DataRepository << Day->returnDay_and_personList();

            while (DataRepository->size() < (ColumnCount - 1)) *DataRepository << *emptyString;

            delete Day;
            Day = 0;

            emit StringListIsReady();
        }
    else if (i==Day_and_person::Next) // >>
        {
            *DataRepository << Day->returnDay_and_personList();

            addNewRecordEmployee();
        }
    else { // Cancel
        delete DataRepository;
        //delete Day;
        //Day = 0;
        return;
    }

    return;

}

Короче было выяснено что падает из-за

Код:

delete Day;

впечатление будто окно раньше удаляется чем заканчивается обработчик событий

[blood_shadow]

Программа большая? Просто по кусочкам ничего толком не понятно, кроме того, что с программированием у тебя не очень. (Извини, но это так). Если есть желание, пришли мне код на мыло, а я по свободе поковыряю его в поисках ошибки, да и постараюсь поправить код до нормального состояния.

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

[Пантер]

И то, и другое. Плюс к этому читать чужие исходники.

[blood_shadow]

И то, и другое. Плюс к этому читать чужие исходники.

пасиба, особенно за чужие исходники - это то что я обычно избегал делать