Discussion :

[sergebourlon]

Bonjour,

On doit faire un devoir pour jeudi, sur les fonctions.
Voici les instructions :

# créez une fonction qui s'appelle :
# joue_du_banjo
# Si votre prenom commence par la lettre "R" ou "r", alors vous jouez du banjo
# La fonction prend un prénom comme seul argument et retourne l'une des chaines suivantes :
# prenom + "Joue du banjo"
# prenom + "ne joue pas du banjo"

Le prof nous a mis le début du code comme suit :

---------------------------------------------------------

def joue_du_banjo(prenom):

if prenom[0]=="r" or prenom[0]=="R":
return prenom + "joue du banjo"
else :
return prenom + "ne joue pas du banjo"
---------------------------------------------------------------

Par la suite, on doit terminer le codage en demandant à l'utilisateur d'entrer son prénom, la fonction fait le reste.

J'ai placé ceci en fin de code, mais ça ne fonctionne pas :

---------------------------------------------------------------

prenom=print("Ecrivez votre prénom: ")
input()
print("Dans ce groupe musical", joue_du_banjo(prenom))

-----------------------------------------------------------------


Pouvez-vous m'aider ?

[umfred]

L'erreur dit que prenom est de type None et que ce type est non "subscriptable" (on ne peut pas accéder à un de ses éléments), il reste à savoir pourquoi prenom serait de type None et non string, donc regarde la doc de print() et ce quelle retourne (vu que tu affectes à prenom le retour du print)
Selon toi, à quoi sert la fonction input() ? (pareil, regarde sa doc)

En regardant ces 2 points, tu devrais trouver ton erreur je pense.

[Sve@r]

Bonjour
umfred a tout dit. Tout vient de ta façon dont tu associes la chaine "Eric" à la variable prenom, variable qui est ensuite passée à la fonction.
Je rajouterais que input() peut afficher une chaine (donc inutile de mettre un print() pour ça) et que les règles du forum demandent que les codes soient placés directement dans les posts plutôt que dans une image inutilisable (perso je ne vais pas m'embêter à recopier ton code manuellement parce que tu as eu la flemme de le faire via un copier/coller qui aurait été pourtant mille fois plus facile que de mettre cette image).
Et pour ta fonction, le prof est bien sympa mais il a fait ça au plus simple. Quand tu auras l'habitude, tu verras que tu peux faire ça en une ligne.

Code python :

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

1
2
def joue_du_banjo(prenom):
    return prenom + ("joue du banjo" if prenom[0]=="r" or prenom[0]=="R" else  "ne joue pas du banjo")

ou mieux encore en utilisant les formatages de strings

Code python :

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

1
2
def joue_du_banjo(prenom):
    return "%s %s du banjo" % (prenom, "joue" if prenom[0] in "rR" else "ne joue pas")