c++ Найти в массиве и вывести на экран сумму абсолютных значений отрицательных элементов массива.(5x7)

[sa1nts_eye]

Здраствуйте друзья , сижу решаю задачки , учусь, ну вот не очень сильно понимаю как доделать .
Вот мой код , если сможете выслать решение данной задачи с объяснением. Буду благодарен

Задача :
Найти в массиве и вывести на экран сумму абсолютных значений отрицательных элементов массива.(5x7)

#include <iostream>

int main() {
    setlocale(LC_ALL, "rus");
    int a[5];
    double s_neg = 0;
    int n_neg = 0, s_pos = 0;
    std::cout << "Введите 5 целых чисел:\n";
    for (int i = 0; i < 5; i++) {
        std::cin >> a[i];
        if (a[i] < 0) {
            s_neg += abs(a[i]);
            n_neg++;
        }
    }
    if (n_neg == 0) {
        std::cout << "Отрицательных чисел нет!!!\n";
        return 1;
    }
    s_neg /= n_neg;
    for (int i = 0; i < 5; i++)
        if (abs(a[i]) > s_neg)
            s_pos += a[i];
    std::cout << "Вот она, сумма:" << s_pos << '\n';
    return 0;
}

[ViktorR]

Сложно понять логику, но:

for (int i = 0; i < 5; i++) {
   std::cin >> a[i];
   if (a[i] < 0) {
      s_neg += abs(a[i]);
      n_neg++;     // это кому-то нужно?
   }
}

В этом цикле вводятся элементы массива и подсчитывается сумма абсолютных значений отрицательных элементов - то, что требуется в задаче.
Зачем считаем?

n_neg++;

Зачем следующее условие?

s_neg /= n_neg;
for (int i = 0; i < 5; i++)
   if (abs(a[i]) > s_neg)
      s_pos += a[i];

Добавить за первым циклом

std::cout << "Вот она, сумма:" << s_neg << '\n';

и радоваться жизни

PS: Только вот массив, по заданию, двухмерный!
Надо использовать вложенный цикл.
Первый (внешний цикл), например, по строкам, а второй - по столбцам.
Что-то вроде такого:

int a[5][7];
...
s_neg = 0;
for (int i = 0; i < 5; i++) {
   for (int j = 0; j < 7; j++) {
   std::cin >> a[i],[j];
   if (a[i],[j] < 0) {
      s_neg += abs(a[i],[j]);
   }
}

PSS: За точность оформления не ручаюсь, так как на этом языке писал очень давно, а проверить неначем.

[BDA]

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

За точность оформления не ручаюсь

Ага, запятая между индексами лишняя.
sa1nts_eye, а еще можно без модуля считать просто "s_neg -= a[i][j];".

[sa1nts_eye]

ViktorR, Могли бы вы целый код написать ?..

[sa1nts_eye]

ViktorR, оно сумму не выдает чисел!((

[BDA]

Так покажите, как исправили исходный код с учетом замечаний.

[sa1nts_eye]

BDA,

#include <iostream>

int main() {
    setlocale(LC_ALL, "rus");
    int a[5][7];
    double s_neg = 0;
    int n_neg = 0, s_pos = 0;
    
    for (int i = 0; i < 5; i++) {
        for (int j = 0; j < 7; j++) {
            std::cin >> a[i][j];
            if (a[i][j] < 0) {
                std::cout << "Сумма";
                s_neg += abs(a[i][j]);
            }
            
            
        }
    }
    

    

}

BDA, сказали как бы по убирать не нужное , я убрал . массив работает как нужно , но вот сумму по условию задачи оно в конце не считает (

[BDA]

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

[sa1nts_eye]

BDA, Если не ошибаюсь должно выглядить вот так

#include <iostream>

int main() {
    setlocale(LC_ALL, "rus");
    int a[5][7];
    double s_neg = 0;
    int n_neg = 0, s_pos = 0;
    
    for (int i = 0; i < 5; i++) {
        for (int j = 0; j < 7; j++) {
            std::cin >> a[i][j];
            if (a[i][j] < 0) {
                std::cout << "Сумма";
                s_neg += abs(a[i][j]);

                if (n_neg == 0) {
                    std::cout << "Отрицательных чисел нет!!!\n";
                    return 1;
                }
                s_neg /= n_neg;
                for (int i = 0; i < 5; i++)
                    if (abs(a[i][j]) > s_neg)
                        s_pos += a[i][j];
                std::cout << "Вот она, сумма:" << s_pos << '\n';
                return 0;
            }
            
            
        }
        
    }
    

    

}

и не работает (

[sa1nts_eye]

BDA, если написать код вот так

#include <iostream>

int main() {
    setlocale(LC_ALL, "rus");
    int a[5][7];
    double s_neg = 0;
    int n_neg = 0, s_pos = 0;
    
    for (int i = 0; i < 5; i++) {
        for (int j = 0; j < 7; j++) {
            std::cin >> a[i][j];
            if (a[i][j] < 0) {
                std::cout << "Сумма";
                s_neg += abs(a[i][j]);

                
                
               
            }
            for (int i = 0; i < 5; i++)
                if (abs(a[i][j]) > s_neg)
                    s_pos += a[i][j];
            std::cout << "Вот она, сумма:" << s_pos << '\n';
            return 0;

            
        }
        
    }
    

    

}

То я вписываю только одно число и после этого сразу код выдает сумму :858993458