Parsing binary search as an arraylist in C#

Question

Is there any possible way to make this code shorter?

            int? index = ListOfPeople.BinarySearch(searchBar.Text);
            int? temp = int.TryParse(index.ToString(), out int i) ? (int?)1 : null;
            MyPeopleGrid.SelectionMode = DataGridViewSelectionMode.RowHeaderSelect;
            MyPeopleGrid.Rows[i].Selected = true;
            MyPeopleGrid.CurrentCell = MyPeopleGrid.Rows[i].Cells[0];

ProgrammingLlama · Accepted Answer · 2018-08-18 05:48:21Z

1

You seem to have seriously overcomplicated things, and you certainly should never convert a number to a string only to parse it back to a number.

int index = ListOfPeople.BinarySearch(searchBar.Text);

This will return a non-negative number when the item isn't found. It does not return int?, it returns int.

So now you can use it:

int index = ListOfPeople.BinarySearch(searchBar.Text);
if (index >= 0)
{
    MyPeopleGrid.SelectionMode = DataGridViewSelectionMode.RowHeaderSelect;
    MyPeopleGrid.Rows[index].Selected = true;
    MyPeopleGrid.CurrentCell = MyPeopleGrid.Rows[index].Cells[0];
}
else
{
    // do something when the item isn't found
}

edited Aug 18, 2018 at 5:48

answered Aug 18, 2018 at 5:24

ProgrammingLlama

39.3k7 gold badges78 silver badges104 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

ProgrammingLlama Over a year ago

OK, what are you actually trying to do then? Why do you need a nullable int? What's that for?

ProgrammingLlama Over a year ago

Can you post some more code? I don't understand why you're not just using ArrayList.BinarySearch(searchBar.Text); without the extra stuff.

ProgrammingLlama Over a year ago

What are you using temp for?

Collectives™ on Stack Overflow

Parsing binary search as an arraylist in C#

Is there any possible way to make this code shorter?

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

Is there any possible way to make this code shorter?

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related