Discussion :

[nouby]

Salut


Ma classe Humain contient bien l'attribut Camp mais une fonction ne trouve pas cet attibut.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
class Humain(C_player) :
    def __init__(self) :
        super().__init__()
        # ***************** Caractéristiques ***************
        self.Nom="Humain"
        self.Camp=0

La fonction qui ne sait pas trouver l'attribut Camp :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
def Cree_vaisseau(nom,arme) :
 
    # Attribue la classe Vaisseau()
    vaisseau = Vaisseau
 
    vaisseau.Nom=nom
 
    # Définit les armes voulues.
    vaisseau.Armes=arme
 
    if joueur.Camp==0 :
        # Etc....

Le message d'erreur :

if joueur.Camp==0 :
AttributeError: type object 'Humain' has no attribute 'Camp'


EDIT :

Apparement, j'ai plus le message d'erreur après avoir ajouté des parenthèses ici : vaisseau = Vaisseau()

Qu'en dites-vous ?

[josmiley]

J'en dit que les majuscules sont mal utilisées et que ça serait plus clair si on avait une idée du type de jeu concerné.
Quelle est l'utilité d'une fonction cree_vaisseau s'il existe déjà une classe Vaisseau ?

[fred1599]

Hello,

Le message d’erreur donne une indication importante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
AttributeError: type object 'Humain' has no attribute 'Camp'

La partie type object 'Humain' signifie que joueur référence probablement la classe Humain elle-même, et non une instance de cette classe.

Vous devez sans doute avoir quelque chose de ce genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
joueur = Humain

Dans ce cas, la méthode init() n’est jamais exécutée, donc l’attribut Camp n’est pas créé.

Il faut instancier la classe avec des parenthèses :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
joueur = Humain()

Votre attribut étant défini ainsi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
self.Camp = 0

il appartient aux instances de Humain, et non directement à la classe Humain.

C’est exactement le même problème avec cette ligne :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
vaisseau = Vaisseau

Elle ne crée pas de vaisseau : elle place simplement la classe Vaisseau dans la variable vaisseau.

La bonne écriture est donc bien :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
vaisseau = Vaisseau()

Mais cette correction ne peut pas, à elle seule, corriger l’erreur concernant joueur.Camp. Il faut également vérifier comment la variable joueur est initialisée.

Votre fonction pourrait par exemple être écrite ainsi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
def cree_vaisseau(nom, arme, joueur):
    vaisseau = Vaisseau()
    vaisseau.nom = nom
    vaisseau.armes = arme
 
    if joueur.camp == 0:
        # ...
        pass
 
    return vaisseau

Et appelée de cette manière :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
joueur = Humain()
vaisseau = cree_vaisseau("Mon vaisseau", arme, joueur)

À noter également que, selon les conventions Python, on utilise généralement :

• Humain et Vaisseau pour les noms de classes ;
  cree_vaisseau, joueur, nom, armes et camp pour les fonctions, variables et attributs.

Enfin, concernant la remarque de josmiley, une fonction cree_vaisseau() peut être utile si elle joue le rôle de fabrique et contient une logique de création particulière. Si elle ne fait qu’appeler Vaisseau() et affecter deux attributs, il serait probablement plus simple de passer ces informations directement au constructeur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
class Vaisseau:
    def init(self, nom, armes):
        self.nom = nom
        self.armes = armes

Puis :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
vaisseau = Vaisseau("Mon vaisseau", arme)

[nouby]

merci beaucoup Fred, comme chaque fois tes réponses sont toujours des supers cours où j'apprend des choses !!! J'ai bien fais de poser la question

Joesmiley, salut !
c'est un shooter, le jeu, et l'utilité de la fonction vaisseau est de créer des instances d'un type de vaisseau mais avec une logique particulière comme l'a ainsi nommée Fred.
vala, vala...

[josmiley]

Je veux bien s'il s'agit de créer un vaisseau de façon interactive, le joueur choisi les caractéristiques par exemple, sinon je ne vois pas vraiment. La fonction retournera une instance de Vaisseau normalement, à moins qu'elle initialise en plus des variables plus globales, auquel cas elle est mal nommée.
Mais bon, je chipote 😁