Discussion :

[Duloup]

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 :

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

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 :

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

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 :

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

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

[Duloup]

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

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
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")

[Guigui_]

A mon avis, l'"erreur" vient de la ligne:

Code :

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

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 :

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

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 :

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

1
2
def copy(self, img):
    self.data=list(img.data)

puis de remplacer la ligne par:

Code :

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

self.copy(res)

au passage:

au lieu d'écrire

Code :

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

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

tu peux faire

Code :

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

 for i in range(size):

et au lieu

Code :

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

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

tu peux faire

Code :

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

self.data=size*[255]

[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 .

[Guigui_]

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é

[Duloup]

Ok merci beaucoup maintenant c'est clair !