Discussion :

[Titi41]

Bonjour,

J'ai un petit problème avec cette fonction, elle ne récupère pas mon élément.

voila le code de ma fonction

Code :

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

1
2
3
4
5
6
7
8
9
10
    function Count(text, lon, textBox) 
    {
        var maxlength = new Number(lon);
 
        if(document.getElementById(textBox).value.length > maxlength)
        {
                    text.value = text.value.substring(0,maxlength);
                    alert(" Only " + lon + " chars");
        }
    }

J'appelle cette fonction lorsque l'utilisateur ajoute ou enlève un caractère dans un textbox. Je pense que le problème vient du fait que j'utilise la fonction dans un user control .ascx affiché sur une page .aspx. (je développe donc en C#)
Il faut peut etre changer le parametre "document" ?

Merci

[tusssss]

Je ne suis pas certain du value.length sur ton document.getElementById

Un document.getElementById(textBox).length devrait suffir non ?

[Titi41]

J'ai testé, et non. Ca ne change rien.
Mon erreur est "Object Required"

Ce code marche bien dans une page aspx simple. Je pense que c'est vraiment un problème avec le getelement

[SpaceFrog]

tu peux nous montrer le html de l'objet visé ???
il a bien un id ... la casse est respectée lors du passage de paramètre ?
il n'y a qu'un objet avec cet id sur la page ?

[E.Bzz]

Bonjour,

Je ne suis pas certain du value.length sur ton document.getElementById

Un document.getElementById(textBox).length devrait suffir non ?

Non non : c'était bien ça (c'est la value qui est une string, pas l'objet lui-même).

Par contre :

Code :

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

1
2
3
4
5
6
7
8
9
10
function Count(text, lon, textBox) 
    {
        var maxlength = new Number(lon);
 
        if(document.getElementById(textBox).value.length > maxlength)
        {
                    text = text.substring(0,maxlength);
                    alert(" Only " + lon + " chars");
        }
    }

Une variable n'a pas de propriété value : elle représente elle-même la value ...

... et il est possible de trouver des noms de variables plus clairs et moins ambigües que "text"

A+

[Titi41]

Voila comment le l'appelle

Code :

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

1
2
3
4
5
6
7
8
9
10
11
            <tr>
                <td>
                    Comment Employee
                </td>
                <td>
                    <asp:TextBox ID="tbCommentEmployee" 
                        textmode="MultiLine" Rows="5" runat="server" 
                        onKeyUp="javascript:Count(this, 100,'tbCommentEmployee');" 
                    onChange="javascript:Count(this, 100,'tbCommentEmployee');"></asp:TextBox>
                </td>
            </tr>

Il n'y a biensur qu'un control avec cet ID. Je ne met pas tout ma page, elle fait plus de 250 lignes.

[SpaceFrog]

oui comme le souligne Ebzz .. il faudait que nous en sachions un peu plus sur les paramètres ???

[Titi41]

Les paramètres :
Text : Texte de la textbox
lon : nombre de caractère maximum
Textbox : id de ma textbox

[E.Bzz]

oui comme le souligne Ebzz .. il faudait que nous en sachions un peu plus sur les paramètres ???

Je corrigeais juste l'utilisation de la variable qui était fausse (variable.value).

Ce n'était peut être que ça le problème

A+

[E.Bzz]

@Titi41 > Tu as corrigé ça ?

A+

[SpaceFrog]

ben déja il y a un paramètre en trop ...

Code :

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

1
2
3
4
5
6
7
8
9
10
function Count( lon, textBox) 
    {
        var maxlength = new Number(lon);
        var text=document.getElementById(textBox).value;
        if(text.length > maxlength)
        {
                    document.getElementById(textBox).value = text.substring(0,maxlength);
                    alert(" Only " + lon + " chars");
        }
    }

pas besoin de passer l'id et le texte ...

Ensuite au niveau ergonomie imagine que le gars fasse une insertion en mileu de chaîne ???

[Titi41]

ca ne marche toujours pas.. Sinon, est-ce qu'il y a une meilleure fonction déjà existante ? Si ce n'est pas ergonomique.

[E.Bzz]

ca ne marche toujours pas..

Si les autres ont compris où tu en étais dans tes modif, ils sont forts

=> ton code actuel ??

A+

[Titi41]

Oui, désolé

En fait, j'ai repris le code de SpaceFrog

Code :

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

1
2
3
4
5
6
7
8
9
10
function Count( lon, textBox) 
    {
        var maxlength = new Number(lon);
        var text=document.getElementById(textBox).value;
        if(text.length > maxlength)
        {
                    document.getElementById(textBox).value = text.substring(0,maxlength);
                    alert(" Only " + lon + " chars");
        }
    }

[E.Bzz]

OK !

Alors : résultat (à part que "ça marche pas") ?
Erreur ?

[SpaceFrog]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<script type='text/javascript'>
function Verif(Obj,MaxL){
		var Texte=Obj.value;
       if(Texte.length > MaxL){
                    Obj.value = Texte.substring(0,MaxL);
                    alert(" Only " + MaxL + " chars");
        				}
        }
</script>
 
<title>Nouvelle page 1</title>
</head>
<body>
<input type='text' onkeyup="Verif(this,5)" value='test' />

Sinon sans javascript mais sans message d'alerte:

Code :

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

1
2
 
<input type='text' maxLength="5"  value='test' />

ou avec une alerte :

Code :

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

1
2
 
<input type='text' maxLength="5" onkeydown="if(this.value.length==this.maxLength){alert('Only '+this.maxLength+' chars')}" value='test' />

[Titi41]

Toujours le meme problème, objet non trouve sur la ligne du GetElementbyID.
Ensuite, Lenght is null

[Titi41]

Je prefererai rester avec des controls asp. Et avec les controls asp, si on ajoute le mode "MultiLine", il ne gère plus la proriété maxlength

[SpaceFrog]

titi tu as testé mon code ...
???

il tourne parfaitement chez moi ...

[Titi41]

Ca ne marche pas chez moi. Je pense vraiment que le problème vient du user control. Et il ne trouve pas ma text box.
la variable "document" fait référence à la page ?
Il n'est pas possible de faire référence au user control ?