idée pour améliorer mon code ?

Version imprimable

21/08/2011, 13h31
Imène_23

idée pour améliorer mon code ?

Bonjour à vous tous

je fait une petite fonction qui me merge deux classes d'une matrice, le résultat y est mais je veux l'améliorer surtout que ca marche sur une petite matrice mais si elle est plus grande j'ai peur que ca consomme beaucoup.

réunir deux classes n'est pas difficile, mais c'est la mise à jour des numéros que j'ai trouvé compliquer, par exemple j'ai 4 classes et je réunie la 1 et 2 j'aurais donc 3 classes 1,3,4 je dois les rendre 1,2,3.

j'ai utilisé

Code:

unique

pour mettre le résultat dans un vecteur, après je parcours, si la valeur du vecteur et égale à l'élément, je donne l'indice du vecteur à l'élément de la matrice.

alors si quelqu'un a une meilleur idée pour arriver au même résultat et surtout moin couteuse je suis preneuse.

et merci d'avance.
21/08/2011, 16h21
duf42

Bonjour,

Pourrais-tu nous montrer le code en question?

Duf

ok Duf le voici:

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
function mer = merg (X,a,b);
r= min (a,b);
g = max(a,b);
[nbrows,nbcols]=size (X);
for i=1:nbrows
    for j=1:nbcols 
        if X(i,j)== g
            X(i,j)=r;
        end
        mer (i,j)=X(i,j);
    end
end
%%%%%%%%%%%%%%%%%%%% et pour mettre à jour
f=unique(X);
f=f';
y=length (f);
for i=1:nbrows
    for j=1:nbcols
        for x=1:y
            if f(x)==X(i,j)
                if X(i,j)~=x
                X(i,j)=x;
                end
            end
            mer (i,j)=X(i,j);
        end
    end
end

23/08/2011, 09h28
duf42
Pour la première partie, ca doit pouvoir s'écrire de cette façon:
Code:

1 2 3 4 5 6 r= min (a,b); g = max(a,b); X(X==g) = r; mer = X;

Citation:

Envoyé par Imène_23

par exemple j'ai 4 classes et je réunie la 1 et 2 j'aurais donc 3 classes 1,3,4 je dois les rendre 1,2,3.

Essaie ceci :
Code:

1 2 3 4 5 6 X = [1 3 4 4 4 ; 3 1 1 1 3 ; 1 4 3 4 4 ] [uX,a,b] = unique(X); X(:) = b(:)
ce qui donne :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 X = 1 3 4 4 4 3 1 1 1 3 1 4 3 4 4 X = 1 2 3 3 3 2 1 1 1 2 1 3 2 3 3

Une autre solution:

Code:

1
2
3
X = [1 3 4 4 4 ; 3 1 1 1 3 ; 1 4 3 4 4 ];
idx = X>1;
X(idx) = X(idx)-1;

24/08/2011, 21h07
Jerome Briot
Une version compacte de la solution que j'ai donnée précédemment :
Code:

1 2 3 X = [1 3 4 4 4 ; 3 1 1 1 3 ; 1 4 3 4 4 ]; [pasbesoin,pasbesoin,X(:)] = unique(X);
27/08/2011, 23h54
Imène_23

Désolé pour le retard et merci infiniment Duf42,Dut et Winjerome pour votre aide, ca marche maintenant.:ccool: