Exception lors du debogage

Version imprimable

Bonjour,

Je souhaiterais me connecter sur une page web via un webBrowser..
J'ai d'on fait un groupBox contenant deux textBox et un bouton login.
Voici mon code :

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
 
	// bouton connexion
	private void btn_connect_Click(object sender, EventArgs e)
        {
            wb_viewer.Navigate("http://monsite.com/login/");
 
        }
        // WebBrowser
        private void wb_viewer_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
        {
            if (wb_viewer.Document != null)
            {
                txt_login.Text = null;
                HtmlDocument doc = wb_viewer.Document;
                HtmlElement elem = doc.GetElementById("login_field");
                HtmlElement elem2 = doc.GetElementById("password_field");
                HtmlElement elem3 = doc.GetElementById("btn_logon");
 
                elem.SetAttribute("value", txt_login.Text);
                elem2.SetAttribute("value", txt_psw.Text);
                elem3.InvokeMember("click");
 
                doc.Body.AppendChild(elem);
                wb_viewer.Navigate("http://monsite.com/mypage.php");
                gb_list.Visible = true;
            }
        }

Mais lorsque je passe le Débogage, l'appli va essayer de se connecter pour finir sur une exception

Citation:

L'exception NullreferenceException n'a pas été géré par le code utilisateur

En m'indiquant cette partie :

Code:

("value", txt_login.Text);

Comment faire?
Merci d'avance.

25/09/2011, 23h18
Olivier782

Bonsoir!

En débuggant, regarde lequel entre "txt_login" "ou "elem" est null.
26/09/2011, 10h23
EvexR

elem s'affiche en vert, le reste de la ligne en jaune.

Code:

elem.SetAttribute("value", txt_login.Text);
26/09/2011, 11h07
Olivier782

Met le curseur sur "elem" pour voir si c'est bien lui qui est null. Si oui, cela veut dire que doc.GetElementById("login_field"); renvoie null.
26/09/2011, 11h13
EvexR

Il est bien null, comment puis je passer ce problème?
26/09/2011, 11h47
Olivier782

Peux-tu nous afficher le code de la page html que tu essayes de parser?

L'évènement DocumentCompleted est appelé quand un WebBrowser a fini de charger un document. J'imagine que tu as une première page où tu remplis ton login et ton password, puis tu peux naviguer sur d'autres pages web. Mais pour chaque page que tu chargeras, ta méthode sera appelée. Si la page est chargée correctement, wb_viewer.Document sera différent de null et rentrera dans ton if. Et tu vas essayer à chaque fois de trouver le "login_field" ou "password_field".

Ais-je bien deviné ce que tu veux faire? Si non, explique moi un peu plus en détail ce que tu souhaites faire.
26/09/2011, 12h20
EvexR
Alors oui j'ai une page de connexion que j'appelle en cliquant sur le bouton de connexion
Code:

1 2 3 4 5 private void btn_connect_Click(object sender, EventArgs e) { wb_viewer.Navigate("http://monsite.com/login/"); }
Ensuite le WebBrowser intervient, affiche la page de de connexion, ou celle ci contient 2 textBox et un bouton ( identifiant, password et le bouton )

Donc mon appli contient deux groupBox une à gauche contenant les informations à remplir et une à droite contenant le WebBrowser.

Lorsque je clique sur mon bouton de connexion de l'application, je souhaiterais que celui ci entre les informations de mon compte utilisateur du site et passe à la page suivante.
qui est celle ci :

Code:

wb_viewer.Navigate("http://monsite.com/mypage.php");

aucune erreur dans le codage de base, l'application va bien charger la page de connexion quand on appui sur le bouton, entré les informations identifiant et mot de passe, cliquer sur le bouton logon du site..
l'application va commencer à passer à la page suivante... "mypage.php" commence à charger et l'erreur apparait.

mais lorsque j'essaye l'application depuis le dossier de ou celle ci est enregistrer, l'erreur n'est pas la mais l'application ne prends pas en compte les données de connexion.

Donc il faudrait que l'application garde les infos de connexion en mémoire et que je puisse naviguer d'une page à l'autre en gardant mes identifiants.

Ok c'est bien ce que je pensais.

A mon avis, au moment où ta page "mypage.php" se charge, ta méthode "wb_viewer_DocumentCompleted" est appelée. Elle va donc analyser la nouvelle page qui ne contient aucun élément de connexion (textBox et bouton).

Il faudrait que tu ajoutes une nouvelle condition, par exemple :
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 // Ce booléen ne se trouve pas dans ta méthode, mais en variable de classe ou globale bool connexion = true; if (wb_viewer.Document != null && connexion) { txt_login.Text = null; HtmlDocument doc = wb_viewer.Document; HtmlElement elem = doc.GetElementById("login_field"); HtmlElement elem2 = doc.GetElementById("password_field"); HtmlElement elem3 = doc.GetElementById("btn_logon"); elem.SetAttribute("value", txt_login.Text); elem2.SetAttribute("value", txt_psw.Text); elem3.InvokeMember("click"); doc.Body.AppendChild(elem); // Si ta page s'est correctement chargée, tu passes ton bool à false. // Et il faut mettre le connexion à false avant d'appeler la méthode Navigate, // sinon tu sera dans le meme cas que précédement. connexion = false; wb_viewer.Navigate("http://monsite.com/mypage.php"); gb_list.Visible = true; }
Tu peux utiliser autre chose qu'un booléen pour tester que la page actuelle n'est pas la page de connexion.

Si tu ne comprends pas mon idée, hésite pas à me redemander. ;)

J'ai effectuer quelques modifications, cela semble fonctionner. un avis?

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
// Bouton de connexion
        private void btn_connect_Click(object sender, EventArgs e)
        {
            wb_viewer.Navigate("http://monsite.com/login.php");
        }
        // WebBrowser
        bool connexion = true;
        private void wb_viewer_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
        {
            if (wb_viewer.Document != null && connexion)
            {
                wb_viewer.Document.GetElementById("login_field").SetAttribute("value", txt_login.Text);
                wb_viewer.Document.GetElementById("password_field").SetAttribute("value", txt_psw.Text);
                wb_viewer.Document.GetElementById("btn_logon").InvokeMember("click");
 
                connexion = false;
                // Redirection direct vers la page d’accueil des utilisateurs enregistrer
                // Affichage d'une groupBox
                gb_list.Visible = true;
            }
            }
        // bouton de deconnexion du site
        private void btn_disconnect_Click(object sender, EventArgs e)
        {
                wb_viewer.Navigate("http://monsite.com/logoff.php");
        }
        // bouton pour acceder à la page perso
        private void btn_mypage_Click(object sender, EventArgs e)
        {
            wb_viewer.Navigate("http://monsite.com/mypage.php");
        }
        }

Cela fonctionnera la première fois que tu voudras te connecter. Si tu te déconnectes puis te reconnecte, "connexion" sera toujours à false et cela ne fonctionnera plus.

Il faut détecter que tu es actuellement sur la page de connexion pour pouvoir atteindre le code suivant :
Code:

1 2 3 4 5 6 7 8 9 wb_viewer.Document.GetElementById("login_field").SetAttribute("value", txt_login.Text); wb_viewer.Document.GetElementById("password_field").SetAttribute("value", txt_psw.Text); wb_viewer.Document.GetElementById("btn_logon").InvokeMember("click"); connexion = false; // Redirection direct vers la page d'acceuil des utilisateurs enregistrer // Affichage d'une groupBox gb_list.Visible = true;
La solution utilisant un booléen n'est peut être pas la meilleure... :?

On pourrait plutôt faire un test sur la propriété "Url" du "wb_viewer.Document" pour déterminer si la page en cours est la page de connexion, et ainsi le test deviendrait qqch du genre :
Code:

1 2 3 4 5 if (wb_viewer.Document != null && wb.Document.Url == "http://monsite.com/login.php") { //... }
Et ne plus utiliser le bool, ce qui me semble plus juste.

26/09/2011, 17h11
EvexR
j'ai essayer avec ceci :
Code:

1 2 3 4 if (wb_viewer.Document != null || wb_viewer.Document.Url.ToString() == "http://monsite.com/login.php") { ..... }
car je ne pouvais utiliser "==" seul ( il y avait une erreur )
mais la j'en reviens avec la même erreur

Code:

wb_viewer.Document.GetElementById("login_field").SetAttribute("value", txt_login.Text);

exception non géré
26/09/2011, 17h26
Olivier782

Toujours une NullReferenceException? Ca plante au premier coup? avant le login? après? Il faut nous donner un peu plus de détails pour que l'on puisse t'aider.

Utilises-tu le débuggeur? Si tu mets un point d'arrêt au niveau de la ligne qui plante tu peux voir quelle le contenu de variables, sinon tu peux ajouter un espion.

Tu devrais décomposer cette grande ligne en 2 ou 3, comme tu avais fait au début afin de voir à quel moment ça plante.

Alors, toujours aucune erreur sur le code lui même, mais lorsque j'utilise le déboguage.

code reformer pour la partie qui plante :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
private void wb_viewer_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
        {
            if (wb_viewer.Document != null || wb_viewer.Document.Url.ToString() == "http://monsite.com/login.php")
            {
                HtmlDocument doc = wb_viewer.Document;
 
                HtmlElement elem = doc.GetElementById("login_field");
                elem.SetAttribute("value", txt_login.Text);
 
                // wb_viewer.Document.GetElementById("login_field").SetAttribute("value", txt_login.Text);
                wb_viewer.Document.GetElementById("password_field").SetAttribute("value", txt_psw.Text);
                wb_viewer.Document.GetElementById("btn_logon").InvokeMember("click");
 
                // Redirection direct vers la page d'accueil des utilisateurs enregistrer
                // Affichage d'une groupBox
                gb_list.Visible = true;
            }
            }

à nouveau la partie

Code:

elem.SetAttribute("value", txt_login.Text);

Elem est null.

edit :

le plantage arrive comme au début, j'entre dans les textBox mes identifiants, je clique sur le bouton "connect" et l'erreur apparait un peu avant le passage à la page suivante.

26/09/2011, 17h49
Olivier782
J'avais pas vu! ^^
Code:

1 2 if (wb_viewer.Document != null || wb_viewer.Document.Url.ToString() == "http://monsite.com/login.php")
Une fois que tu as validé ta page de connexion, cela change de page. "wb_viewer.Document" est bien différent de null, et donc on entre dans le contenu du if.

Il suffit de remplacer le || par && dans la condition :), comme ça on ne rentrera dans le code de la condition que si le document n'est pas null ET la page est différente de celle indiquée.
26/09/2011, 17h59
EvexR

le problème avec l'opérateur && c'est que... rien ne se passe au débogage =/
pas même l'insertion des mes infos dans les textBox de la page web.
26/09/2011, 18h08
EvexR
Code:

if (wb_viewer.Document != null && wb_viewer.Document.Url == "http://monsite.com/login.php")

erreur :

Citation:

L'opérateur '==' ne peut pas être appliqué aux opérandes de type 'System.Uri' et 'String'

Code:

if (wb_viewer.Document != null && wb_viewer.Document.Url.ToString() == "http://monsite.com/login.php")

Aucune erreur de codage mais rien ne se passe au debogage.
Code:

1 2 if (wb_viewer.Document != null || wb_viewer.Document.Url.ToString() == "http://monsite.com/login.php")
Aucune erreur de codage, au debogage, insertion de mes identifiants, et erreur NullReferenceException.
26/09/2011, 18h15
Olivier782
Ajoute des parenthèses pour bien séparer chaque condition (par sûr que ça soit la solution) :
Code:

1 2 if (wb_viewer.Document != null && (wb_viewer.Document.Url.ToString() == "http://monsite.com/login.php"))
Et met un point d'arrêt (ou un espion) sur cette ligne pendant que tu débugges pour voir la valeur de "wb_viewer.Document.Url.ToString()", s'il n'y a pas un caractère différent (majuscule/minuscule...)

Voila, donc le code fonctionne correctement ainsi. Merci pour ton aide.

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
 
        // bouton de connexion.
        private void btn_login_Click(object sender, EventArgs e)
        {
            wb_viewer.Navigate("http://monsite.com/login/");
 
        }
 
        private void wb_viewer_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
        {
            // Ajout de la condition if et vérification de la page de connexion
            if (wb_viewer.Document != null && (wb_viewer.Document.Url.ToString() == "http://monsite.com/login/"))
            {
 
                HtmlDocument doc = wb_viewer.Document;
 
                HtmlElement elem = doc.GetElementById("login_field");
                elem.SetAttribute("value", txt_login.Text);
 
                HtmlElement elem2 = doc.GetElementById("password_field");
                elem2.SetAttribute("value", txt_psw.Text);
 
                HtmlElement elem3 = doc.GetElementById("btn_logon");
                elem3.InvokeMember("click");
            }
 
        }
        // Ajout du bouton de déconnexion
        private void btn_logout_Click(object sender, EventArgs e)
        {
            wb_viewer.Navigate("http://monsite.com/logoff.php");
        }