C dificuldade em completar exercício em C

Helio Junnior Sartor Dalmolin · 9 de junho de 2021

Crie um registro para armazenar informações de um jogador de futebol contendo:

nome;
idade;
número de jogos;
número de gols.

Escreva um programa que obtenha os dados para jogadores titulares de um time de futsal (5 jogadores; utilize um vetor de registros).

Por fim imprima o nome e a média de gols de cada jogador.

Eu não consegui fazer a media, alguém poderia me mostrar como é?

#include<stdio.h>
#include<stdlib.h>
struct jogador{
	char nome [50];
	char idade [3];
	int njogos [50];
	int ngols [6];
}atleta[5];

 main(){
 	
	int i;
	float media;
	
	for(i=0;i<5;i++){
		printf("Digite o nome do jogador: \n");
		scanf("%s", &atleta[i].nome);
		printf("Digite a idade do jogador: \n");
		scanf("%s", &atleta[i].idade);
		printf("Digite o numero de jogos do jogador: \n");
		scanf("%d", &atleta[i].njogos);
		printf("Digite o numero de gols do jogador: \n");
		scanf("%d", &atleta[i].ngols);
	}
	printf("Numeros de gols de cada jogador: \n");
	for(i=0;i<5;i++){
		media=(atleta[i].ngols)/(atleta[i].njogos);
		printf("%s:",atleta[i].nome); 
		printf("%f gol(s).\n",media);
	}
}

devair1010 · 9 de junho de 2021

@Helio Junnior Sartor Dalmolin na struct voce nao precisa colocar vetor , nao , pois voce irah adicionar um vettor para a struct toda , e a media eh assim mesmo o total de gols dividido pela qtd de jogos , e ai o que esta errado eh por que voce colocou a variavel ngols como vetor e no calculo usou como variavel simples , e ate funcionaria assim mesmo se voce colocar um colchete depois da variavel ngols assim :

scanf("%d",&atleta[ i ] .ngols[0]);

media = atleta[ i ] .ngols[ 0 ];

Helio Junnior Sartor Dalmolin · 10 de junho de 2021

@devair1010

Mudei do jeito como entendi, agora ele esta calculando, porém aparece a media como 0 dos cinco atletas. o que estou errando ainda?

#include<stdio.h>
#include<stdlib.h>
struct jogador{
	char nome [50];
	char idade [3];
	int njogos [50];
	int ngols [6];
}atleta[5];

 main(){
	int i;
	float media;
	
	for(i=0;i<5;i++){
		printf("Digite o nome do jogador: \n");
		scanf("%s", &atleta[i].nome);
		printf("Digite a idade do jogador: \n");
		scanf("%s", &atleta[i].idade);
		printf("Digite o numero de jogos do jogador: \n");
		scanf("%d", &atleta[i].njogos[i]);
		printf("Digite o numero de gols do jogador: \n");
		scanf("%d", &atleta[i].ngols[i]);
	}
	printf("Numeros de gols de cada jogador: \n");
	for(i=0;i<5;i++){
		media=(atleta[i].ngols[i])/(atleta[i].njogos[i]);
		printf("%s:",atleta[i].nome); 
		printf("%f gol(s).\n",media);
	}
}

devair1010 · 10 de junho de 2021

@Helio Junnior Sartor Dalmolin sempre na posição zero do vetor ngols ,assim :

printf("Digite o numero de jogos do jogador: \n");
scanf("%d", &atleta[i].njogos[0]);
printf("Digite o numero de gols do jogador: \n");
scanf("%d", &atleta[i].ngols[0]);
media = atleta[i].ngols[0]) / atleta[i].njogos[0];

Helio Junnior Sartor Dalmolin · 10 de junho de 2021

@devair1010

fiz isso e ainda continua a media dando 0.

arfneto · 10 de junho de 2021

2 horas atrás, Helio Junnior Sartor Dalmolin disse:

Mudei do jeito como entendi, agora ele esta calculando, porém aparece a media como 0 dos cinco atletas

Mas...

struct jogador{
	char nome [50];
	char idade [3];
	int njogos [50];
	int ngols [6];
}atleta[5];

Entenda o que você declarou:

O que é atleta?

atleta é jogador[5]. Um vetor de 5 jogador, de jogador[0] a jogador[4]

E o que é njogos?

njogos é int[50], 50 int numerados de njogos[0] a njogos[49] !

E o que é ngols?

ngols é int[6], 6 int numerados de ngols[0] a ngols[5]

E acho que sabe que não é isso que o enunciado diz. ngols é um número, quantos gols o cara marcou. njogos é um número, o número de jogos que o cara jogou. A média, claro, é o popular quociente de um pelo outro, ngols / njogos.

De seu programa

	for(i=0;i<5;i++){
		media=(atleta[i].ngols[i])/(atleta[i].njogos[i]);
		printf("%s:",atleta[i].nome); 
		printf("%f gol(s).\n",media);
	}

Não escreva assim. Declare i DENTRO do for. Nunca deixe uma variável com um nome ingênuo como i solta pelo programa. Nem todo programa que você escrever vai ter 10 linhas e isso sempre cai na sua cabeça. NÃO use variáveis globais. Nunca.

Escreva

	for(int i=0;i<5;i++) // ...

Esse cálculo da média não faz sentido... o índice que controla o acesso a atleta[] nada tem a ver com os índices dos vetores internos ngols e njogos, que já estão errados.

Calcular a média é apenas dividir o número de gols pelo número de jogos e pode mostrar isso direto no printf() e encerrar o programa. Muito mais simples

devair1010 · 10 de junho de 2021

@Helio Junnior Sartor Dalmolin fazendo cálculos com variáveis int e float misturadas , precisa usar um "cast" , mais ou menos assim :

media = ((float)atleta[i].ngols[0] / (float)atleta[i].njogos[0]); 
// media já é float , então converte os dois int's em dois float's
// mas talvez os ++ experientes não gostem de usar isso , e com 
// certeza exitem outros modos de se fazer , mas Ainda sou iniciante
// i vou deixar desse jeito mesmo

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <math.h>
struct jogador
{
    char nome [50];
//	char idade[3];  normalmente seria um int
    int idade     ;
    int njogos[50];
    int ngols  [6];
}atleta        [5];
int  main        ()
{
    int i;
    float media;
    strcpy(atleta[0].nome ,"lolipop");
    atleta[0].idade     = 21;
    atleta[0].njogos[0] =  2;
    atleta[0].ngols [0] = 15;

    strcpy(atleta[1].nome ,"kiko");
    atleta[1].idade     = 54;
    atleta[1].njogos[0] = 30;
    atleta[1].ngols [0] = 11;

    // aqui só para testar
/*
    for( i=0; i<2;i++ )
    {
        printf("Digite o nome do jogador ------------: ");
        scanf("%s", atleta[i].nome); //  não precisa desse & pois é string
        printf("Digite a idade do jogador -----------: ");
        scanf("%d",& atleta[i].idade);
        printf("Digite o numero de jogos do jogador %s -: ",atleta[i].nome);
        scanf("%d", &atleta[i].njogos[0]);
        printf("Digite o numero de gols do jogador %s --: \n",atleta[i].nome);
        printf("nesses %d JoGos ",atleta[i].njogos[0]);
        scanf("%d", &atleta[i].ngols[0]);
       }
*/
    printf("\nNumeros de gols de cada jogador\n\n");
    for( i=0; i<2; i++ )
    {
        media = ((float)atleta[i].ngols[0] / (float)atleta[i].njogos[0]);
        printf("-------------------------------\n");
        printf("Nome ------------: %s\n" , atleta[i].nome     );
        printf("IdaDe -----------: %d\n" , atleta[i].idade    );
        printf("numero de Gols --: %d\n" , atleta[i].ngols[0] );
        printf("numero de joGos -: %d\n" , atleta[i].njogos[0]);
        printf("MediA -----------: %.2f gol(s).",    media    );
        printf(" por PartiDa\n\n");
    }
    printf("\n\n\n");
    return 0;
}

aqui no Dev C++ e no codeblocks , @arfneto , precisa declarar a variável int i , antes do loop for , pois dentro dele o compilador dá mensagem de erro e não nem compila, não .

arfneto · 10 de junho de 2021

5 horas atrás, devair1010 disse:

aqui no Dev C++ e no codeblocks , @arfneto , precisa declarar a variável int i , antes do loop for , pois dentro dele o compilador dá mensagem de erro e não nem compila, não

Isso que citou, Dev C++ e Code::Blocks, são IDE (sim, singular). Não são compiladores. Então tem que procurar a razão do erro no seu compilador e em como está usando dentro desses ambientes.

O que te expliquei e sempre escrevo aqui

11 horas atrás, arfneto disse:

Não escreva assim. Declare i DENTRO do for. Nunca deixe uma variável com um nome ingênuo como i solta pelo programa. Nem todo programa que você escrever vai ter 10 linhas e isso sempre cai na sua cabeça. NÃO use variáveis globais. Nunca.

Isso levou uma década para ser corrigido em C. Só que foi a década de 80. Se você continuar usando seu compilador configurado para a década de 80 nada vai mudar. Esses ambientes, Dev-C++ e Code::Blocks, não são assim uma beleza hoje em dia. E tem problemas sérios. Mas permitem ao menos que se possa configurar o compilador.

Imagino que esteja usando o gcc. Ele tem dezenas e dezenas de opções de compilação, e MUITAS delas são bem importantes, para quem está aprendendo, ensinando ou apenas precisa do resultado, um código operante.

Usando seu programa como exemplo

Isso

    strcpy(atleta[0].nome ,"lolipop");
    atleta[0].idade     = 21;
    atleta[0].njogos[0] =  2;
    atleta[0].ngols [0] = 15;

    strcpy(atleta[1].nome ,"kiko");
    atleta[1].idade     = 54;
    atleta[1].njogos[0] = 30;
    atleta[1].ngols [0] = 11;

    // aqui só para testar

pode ser escrito assim nos tempos modernos

    Jogador     atleta[5] =
    {
        [1] = {
            .idade = 21, .nome = "lolipop",
            .njogos = 2, .ngols = 15
        },
        [0] = {
            .nome = "kiko", .njogos = 30,
            .ngols = 11, .idade = 54,
        }
    };

Você pode escrever os valores direto na declaração, para todos ou alguns campos, em qualquer ordem.

Isso

    {
        media = ((float)atleta[i].ngols[0] / (float)atleta[i].njogos[0]);
        printf("-------------------------------\n");
        printf("Nome ------------: %s\n" , atleta[i].nome     );
        printf("IdaDe -----------: %d\n" , atleta[i].idade    );
        printf("numero de Gols --: %d\n" , atleta[i].ngols[0] );
        printf("numero de joGos -: %d\n" , atleta[i].njogos[0]);
        printf("MediA -----------: %.2f gol(s).",    media    );
        printf(" por PartiDa\n\n");
    }

provavelmente seria escrito assim (mesmo nos 80)

        printf("\
\t-------------------------------\n\
\tNome ------------: %s\n\
\tIdade -----------: %d\n\
\tNumero de gols --: %d\n\
\tNumero de jogos -: %d\n\
\tMedia de gols por jogo: %.2f\n",

               // ...

É mais fácil de ler e alterar e não há sentido em chamar 5 vezes a mesma função.

Isso

        media = ((float)atleta[i].ngols[0] / (float)atleta[i].njogos[0]);

seria algo como

    (float)atleta[i].ngols / atleta[i].njogos // media so vai ser usada aqui afinal

direto no printf() já que só é usado uma vez e basta um cast para float.

A estrutura como você mesmo já disse está declarada errado: ngols e njogos são escalares e não vetores. Usar apenas o primeiro elemento do vetor não corrige o problema.

A estrutura podia ser declarada assim hoje em dia

typedef struct
{
    char nome [50];
    int idade     ;
    int njogos;
    int ngols;

}   Jogador;

E seria bem mais conveniente ter um time assim declarado

typedef struct 
{
    unsigned    n_jogadores;
    Jogador     atleta[5];

}   Equipe;

Escrevendo em torno dos dados e entendendo que os atletas estão organizados em times fica tudo mais fácil.

Um exemplo completo usando seu código como protótipo:

#include <stdio.h>
#include <memory.h>

typedef struct
{
    char nome [50];
    int idade     ;
    int njogos;
    int ngols;

}   Jogador;

typedef struct 
{
    unsigned    n_jogadores;
    Jogador     atleta[5];

}   Equipe;


int  main(void)
{
    Jogador     atleta[5] =
    {
        [1] = {
            .idade = 21, .nome = "lolipop",
            .njogos = 2, .ngols = 15
        },
        [0] = {
            .nome = "kiko", .njogos = 30,
            .ngols = 11, .idade = 54,
        }
    };

    // outro jeito, mais flexivel
    Equipe      alfa = { .n_jogadores = 0 };
    // coloca os jogadores do vetor atleta na tal equipe
    alfa.atleta[0] = atleta[0];
    alfa.atleta[1] = atleta[1];
    alfa.n_jogadores = 2; // claro, acabou de copiar os dois

    // testa
    printf( "Nome do primeiro jogador na equipe: %s\n", alfa.atleta[0].nome);
    printf( "O ultimo jogador da equipe --- %s --- marcou %d gols em %d partidas\n", 
        alfa.atleta[alfa.n_jogadores-1].nome, 
        alfa.atleta[alfa.n_jogadores-1].ngols, 
        alfa.atleta[alfa.n_jogadores-1].njogos);

    // completa a equipe com os mesmos caras, so um teste afinal
    alfa.atleta[alfa.n_jogadores++] = atleta[0]; // o 3
    alfa.atleta[alfa.n_jogadores++] = atleta[0]; // o 4
    alfa.atleta[alfa.n_jogadores++] = atleta[0]; // o ultimo

    // redefine os gols a partir de 10 para ficar arrumadinho no loop
    for( int i=0; i<alfa.n_jogadores; i+=1 ) alfa.atleta[i].ngols = 10+i;
    
    printf("\nA equipe (%d atletas):\n\n", alfa.n_jogadores );
    for( int i=0; i<alfa.n_jogadores; i+=1 )
    {
        printf("\
\t[%d] ------------------------------------\n\
\tNome -----------------: %s\n\
\tIdade ----------------: %d\n\
\tNumero de gols -------: %d\n\
\tNumero de jogos ------: %d\n\
\tMedia de gols por jogo: %.2f\n",
            1+i,
            alfa.atleta[i].nome,
            alfa.atleta[i].idade,
            alfa.atleta[i].ngols,
            alfa.atleta[i].njogos,
            (float) alfa.atleta[i].ngols/alfa.atleta[i].njogos // media so vai ser usada aqui afinal
        );
    };  // for()
    return 0;
}

E pode ver como usar a estrutura sózinha e também o Jogador dentro da Equipe e como copiar e inicializar do jeito mais comum nesse século.

Eis a saída do programa

Nome do primeiro jogador na equipe: kiko
O ultimo jogador da equipe --- lolipop --- marcou 15 gols em 2 partidas

A equipe (5 atletas):

        [1] ------------------------------------
        Nome -----------------: kiko
        Idade ----------------: 54
        Numero de gols -------: 10
        Numero de jogos ------: 30
        Media de gols por jogo: 0.33
        [2] ------------------------------------
        Nome -----------------: lolipop
        Idade ----------------: 21
        Numero de jogos ------: 2
        Media de gols por jogo: 5.50
        [3] ------------------------------------
        Nome -----------------: lolipop
        Idade ----------------: 21
        Numero de gols -------: 12
        Numero de jogos ------: 2
        Media de gols por jogo: 6.00
        [4] ------------------------------------
        Nome -----------------: lolipop
        Idade ----------------: 21
        Numero de gols -------: 13
        Numero de jogos ------: 2
        Media de gols por jogo: 6.50
        [5] ------------------------------------
        Nome -----------------: lolipop
        Idade ----------------: 21
        Numero de gols -------: 14
        Numero de jogos ------: 2
        Media de gols por jogo: 7.00

Preste atenção ao código para entender a saída...

Como compilar isso?

Acrescente os parâmetros corretos na compilação e use um compilador dessa década ao menos.

PS C:\Users\toninho\source\repos\chc21-0606-pilha> gcc -v
Using built-in specs.
...
COLLECT_GCC=C:\bin\sys64\mingw64\bin\gcc.exe
...
COLLECT_LTO_WRAPPER=C:/bin/sys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.2.0/lto-wrapper.exe       
Target: x86_64-w64-mingw32
...
Supported LTO compression algorithms: zlib zstd
gcc version 10.2.0 (Rev6, Built by MSYS2 project)  
PS C:\Users\toninho\source\repos\chc21-0606-pilha>

gcc -v mostra a versão do compilador que está usando. Apenas rode isso em sua m'`aquina e saberá o que tem.

No site do gcc pode ver as datas de liberação das versões, mas acho que algo além de 7 estará bem. 9.3 é uma versão comum.

Os ... são porque eu cortei umas linhas da saída porque não acrescentam nada aqui.

Compilando com gcc 10.2 por exemplo

PS C:\Users\toninho\source\repos\chc21-0606-pilha> gcc -c -Wall  main.c        
PS C:\Users\toninho\source\repos\chc21-0606-pilha>

-c é para não gerar código, só compilar. E -Wall é para mostrar todos os erros, mesmo aqueles de baixa prioridade, os tais avisos.

Mas usando a versão de C de 89...

PS C:\Users\toninho\source\repos\chc21-0606-pilha> gcc -c -Wall -std=c89 main.c
main.c: In function 'main':
main.c:35:5: error: C++ style comments are not allowed in ISO C90
   35 |     // outro jeito, mais flexivel
      |     ^
main.c:35:5: note: (this will be reported only once per input file)
main.c:55:5: error: 'for' loop initial declarations are only allowed in C99 or C11 mode
   55 |     for( int i=0; i<alfa.n_jogadores; i+=1 ) alfa.atleta[i].ngols = 10+i;
      |     ^~~
main.c:55:5: note: use option '-std=c99', '-std=gnu99', '-std=c11' or '-std=gnu11' to compile your code
main.c:58:14: error: redefinition of 'i'
   58 |     for( int i=0; i<alfa.n_jogadores; i+=1 )
      |              ^
main.c:55:14: note: previous definition of 'i' was here
   55 |     for( int i=0; i<alfa.n_jogadores; i+=1 ) alfa.atleta[i].ngols = 10+i;
      |              ^
main.c:58:5: error: 'for' loop initial declarations are only allowed in C99 or C11 mode
   58 |     for( int i=0; i<alfa.n_jogadores; i+=1 )
      |     ^~~

Pode ver tudo que deu erro e também como corrigir... Então faça isso...

É preciso acompanhar a evolução da linguagem nesses 60 anos para poder produzir e ler os programas modernos nas empresas e escolas de hoje em dia.

Se certificando...

PS C:\Users\toninho\source\repos\chc21-0606-pilha> gcc -c -Wall -std=c17 main.c
PS C:\Users\toninho\source\repos\chc21-0606-pilha>

RESUMO: acrescente -std=c17 e -Wall nas opções de compilação do Dev-C++ e desse Code::Blocks.

Considere usar ambientes IDE mais modernos.

https://gcc.gnu.org/releases.html

Esse é o site do compilador gcc. A versão corrente é 11.3 lançada em abril de 2021. Isso nada tem a ver com o IDE.

outros compiladores importantes são o clang, o CL, os compiladores da Intel e da IBM

Entrar

C dificuldade em completar exercício em C

Posts recomendados

Helio Junnior Sartor Dalmolin

devair1010

Helio Junnior Sartor Dalmolin

devair1010

Helio Junnior Sartor Dalmolin

arfneto

devair1010

arfneto

Crie uma conta ou entre para comentar

Criar uma conta

Entrar

Quem está navegando aqui 0 usuários estão online

Últimos tópicos

Livros

Cursos

Conteúdo recente

Sobre o Clube do Hardware

Direitos autorais

Mais

Fórum

Atividades

Artigos

Análises

Livros

Cursos