Sauf erreur, tes variables source et source2 sont déclarées comme String alors que tu leurs assignent False ou True dans un premier temps, puis le code source dans un second ... Y'a une logique de type à revoir je pense...
Même dans ta programmation y'a certaines logiques que je comprend pas ... Tu fais tes 2 chargements de code source en parallèle et ce en doublant toutes tes variables alors qu'il n'y en a pas l'utilité ... Tu alourdis ton code et occupe des réfs mémoires pour rien ...
En essayant de garder ta logique d'algo de base on peut raccourci un peu tout ça ...
Code:
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17
| Dim source As String
Dim website As String
Dim wcli As WebClient
For i As Integer = 0 To NumericUpDown1.Value
If ComboBox1.SelectedItem = "premier" Then
website = "http://premiersite.com=" & i
ElseIf ComboBox1.SelectedItem = "second" Then
website = "http://secondsite.com=" & i
End If
wcli = New webclient
source = wcli.DownloadString(website)
Invoke(New MethodInvoker(Sub() nameList.Items.Add(source)))
Next |
Et n'hésite pas à mettre un try catch dès fois que
.DownloadString retourne une exception (ou même pour le reste). Et également pense qu'il est possible que l'utilisateur ne sélectionne rien dans la combo mais qu'il clique quand même sur Button1, ce faisant ta variable website ne serait pas affecté avec "http://[...]" et donc ton downloadstring planterait ...