Providing variable values as double[] to Dataset fails at runtime

[abeham]

In the Dataset class an IList is accepted as variable values, however later on this IList is cast as a List<T>.

public Dataset(IEnumerable<string> variableNames, IEnumerable<IList> variableValues)
  : base() {
}

  // line 199:
  List<T> values = list as List<T>;
  if (values == null) throw new ArgumentException("The variable " + variableName + " is not a " + typeof(T) + " variable.");

Now, a double[] is an IList, but not a List<double>. The compiler is happy, but the program crashes at runtime. In my opinion, it should be checked if it's a T[] in case List<T> fails or otherwise prevent accepting double[] in the constructor.

[bburlacu]

r14857: Add DatasetUtil static class with useful methods for dealing with Datasets. Add input validation to Dataset constructor.

[gkronber]

I think r14857 is not related to the ticket description.

[abeham]

I agree with gkronber, this change does not adress the ticket.

[bburlacu]

r15013: Found a different solution to this issue, by using IList<T> as the return type for the GetValues<T> method.

[bburlacu]

r15015: Fix exception in Dataset.cs when calling the ToModifiable() method.

[gkronber]

r15152: merged r14857 from trunk to stable

Premature merge to stable to fix a compilation fail of the stable branch. The remaining changesets need to be reviewed an merged.

[gkronber]

Reviewed r14857, r15013, r15015 and tested this with the following C# script.

    var names = new string[] {"a", "b", "c"};
    var @as = Enumerable.Repeat(1.0, 10).ToArray();
    var bs = Enumerable.Repeat("abc", 10).ToArray();
    var cs = Enumerable.Repeat(DateTime.Now, 10).ToArray();
    var values = new System.Collections.IList[] {@as, bs, cs};
      
    var ds = new Dataset(names, values);
    vars["ds"] = ds;

    Console.WriteLine(ds.GetDoubleValue("a", 0));
    Console.WriteLine(ds.GetStringValue("b", 0));
    Console.WriteLine(ds.GetDateTimeValue("c", 0));
    
    var columnA = ds.GetReadOnlyDoubleValues("a");
    Console.WriteLine(columnA[0]);
    
    var columnB = ds.GetReadOnlyStringValues("b");
    Console.WriteLine(columnB[0]);

    var columnC = ds.GetReadOnlyDateTimeValues("c");
    Console.WriteLine(columnC[0]);
    
    var aEnum = ds.GetDoubleValues("a");
    Console.WriteLine(aEnum.First());

    var bEnum = ds.GetStringValues("b");
    Console.WriteLine(bEnum.First());

    var cEnum = ds.GetDateTimeValues("c");
    Console.WriteLine(cEnum.First());

[gkronber]

r15159: merged r15013 and r15015 from trunk to stable (all changesets merged)