Check if Characters in ArrayList C# exist - C# (2.0)

Question

I was wondering if there is a way in an ArrayList that I can search to see if the record contains a certain characters, If so then grab the whole entire sentence and put in into a string. For Example:

list[0] = "C:\Test3\One_Title_Here.pdf";
list[1] = "D:\Two_Here.pdf";
list[2] = "C:\Test\Hmmm_Joke.pdf";
list[3] = "C:\Test2\Testing.pdf";

Looking for: "Hmmm_Joke.pdf"
Want to get: "C:\Test\Hmmm_Joke.pdf" and put it in the Remove()

    protected void RemoveOther(ArrayList list, string Field)
    {
        string removeStr;

        -- Put code in here to search for part of a string which is Field --
        -- Grab that string here and put it into a new variable --
        list.Contains();
        list.Remove(removeStr);

    }

Hope this makes sense. Thanks.

You are blanking out the Field variable. This will clear any value passed into the function. Just to be clear, in your example you want to remove the element that contains "Hmmm_Joke.pdf" from the arrayList? — Joseph Carrigan
– Joseph Carrigan, Commented Sep 25, 2013 at 20:18
Yeah I understand that. But what I am looking for is in my list how can I search for that "Hmmm_Joke.pdf" and then grab that whole sentence "C:\Test\Hmm_Joke.pdf" and then put it in the Field string then Remove it. from the list. — user2800287
– user2800287, Commented Sep 25, 2013 at 20:20
Does it have to be case sensitive? Does it have to delete all files with the same name in different folders? — Dorival
– Dorival, Commented Sep 25, 2013 at 20:25

Karl Anderson · Accepted Answer · 2013-09-25 20:27:23Z

4

Loop through each string in the array list and if the string does not contain the search term then add it to new list, like this:

string searchString = "Hmmm_Joke.pdf";
ArrayList newList = new ArrayList();

foreach(string item in list)
{
    if(!item.ToLower().Contains(searchString.ToLower()))
    {
        newList.Add(item);
    }
}

Now you can work with the new list that has excluded any matches of the search string value.

Note: Made string be lowercase for comparison to avoid casing issues.

edited Sep 25, 2013 at 20:27

answered Sep 25, 2013 at 20:20

Karl Anderson

34.9k12 gold badges69 silver badges83 bronze badges

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

6 Comments

Joseph Carrigan Over a year ago

That is what I was going to propose. If you want to do something else with it return a to the string with return item;

Dorival Over a year ago

Perhaps it's good to make sure if it's case sensitive. I would also use LINQ to traverse the list. Since it's unlikely to know if we will have different Hmm_Joke.pdf in different folders. Ref: codeproject.com/Questions/411787/…

JG in SD Over a year ago

This code will not work, it will throw an InvalidOperationException, the ArrayList cannot be modified while it is being iterated upon.

user2800287 Over a year ago

On the foreach().... Exception Details: System.InvalidOperationException: Collection was modified; enumeration operation may not execute.

Karl Anderson Over a year ago

Updated answer to account for InvalidOperationException.

|

JG in SD · Accepted Answer · 2013-09-25 20:26:05Z

1

In order to remove a value from your ArrayList you'll need to loop through the values and check each one to see if it contains the desired value. Keep track of that index, or indexes if there are many.

Then after you have found all of the values you wish to remove, you can call ArrayList.RemoveAt to remove the values you want. If you are removing multiple values, start with the largest index and then process the smaller indexes, otherwise, the indexes will be off if you remove the smallest first.

answered Sep 25, 2013 at 20:26

JG in SD

5,6373 gold badges40 silver badges47 bronze badges

Comments

Casperah · Accepted Answer · 2013-09-25 20:29:29Z

0

This will do the job without raising an InvalidOperationException:

string searchString = "Hmmm_Joke.pdf";
foreach (string item in list.ToArray())
{
    if (item.IndexOf(searchString, StringComparison.OrdinalIgnoreCase) >= 0)
    {
        list.Remove(item);
    }
}

I also made it case insensitive.

Good luck with your task.

answered Sep 25, 2013 at 20:29

Casperah

4,5841 gold badge21 silver badges13 bronze badges

Comments

Community · Accepted Answer · 2017-05-23 10:31:56Z

0

I would rather use LINQ to solve this. Since IEnumerables are immutable, we should first get what we want removed and then, remove it.

        var toDelete = Array.FindAll(list.ToArray(), s =>
            s.ToString().IndexOf("Hmmm_Joke.pdf", StringComparison.OrdinalIgnoreCase) >= 0
            ).ToList();

        toDelete.ForEach(item => list.Remove(item));

Of course, use a variable where is hardcoded.

I would also recommend read this question: Case insensitive 'Contains(string)' It discuss the proper way to work with characters, since convert to Upper case/Lower case since it costs a lot of performance and may result in unexpected behaviours when dealing with file names like: 文書.pdf

edited May 23, 2017 at 10:31

CommunityBot

11 silver badge

answered Sep 25, 2013 at 20:56

Dorival

6894 silver badges19 bronze badges

Collectives™ on Stack Overflow

Check if Characters in ArrayList C# exist - C# (2.0)

4 Answers 4

6 Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

6 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related