Discussion :

[annie123456]

Bonjour,
je dois faire une fonction qui retire les elements dun vecteur divisible par un diviseur donner... voici lenonce
parametres dentree: x,a,b, c
Je dois retirer les elements du vecteur x divisible soit par a b ou c

Code :

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

1
2
 
y = retrait(x,a,b,c).

Voici ce que jai fait mais aucun resultat...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
function y=retrait(x,a,b,c)
%x=vecteur initial
%a,b,c= diviseurs de x
%y=vecteur resultant apres retrait des nbrs divisibles par a,b ou c
n=length(x);
for i=1:n
if rem(x(i),a)==0 || rem(x(i),b)==0 || rem(x(i),c)==0
x(i)=[];
end
end
y=x;

Merci !!!

[Invité]

Bonjour,

Ton code présente un problème: lorsqu'il y a effectivement des valeurs divisibles par a, b ou c, la taille de ton vecteur diminue, et de ce fait la boucle for ne peux plus aller jusqu'à n (longueur initiale de ton vecteur) et tu obtiens une erreur "index out of bounds".

Pour remédier à ce problème, tu peux passer soit par une boucle while

Code :

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

1
2
3
4
5
6
7
8
i = 1;
while i<=length(x)
    if rem(x(i),a)==0 || rem(x(i),b)==0 || rem(x(i),c)==0
        x(i)=[];
    else
        i = i+1;
    end
end

Soit juste remplacer ta ligne for par

Code :

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

for i=n:-1:1

Je te propose aussi un code optimisé utilisant l'indexage logique:

Code :

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

x(~rem(x,a) | ~rem(x,b) | ~rem(x,c)) = [];

[tubaas]

ou (équivalent à méthode d'indexage logique de winjerome)

Code :

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

y = x(rem(x,a) & rem(x,b) & rem(x,c));