Question

I have fixed the problem of it not listing the data which was related to a wrong field on the spawn dictionary but still i would like to question if the way I am approaching it is ok ?

I have a few checkboxs that if true should add check the item status and include it on the list.

So not really knowing what to do, this is what I came up with:

        HashSet<int> status = new HashSet<int>();
        if (OptionsForm.filterPlayer.Checked) status.Add(0);
        if (OptionsForm.filterEnemy.Checked) status.Add(1);
        if (OptionsForm.filterSummon.Checked) status.Add(2);
        if (OptionsForm.filterNPC.Checked) status.Add(3);
        if (OptionsForm.filterObject.Checked) status.Add(4);
        if (OptionsForm.filterMonster.Checked) status.Add(5);
        if (OptionsForm.filterGatherable.Checked) status.Add(6);
        if (OptionsForm.filterUnk.Checked) status.Add(7);

        var query = from SpawnList item in spawnEntities.Values
                    where status.Contains(item.Status)
                    orderby item.Name ascending
                    select item;

But currently it is not returning me any errors or items that should have been returned.

spawnEntities is a dictionary with uint, SpawnList.

SpawnList is a simple class:

public class SpawnList
{
    public string Name { get; set; }
    public int Status { get; set; }
    // some more data not needed for the question
}

Answer 1

it was my mistake, the status on the
dictionary was wrong in 1 field and i
was not getting what I wanted after
listing all the data I noticed it heh,
but is it ok to do it like I am doing
?

To avoid such mistakes, I would use enumerations instead of ints – just define

enum SomeStatus
{
    Player,
    Enemy,
    <.etc.>
}

and change

public int Status { get; set; }

into

public SomeStatus Status { get; set; }