Помочь исправить

@Akaufo · Регистрация: 21.03.2009

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

Здравствуйте, в с++ я новичок, программирую 2 день, дошел до одномерных массивов, помогите разобраться в чем ошибка.
Задача: В одномерном массиве из н элементов, вычислить:
1. произведение элементов с четным индексом
2. сумму между первым нулевым и последним элементам
Прошу не исправлять, а на словах объянить в чем ошибка!При отладке указывает на строку 45!

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
#include <iostream.h>
#include <stdlib.h>
using namespace std;
int main()
{
    int const n=6;
    int a[n];
    int i;
    cout << "Введите элементы массива: ";
    for (i=0; i<n; i++)
        {
              cout << "a[" << i << "]= ";
              cin >> a[i];
        };
    int p=1; 
    for (i=0; i<n; i++)
        if (i%2)
           p*=a[i];
    int n1, n2, k;
    for (i=0; i<n; i++)
              if (a[i]==0)
                 {
                         n1=i;
                         break;
                 }
              else
                  k=1;
     //cout << "n1= " << n1;             
              
    if (k!=1)
       {
             for (i=n1+1; i<n; i++)
                 if (a[i]==0)
                    n2=i;
                 else
                     k=2;
       }
    else
       cout << "Error!"; 
    if (k==2)
       cout << "Error!";
    else
       {
              int sum;
              for (i=n1+1, sum=0; i<=n2; i++)
                  sum+=a[i]; //отладка
              cout << "\nSum " << sum;         
       };
    cout << "\nP= " << p;
    system("\npause");
    return 0;
}

@Devioso · 05.01.2011, 22:02

Нашел ошибку. Проблема в n1. Данной переменной вы не присваиваете никакого значения, но используете в цикле подсчета суммы, где и вылетает ошибка.

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

int n1, n2, k;

Если обнулите, то ошибка исчезнет. (int n1=0,n2=0,k)

@Sokolov · 05.01.2011, 22:10

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

#include <iostream.h>

а разве там не просто <iostream>
или нету разницы?

@Devioso · 05.01.2011, 22:11

Это зависит от компилятора. .h - это расширение файла

@apachan · 05.01.2011, 22:12

или нету разницы?

В данной программе не особо важно.

@Алексaндр · 05.01.2011, 22:14

Sokolov, разница есть. Если бы было <iostream>, нужно было бы дописать
using namespace std;
или вывод оформлять так
std::cout<<a[i];

Добавлено через 1 минуту
Но в данной программе это действительно не особо важно.

@Sokolov · 05.01.2011, 22:14

У меня visual studio 2008 вообще не воспринимает <iostream.h> ругается на нее

@Akaufo · 05.01.2011, 22:16 **[ТС]**

Devioso, спасибо, обнуление исправило вылетание ошибки, но появилась другая проблема, теперь не корректно считается сумма, если нулевых элементов вообще нет, то выводит и сообщение Error и значение суммы...

@Sokolov · 05.01.2011, 22:16

А еще вопрос:
часто на форуме встречал что пишут
std::cout<< ;
вместо
using namespace std;
разве это удобно?
или есть какие-то плюсы в таком написании?

@Devioso · 05.01.2011, 22:17

А вы бы не могли поточнее сказать сумму чего нужно подсчитать. Программа вроде работает как нужно.

@Akaufo · 05.01.2011, 22:39 **[ТС]**

всем спасибо, программу исправил, убрав все проверки.Программа работает и считает все верно, но мне кажется что такой метод не совсем корректен!

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
#include <iostream.h>
using namespace std;
int main()
{
    int const n=6;
    int i, a[i];
    for (i=0; i<n; i++)
        {
              cout << "a[" << i << "]= ";
              cin >> a[i];
        };              
    for (i=0; i<n; i++)
        cout << a[i] << "  ";
    int p=1; 
    for (i=0; i<n; i++)
        if (i%2)
           p*=a[i];
    int n1=0, n2=0;
    for (i=0; i<n; i++)
        if (a[i]==0)
           {
                    n1=i;
                    break;
           };
    for (i=n1+1; i<n; i++)
        if (a[i]==0)
           {
                    n2=i;
                    break;
           };
    int sum;
      for (i=n1+1, sum=0; i<n2; i++)
            sum+=a[i];
    cout << "\nn1= " << n1;
    cout << "\nn2= " << n2;
    cout << "\np= " << p;
    cout << "\nsum= " << sum;
    system("pause");
    return 0;      
}

Новые блоги и статьи Все статьи Все блоги /
Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .	Сумматор с применением элементов трёх состояний. Hrethgir 26.03.2026 Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .	Автозаполнение реквизитов при создании документа Maks 26.03.2026 Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .	Команды формы и диалоговое окно Maks 26.03.2026 1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .
Кому нужен AOT? DevAlt 26.03.2026 Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .	Отправка уведомления на почту при изменении наименования справочника Maks 24.03.2026 Программная отправка письма электронной почты на примере изменения наименования типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной. . .	модель ЗдравоСохранения 5. Меньше увольнений- больше дохода! anaschu 24.03.2026 Теперь система здравосохранения уменьшает количество увольнений. 9TO2GP2bpX4 a42b81fb172ffc12ca589c7898261ccb/ https:/ / rutube. ru/ video/ a42b81fb172ffc12ca589c7898261ccb/ Слева синяя линия -. . .	Midnight Chicago Blues kumehtar 24.03.2026 Такой Midnight Chicago Blues, знаешь?. . Когда вечерние улицы становятся ночными, а ты не можешь уснуть. Ты идёшь в любимый старый бар, и бармен наливает тебе виски. Ты смотришь на пролетающие. . .

@Devioso 4 / 4 / 1 Регистрация: 13.12.2009 Сообщений: 60
	05.01.2011, 22:11
	Это зависит от компилятора. .h - это расширение файла 1

@Алексaндр 131 / 108 / 12 Регистрация: 04.12.2010 Сообщений: 313
	05.01.2011, 22:14
	Sokolov, разница есть. Если бы было <iostream>, нужно было бы дописать using namespace std; или вывод оформлять так std::cout<<a[i]; Добавлено через 1 минуту Но в данной программе это действительно не особо важно. 1

@Sokolov 42 / 42 / 13 Регистрация: 04.01.2011 Сообщений: 125
	05.01.2011, 22:14
	У меня visual studio 2008 вообще не воспринимает <iostream.h> ругается на нее 0

@Akaufo 13 / 0 / 0 Регистрация: 21.03.2009 Сообщений: 100
	05.01.2011, 22:16 [ТС]
	Devioso, спасибо, обнуление исправило вылетание ошибки, но появилась другая проблема, теперь не корректно считается сумма, если нулевых элементов вообще нет, то выводит и сообщение Error и значение суммы... 0

@Sokolov 42 / 42 / 13 Регистрация: 04.01.2011 Сообщений: 125
	05.01.2011, 22:16
	А еще вопрос: часто на форуме встречал что пишут std::cout<< ; вместо using namespace std; разве это удобно? или есть какие-то плюсы в таком написании? 0

@Devioso 4 / 4 / 1 Регистрация: 13.12.2009 Сообщений: 60
	05.01.2011, 22:17
	А вы бы не могли поточнее сказать сумму чего нужно подсчитать. Программа вроде работает как нужно. 0