which is the best way

Question

0.00/5 (No votes)

See more:

I need to add onclick method to textbox inside grid.In this scenerio i can bind the client id the get the value of that grid item inside javascript method or can bind directly the value to the javascript method which is the best way for implementation

1. binding the client id and getting the value
txtbx1.attributes.add("onclick","Getvalues('txtbx1')");

If i am passing id javascript will search complete html,so performance will be less

2. Bind value directly to the method
txtbx1.attributes.add("onclick","Getvalues('20')");

No need to search get the value directly performance will be good

My answer is 2 .Please suggest your answers

Posted 20-May-14 2:06am

Srikanth954

Updated 20-May-14 2:20am

v2

Add a Solution

Comments

[no name] 20-May-14 8:10am

If you already know the answer, then why are you asking?

Srikanth954 20-May-14 8:15am

No i implemented the second one as per the performance i want to confirm whether my assumption is correct.please answer your view

[no name] 20-May-14 8:28am

Why? You asked and answered your own question so there is nothing here to give an answer to. And this is not a proper place to perform code reviews IMO.

Srikanth954 20-May-14 8:32am

Not like that i did the code and hosted in live environment.It is working fine but just my doubt is whether my assumption is correct or not.It should fail in any cases.

[no name] 20-May-14 8:58am

So in other words, you are obsessing about a non-existent problem?

Stephen Hewison 20-May-14 8:10am

This is swings and roundabouts really. I don't think it makes much difference. I'd choose option 1 because then you could update the value in the client without having to rewrite the event binding.

Srikanth954 20-May-14 8:16am

Thankyou for the reply.All are read only fields. just i need to validate

Stephen Hewison 20-May-14 8:51am

I think you're worrying too much. Most people are pragmatic. They'll ask two questions about your end product. Does it do what they want and does it do it fast enough? If the answer to both is yes then it doesn't really matter. If you're only doing something, such as raising a click event, once as part of a process then it really doesn't matter. If you're doing something thousands of times in quick succession then performance becomes more important. In which case, sent a up your own bench mark tests. Try each method 1000s of times until you see a difference in performance and then pick the best one.

Richard MacCutchan 20-May-14 9:02am

Why have you tagged this as C#3.5 when it is a Javascript question?

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

thatraja · Answer 1 · 2014-05-20T05:47:00

You're doing Option.1 in wrong way. You were sending only control id to javascript function. I think you're creating object there to get value from textbox after matching the element. That's why the slow.

Here better way(than existing one). I'm just getting the value & sending to javascript function.

C#

txtbx1.attributes.add("onclick","Getvalues('" + txtbx1.Text + "')");

One more suggestion(alternative).
Go with jQuery instead of javascript. Using Click()[^] event you could reduce the code.