Dificuldade em exercício

[ph1 0]

Calcular e imprimir a média aritmética dos números inteiros positivos ímpares inferiores ou iguais a um número N lido inicialmente.

 

#include <cmath> // ou <math.h>
#include <iostream>
#include <cstdlib>
#include <locale.h>
#include <iomanip>
using namespace std;
int main()
{
setlocale(LC_ALL, "Portuguese");
float m,acm;
int cont,n;
acm = 0;
cout << "Entre com um número qualquer.:\t";
cin >> n;
  for (cont = 0;cont <= n;n++)
    {
    if ((n % 2)==1)
      {
        acm = acm + 1;
        m = acm / n;
      }
    else if ((n % 2) == 0)
        {
         cout << "O número não é ímpar.";
        }
   }
cout << "A média dos" << n << " números ímpares é de:\t" << m;
}
        

Estou com com dificuldade nesse exercício. Porque está dando loop infinito? Não descubro o erro.

[Nefisto 0]

Ola @ph1, seu loop n quebra por duas razoes:

1) Voce ta somando o N, logo cont NUNCA vai ser maior que N e seu N vai pro infinito, daew crasha hehe

2) A sua condição exige que CONT seja maior que N, porem você nunca altera o valor inicial de CONT

 

Mas o problema n ta so no loop, sua matematica tbm ta errada, na media aritmetica (q eu suponho ser a simples), você soma os valores e dividi pela quantia de valores que foram somados, você ta ta fazendo não só a divisão errada (você ta dividindo a quantia de numeros pelo seu limite da pesquisa, o que nao faz sentido), como tbm a quantia de vezes que você dividi, ja q você dividindo a cada loop... da uma olhada no codigo abaixo

 

Espero ter ajudado, qualquer coisa da um grito, abraços

 

EDIT:

Esqueci de dizer, procura ir chamando as bibliotecas só conforme você for usando, assim você evita de ter uma centena de bibliotecas e só usar uma, q é o seu caso

#include <iostream>

using namespace std;

int main()
{
	float soma = 0;
	int acm = 0;
	int i, n;

	cin >> n;
	
	for (i = 1;i <= n; i++)
    {
		if ((i % 2)==1)
		{
			soma += i;
			acm++;
		}
	}
	
	cout << "A media dos números ímpares é de: %f\t" << soma/acm;
	
	return (0);
}

 

[ph1 0]

Perfeito, entendi a lógica do programa. Tudo é feito encima de i e não de n, n é só para colocar um limite no número de iterações. 

Muito obrigado pela ajuda. Fiz um rascunho e um tipo de teste de mesa.

Obrigado.