[C#] [Débutant] Optimisation ou bonne pratique en C#

Bonjour à tous.

Je débute et j'ai écris ce morceau de code :

Code:

---

1 2 3 4 5 6 7 8 9

string Adresse = BarreAdresse.Text; if (Adresse.Length < 4) { Adresse = "http://" + Adresse; } else { if (Adresse.Substring(0, 4) != "http") { Adresse = "http://" + Adresse; }; }

---

Je me dit que par deux foix j'utilise la même ligne "Adresse = "http://" + Adresse;".

L'un d'entre vous peut-il me montrer quel code équivalant faut-il écrire pour obtenir le même résultat et ceci optimisé pour qu'il soit le plus court possible ?

Merci.

Je dirais avec l'opérateur ou:

Code:

---

1 2 3 4 5

string Adresse = BarreAdresse.Text; if ((Adresse.Length < 4)||(Adresse.Substring(0, 4) != "http")) { Adresse = "http://" + Adresse; }

---

Merci cela marche et en plus la réponse est rapide !

J'avais bien pensé au "ou" mais je n'avais mis qu'un seul "|".
Donc ce ne marchait pas, je comprends maintenant pourquoi.

Ca marche aussi en mettant un seul |.
L'intérêt de || et de &&, c'est que le deuxième objet n'est pas évalué si c'est pas nécessaire. Par exemple (true || machin) renverra true sans jamais chercher à savoir si machin est vrai ou faux. Alors qu'avec (true | machin), on évalue machin, et on fait ensuite (true | true) ou (true | false) selon la valeur de machin.

Ce truc est au moins aussi vieux que le langage C :)

Citation:

---

Envoyé par Guulh

Ca marche aussi en mettant un seul |.

---

No cela ne marche pas, car si la chaîne contenue dans Adresse est inférieure à 4 caractéres cela provoque une erreur, voilà pourquoi je disais que cela ne marchait pas.

Soyons plus précis :) Ca compile, mais effectivement à l'exécution, le Substring lance une exception. Un exemple typique de l'intérêt du || par rapport au simple |. :)