Форум программистов, компьютерный форум CyberForum.ru

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

Войти
Регистрация
Восстановить пароль
 
Valerko
17 / 17 / 1
Регистрация: 30.11.2010
Сообщений: 164
#1

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

07.02.2011, 01:38. Просмотров 428. Ответов 6
Метки нет (Все метки)

Программа компилируется нормально.
Спрашивает кол-во элементов массива.
Вводятся элементы массива
Считает сумму от первого до последнего нулевого(если его нет- сумму всех элементов)
И должна считать произведение чётных элементов, НО не считает ( всегда результат единица)
Насколько могу понять, ошибка где-то в 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;
}
Вот пример работы программы
Миниатюры
Операции с массивами (есть логическая ошибка, не пойму где)  
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
07.02.2011, 01:38     Операции с массивами (есть логическая ошибка, не пойму где)
Посмотрите здесь:

Не пойму где ошибка - C++
Всем привет, вот решил написать простенький класс для работы с интернетом и выскакивают какие-то ошибки, не пойму в чем дело, вот эти...

Не пойму где ошибка - C++
Помогите исправить код, ошибок он не выдает, но программа почему то не работает, после вводы данных ничего не происходит, ниже условие и...

Не пойму где ошибка - C++
#include &lt;iostream&gt; using namespace std; int main() { setlocale(0, &quot;&quot;); double a, b; char f; a = 0; f = &quot;+&quot;; ...

Не пойму, где ошибка - C++
Здравствуйте, что-то никак не разберусь в этом куске кода, и почему компилятор нашел там narrowing conversion во второй фигурной скобке...

Не пойму, где ошибка - C++
Всем добрый день. Недавно начал изучать С++ вот столкнулся с проблемой в одной из задач: Из матрицы размером NxM получить вектор B,...

Не пойму где ошибка в коде - C++
Задача: (В задаче можно использовать только такие библиотеки и решить надо только через условные операторы.) Все проходит и решение...

После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
almazsr
232 / 169 / 7
Регистрация: 13.11.2010
Сообщений: 425
07.02.2011, 01:56     Операции с массивами (есть логическая ошибка, не пойму где) #2
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
17 / 17 / 1
Регистрация: 30.11.2010
Сообщений: 164
07.02.2011, 02:02  [ТС]     Операции с массивами (есть логическая ошибка, не пойму где) #3
Цитата Сообщение от 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;
}
вот ошибка при компиляции
Миниатюры
Операции с массивами (есть логическая ошибка, не пойму где)  
Евгений М.
1035 / 976 / 54
Регистрация: 28.02.2010
Сообщений: 2,829
Завершенные тесты: 2
07.02.2011, 08:15     Операции с массивами (есть логическая ошибка, не пойму где) #4
Исправьте заголовок.
C++
1
int proiz(int s[]);
Valerko
17 / 17 / 1
Регистрация: 30.11.2010
Сообщений: 164
07.02.2011, 08:29  [ТС]     Операции с массивами (есть логическая ошибка, не пойму где) #5
Цитата Сообщение от Евгений М. Посмотреть сообщение
Исправьте заголовок.
C++
1
int proiz(int s[]);
если такой заголовок, то произведение всегда равно единице(не могу понять почему)
Евгений М.
1035 / 976 / 54
Регистрация: 28.02.2010
Сообщений: 2,829
Завершенные тесты: 2
07.02.2011, 09:17     Операции с массивами (есть логическая ошибка, не пойму где) #6
Цитата Сообщение от Valerko Посмотреть сообщение
если такой заголовок, то произведение всегда равно единице
Цитата Сообщение от Евгений М. Посмотреть сообщение
Исправьте заголовок.
Я пояснил что исправить. Как исправить - не говорил.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
07.02.2011, 09:24     Операции с массивами (есть логическая ошибка, не пойму где)
Еще ссылки по теме:

Своя реализация strcpy - не пойму где ошибка! - C++
void copy(char *p, char *q) throw (char*) { int l2 = strlen(q); //cout&lt;&lt;&quot;strlen(q)=&quot;&lt;&lt;strlen(q)&lt;&lt;&quot; strlen(p)=&quot;&lt;&lt;strlen(p)&lt;&lt;endl; ...

Конечные суммы и произведение (не пойму где ошибка в программе) - C++
Создать программу для вычисления конечной суммы и произведения. #include &lt;iostream.h&gt; #include &lt;math.h&gt; int main() { int i,...

Где здесь логическая ошибка? Второй час уже не могу найти - C++
void ReadFromFile( Person*&amp; pers, int&amp; rCount ) { FILE* file = fopen(&quot;kakhochy.pam&quot;, &quot;r&quot;); //Если не может открыть файл if...

Не пойму где ошибка при вычислении площади треугольника формулой Герона - C++
Иногда результат 0.0... Иногда -1.#j Иногда -1.#ind00 #include&lt;stdio.h&gt; #include&lt;math.h&gt; float square(float a, float b,...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
NNN777
299 / 157 / 19
Регистрация: 27.02.2010
Сообщений: 317
07.02.2011, 09:24     Операции с массивами (есть логическая ошибка, не пойму где) #7
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 останется без инициализации, что приведет к неправильному подсчету суммы или к аварийному выходу программы.
Yandex
Объявления
07.02.2011, 09:24     Операции с массивами (есть логическая ошибка, не пойму где)
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru