how can improve my code to work

Question

0.00/5 (No votes)

See more:

This code for getting checked rows in gridview

C#

private List<GridViewRow> GetCheckedRows(GridView grd)
    {
        List<GridViewRow> result = new List<GridViewRow>();
        foreach (GridViewRow gridRow in grd.Rows)
        {
            CheckBox chkBox = gridRow.FindControl("checkBoxName") as CheckBox ;
            if (chkBox != null && chkBox.Checked  )
            {
                result.Add(gridRow);
            }
        }
        return result;
    }

When I run it in the browser it dose not give me any response

How can I improve it to work?

and should be put in gridview event ?

Thank you

Posted 10-Apr-11 5:16am

dedoooo

Updated 10-Apr-11 5:23am

dasblinkenlight

v2

Add a Solution

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)

Sandeep Mewara · Answer 1 · 2011-04-10T05:46:00

Solution 1

When I run it in the browser it dose not give me any response
And what kind of response you are expecting from the above code?
All your code does is creates a set of GridRows whose checkbox was found as checked. Done.

Based on what the method looks like doing, all is fine and does not need any change/improvement.

It might be that you are looking for somthing else but you totally failed in expressing on what you were trying and needed here. For now, the code is fine and does not look like an issue in it.

Posted 10-Apr-11 5:46am

Sandeep Mewara

Comments

dedoooo 10-Apr-11 12:27pm

I want from the code to display the checked items in the page in order to print out in paper.

Sandeep Mewara 10-Apr-11 12:38pm

Well, your code has given you the rows that was selected. Now, use the List and do the rest - code for it.

dedoooo 10-Apr-11 12:46pm

What is the List? Did you mean an array?

Sandeep Mewara 10-Apr-11 12:50pm

Is it a copied code from somewhere? I ask as, this is your code where you made a method that returns "List<gridviewrow>" => Based on logic, a list of rows that has it's checkbox checked.

Not sure now, what you know, wrote and trying here, seriously.

Espen Harlinn 10-Apr-11 12:39pm

I believe I mentioned something about part of the code running in the browser, and part of it running on the server. Your c# codebehind runs on the server, so you have to do something that triggers communication between what you've got in the browser and your serverside code ... like a postback

Sandeep Mewara 10-Apr-11 12:49pm

Sounds like, you guys had a discussion on the same earlier and this is a followup question... is it?

Espen Harlinn 10-Apr-11 13:00pm

Encounter OP for the first time here:
http://www.codeproject.com/Answers/179439/Give-me-a-hint-to-my-project.aspx#answer3
As he indicated he was working on a "medical system", I thought he had progressed beyond fireing up Visual Studio :)
The related question can be found here though : http://www.codeproject.com/Answers/179571/correcting-my-small-code-in-csharp.aspx#answer1

Sandeep Mewara 10-Apr-11 13:08pm

Oh ok. Got it.

But, based on the interation with OP here, I really doubt he understands the things. Looks like copy-paste work going on istead of learning.

dedoooo 10-Apr-11 13:13pm

my school learn us the first trying to learn is searching about codes and test them and from small point can go to big one

Sandeep Mewara 10-Apr-11 13:30pm

first trying to learn is searching about codes and test them
You mean without understanding? Common... thats the wrong way in your school that you follow.

Copy-Paste->Test... move on to BIG. I diagree and discourage the approach. Sorry.

Espen Harlinn 10-Apr-11 14:10pm

I guess I gave you a long list of quality examples to work your way up from like:
http://msdn.microsoft.com/en-us/library/ms229601.aspx and http://www.asp.net/web-forms.
These are made by professionals to help you learn the .Net platform.

dedoooo 10-Apr-11 13:09pm

This is another code

all comments I read, but still I am not understand

and I tried to test small codes to start the big one.

where is the problem?

Ok , I will never ask again :(

Sandeep Mewara 10-Apr-11 13:32pm

Ok , I will never ask again :(
I would have encouraged, tried and would had helped, had you said something like "Ok, I will to learn and ask when I get stuck"

I am not saying you should not ask. Surely you should if you get stuck, but you should be *aware* of what you are doing and what the code is doing. If you have no clue and you ask for help - it simply means nothing. It would not help you or anyone who helps you.

Espen Harlinn 10-Apr-11 12:37pm

Nice effort :)

dedoooo 10-Apr-11 12:45pm

Espen
like what communiction between server and browser?

to understand this concept
thank you

Espen Harlinn 10-Apr-11 14:19pm

Here is a fairly nice article that may help you:
http://webtrends.about.com/od/webapplications/a/web_application.htm