Bonsoir,

Je n'utilise peut-être pas la bonne manière d'initialiser un 'struct' à la ligne 14, il y a moyen de faire plus simple ?

Par exemple, est-il possible de résumer les lignes 14, 15 et 16 en une seule instruction ?

#include <stdlib.h>

typedef struct Player
{
	int id;
	int points;         // points de son dernier match
	float total_score;

} player;

player* new_player(int id)
{
	player* p = (player*) malloc(sizeof(player));
	p->id = id;
	p->points = 0;
	p->total_score = 0;

	return p;
}

int main(void)
{
	player* Alice = new_player(45);
	
	// ...
	
	free(Alice);
	return 0;
}

Variantes, depuis C99

player* new_player(int id)
{
    player* p = malloc(sizeof(player));
    *p = (player) { id, 0, 0 };
    return p;
}

ou

player* new_player(int id)
{
    player* p = malloc(sizeof(player));
    *p = (player) {.id = id };
 
    return p;
}

(designated initializers,  - les champs non mentionnés sont initialisés à 0 pour les entiers, NULL pour les pointeurs etc. C11 6.7.9/10)

Le compilateur, qui n'est pas la moitié d'un idiot pour peu qu'on lui demande gentiment d'optimiser, ne fabrique évidemment pas un "player" intermédiaire pour le copier ensuite.

-
Edité par michelbillaud 22 octobre 2019 à 14:48:30

Bonjour,

Ta fonction peut aussi retourner une simple copie

player new_player(int id)
{
  return (player){id, 0, 0};
}

int main(void)
{
  player Alice = new_player(45);

  //...

  return 0;
}