В чём ошибка? - C++

@Kranxx · Регистрация: 16.04.2013

C++

#include <std_lib_facilities.h>
int func1(int a)
{
    int chisla[10];
    for (int i=0; i<10; i++)
    {
        cin >> chisla[i];
        if (chisla[i] > chisla[i-1])
            a=chisla[i];
    }
    return a;
}
int main()
{
    setlocale(0, "");
    int b=0;
    func1(b);
    cout << "Самое большое число: " << b << endl;
    system("pause");
    return 0;
}

Всегда выдаёт 0 на выходе. В чём ошибка?
Я полагаю, что это происходит из-за присвоении нуля переменной b. Если не присвоить переменной b значение, то выдаёт ошибку о не инициализированной переменной b. Как исправить?

@Catstail · 19.04.2013, 18:21

Ошибок куча. Вот правильный код:

C++

#include <iostream.h>
 
int func1()
{
    int chisla[10],a;
 
    for (int i=0; i<10; i++)
    {
        cin >> chisla[i];
        if (i == 0)
            a=chisla[0];
        else
            if (chisla[i] > a) a=chisla[i];
    }
    return a;
}
int main()
{
    setlocale(0, "");
    int b=0;
    b=func1();
    cout << "Самое большое число: " << b << endl;
    system("pause");
    return 0;
}

Добавлено через 4 минуты
А если конкретно, то:

1) параметр у func1 не нужен. a лучше сделать локальной переменной (хотя это - не ошибка)
2) вызов func1 нужно выполнять так: b=func1() ведь при передаче по значению функция не может изменить фактический параметр из вызывающей функции
3) каждое введенное число нужно сравнивать не с предыдущим (кстати, на первом витке у тебя будет сравнение с -1-м элементом

), а с текущим максимумом.

@Kranxx · 19.04.2013, 18:27 **[ТС]**

Спасибо, всё понял)

@zss · 19.04.2013, 19:44

Гораздо правильнее в функцию выносить некое осмысленное действо:

C++

// Нахождение максимума в массиве chisla размером n
int MyMax(int* chisla,int n)
{
    int a=chisla[0];
    for (int i=1; i<n; i++)
        if (chisla[i] > chisla[i-1])
            a=chisla[i];
    return a;
}
int main()
{
    setlocale(0, "");
    int b=0;
    const int SIZE=10;
    int chisla[SIZE];
// заполнение массива
    for (int i=0; i<SIZE; i++)
        cin >> chisla[i];
// вызов функции нахождения максимума
    b=MyMax(chisla,SIZE);
    cout << "Самое большое число: " << b << endl;
    system("pause");
    return 0;
}

@Catstail · 19.04.2013, 20:23

Сообщение от zss

Гораздо правильнее

- будет не ограничиваться массивом в 10 элементов, а читать данные из потока до EOF. Массив вообще не нужен.

@zss · 19.04.2013, 21:29

Сообщение от Catstail

читать данные из потока до EOF

А что такое EOF для потока cin?

@Catstail · 19.04.2013, 21:39

Сообщение от zss

А что такое EOF для потока cin?

- это CTRL-Z + Enter

@Kranxx · 19.04.2013, 21:49 **[ТС]**

В этом случае мне не надо было добиваться максимально правильного эффекта, я эксперементировал с массивом и функцией, чтобы лучше понять их в действии, но всё равно спасибо.

@Kranxx 0 / 0 / 0 Регистрация: 16.04.2013 Сообщений: 18
	19.04.2013, 18:27 [ТС]	#3
	Спасибо, всё понял) 0

@Catstail Модератор 23401 / 11528 / 1879 Регистрация: 12.02.2012 Сообщений: 18,833
	19.04.2013, 20:23	#5
	Сообщение от zss Гораздо правильнее - будет не ограничиваться массивом в 10 элементов, а читать данные из потока до EOF. Массив вообще не нужен. 1

@zss Модератор 6840 / 6402 / 2137 Регистрация: 18.12.2011 Сообщений: 16,800 Завершенные тесты: 1
	19.04.2013, 21:29	#6
	Сообщение от Catstail читать данные из потока до EOF А что такое EOF для потока cin? 0

@Catstail Модератор 23401 / 11528 / 1879 Регистрация: 12.02.2012 Сообщений: 18,833
	19.04.2013, 21:39	#7
	Сообщение от zss А что такое EOF для потока cin? - это CTRL-Z + Enter 2

@Kranxx 0 / 0 / 0 Регистрация: 16.04.2013 Сообщений: 18
	19.04.2013, 21:49 [ТС]	#8
	В этом случае мне не надо было добиваться максимально правильного эффекта, я эксперементировал с массивом и функцией, чтобы лучше понять их в действии, но всё равно спасибо. 0

Опции темы

С++ для начинающих

В чём ошибка? - C++