Discussion :

[camvac]

Salut,

Je récupère une liste de fichiers et j'aimerais les mettre en forme. Je me suis donc créé le fichier de test suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
fileSystTbl = ['/', '/usr', '/var']
 
def convElement(list):
    for element in list:
        if element == '/':
            '/'.replace('/','root')
        else:
            element.strip('/')
 
print convElement(fileSystTbl)

Pour l'instant, il me renvoie "None", mais j'aimerais qu'il puisse me renvoyer la liste: root, usr, var.

Comment faire?

Merci d'avance pour vos réponses

[Guyome41]

C'est normale que python te renvoi "none" puisque ta fonction ne retourne rien.

[Gakusei]

Bonjour,

Aucun return dans votre fonction donc c'est normale que None soit renvoyé.

J'ai modifié votre code sur des détails:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
fileSystTbl = ['/', '/usr', '/var']
 
def convElement(list):
    for ind, element in enumerate(list):
        if element == '/':
            list[ind] = 'root'
        else:
            list[ind] = element.strip('/')
    return list
 
print convElement(fileSystTbl)

[dariyoosh]

Bonjour,


En python, list est un objet mutable et les objets mutables sont passés par référence lors d'appel de fonction. En conséquence, la fonction modifie directement l'objet. L'expression return n'est pas nécessaire dans ce contexte.

Comme nos amis ont indiqué, puisque vous n'avez pas de clause return dans votre code, Python ajoute implicitement une expression "return None". Étant donné que vous avez inclu l'appel de fonction directement dans l'appel de "print" c'est donc "None" que print va récupérer. La solution consiste donc à séparer ces deux appels.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
def convElement(list):
    for ind, element in enumerate(list):
        if element == '/':
            list[ind] = 'root'
        else:
            list[ind] = element.strip('/')
 
fileSystTbl = ['/', '/usr', '/var']			
convElement(fileSystTbl)
print(fileSystTbl)

Et on obtient

Code :

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

1
2
 
>>> ['root', 'usr', 'var']

[Gakusei]

En effet, list est un objet mutable d'où les détails de modification précédent.
En revanche, il est toujours bon de mettre un return de ce que l'on veut.
Car comme vous l'expliquez ne rien mettre est équivalent à mettre un return ou return None donc peu d'intérêt (qui peu le plus peu le moins).

[dariyoosh]

... En revanche, il est toujours bon de mettre un return de ce que l'on veut ...

Tout à fait d'accord avec vous Il faut toujours le mettre pour améliorer la lisibilité et la clarté du code. En C et en Java on peut utiliser le mot "void" afin de préciser que la fonction/méthode en question ne renvoie rien, mais en python ce n'est pas le cas pour les fonctions et d'ailleurs je me demande pourquoi il n'ont pas fait