I have a string like
string Text = "012345678901234567890123456789";
and a List<int>
with indexes
List<int> Indexes = new List<int>() { 2, 4, 7, 9, 15, 18, 23, 10, 1, 2, 15, 40 };
with following restrictions
Text.length
what's the best way to remove characters from the text which are in the index list?
expected output:
035681234679012456789
Is there a more efficent way than
foreach (int index in Indexes
.OrderByDescending(x => x)
.Distinct()
.Where(x => x < Text.Length))
{
Text = Text.Remove(index, 1);
}
Update:
Here are the Benchmarks of the current answers (string
with 100.000 characters and List<int>
with length 10.000:
Gallant: 3.322 ticks
Tim Schmelter: 8.602.576 ticks
Sergei Zinovyev: 9.002 ticks
rbaghbanli: 7.137 ticks
Jirí Tesil Tesarík: 72.580 ticks
Here's a more or less elegant LINQ way:
Text = new string(Text.Where((c, index) => !Indexes.Contains(index)).ToArray());
It uses the overload of Enumerable.Where
that projects the index of the item in the sequence.
If you want the most efficient and not the most readable way and the text is really large you could use a HashSet<int>
instead of the list which doesn't allow duplicates and a StringBuilder
to create the new string:
var indexSet = new HashSet<int>(Indexes); // either create from the list(as shown here) or use it without your list
var textBuilder = new StringBuilder(Text.Length);
for(int i = 0; i < Text.Length; i++)
if (!indexSet.Contains(i))
textBuilder.Append(Text[i]);
Text = textBuilder.ToString();
Of course you could also use the HashSet<int>
in the LINQ approach to make it more efficient.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With