Alguém pode avaliar esse meu código? é pequeno.

[doodoh]

Cálculo de Média Aritmética de 4 valores: Esta opção deverá ser implementada através do uso de uma função que receba como parâmetro 4 valores e calcule e escreva o resultado da média aritmética entre os mesmos.

#include <iostream>
using namespace std;
float n1,n2,n3,n4,m;
calcularmedia (float a,float b,float c,float d){


m=(n1+n2+n3+n4)/4; 

cout<<"Media: "<<m;

}


main () {
	cout<<"Informe os 4 valores: ";
	cin>>n1>>n2>>n3>>n4;

	calcularmedia (n1,n2,n3,n4);
}

 

Editado Dezembro 21, 2016 por doodoh

[vangodp]

Se você recebeu float a,float b,float c,float d por parametro por que você usa n1+n2+n3+n4 para fazer o calculo dentro da sua função? Troque m=(n1+n2+n3+n4)/4; por m=(a+b+c+d)/4;.

E já vai acostumando a nunca nunca nunca, usar variáveis globais. Quantos antes acostumar antes vai doer menos >_<.
https://forum.zwame.pt/threads/c-variaveis-globais-porque-devemos-evitar-variavies-globais.794001/