System.ArgumentOutOfRangeException linq c#

Question

I have to make a program that takes the domain for each columns.

My code is this, but when i want show throws an exception (System.ArgumentOutOfRangeException).

string[] all_lines =  {  "1, 2, 3" , "4, 5, 6", "4, 2, 3", "4, 2, 6", "9, 8, 7" };


string[] separator = new string[] { ", " };
int nAtt = all_lines[0].Split(separator, StringSplitOptions.RemoveEmptyEntries).Count() ;
List<IEnumerable> dom = new List<IEnumerable>();

for (int i = 0; i < nAtt; i++)
{
    var ele = (from lines in all_lines
                let data = lines.Split(separator, StringSplitOptions.RemoveEmptyEntries)
                select data.ElementAt(i)).Distinct();
    dom.Add(ele);
}

foreach (var row in dom)
{
    Console.Write("( ");
    foreach (var ele in row)
        Console.Write("{0} ", ele);
    Console.WriteLine(")");
}

The output of this code should be:

( 1 4 9 )
( 2 5 8 )
( 3 6 7 )

Is there any solution or alternative for this?

there can be alternatives, if you explain what this code is supposed to do. which output it should produce? — ASh
– ASh, Commented Nov 23, 2015 at 18:08

M.kazem Akhgary · Accepted Answer · 2015-11-23 18:10:20Z

4

There is closure problem. see ElementAt(i) here.

for (int i = 0; i < nAtt; i++)
{
    var ele = (from lines in all_lines
                let data = lines.Split(separator, StringSplitOptions.RemoveEmptyEntries)
                select data.ElementAt(i)).Distinct();
    dom.Add(ele);
}

This query will use the variable i it self. its not a copy. so after loop finishes i becomes nAtt and you get exception when you try to use it. so you have to store variable before using it.

var i1 = i;
var ele = (from lines in all_lines
        let data = lines.Split(separator, StringSplitOptions.RemoveEmptyEntries)
        select data.ElementAt(i1)).Distinct();
dom.Add(ele);

answered Nov 23, 2015 at 18:10

M.kazem Akhgary

19.3k8 gold badges68 silver badges126 bronze badges

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

Comments

spender · Accepted Answer · 2015-11-23 18:24:37Z

1

Your technique is terribly inefficient because, by using .Count() and .ElementAt() repeatedly, you are reiterating your enumerables unnecessarily, many times over.

A considerably more efficient, elegant and less error-prone approach might be:

var dom = all_lines
    .SelectMany(line => line
                         .Split(new[]{", "}, StringSplitOptions.RemoveEmptyEntries)
                         .Select((x, i) => new{x, i}))
    .GroupBy(x => x.i)
    .Select(g => g.Select(gi => gi.x).Distinct())
    .ToList();

edited Nov 23, 2015 at 18:24

answered Nov 23, 2015 at 18:15

spender

121k36 gold badges245 silver badges369 bronze badges

5 Comments

spender Over a year ago

@M.kazemAkhgary I got it when they showed us the desired output. Was tricky to spot the pattern though!

Jesus Perez Limon Over a year ago

but also keeps the comma

spender Over a year ago

@JesusPerezLimon. Um. No it doesn't. What makes you think that?

juharr Over a year ago

This has the benefit of working even if the number of comma separated values in each "line" is different.

Jesus Perez Limon Over a year ago

my fault, it's perfect

Collectives™ on Stack Overflow

System.ArgumentOutOfRangeException linq c#

2 Answers 2

Comments

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related