Как обработать аргументы при вызове функции?

Question

[dvedoski]

Приветствую. Начала изучать C++. Читаю Страуструпа. Заголовочный файл, что прикладывался к книге - не работает. Так что некоторые вопросы приходиться гуглить. К примеру с функцией sort() всё решила сама, с русскими буквами в консоли тоже.
Сейчас возникла проблема решение которой найти ну никак не получается. HELP PLZ!
Пытаюсь обработать входные параметры в функции square(). Если значение отрицательное то должно быть просто некорректный ввод. С положительными всё норм. Если 2 и 5 - возвращает 10. С отрицательными типа работает, но в конце плюсом возвращается значение которое непонятно откуда появляется. Вот код.

int square(int x, int y) {
	if (x <= 0 || y <= 0) {
		cout << "некорректный ввод \n";
	}
	else
	{
		return  x * y; 
	}
}
int main(void)
{
	cout << square(-2, 5);

вывод такой:

некорректный ввод
254589920

------------------------------------------
Откуда берется цифра 254589920 ?оО Скрин есть.

Answer 1

[profesor08]

Вместо cout должен быть return, который будет что-то возвращать, например -1. При вызове функции проверяй, если результат -1, то выводи сообщение об ошибке, если 0 либо больше, то все хорошо. Я бы сделал так либо вообще бросил бы исключение, либо перемножал бы даже отрицательные, возвращая значение по модулю.

int square(int x, int y) {
  if (x < 0 || y < 0) {
    return -1;
  }
  else
  {
    return  x * y; 
  }
}

int main(void)
{
  int result = square(-2, 5);

  if (result == -1) {
    cout << "некорректный ввод \n";
  }
  else {
    count << result;
  }

}

Comments

[dvedoski]

Благодарю! Это решает задачу)
В конце концов всё заработало с таким кодом. Как я того и хотела в самом начале чтобы всё проверялось в теле функции. Но откуда бралось 254589920 непонятно оО. Но это уже не важно)

int square(int x, int y) {
if (x < 0 || y < 0) cout << "incorrect input";
else {
cout << x * y ;
}
return x * y;
}
int main(void)
{
square(-22, 5);

Хотя не... не особо. если писать cout << square(-22, 5); то из потока вывода он подхватывает -110... не суть кароч.

[profesor08]

dvedoski, Это мусор. Компилятор автоматически достроил функцию и добавил недостающий return, который возвращает что-то там, и вот это "что-то там", это и есть мусор из памяти.

Answer 2

[Vitaly]

В первой ветке if у вас функция ничего не возвращает. Возможно, причина в этом. Возвращать можно любое неверное значение для теста, допустим, -1.