copie de tableau boucle for

Bonjour à tous,
Encore un petit soucis avec une boucle for pour copier un tableau de string dans un autre tableau ..

Code:

---

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

int j=0; string[] sydm = new string[6] {"A", "", "B", "", "C", "D"}; string[] valX = new string[7];   for (var i = 0; i < sydm .GetUpperBound(0) + 1; i++) { if (sydm [i] != "") { j = j + 1; valX[j] = sydm [i];   } }

---

J'essaie de ranger le tableau valx de la manière suivante :
{"A", "B", "C", "D", "", ""};
J'arrive à copier A mais pas les autres..B ou C ..

En VB.net ça donne un truc comme ça

Code:

---

1 2 3 4

For i = 0 To 5 If sydm(i) <> "" Then k = k + 1 : valX(k) = sydm(i) Next i

---

pouvez vous me dire d’où vient l'erreur dans le code c#
Merci de votre aide.
Greg

Voici la solution toute bête

Code:

---

Symdme.CopyTo(valX, 0);

---

:oops:

Parfois faut savoir laisser un problème de coté et y revenir après!!!
Greg

Bon bas çà fonctionne pas!!!!!!

Bonjour,

Il ne faut pas forcement chercher à faire la traduction littérale du code entre deux langages :).

Donc ce que tu cherches à faire, c'est copier sydm dans valX, et ensuite trier ce dernier.

C'est bien tu as réussi la copie par le bout de code que tu as montré, il reste ensuite à le trier.

Code:

---

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

sydm.CopyTo(valX, 0);   //Tri du tableau Array.Sort(valX, (a, b) => { if (string.IsNullOrEmpty(a)) return 1; else if (string.IsNullOrEmpty(b)) return -1;   return a.CompareTo(b); });

---

J'utilise le string.isNullOrEmpty() car le tableau contient une valeur nul ( valX à une taille supérieur de 1) .

La documentation du Array.Sort => http://www.csharp-examples.net/sort-array/

Bon code,

Merci WaterTwelve21 pour t'on aide..
petit problème le tableau valx est rangé par ordre alphabétique, il faut qu'il garde l'ordre d'origine du tableau sydm.

Si string[] sydm = new string[6] {"B", "", "D", "", "A", "E"}; alors valx {"B", "D", "A", "E", "", ""}.

Le a.CompareTo(b) range les éléments du tableau par ordre alphabétique.
J'essaie de trouvé un contournement au problème dans la doc que tu m'as fournis..

Encore merci pour ton aide
Greg

Linq :

Code:

---

1 2	var array = new[] { "A", "", "\t", "E", null, "C", "", "B", " " }; var result = array.GroupBy(s => string.IsNullOrWhiteSpace(s)).SelectMany(g => g).ToArray(); // ["A", "E", "C", "B", "", "\t", null, "", " "]

---

Merci Noxen de t'on aide, je connais pas trop linq mais le code ne fonctionne pas dans tous les cas de figures:
Si je fais ça:

Code:

---

1 2 3

string[] array = new[] { "", "", "", "E", "", "C", "A", "B", "" }; string[] result = array.GroupBy(s => string.IsNullOrWhiteSpace(s)).SelectMany(g => g).ToArray();

---

le code renvoie "";"";"";"";"";"E";"C";"A";"B" alors qu'il me faudrait "E";"C";"A";"B";"";"";"";"";""

j'ai essayé avec le code suivant

Code:

---

var result = array.Where(s => !string.IsNullOrWhiteSpace(s)).ToArray();

---

mais là il m'efface les "vides" et redimensionne le tableau , ça donne "E";"C";"A";"B"

Je suis donc repartis sur une boucle for

Code:

---

1 2 3 4 5 6 7

for (int i = 0; i < array.Length; i++) { if (array[i] != "") { j = j + 1; valX[j] = array[i]; } }

---

qui me donne
""; "E";"C";"A";"B";"";"";""
On dira que c'est un pis-aller !!!!
Encore merci a tous pour votre aide
Greg

Citation:

---

Envoyé par gregm954

Code:

---

1 2 3 4 5 6 7 8

for (int i = 0; i < array.Length; i++) { if (array[i] != "") { valX[j++] = array[i]; } }

---

---

Bonjour,

Vous pouvez essayer d'incrémenter la variable j après, et non avant, d'affecter la valeur courante.
Actuellement, étant donné que vous incrémentez j avant, le premier élément est vide. Ou alors, initialisez j à -1 :)

Cordialement.

Bien vu. Effectivement, j'ai été trompé parce-que le premier élément de mon tableau n'était pas une chaîne vide, du coup c'était le premier groupe créé. Ceci-dit la correction n'est pas très compliquée (j'aurais d'ailleurs du l'envisager en premier) :

Code:

---

1 2	string[] array = new[] { "", "", "", "E", "", "C", "A", "B", "" }; string[] result = array.OrderBy(string.IsNullOrWhiteSpace).ToArray();

---

Apparemment en C# true est avant false ; ce serait probablement différent en C / C++ qui les traiteraient comme des entiers (à vérifier...). Par ailleurs en cas d'égalité sur l'évaluation de deux éléments OrderBy préserve l'ordre actuel.

Citation:

---

J'essaie de ranger le tableau valx de la manière suivante :
{"A", "B", "C", "D", "", ""};

---

Il me semblait à la lecture que tu voulais trier dans l'ordre alphabétique, pas de soucis alors, il suffit de remplacer le compareTo par 0

Code:

---

1 2 3 4 5 6 7 8 9

Array.Sort(valX, (a, b) => { if (string.IsNullOrEmpty(a)) return 1; else if (string.IsNullOrEmpty(b)) return -1;   return 0; });

---

Citation:

---

J'essaie de ranger le tableau valx de la manière suivante :
{"A", "B", "C", "D", "", ""};
J'arrive à copier A mais pas les autres..B ou C ..

---

Deja le code vb.net comporte un erreur...l'indice j devrait être incrémenter après l'assignation au tableau valX...
Ensuite il faut initialiser valX avec des "whitespace" ou espace blanc (< "">....
Le tri exige une ligne de code : Array.Sort<string>(tableau);

code behind .cs revu (ungredients: un bouton et 2 textbox):

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

namespace WindowsFormsApplication1 { public partial class Form2 : Form { public Form2() { InitializeComponent(); }   private void button1_Click(object sender, EventArgs e) { string[] sydm = new string[6] { "A", "", "C", "", "D", "B" }; textBox1.Clear(); //on affiche ton sydm: for (var i = 0; i < sydm.GetUpperBound(0) + 1; i++) { textBox1.Text += sydm[i];   }   // on remplit valX de "whitespace" comme ils disent à microsoft string blancheneige = ""; string[] valX = new string[sydm.Length]; for (var i = 0; i < valX.GetUpperBound(0) + 1; i++) { valX[i] = blancheneige; }     // copie des lettres int j = 0; for (var i = 0; i < sydm.GetUpperBound(0) + 1; i++) { if (sydm[i] != blancheneige) { valX[j] = sydm[i]; // indice deporté apres j = j + 1; } }   //on affiche ton valX: textBox1.Text += Environment.NewLine; for (var i = 0; i < valX.GetUpperBound(0) + 1; i++) { textBox1.Text += valX[i]; }   // on trie ton valX: textBox2.Clear();   textBox2.Text += "valx trie"; textBox2.Text += Environment.NewLine; //le maudit tri Array.Sort<string>(valX); for (var i = 0; i < valX.GetUpperBound(0) + 1; i++) { textBox2.Text += valX[i]; } }     } }

---

bon code...

@MABROUKI

Cela fait beaucoup de bricolage pour un simple tri non ? :D

Et je précise que son tableau de copie n'a pas la même taille que celui d'origine ( pourquoi ? aucune idée )

Citation:

---

string[] sydm = new string[6] {"A", "", "B", "", "C", "D"};
string[] valX = new string[7];

---

Bonjour à tous,
WaterTwelve21, pour la taille du tableau , au départ la boucle for que j'avais écrite étant mal foutu, elle écrivait à l'indice 1 du tableau et non pas 0...je sais pas si je suis bien claire!!!
Bon en tous cas, le code de Noxen , fonctionne bien..en tous cas pour l'instant je n'ai pas détecté d'erreur .

Merci encore à tous d'avoir pris un peu de temps pour mon problème.
Bonne journée
Greg