Discussion :

[Aspic]

Si ton code marchait ^^ mais pas pour le cas où il n'y avait rien dans la case.

C'est vraiment gentil de m'avoir donné le code. Vu que je suis débutante, j'aurais jamais pu trouver toute seule. Je dois rendre le projet avant lundi, et le programme à encore quelques soucis.
Donc j'apprend aussi vite que je peux..

Il fallait simplement rajouter le code sperot51:

Code :

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

If text_box.text = "" then exit sub

Pour que cela marche dans tous les cas =)

Mais il est sur que lorsqu'il propose son aide, il pourait etre beaucoup plus délicat, surtout avec les jeunes demoiselles qui sont dans mon cas.

Encore une fois, merci Aspic, et quand meme merci sperot, meme si t'aurais pu prendre des gants pour adoucir ton language épineux qui risque d'en blesser certains.

Tu aurais pu mettre une condition du style

Code :

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

1
2
3
If Text_Box.Text <> "" then
 'executer mon code
Endif

Comme ca plus de probleme ! Le genre de condition avec des exit sub je n'aime pas trop c'est moche dans le code mais c'est vrai que pas mal de gens l'utilisent (je ne critique pas ton code Sperot51 )

Sinon ce forum est ouvert à tous donc pas de soucis pour les débutant(e)s s'ils font un minimum de recherches par eux meme (et ce n'est pas souvent le cas ).

Je pense que Sperot51 a été un peu dur mais ce n'est pas la mort ! Ca arrive a tout le monde de se lacher

[Lutine]

Merci pour tes conseils =)

J'y avais pensé, mais je ne savais pas trop manipuler.. Les hésitations d'une débutante..

Mais bon, maintenant ca marche, et j'en suis contente ^^

Pour le tag résolu, il me semble que je l'avais déjà mis ^^

Merci encore !

[Pol63]

les exit sub sont pourtant recommandés pour diverses raisons
clarté du code
ca évite d'executer du code pour rien en testant les cas ou il n'y a rien à faire au début et en sortant tout de suite
plutot que de passer sur tous les ifs (pas dans ce cas où il n'y en a qu'un) ...

[SaumonAgile]

Tu aurais pu mettre une condition du style

Code :

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

1
2
3
If Text_Box.Text <> "" then
 'executer mon code
Endif

Le problème est que lorsque tu as plusieurs conditions d'arret comme ça et que tu ne peux pas les regrouper sous une seule, tu obtiens une indentation pas possible avec 14 niveaux de code avant d'avoir un morceau de code utile.
Par exemple :

Code :

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

1
2
3
4
5
6
If conditionDeContinuite1 then
    ' Faire quelquechose
    If conditionDeContinuite2 then
        ' Faire quelquechose
    Endif
Endif

Quand tu as plus de 2 ou 3 conditions, tu peux remplacer par :

Code :

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

1
2
3
4
5
6
If conditionArret1 then exit sub
' Faire quelquechose
If conditionArret2 then exit sub
' Faire quelquechose
If conditionArret3 then exit sub
' Faire quelquechose

C'est tout de même plus agréable que de faire défiler 2 écrans horizontalement.

Mais après ça sors complètement du sujet

PS : remarque Aspic que j'ai écrit du VB.NET . Comme quoi, tout arrive

[Aspic]

Oui ca dépens des cas, pour les cas simples (avec une seule condition) j'evite le exit sub. Mais c'est mon choix perso

PS : remarque Aspic que j'ai écrit du VB.NET . Comme quoi, tout arrive

Oui je vois ca ^^ tu apprends vite

PS 2 : Remarque que moi je poste aussi en C# (pas tout le temps) dans les forums adequates

NB : Je suis en bas débit dans la campagne pommé au milieu de nulle part c'est pas le top pour editer mes posts