Discussion :

[neon29]

Bonsoir à tous,

Alors voilà, actuellement en 1ere année de licence info je dois créer un petit programme pour résoudre les équations du second degré.

J'ai çà:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
Public Class Form1
Dim a, b, c As Integer
Dim dt, x0, x1, x2 As String
a = InputBox ( " Valeur de a ? " )
b = InputBox ( " Valeur de b ? " )
c = InputBox ( " Valeur de c ? " )
dt = b^2-(4*a*c)
If (dt>0) Then
x1 = (-b+dt^(1/2))/(2*a)
x2 = (-b-dt^(1/2))/(2*a)
MsgBox( " Les solutions sont " & x1 & x2 )
Else
If (dt=0) Then
x0 = -b/(2*a)
MsgBox ( " La solution est " & x0 )
Else
MsgBox ( " Pas de solution " )
EndIf
EndIf
 
End Class

Mais j'ai 16 erreurs... Sans savoir d'où çà peut venir.

J'utilise VB express 2008.
Les erreurs apparaissent quand je tentent un debugage.

[smyley]

Quelles sont ces 16 erreurs ?
ps. :

Code :

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

MsgBox( " Les solutions sont " & x1 & x2 )

ça ne serait pas plutôt

Code :

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

MsgBox( " Les solutions sont " & x1 & " et " & x2 )

?

Code :

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

1
2
3
a = InputBox ( " Valeur de a ? " )
b = InputBox ( " Valeur de b ? " )
c = InputBox ( " Valeur de c ? " )

C'est laid. Peut être que VB.Net le permet en convertissant à la volée de String vers Integer (j'en sais rien, je ne l'aime pas ce langage) mais si on rentre comme valeur "maman", quelle nombre va-t-il avoir ? Il faut donc vérifier les valeurs rentrées ou au moins mettre à Try-Catch pour afficher l'erreur avec grâce.

Pareil, rien ne te dit que a, b, c et x1 x2 x0 sont des entiers. D'ailleurs, ils sont pas nombreux les polynômes qui admettent des entiers comme racines. Même s'il y en a une infinité, ce ne sera sans doute pas toujours le cas. Donc il faudrait voir peut être en Double ou Float.

Et enfin c'est plutôt dommage qu'on ne t'ai pas demandé de résoudre aussi les polynômes dans C

[Pol63]

si tu as 16 erreurs, elles sont affichées et expliquées dans la fenetre "liste d'erreurs" (menu affichage)

et en effet c'est pas très réfléchi ce code

@smyley : vb autorise pas mal de choses par défaut ... mais c'est paramétrable

[smyley]

@smyley : vb autorise pas mal de choses par défaut ... mais c'est paramétrable

C'est vraiment mal. Aucune erreur de compilation à convertir "vache" en Integer, et à l'exécution on mange les pissenlits par la racine.

Je crois qu'il y a un truc genre

Code :

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

1
2
3
 
Option Strict
Option Explicit On

Qu'il faudrait utiliser pour éviter de caster des vaches en integer ...

[harz62]

Moi je dis il faut éviter d'utiliser les fonctions spécifiques au langage à savoir msgbox, inputbox... pour 2 raisons:

--> Ces fonctions représente un frein (selon moi) pour migrer de VB6 à VB.NET dans le sens où le programmeur va croire qu'il n'y pas beaucoup de changements et continuer d'essayer de programmer en procédural
--> Ces fonctions ne sont valables que pour ce langage donc quand on change de langage managé bah ça marche pas car en C#, C++/CLI ça existe pas.

Je serais plutôt d'avis d'utiliser les classes du framework qui sont elles compatibles avec tous les langages managés.

Pour ce qui est du casting, je suis d'accord avec smyley autoriser une conversion explicite entre un object et un int sans présence de cast représente une erreur classique de compilation tout à fait logique si on suit bien le concept objet.

[neon29]

Moi je dis il faut éviter d'utiliser les fonctions spécifiques au langage à savoir msgbox, inputbox (...)

J'ai pas le choix pour l'instant c'est les deux seules fonctions que l'on a droit d'utiliser.

[OhMonBato]

Je crois qu'on s'écarte un peu du problème d'origine.
Il y a peut etre un petit quelquechose qui cloche, dans quel événement est écrit ton code ? Aucun. Dans quelle procédure ? Aucune

Amuse toi à couper coller ton code pour le mettre dans l'événement FormLoad par exemple et regarde combien d'erreurs tu auras encore.

Evidemment, le FormLoad n'est peut etre pas le meilleur endroit ou mettre un code de ce type, ce serait peut etre mieux sur le click d'un bouton mais c'est juste pour le principe.

[yroubag]

J'ai trouvé tes 16 erreurs.

Le seul problème, techniquement parlant, est que ton code n'est pas dans une procédure.