une ligne qui ne s'exécute pas

Version imprimable

Bonjour, dans mon programme, j'ai ce bout de code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 protected void go_Click(object sender, EventArgs e)
    {
        ModalPopupExtender1.Hide();
        try
        {
            connection.Open();
            .....
            MyCmd.ExecuteNonQuery();
            connection.Close();
            fill();
            btnLocaliser.Attributes.Add("onClick", "javascript:findAddress('" + adresse.Text +  "','" + statut.Text + "'); return false;");
 
        }
        catch (SqlException ex)
        {
            go.Attributes.Add("onclick", "javascript: return alert('Erreur :"+ex.ToString()+"');");
        }
        finally
        {
            connection.Close();
        }
    }

tout marche bien sauf que la ligne :

Code:

btnLocaliser.Attributes.Add("onClick", "javascript:findAddress('" + adresse.Text +  "','" + statut.Text + "'); return false;");

ne s'exécute pas

avez vous une idée ?

02/06/2014, 16h44
yinyang

Bonjour,

Qu'est-ce que donne le HTML généré au niveau de ton "btnLocaliser" ?
Est-ce que tu retrouves ton appel JS ou le classique "doPostBack" ?
Quand tu cliques sur ton bouton, il n'y a pas de message d'erreur ?

Seb
03/06/2014, 10h08
Msysteme

mon bouton localiser contient :

Code:

<input type="submit" name="btnLocaliser" value="Localiser" onclick="javascript:findAddress('Avenue Said Daoudi, Kenitra, Morocco','Station test','34.26140183944268','-6.600465774536133','PK105, P4266','Mr X','Société','Régularisé'); return false;WebForm_DoPostBackWithOptions(new WebForm_PostBackOptions("btnLocaliser", "", true, "", "", false, false))" id="btnLocaliser" class="button" />

normalement si la ligne s'exécute, elle va m'afficher une infos sur ma page, mais rien ne se passe, et pas de message d'erreur
03/06/2014, 10h35
nid4mail
je ne m'y connais pas trop en javascript mais une chose m'etonne dans le code que tu as posté : il faudrait que tu vérifies la ponctuation (si dans ton adresse tu as une quote il faut penser à la "préparer" pour pas que elle soit interpretee comme du code dans ton bouton localiser).

moi j'utiliserais la fonction suivante pour éviter cela sur l'adresse qui est passée en parametre avant ton bouton : ca remplace ' par ' ' (sans l'espace)
Code:

1 2 3 4 public static string NettoieString(string strValeur) { return strValeur.Replace("'", "''").ToUpper(); }
de la même manière vérifie les virgules, doubles quotes etc...
03/06/2014, 11h24
yinyang

Je suis d'accord avec la remarque de Nid4Mail : ta méthode JS attend 2 paramètres mais dans le code que tu nous montres entre les "'" et les virgules, il y a > 2 paramètres passés.

Seb

j'ai revérifié le nombre de paramétres avec les valeurs envoyées, il n 'a ya pas d'erreur,
Je vous ai envoyé uniquement un exemple mais ma fonction réel est la suivante :

c#
Code:

1 2 btnLocaliser.Attributes.Add("onClick", "javascript:findAddress('" + adresse.Text + "','" + nmStation.Text + "','" + latitude.Text + "','" + longitude.Text + "','" + emplacement.Text + "','" + prop.Text + "','" + tpProp.Text + "','" + statut.Text + "'); return false;");
js :
Code:

1 2 function findAddress(adresse,nom ,lat,lgt,emplacement,prop,TP,statut){ ...}
html

Code:

<input type="submit" name="btnLocaliser" value="Localiser" onclick="javascript:findAddress('Avenue Chakib Arsalane','param1','34.2533146152774','-6.58913612365723','Param2','param3','Société ','Régularisé '); return false;WebForm_DoPostBackWithOptions(new WebForm_PostBackOptions("btnLocaliser", "", true, "", "", false, false))" id="btnLocaliser" class="button" />

09/06/2014, 15h29
DotNetMatt

Il y a un bug connu de longue date qui provoque l'encodage HTML des attributs, donc une apostrophe (simple quote) est encodée en ' ce qui rend impossible l'évaluation de l'expression javascript dans ton cas. Quelle version du .NET Framework utilises-tu ?

Merci pôur votre réponse

Code:

1
2
3
4
5
6
7
8
9
<compilation debug="true" targetFramework="4.0">
			<assemblies>
				<add assembly="System.Windows.Forms, Version=4.0.0.0, Culture=neutral, PublicKeyToken=B77A5C561934E089"/>
				<add assembly="System.Windows.Presentation, Version=4.0.0.0, Culture=neutral, PublicKeyToken=B77A5C561934E089"/>
				<add assembly="PresentationCore, Version=4.0.0.0, Culture=neutral, PublicKeyToken=31BF3856AD364E35"/>
				<add assembly="System.Data.Services.Client, Version=4.0.0.0, Culture=neutral, PublicKeyToken=B77A5C561934E089"/>
				<add assembly="System.Design, Version=4.0.0.0, Culture=neutral, PublicKeyToken=B03F5F7F11D50A3A"/>
				<add assembly="System.Web.Extensions.Design, Version=4.0.0.0, Culture=neutral, PublicKeyToken=31BF3856AD364E35"/></assemblies>
		</compilation>

Ok effectivement c'est sur le 4.0 que ce comportement étrange existe. En fait Microsoft est parti du principe que pour ajouter du script côté client, il ne faut pas passer par les attributs, mais par le ClientScriptManager. On peut trouver principalement 3 solutions sur le net :

Utiliser le ClientScriptManager
Surcharger la façon dont l'encodage est géré
Créer son propre control

Pour la première, il faut passer par quelque chose de ce genre :
Code:

1 2 Page.ClientScript.RegisterStartupScript(Me.GetType(), "callFindAddress", "function callFindAddress() { findAddress('" + adresse.Text + "','" + nmStation.Text + "','" + latitude.Text + "','" + longitude.Text + "','" + emplacement.Text + "','" + prop.Text + "','" + tpProp.Text + "','" + statut.Text + "'); }") btnLocaliser.Attributes.Add("onclick", "callFindAddress();");
Pour la seconde, il faut surcharger l'encodage des attributs HTML, et éventuellement celui du code HTML :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 public class HtmlEncodingOverride : HttpEncoder { protected override void HtmlAttributeEncode(string value, TextWriter output) { var sb = new StringBuilder(); var sw = new StringWriter(sb); base.HtmlAttributeEncode(value, sw); output.Write(sw.ToString().Replace("'","'")); } protected override void HtmlEncode(string value, TextWriter output) { var sb = new StringBuilder(); var sw = new StringWriter(sb); base.HtmlEncode(value, sw); output.Write(sw.ToString().Replace("'", "'")); } }
La seconde méthode (HtmlEncode) n'est pas obligatoire, mais dans certains cas elle est nécessaire. Ensuite, il faut déclarer ceci dans le Web.config :

Code:

<httpRuntime encoderType="MyNamespace.HtmlEncodingOverride, MyAssembly" />

Enfin, pour la troisième solution, il y a plein de tuto sur le net, donc je te laisse le soin de chercher.

Est-ce que ça solutionne ton problème ?

Bonjour, merci pour votre reponse, j'ai opté pour la premiere solution, d'ailleurs c'est cella que j'ai comprise :

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
 ModalPopupExtender1.Hide();
        try
        {
            connection.Open();
            string InsertQ = "INSERT INTO dbo.StationsServices(Name_station,Latitude,Longitude,Emplacement,Propriétaire,Type_Prop,Adresse,Ville,Statut) VALUES('"
                + nmStation.Text + "', '" + latitude.Text + "','" + longitude.Text + "','" + emplacement.Text + "','" + prop.Text + "','" + tpProp.Text + "','" + adresse.Text +
                "','ville','" + statut.Text + "');";
            SqlCommand MyCmd = new SqlCommand(InsertQ, connection);
            MyCmd.ExecuteNonQuery();
            connection.Close();
            fillStations();
 
                       Page.ClientScript.RegisterStartupScript(this.GetType(), "callFindAddress", "function callFindAddress() { findAddress('" + adresse.Text + "','" + nmStation.Text +
            "','" + latitude.Text + "','" + longitude.Text + "','" + emplacement.Text + "','" + prop.Text + "','" + tpProp.Text + "','" + statut.Text + "'); }");
 
            go.Attributes.Add("onclick", "callFindAddress();");
 
 
        }
        catch (SqlException ex)
        {
            go.Attributes.Add("onclick", "javascript: return alert('Erreur :"+ex.ToString()+"');");
        }
        finally
        {
            connection.Close();
        }

mais rien ne se passe,le bouton go ne veut pas se valider et quand j'enleve les deux lignes :

Code:

1
2
3
4
            Page.ClientScript.RegisterStartupScript(this.GetType(), "callFindAddress", "function callFindAddress() { findAddress('" + adresse.Text + "','" + nmStation.Text +
            "','" + latitude.Text + "','" + longitude.Text + "','" + emplacement.Text + "','" + prop.Text + "','" + tpProp.Text + "','" + statut.Text + "'); }");
 
            go.Attributes.Add("onclick", "callFindAddress();");

mon champ s'ajoute bien a ma base de données

13/07/2014, 18h00
popoels

Bonjour,

ne serait-ce pas la balise <script> </script> manquante ?
quand tu enregistres un script via le scriptmanager, il y'a un boolean pour indiquer s'il faut rajouter la balise <script ..> à ton bout de code. dans le cas contraire,

ClientScript.RegisterStartupScript(Me.GetType, "testkey", "function test() { alert('testfunction');}", True)