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

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

Войти
Регистрация
Восстановить пароль
Другие темы раздела
C++ вызов функции overload http://www.cyberforum.ru/cpp-beginners/thread239222.html
Что я делаю не так, где ошибка? . #include "stdafx.h" #include<iostream> #include<conio.h> using namespace std; int max_element(int n,int array) { int hold;
C++ Прога с++ Помогите плиз создать прогу: Двоичный файл содержит данные о студентах, фамилия, группа, оценка по программированию. Создать на его основе текстовый файл с записями такого же вида, помещенных в него только тех студентов, у которых оценка по программированию выше среднего в группе. http://www.cyberforum.ru/cpp-beginners/thread239203.html
Программа на описание классов C++
Помогите плиз создать прогу: Описать класс, содержащий строковое поле, целочисленное поле и указатель на массив вещественных чисел. Реализовать два произвольных конструктора, инициализировать все поля.
C++ Файлы, работа с файлами
В общем такая трабла. Задание:создать файл ф1 содержащий не менее 10 строк,скопировать из файла ф1 в ф2 все строчки не содержащие цифр, подсчитать строки в файле ф2, которые начинаются с А. Вот мой код. #include "stdafx.h" #include <stdio.h> #include <conio.h> #include <iostream> using namespace std; int _tmain(int argc, _TCHAR* argv)
C++ Ввод длинных строк с пробелами http://www.cyberforum.ru/cpp-beginners/thread239191.html
Доброго времени суток!!! Собственно как организовать ввод строки вида "один два три 4 5 6 7..." - где вперемешку будут и символы, и буквы. Пытался сделать это через gets() - при вводе только букв, проблем не возникает, но если ввести буквы и цифры, возникает ошибка access violation.
C++ Массивы. Движение при сравнении значений индекса Задание: Написать программу, которая находит в массиве значения, повторяющиеся два и более раз, и показывает их на экран. Поиском находил задачи, но там либо мне было непонять их, либо через динамические массивы. Вроде ничего сложного, но чего то не хватает. Вот основной код: #include <iostream> using namespace std; int main(){ const int a_size=8; int a={1,5,7,2,7,3,5,7}; подробнее

Показать сообщение отдельно
NNN777
299 / 157 / 19
Регистрация: 27.02.2010
Сообщений: 317
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 останется без инициализации, что приведет к неправильному подсчету суммы или к аварийному выходу программы.
 
Текущее время: 11:43. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru