C++ как исправить ошибки в простейшем коде?

Question

[Tim Tatman]

Добрый вечер господа. С сегодняшнего дня начал постигать основы программирования по книге Р. Лафоре, ООП C++ и тут же возникли проблемы на первых шагах в написании кода. Сам компилятор при отладке ошибок не показывает, но при запуске программы я получаю не тот результат которые описывается в книге, возможно я где то ошибся. Прошу помощи) огромное желание освоить программирование, но без помощи опытных и закаленных профи, чувствую не обойдусь, заранее спасибо

#include <iostream>
using namespace std;
int main()
{
	int ftemp; 
	cout << "Fahrenheit temperature: 212 ";
	cin >> ftemp;
	int ctemp = (ftemp-32)*5 / 9;
	cout << "Celsius temperature " << ctemp << endl;
	return 0;
}

И так же возникла проблема с символьными константами

#include <iostream>
using namespace std;
int main()
{
	char charvar1 = 'A ';
	char charvar2 = '\t ';
	cout <<charvar1;
	cout <<charvar2;
	charvar1 = 'B';
	cout <<charvar1;
	cout <<'\n '; 
	return 0;
}

Answer 1

[tsarevfs]

Значения температуры могут получаться не правильными из-за округления при делении. int хранит только целые числа, поэтому при делении на 9 дробная часть отбрасывается. Возможные варианты исправления:

double ctemp = (double)(ftemp - 32) * 5 / 9; //явно привести тип
double ctemp = static_cast<double>(ftemp - 32) * 5 / 9; //C++ style cast - безопаснее первого (ругнется если вы пытаетесь привести что-то не то)
double ctemp = (ftemp - 32) * 5 / 9.0; //делить на константу типа double

Во втором примере char представляет только один символ (в отличие от строки). У вас же 2 вместе с пробелом.

char charvar1 = 'A '; //пробел после A
char charvar2 = '\t '; //пробел после \t
charvar1 = 'B'; //OK
cout <<'\n '; //пробел после \n

При запуске из студии консоль с выводом программы закрывается сразу же при завершении программы. Вы можете не успеть ничего увидеть.

#include <cstdlib>
...
system("pause"); //перед return 0;

Comments

[Tim Tatman]

Изменений никаких не произошло. Я вас правильно понял?

#include <iostream>
using namespace std;
int main()
{
	int ftemp; 
	cout << "Fahrenheit temperature: 212 ";
	cin >> ftemp;
	double ctemp = (ftemp - 32) * 5 / 9.0;
	cout << "Celsius temperature " << ctemp << endl;
	return 0;
}

[Армянское Радио]

Какой интересный пример. Вы приводите значение температуры к float, а потом сохраняете в double. И да, это же C++. Тут должен быть static_cast(ftemp-32), а не ваш привет из C.

[tsarevfs]

Армянское Радио: Ну да, в скобках double конечно. Во втором варианте все правильно. Для элементарных типов c-style приведение мне кажется вполне допустимо (и гораздо короче).

[tsarevfs]

maybeivan: А какой вывод есть/ожидается?

[Tim Tatman]

tsarevfs: вывод простой Fahrenheit temperature: 212 и все , хотя ниже еще должно быть Celsius temperature 100, но его как такового там нет...

[tsarevfs]

maybeivan: На самом деле есть. Прочитайте последнюю часть моего ответа.

[Mercury13]

system("pause"); — пушка для муравья! Я бы сделал это так.

cin.sync(); // сбросить буфер
cin.ignore(1); // считать байт в никуда
cin.sync(); // снова сбросить

[tsarevfs]

Mercury13: А это точно будет всегда работать? Если перед этим считать int, а ввести букву, то паузы нет (предположим, при неправильном вводе хотим написать сообщение об этом).

[Mercury13]

tsarevfs: И верно, нашли проруху.

[Tim Tatman]

tsarevfs: Вот код,

#include <iostream>
using namespace std;
int main()
{
	int ftemp;
	cout << "Farhenheit temperature: 212";
	cin >> ftemp;
	double ctemp = static_cast<double>(ftemp - 32) * 5 / 9;
	cout <<"Celsius temperature" <<ctemp<< '\n';
	return 0;
}

У меня в студии, все стабильно, ничего не закрывается, консоль висит вот такая

[marked-one]

maybeivan: введите с клавиатуры пару цифр, нажмите "Enter" и прочитайте еще раз, что такое cin :-)

Answer 2

[Армянское Радио]

Уберите дорой русские буквы из имен файлов.
Если вы списали пример со "\n" вместо endl в качестве окончания строки из учебника, учебник - на помойку.

Одинарные кавычки должны окружать ровно один символ 'a' или спецсимвол '\n'
У вас же за каким-то чертом там еще и пробел завелся, перед закрывающей кавычкой.

Comments

[Tim Tatman]

Не могли бы вы подсказать, хорошую литературу по программированию с 0?

[Армянское Радио]

maybeivan: Нет такой. C++ можно считать изученным только после достижения понимания. Эту задачу вам ни одна книжка не решит.

[Tim Tatman]

Армянское Радио: Тогда каким образом достигать этого понимания? Путем практики? Но если нет теории, о чем можно говорить вообще?

[Армянское Радио]

maybeivan: Это означает, что можно брать любой учебник для начала и учится по нему. Потом взять еще один учебник, понять, что первый кривой и учится еще раз. Потом прочитать Голуба, понять, что первые два учебника - помои, но рациональное зерно в них есть.

Подчеркиваю, Голуба надо читать после того, как вопрос "исключения, оставленные в деструкторе" будет вызывать у вас не "ой что это?!" а "какая скука, сколько можно объяснять"

Потом наработать пару пятилеток стажа, понять, что Голуб местами кривит душой. Написать свой учебник.

[Tim Tatman]

Армянское Радио: Большое спасибо, за подробный ответ)

[tsarevfs]

Армянское Радио: maybeivan: Судя по отзывам, книга достаточно приличная. Использовать std::endl везде не лучший совет. Нужно понимать, что вывод std::endl дополнительно делает flush потоку (непосредственно отрисовка на экране или запись в файл). Если это не требуется, например при записи 1000000 строк в файл, то запись '\n' может быть на порядки быстрее.

[Армянское Радио]

tsarevfs: Именно в том месте у автора FLUSH сделать стоит.
Пример с endl хорош своей надежностью, вероятность непоняток с ним меньше, чем с ''\n''. В сноске в углу можно написать про производительность. Впрочем, у streams она и так хромая: писать миллионы строк в файл лучше и вовсе без них.

[tsarevfs]

Армянское Радио: В первоначальном виде, без паузы в конце не нужно. Деструктор std::cout сам сделает flush. Если запускать из под консоли все будет ок.

[Don Kaban]

1. Что именно "не так" выводит пример, вот что надо спрашивать.
2. Советовать std::endl вместо "\n" - это примерно то же самое, что советовать std::shared_ptr вместо сырого указателя. Можно, но не стоит. Стоит, но не всегда. И все такое. Сами же понимаете.
3. Читать надо Страутрупа, режьте меня, но это остается фактом. Сначала основную книжку, потом "Дизайн и эволюция".
4. Изучать (нелепое слово "учить") надо актуальный стандарт, енстественно, он проще.

[Армянское Радио]

Don Kaban: endl на первых порах дает меньше риска расстрела собственных ног.

[Don Kaban]

Не очень понимаю. На первых порах, в принципе разницы нет. Приведите юзкейс выстрела в ногу с \n?

[Армянское Радио]

Don Kaban: использовать его для вывода отладочной информации перед падением.
cout << "возможно сейчас упаду, баюс баюс!\n";
/0; //образно конечно же
программа упала, но деструктора не было, строка не вывелась, программист - в непонятках.

[tsarevfs]

Don Kaban: Думаю речь идет о "Я печатаю, а оно не печатается!".

[Don Kaban]

А. Ну ок, согласен. Есть правда подозрение, что ошибку этот предполагаемый программист будет выводить с cerr :) Но аргумент понятен.

[Армянское Радио]

Don Kaban: принципиальной разницы не будет между cerr и cout

[Don Kaban]

cerr unbuffered же, разве нет? Пошел гуглить

[Don Kaban]

ideone.com/sQs6kO - похоже, что да

[Армянское Радио]

Don Kaban: ваша правда, у cerr flush делается на каждый чих.

Answer 3

[kstyle]

одинарные кавычки на двойные замените (там где \n)

Comments

[Don Kaban]

Отчего же? Можете обосновать? :)