Как сократить код с подпрограмой?

Question

[Mnsweet]

Кароче я написал код он работает но моему преподавателю не понравилось что три подпрограммы каторые считывают сумы отрицальных чисел трёх масивов выполняют одно и тоже действия . Как сократить код чтоб три подпрограмы выполнялись в одной и чтоб код работал до сокращения.

#include <iostream>
using namespace std;
int masivA(int* a) {
    int sA = 0;
	for (int i = 0; i < 20; i++)
	if(a[i]<0)	sA += a[i];

	return sA;
}
int masivB(int* b) {
	int sB = 0;
	for (int i = 0; i < 20; i++)
		if (b[i] < 0)	sB += b[i];
	
	return sB;
}
int masivC(int* c) {
	int s = 0;
	for (int i = 0; i < 20; i++)
		if (c[i] < 0)	s += c[i];
	
	return s;
}
int fun(int f, int g, int h) {
 return f + g + h;

}

int main()
{
	int a[20], b[20], c[20], i, e,  x, y, z;

	for (i = 0; i < 20; i++) {
		cout << "\t a=";
		cin >> a[i];
	}
	for (i = 0; i < 20; i++) {
		cout << "\t b=";
		cin >> b[i];
	}
	for (i = 0; i < 20; i++) {
		cout << "\t c=";
		cin >> c[i];

	}

	x=masivA(a);
	y = masivB(b);
	z = masivC(c);
	fun(x, y, z);
	e = fun(x, y, z) / 3;
	cout << "sumaA=" << masivA(a) << endl;
	cout << "sumaB=" << masivB(b) << endl;
	cout << "sumaC=" << masivC(c) << endl;
	cout << "suma=" << e << endl;
	cin.get();
	return 0;
}

Answer 1

[Wataru]

Эти три функции идентичны, с точностью до переименования локальных переменных. Вы три раза написали одно и тоже. Можно 2 функции просто удалить и использовать отсавшуюся три раза.

Ну какая разница, как у вас там переменная называется sA или sB - результат будет один и тот же.

Да, может вы путаетесь, но аргумент в функции можно тоже переменовать. Хоть там и написано int masivA(int* a), этот a - это аргумент. Он никак не привязан к массиву a в main(). Туда можно передать и a и b и любой другой массив.

Comments

[Mnsweet]

Не когда мне в жизни нравились такие люди каторые дают очень расплычавты ответ.
Замест таго чтоб дать хотя бы наводку что надо использовать вы нам даёте какой-то ребус.

[Модератор]

Mnsweet, за готовыми рецептами надо обращаться на фриланс. Сюда надо приносить свою попытку решения, показать что ожидалось и что получилось. Тогда, возможно, помогут.
В данном виде это не вопрос, а задание, и может быть удален за нарушение п.5.12 Регламента.

[Wataru]

Mnsweet, ЧТО?! Ничего более конкретного, чем указание удалить 2 одинаковые функции, на ваш "вопрос" не ответить никак. Ну только если прямо код за вас весь исправить. Плюс я попытался объяснить момент, из-за которого вы не видите что у вас 3 функции идентичны. Но, похоже, вам сначала надо объяснить всю методичку по С++, с полного нуля, раз вам эти конкретные и развернутые объяснения кажутся ребусами. Но этим тут никто заниматься не будет.

[Евгений Шатунов]

Mnsweet, если тебе не понятно написанное в ответе, значит мы в целом не понимаем друг друга. Для меня, как для профессионала, написанное в ответе полностью понятно и дает исчерпывающее решение твоего вопроса.
Стало быть, суть непонимания сейчас сокрыта в тебе. Я хочу задать тебе несколько вопросов и понять твои ответы.

int masivA(int* a) {
    int sA = 0;
  for (int i = 0; i < 20; i++)
  if(a[i]<0)	sA += a[i];

  return sA;
}
int masivB(int* b) {
  int sB = 0;
  for (int i = 0; i < 20; i++)
    if (b[i] < 0)	sB += b[i];
  
  return sB;
}
int masivC(int* c) {
  int s = 0;
  for (int i = 0; i < 20; i++)
    if (c[i] < 0)	s += c[i];
  
  return s;
}

Это разные функции? Чем они отличаются, если да? Что у них общего, если нет?

Answer 2

[mayton2019]

Исходник стартовал с включения заголовка
#include <iostream>
Тоесть поднят флаг С++. А дальше идет типичный С-код. Зачем так?
По идее надо затаскивать vector и лямбды для суммирования.
Это было бы идеологически правильно.