Работа с доступом к нулевому указателю

Question

Работа с доступом к нулевому указателю

0

В моем классе у меня есть, среди прочего, указатель:

Class GSM
{
//...
private:
    char *Pin;
//...
}

Мой конструктор инициализирует его как:

GSM::GSM()
{
//...
    Pin = NULL; 
//...
}

Теперь я хочу установить значение по умолчанию ("1234") для моего ПИН-кода. Я пробовал очень простой способ:

bool GSM::setDefaultValue()
{
    lock();
    Pin = "0";
    for (uint8 i =0; i < 4; ++i)
    {
        Pin[i] = i+1;
    }
    unlock();
    return true;
}

Но это не сработало. Когда я запускаю свою программу (я использую Visual Studio 2010), возникает ошибка:

Access violation writing location 0x005011d8

Я попытался удалить строку

Pin = "0";

Но это не помогло. Я должен инициализировать его как NULL в конструкторе. Это часть более крупного проекта, но я думаю, что код выше, что заставляет меня беспокоиться. Я все еще довольно новичок в C++/ООП, и иногда меня все еще путают указатели.

Что я должен сделать, чтобы улучшить свой код и то, как я думаю?
EDIT: по запросу я должен добавить, что я не могу использовать std :: string. Я стажер в компании, проект довольно большой (например, тысячи файлов), и я не видел никакого std здесь, и мне не разрешено использовать его.

Photon Light 17 сен. 2014, в 09:49

Источник

4

Вам нужно сделать так, чтобы Pin указывал на массив char вы можете писать. Кстати, это не имеет абсолютно никакого отношения к ООП.
juanchopanza 17 сен. 2014, в 07:17
1

Почему вы игнорируете предупреждения вашего компилятора и переходите прямо к переполнению стека? Существует множество автоматических средств диагностики, которые могут помочь вам, не требуя внимания человека.
Kerrek SB 17 сен. 2014, в 07:19
2

Почему вы не используете std::string или boost::optional<std::string> если вам нужно отличить отсутствие строки от пустой строки? В любом случае ... если вы хотите использовать указатель, вам нужно указать ему какой-то new -ed - или, возможно, массив символов члена класса - прежде чем пытаться перезаписать его.
Tony Delroy 17 сен. 2014, в 07:19
0

Я не использую std :: string, потому что мне не разрешено. Вся программа построена без нее, и она намного больше (например, ~ 700 Мб), и начальство (я студент-стажер, банкомат) сказал, что не следует использовать std :: string. Если кто-то испытывает отвращение к не очень профессиональному вопросу, можете проголосовать за него. Мне нравится говорить с человеком, когда я не могу понять компилятор, потому что я тоже человек.
Photon Light 17 сен. 2014, в 07:30
1

Это нормально для обучения. Существуют также системы, в которых в любом случае невозможно использовать стандартную библиотеку. Но вы можете добавить эту информацию к вопросу, чтобы прояснить ситуацию.
juanchopanza 17 сен. 2014, в 07:32
1

@Kappa: Тогда спросите , какой тип строки они используют. Для проектов, которые существуют уже 20 лет (что не редкость для C ++), возможно, что они могут использовать свой собственный класс String . Но если они не используют какой-либо строковый класс, вы должны узнать из этого назначения, что не нужно делать.
MSalters 17 сен. 2014, в 08:23

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

Теги:

c++

pointers

oop

nullpointerexception

null

3 ответа

Ещё вопросы

Вам нужно сделать так, чтобы Pin указывал на массив char вы можете писать. Кстати, это не имеет абсолютно никакого отношения к ООП.
Почему вы игнорируете предупреждения вашего компилятора и переходите прямо к переполнению стека? Существует множество автоматических средств диагностики, которые могут помочь вам, не требуя внимания человека.
Почему вы не используете std::string или boost::optional<std::string> если вам нужно отличить отсутствие строки от пустой строки? В любом случае ... если вы хотите использовать указатель, вам нужно указать ему какой-то new -ed - или, возможно, массив символов члена класса - прежде чем пытаться перезаписать его.
Я не использую std :: string, потому что мне не разрешено. Вся программа построена без нее, и она намного больше (например, ~ 700 Мб), и начальство (я студент-стажер, банкомат) сказал, что не следует использовать std :: string. Если кто-то испытывает отвращение к не очень профессиональному вопросу, можете проголосовать за него. Мне нравится говорить с человеком, когда я не могу понять компилятор, потому что я тоже человек.
Это нормально для обучения. Существуют также системы, в которых в любом случае невозможно использовать стандартную библиотеку. Но вы можете добавить эту информацию к вопросу, чтобы прояснить ситуацию.
@Kappa: Тогда спросите , какой тип строки они используют. Для проектов, которые существуют уже 20 лет (что не редкость для C ++), возможно, что они могут использовать свой собственный класс String . Но если они не используют какой-либо строковый класс, вы должны узнать из этого назначения, что не нужно делать.

Mats Petersson · Answer 1 · 2014-09-17T06-21-00.000Z

Вы должны дать Pin память. Что-то вроде этого:

Pin = new char[5];  // To make space for terminating '\0';
for(...)
{
   Pin[i] = '0' + i + 1;
}
Pin[4] = '\0';      // End of the string so we can use it as a string.

...

Затем вы должны использовать delete [] Pin; где-то тоже (как правило, в деструкторе класса, но в зависимости от того, как он используется, он может понадобиться в другом месте, например, оператор присваивания, и вам также нужно написать конструктор-копию, см. правило три).

В правильном C++ вы должны использовать std::string вместо этого, и тогда вы можете:

Class GSM
{
//...
private:
    std::string Pin;
....

Pin = "0000";
for (uint8 i =0; i < 4; ++i)
{
    Pin[i] += i+1;
}

Использование std::string позволяет избежать большинства проблем выделения/освобождения памяти и "просто работает" при копировании, назначении или уничтожении класса, поскольку реализация std::string и компилятор выполняет эту работу за вас.

delete должно появиться в деструкторе, не так ли? Я полагаю, что это может быть и в других случаях; до тех пор, пока указатель ( Pin ) установлен в нуль после освобождения памяти, это не вызовет никаких проблем.

GHugo · Answer 2 · 2014-09-17T04-40-00.000Z

1

Вам нужно выделить блок памяти для хранения "1234". Этот блок памяти будет указываться указателем Pin. Можешь попробовать:

bool GSM::setDefaultValue()
{
    lock();
    Pin = new char[4];
    for (uint8 i =0; i < 4; ++i)
    {
        Pin[i] = '0' + (i + 1);
    }
    unlock();
    return true;
}

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

GSM::~GSM()
{
    delete [] Pin;
}

GHugo 17 сен. 2014, в 04:40

0

почему я + 1? я уже увеличен
sajas 17 сен. 2014, в 07:21
0

Потому что он хочет «1234», а не «0123». Вы также можете изменить цикл for для достижения того же.
GHugo 17 сен. 2014, в 07:22
1

Вы должны добавить примечание, что класс теперь требует деструктора. Также может быть лучше выделить дополнительный символ, чтобы строка могла завершаться нулем, а затем завершать ее нулем.
Jonathan Leffler 17 сен. 2014, в 07:24
0

Для деструктора я согласен, для строки, если ему нужно только сохранить 4 символа и он проверит проверку правильности ПИН-кода для этих 4 символов, то я думаю, что строка с нулевым символом в конце не нужна (он не сказал, что ему нужно хранить строку , поэтому я предполагаю, что массив char достаточно).
GHugo 17 сен. 2014, в 07:27
0

Нет необходимости добавлять if (Pin != NULL) - удаление гарантированно обрабатывает указатели NULL. В этом случае это может не иметь большого значения, но если у вас есть 1000 классов, которые все это делают, это еще 2-3000 инструкций, которые не нужны (потому что внутри самого delete выполняется такая же проверка)
Mats Petersson 17 сен. 2014, в 07:43
0

Вам не нужно проверять ненулевое значение Pin ; Вы можете безопасно удалить нулевой указатель (см. Безопасно ли удалять нулевой указатель? ). Хотя я признаю, что в вопросе не говорилось, что нужна строка с нулевым символом в конце, я думаю, что было бы разумнее предположить, что она нужна. Тем не менее, это ваш ответ.
Jonathan Leffler 17 сен. 2014, в 07:43
0

Не знал для проверки удаления указатель NULL. Я отредактировал свой ответ.
GHugo 17 сен. 2014, в 07:48
0

Пока вы в этом, у вашего класса теперь есть конструктор копирования по умолчанию, который делает неправильные вещи. Вы можете захотеть `= удалить` его, а также удалить оператор присваивания по той же причине (или реализовать его, конечно). std::string просто работает.
MSalters 17 сен. 2014, в 08:25

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

Baldrickk · Answer 3 · 2014-09-17T07-38-00.000Z

Простой ответ:

Вместо того, чтобы использовать кучу (new delete), просто выделите место в своем классе для четырехзначного вывода:

Class GSM
{
//...
private:
    char Pin[5];
//...
}

Длина фиксируется в 5 (чтобы разрешить пробел для 4 символов и завершение null ('\0'), но до тех пор, пока вам нужно хранить максимум 4 символа, вы в порядке.

Конечно, если вы хотите, чтобы это было легко изменить в будущем:

Class GSM
{
//...
private:
    const int pin_length = 4;
    char Pin[pin_length+1];
//...
}

Ваша функция для установки значения будет выглядеть так:

bool GSM::setDefaultValue()
{
    lock();
    for (char i = 0; i < pin_length; ++i)
    {
        Pin[i] = i+1;
    }
    Pin[pin_length]=0;
    unlock();
    return true;
}

или даже:

bool GSM::setDefaultValue()
{
    lock();
    strcpy(Pin,"1234");  //though you would have to change this if the pin-length changes.
    unlock();
    return true;
}

const int будет лучше, чем определение препроцессора, учитывая отсутствие области видимости с последним. Вы оставили место для NUL (то есть \0 а не /0 ), но не добавили его в setDefaultValue() .
@TonyD Спасибо, я обновил свой ответ.