C++ за 21 день седьмое издание

char UserInput = 'y';

0. 
   cin » Userlnput;
   

1 1 :

0. 
   if (Userlnput== 'у ')
   

0. 
   {
   

0. 
   pTemperature = new int;
   

0. 
   *pTemperature = 30;
   

0. 
   }
   

17:

0. 
   // pTemperature содержит недопустимое значение,
   

1. 
   если пользователь ввел 'п '
   

188 ЗАНЯТИЕ 8. Указатели и ссылки

19: cout « "Temperature is: " « *pTemperature;


2 0 :

0. 
   // оператор delete также может быть вызван для указателя
   

1. 
   без применения оператора new
   

0. 
   delete pTemperature;
   

23:

0. 
   return 0;
   

0. 
   }
   

Результат


Is it sunny (y/n)? у


Temperature is: 30

Второй запуск:

Is it sunny (y/n)? n


<АВАРИЯ!>


Анализ

1. 
   программе много проблем, некоторые из которых уже прокомментированы в коде. Обратите внимание, что память в строке 14 резервируется и ее адрес присваивается ука­ зателю в блоке i f только тогда, когда его условие выполняется, т.е. когда пользователь, соглашаясь, нажимает клавишу <у>. При любом другом нажатии пользователем этот блок
   

0. 
   f не выполняется, и указатель p T e m p e ra tu re остается недопустимым. Таким образом, когда во второй раз пользователь нажимает клавишу <п>, приложение завершается отка­ зом, поскольку указатель p T e m p e ra tu re содержит недопустимый адрес области памяти и обращение к значению по недопустимому указателю в строке 19 создает проблему.
   

Аналогично вызов оператора d e l e t e в строке 22 для этого указателя, который не был инициализирован оператором new, также является неправильным. Обратите внимание на то, что, если у вас есть копия указателя, вызов оператора d e l e t e необходим только для одного из них (наличие неконтролируемых копий указателей также следует избегать).
Лучшая (более безопасная и стабильная) версия программы из листинга 8.13 подразу­ мевала бы инициализацию указателей, проверку допустимости их значений и освобожде­ ние только однажды и только когда допустимо.