Méthode de classe et copie d'objets

Version imprimable

Bonjour à tous,

Je suis débutant en python et j'ai un ptit problème :
je viens de créer une classe Image et de lui ajouter une méthode qui devrait lisser l'image courante (à l'aide d'une bête moyenne).
Le problème vient du fait que les modifications faites au sein de la classe ne semble pas être répercutés sur l'instance de celle ci ...
Voici le code de la méthode :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 def lissage(self): res = copy.deepcopy(self) for j in range(1, self.height-2): for i in range(1, self.width-2): val = self.getPixel(i,j) + self.getPixel(i+1,j) + self.getPixel(i-1,j) +\ self.getPixel(i,j+1) + self.getPixel(i,j-1) + self.getPixel(i+1,j+1) +\ self.getPixel(i+1,j-1) + self.getPixel(i-1,j+1) + self.getPixel(i-1,j-1) res.setPixel(i,j,val/9) print "Self : ", self.getPixel(109,100), ", Res : ", res.getPixel(109,100) self = copy.deepcopy(res) print "Self : ", self.getPixel(109,100), ", Res2 : ", res.getPixel(109,100) self.writeToPGM("moyenne1.pgm")
et son utilisation :
Code:

1 2 3 4 5 img = Image("test.pgm") img.lissage() print "Img : " , img.getPixel(109,100) img.writeToPGM("moyenne2.pgm")
Le problème est donc que le programme me produit la sortie suivante :
Code:

1 2 3 4 Self : 108 , Res : 109 Self : 109 , Res2 : 109 Img : 108
on voit donc que les modifications faites au sein de la méthode ne sont pas effectives (ceci est confirmé par le fait que l'image moyenne1.pgm est bien lissée alors que moyenne2.pgm ne l'est pas ...).

Voilà d'où pensez vous que le problème peut venir ?
Sans doute des copies ?

Merci d'avance

Voici le code de ma classe Image en entier si besoin :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
import string
import copy
 
class Image:
    """ Représente une image."""
 
    def __init__(self, w, h):
        self.width  = w
        self.height = h
        size = self.width * self.height
        for i in range(0, size - 1):
            self.data.append(255)
 
    def __init__(self, fileName):
        self.readPGM(fileName)
 
    def getId(self, i, j):
        return (self.width*i + j)
 
    def setPixel(self, i, j, val):
        self.data[self.getId(i,j)] = val
 
    def getPixel(self, i, j):
        return self.data[self.getId(i,j)]
 
    def writeToPGM(self, fileName):
        f = open(fileName,'w')
        f.write("P2\n")
        f.write("# Created with Python !\n")
        f.write(str(self.width) + " " + str(self.height) + "\n")
        f.write("256\n")
        size = self.width * self.height
        for i in range(0, size - 1):
            f.write(str(self.data[i]) + "\n")
        f.close()
 
    def readPGM(self, fileName):
        f = open(fileName, 'r')
        f.readline() # Le magic number
        f.readline() # Les commentaires        
        line = f.readline() # Les dimensions de l'image
        l = string.split(line)
        self.width  = int(l[0])
        self.height = int(l[1])
        line = f.readline()
        # Les données
        size = self.width * self.height
        self.data = []
        for i in range(0, size - 1):
            line = f.readline()
            self.data.append(int(line))
        f.close()
 
    def lissage(self):
        res = copy.deepcopy(self)
        for j in range(1, self.height-2):
            for i in range(1, self.width-2):
                val = self.getPixel(i,j)     + self.getPixel(i+1,j)   + self.getPixel(i-1,j)   +\
                      self.getPixel(i,j+1)   + self.getPixel(i,j-1)   + self.getPixel(i+1,j+1) +\
                      self.getPixel(i+1,j-1) + self.getPixel(i-1,j+1) + self.getPixel(i-1,j-1)
                res.setPixel(i,j,val/9)
        print "Self : ", self.getPixel(109,100), ", Res  : ", res.getPixel(109,100)
        self = copy.deepcopy(res)
        print "Self : ", self.getPixel(109,100), ", Res2 : ", res.getPixel(109,100)
        self.writeToPGM("moyenne1.pgm")

11/04/2005, 16h46
Guigui_
A mon avis, l'"erreur" vient de la ligne:

Code:

self = copy.deepcopy(res)

self n'est plus après cette instruction égale à img (je n'en suis pas sûr du tout). Mais je regarderais plutôt de ce côté-là.

essaie en créant toi-même la fonction copy du genre:
Code:

1 2 3 4 def copy(self, img): for i in range(img.height): for j in range(img.width): self.SetPixel(i,j,img.getPixel(i,j))
ou plus simple
Code:

1 2 def copy(self, img): self.data=list(img.data)
puis de remplacer la ligne par:

Code:

self.copy(res)

au passage:

Citation:

au lieu d'écrire

Code:

for i in range(0, size - 1):

tu peux faire

Code:

for i in range(size):
Citation:
et au lieu

Code:

1 2 for i in range(0, size - 1): self.data.append(255)

tu peux faire

Code:

self.data=size*[255]
11/04/2005, 17h12
Duloup

Merci ça fonctionne bien.
Je vais continuer à regarder pour essayer de bien comprendre pourquoi la copie ne fonctionnait pas.

Merci également pour les petites astuces pour les boucles :D .
11/04/2005, 17h17
Guigui_

Citation:

Envoyé par Duloup

Je vais continuer à regarder pour essayer de bien comprendre pourquoi la copie ne fonctionnait pas.

C'est pas que la copie ne fonctionne pas, c'est que faire
self = ... en gros recrée un objet self (donc à une nouvelle adresse mémoire), ce qui fait que lorsque tu quittes la fonction, ton objet est en fait inchangé

Edit:
il ne faut pas mettre le "-1" => for i in range(0, size), la borne supérieure est exclue du range. Dans tes images de test, il doit manquer une ligne à priori quand tu as sauvegardé
11/04/2005, 17h27
Duloup

Ok merci beaucoup maintenant c'est clair !