Skip to main content
2 de 3
adicionou 268 caracteres ao conteúdo
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil

Um dos motivos do problema é que você não preza muito por organizar o código e isto pode parecer bobagem, mas faz diferença até cognitiva no que está aprendendo e fazendo. Veja que escrevendo melhor fica mais fácil entender porque o erro ocorre.

Pense no problema da forma mais simples possível.

Se quer ler uma quantidade específica de itens o for sempre é mais recomendado e simples. Vira até algo padrão para se usar.

Se quer somar algo, basta ir acumulando como fez. Mas tem um erro, está incrementando o número o número de lidos condicionalmente, ou seja, só quando lê 20 positivos que o laço encerra. Se tivesse usado um for provavelmente evitaria esse problema. E note que está usando um else if onde um else já basta, um é condição exatamente contrária ao outro.

Só no caso de negativos precisa acumular e contar quanto são porque para calcular a média precisa do total e de quantos são para fazer a divisão.

Outra coisa que eu sempre digo que se declarar a variável mais próxima de onde será usada fica mais fácil lidar com ela e saber porque ela existe.

Eu transformei o total em float para dar um número quebrado, mas isso não é requisito colocado no enunciado, então sem ele deveria funcionar.

#include <stdio.h>

int main() {
    int somaPositivos = 0, somaNegativos = 0, negativos = 0;
    for (int i = 0; i < 20; i++) {
        printf("\nDigite um numero");
        int numero;
        scanf("%d", &numero);
        if (numero < 0) {
            somaNegativos += numero;
            negativos++;
        } else somaPositivos += numero;
    }
    printf("A media dos numeros negativos é de:  %f\n", (float)somaNegativos / negativos);
    printf("A soma dos numeros positivos é: %d\n", somaPositivos);
}

Veja funcionando no ideone. E no Coding Ground. Também coloquei no GitHub para referência futura.

Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil