Операции с массивами (есть логическая ошибка, не пойму где)

@Valerko · Регистрация: 30.11.2010

Студворк — интернет-сервис помощи студентам

Программа компилируется нормально.
Спрашивает кол-во элементов массива.
Вводятся элементы массива
Считает сумму от первого до последнего нулевого(если его нет- сумму всех элементов)
И должна считать произведение чётных элементов, НО не считает ( всегда результат единица)
Насколько могу понять, ошибка где-то в 42-52 строке, но я не пойму где.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
#include <iostream>
using namespace std;
 
int summa(int s[]);
int proiz(int s[]);
 
int main()
{
setlocale(0,"Rus");
int i;
int n;
int *s;
s = new int[n]; 
cout<<"Программма для подсчёта произведения чётных элементов массива,"<<endl;
cout<<"а так же суммы элементов от первого до последнего нулевого"<<endl;
cout<<"(если такового нет- сумма всех элементов массива)."<<endl;
cout<<"\n\n";
cout<<"Введите количество элементов массива:"<<endl;
cin>>n;
 
for (i=0;i<n;i++)
    {
    cout<<"Введите "<<i+1<<" из "<<n<<" элементов массива"<<endl;
    cin>> s[i];
    }
 
cout<<"\n\n";
cout<<"Ваш массив:"<<endl;       
 
for(i=0;i<n;i++)
    {
    cout << s[i]<<"  ";
    }
 
cout<<"\n\n";
    
cout<<"\nПроизведение чётных элементов равно:\n"<<proiz(s)<<endl;
cout<<"Сумма элементов от первого до последнего нулевого (либо всех элементов) равна:\n"<<summa(s)<<endl;
return 0;
}
 
int proiz ( int s[])
{
    int pr = 1;
    int n;
    int i;
    for(i=1; i<n; i+=2)
        {
        pr*=s[i];
        }
    return pr;
}
 
int summa( int s[])
{
    int summ=0;
    int nul;
    int i;
    int n;
 
    for ( i=n;i>=0;i--) 
    {
        if(s[i]==0) 
        {
            nul = i; 
            break; 
        }
    }
 
    for (i=0;i<nul;i++)
    summ=summ+s[i];
 
    return summ;
}

Вот пример работы программы

@almazsr · 07.02.2011, 01:56

C++
1
2
3
4
5
6
7
8
9
int proiz ( int* s, int n)
{
        int pr = 1;
        for(int i=0; i<n; i+=2)
                {
                pr*=s[i];
                }
        return pr;
}

@Valerko · 07.02.2011, 02:02 **[ТС]**

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

C++
1
2
3
4
5
6
7
8
9
int proiz ( int* s, int n)
{
        int pr = 1;
        for(int i=0; i<n; i+=2)
                {
                pr*=s[i];
                }
        return pr;
}

вот ошибка при компиляции

Евгений М. · 07.02.2011, 08:15

Исправьте заголовок.

C++
1
int proiz(int s[]);

@Valerko · 07.02.2011, 08:29 **[ТС]**

Сообщение от Евгений М.

Исправьте заголовок.

C++
1
int proiz(int s[]);

если такой заголовок, то произведение всегда равно единице(не могу понять почему)

Евгений М. · 07.02.2011, 09:17

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

если такой заголовок, то произведение всегда равно единице

Сообщение от Евгений М.

Исправьте заголовок.

Я пояснил что исправить. Как исправить - не говорил.

@NNN777 · 07.02.2011, 09:24

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
#include <iostream>
char* Rus(const char* text);
using namespace std;
 
int summa(int s[], int n);
int proiz(int s[], int n);
 
int main()
{
setlocale(0,"Rus");
int i;
int n;
int *s;
 
cout<<"Программма для подсчёта произведения чётных элементов массива,"<<endl;
cout<<"а так же суммы элементов от первого до последнего нулевого"<<endl;
cout<<"(если такового нет- сумма всех элементов массива)."<<endl;
cout<<"\n\n";
cout<<"Введите количество элементов массива:"<<endl;
cin>>n;
s = new int[n]; 
for (i=0;i<n;i++)
        {
        cout<<"Введите "<<i+1<<" из "<<n<<" элементов массива"<<endl;
        cin>> s[i];
        }
 
cout<<"\n\n";
cout<<"Ваш массив:"<<endl;       
 
for(i=0;i<n;i++)
        {
    cout << s[i]<<"  ";
    }
 
cout<<"\n\n";
        
cout<<"\nПроизведение чётных элементов равно:\n"<<proiz(s, n)<<endl;
cout<<"Сумма элементов от первого до последнего нулевого (либо всех элементов) равна:\n"<<summa(s, n)<<endl;
return 0;
}
 
int proiz ( int s[], int n)
{
        int pr = 1;
       // int n;
        int i;
        if (n<=1)
            return 0;
        for(i=1; i<n; i+=2)
        {
            pr*=s[i];
        }
        return pr;
}
 
int summa( int s[], int n)
{
        int summ=0;
        int nul=n;
        int i;
        //int n;
 
        for (i=n-1;i>=0;i--) 
        {
                if(s[i]==0) 
                {
                        nul = i; 
                        break; 
                }
        }
        for (i=0;i<nul;i++)
        summ=summ+s[i];
 
        return summ;
}

У вас n не глобальная переменная, а это значит, что в функции ее передавать обязательно, чтобы функция знала размер массива. А вы ее просто объявляете еще раз в функции и используете без инициализации. Как следствие неправильый подсчет произведения.
А если у вас будет массив из 1 элемента, то произведение по идее должно быть равно 0, а у вас бедет =1.
summ. вы ищете последний элемент, равный 0. А если такого не будет?... nul останется без инициализации, что приведет к неправильному подсчету суммы или к аварийному выходу программы.

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение/ Перевод Сайт называется reddit: The Thinkpad X220 Tablet is the best budget school laptop period. Это. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .