Discussion :

[See Tenkay]

Bonjour voici mon premier message !

J'ai créé le code suivant pour chercher si un mot est dans une phrase.
Le code fonctionne bien mais j'ai un résultat en trop : None.
J'ai ce problème en local (python 3.9) et sur le site d'auto-formation que je suis.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
text = "this is awesome text"
word = "awesome"
 
def search(text, word):
    list = text.split(" ")
 
    for i in range(0,len(list)):
        if word == list[i]:
            return print("Word found")
        else:
            continue
 
    return print("Word not found")
 
print(search(text, word))

J'obtiens le résultat:

Code :

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

1
2
3
 
Word found
None

Je ne comprend absolument pas d'où viens le "None"..
La liste ne comporte que des mots, pas de valeurs vides
La boucle for fait son travail..

Des idées ?
Merci d'avance pour toute information qui me permettra d'avancer !

[See Tenkay]

Ben ça commence fort!..
Je viens de trouver la réponse,
je pense que le problème était présent dès le départ dans la question:

l'appel de la fonction était présenté de la manière suivante "print(search(text, word))"
si vous enlevez le print, le none disparait...
je pense que je ne suis pas le seul à qui cette question liée à une erreur d'énoncé (volontaire?) a posé un long temps de recherche

"hope it will be useful"

résolu.

[jmv]

Bonjour,

Erreur d'énoncé ? pas si sûr !
return print("n'importe quoi") n'a aucun sens puisque print() ne renvoie jamais rien.
Que devait faire la fonction search() ? retourner ou afficher une message ?
C'est une erreur fréquente chez les débutants de confondre print() et return.
Peux-tu poster l'énoncé ?

Cordialement,
jmv

[BioKore]

En fait, tout va dépendre de l'utilisation que tu souhaites faire de ta fonction.
Si le but est UNIQUEMENT d'afficher la valeur à l'écran, dans ce cas là, retire les return et laisse juste print(...).
Si tu comptes utiliser ta fonction comme étape intermédiaire pour d'autres utilisations, dans ce cas, laisse les return mais supprime les print.
Si tu souhaites absolument faire les deux, dans ce cas, fais un print et en suite retourne la valeur

[wiztricks]

Salut,

l'appel de la fonction était présenté de la manière suivante "print(search(text, word))"

L'énoncé dit juste que la fonction search retourne une chaine de caractère qui sera affichée par "print". Vous avez écrit une fonction qui "affiche" et retourne None.

La différence peut paraître subtile car le résultat est le même... mais c'est tout ce qui fait la complexité des fonctions: paramètres d'appels, objets retournés...

- W

[See Tenkay]

Effectivement, cela semble plus correct de faire un print d'abord et un return pour terminer.
Merci à tous pour vos commentaires intéressants! J'en tiens déjà compte.

[Sve@r]

Bonjour

l'appel de la fonction était présenté de la manière suivante "print(search(text, word))"
si vous enlevez le print, le none disparait...
je pense que je ne suis pas le seul à qui cette question liée à une erreur d'énoncé (volontaire?) a posé un long temps de recherche

Ce n'est pas une erreur d'énoncé, c'est juste un énoncé volontairement simplifié dans le but de t'apprendre à faire des déductions.
Si on te dit que tu dois utiliser la fonction (parce que dans print(search(...) c'est bien ce qui se passe, tu fais bel et bien une "utilisation" de la fonction ou plus précisément de ce qu'elle retourne), alors il faut que ta fonction "retourne" une valeur. Ca reste impératif. Tu débutes donc tu as tenté un return print(...) mais ce faisant tu as retourné ce que "print" retourne, c'est à dire rien (donc None par défaut) d'où le "None" que tu affiches au final.
C'est la subtilité de l'apprentissage des fonctions. Généralement une fonction fait un travail et retourne le résultat de son travail, rien de plus. Et tout ce qu'elle peut afficher à l'écran entre temps ce n'est pas assimilable à un quelconque "retour", c'est juste un affichage.
Exemple: def carre(n): return n*n. Cette fonction n'affiche rien, elle effectue juste son job. Si maintenant l'apelant veut afficher ce qu'elle produit, alors c'est à lui de demander explicitement cet affichage => print(2 * carre(8) + 12).
Et quand on va plus loin, on se rend compte alors qu'en réalité, les affichages intermédiaires deviennent parfois plus une gêne qu'autre chose et donc deviennnent de moins en moins importants. Et dans les gros projets, on sépare alors bien distinctement les fonctions destinées aux travaux des fonctions destinées à afficher le résultat de ces travaux.

Effectivement, cela semble plus correct de faire un print d'abord et un return pour terminer.

Comme je viens de le dire, tu verras à l'usage que les affichages intermédiaires te seront de moins en moins utiles.