sortedlist question

[Crash893]

Hi all

I'm totaling up a list of items in a Binding source and I am just having a small problem with syntax for incrementing the value of a key if it already exists


any help is greatly appreciated line 16 below

SortedList mySortedList = new SortedList();
            foreach (DataRowView view in myBindingSource)
            {
                if ((string)view["group_name"] == "USAT-Desktop") 
                {
                    view["group_name"] = "TEST TEST";
                }
               
                
               if (mySortedList.Contains(view["category_name"])== false)
               {
                   mySortedList.Add(view["category_name"], 1);
               }
               else
               {
               //how do I ++ the value of the key if it already exists?
               //mysortedlist[view["category_name"]].value +=1? 
               }

[penagate]

++mysortedlist[view["category_name"]].value;

surely?

[Crash893]

i got

mysortedlist[view["category_name"]] = 1+(int)mysortedlist[view["category_name"]];

[Crash893]

++mysortedlist[view["category_name"]].value;

surely?

there is not .value option for this did you get this to work or are you just sudo coding

Either one is fine i appreciate the help i just want to know if im missing something or not.

[jmcilhinney]

This is obviously C# 2005 if you are using a BindingSource, but you should still be specifying when you post. After more than 550 posts you should be able to do that without asking. It is one click of a radio button.

As it's C# 2005 you should be using a generic collection, not the standard type. That way your values will be typed and no casting is required:

SortedList<string, int> list=new SortedList<string,int>();
string name;

foreach (DataRowView row in this.bindingSource1)
{
	name = (string)row["Name"];

	if (list.ContainsKey(name))
	{
		list[name]++;
	}
	else
	{
		list.Add(name, 1);
	}
}

With a standard SortedList it would have to look like this:

SortedList list = new SortedList();
string name;

foreach (DataRowView row in this.bindingSource1)
{
	name = (string)row["Name"];

	if (list.ContainsKey(name))
	{
		list[name] = (int)list[name] + 1;
	}
	else
	{
		list.Add(name, 1);
	}
}

[Crash893]

This is obviously C# 2005 if you are using a BindingSource, but you should still be specifying when you post. After more than 550 posts you should be able to do that without asking. It is one click of a radio button.

As it's C# 2005 you should be using a generic collection, not the standard type. That way your values will be typed and no casting is required:

thanks

I never thought about it before I will make sure to note that in the future.