Need Help On An Error In One Line Of Code And Something Else.

Question

4.00/5 (1 vote)

See more:

So I need help fixing this line of code, and I just can't figure it out..

C#

private void btnCheck_Click(object sender, EventArgs e)
        {

            if (lbAccounts.SelectedIndex < 0)
                return;

            if (!string.IsNullOrEmpty(txtUser.Text) && !string.IsNullOrEmpty(txtUser.Text))
                return; // stop processing if they have some data

            // otherwise get acccount:password, 
            // split it into acc and pass and write them to TextBoxes
            string value = lbAccounts.Items[lbAccounts.SelectedItem];
            // I am getting the error above
            string[] values = value.Split(':');
            if (values.Length != 2)
                return;  // wrong format        

            var acc = values[0];
            var pass = values[1];

            txtUser.Text = acc;
            txtPass.Text = pass;
            SendTestMail();
        }

If someone can help me do this that would be great... And also point me in the direction of how to make it keep reading and sending the next accounts in line in the listbox to the textbox. Thanks in advance :)

Posted 5-Jan-13 13:41pm

purps

Add a Solution

Comments

Jibesh 5-Jan-13 19:59pm

what error you are getting? you need to mention the error otherwise we cannot able to give a proper solution.

Jibesh 5-Jan-13 20:05pm

do not post duplicate questions.
http://www.codeproject.com/Questions/522381/5bC-23-5dNeedplusHelpplusOnplusAnplusErrorplusInpl

purps 5-Jan-13 20:19pm

The solution below worked, and now I face another problem. After the SelectedItem is checked, I want it to move down the list and keep going until there is no more to check. All help is appreciated

Jibesh 5-Jan-13 21:23pm

thats a different problem can you update your question with what you have done so far. you can use 'Improve Question' to update your question. see my comments to the earlier solution.

purps 5-Jan-13 21:58pm

Thank you so much :) I really appreciate it!

Jibesh 6-Jan-13 0:19am

You welcome!

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)

Jibesh · Accepted Answer · 2013-01-05T14:04:00

Solution 1

If you want to get the selected item from the list the below code is sufficient, hope SelectionMode is one and not multy.

C#

string value = Convert.ToString(lbAccounts.SelectedItem);

Posted 5-Jan-13 14:04pm

Jibesh

Comments

purps 5-Jan-13 20:26pm

Here is a screenshot of what I am talking about..
http://puu.sh/1K6y6
I want it to be so that when the Check button is pressed, all the accounts the the lbAccounts listbox get checked one by one, and there is no need to select the index and click button manually..

Jibesh 5-Jan-13 20:36pm

If you want to process all the items in the list dont go for Selected Item, instead iterate through the lbAccount.Items property something like this

foreach(object item in lbAccount.Items)
{
string listItem = Convert.ToString(item)
//add your logic
}