C#

harold aptroot wrote:

It cheated. I'm afraid I suggested an ugly work-around that is at least "bad taste".

Oh well. An explanation.

It made the "temporary" object static. That is, it is shared between all instances of the containing class (or struct). It shouldn't be (shared, that is), but it works.. But it also has the unfortunate side effect that a reference to that array is kept - forever (well, at least for a long time).

I was partially under the impression that this initialization code was for a local variable, if that were true there would be no need of such ugly hacks.
And I should have suggested something better, which I did later.. Initializing that field in the constructor.

Now, I just realized something. Code like that should almost never initialize a non-static field. Because it looks like that (jagged) array is meant to be a look-up table. If it isn't then don't worry about it, but if it is, consider making it static as well. Or even static readonly. But what am I saying here, it's your code after all and I don't know how you're using it. Do what you want Smile

Ennis Ray Lynch, Jr. wrote:

UPDATE FROM

SqlCeConnection conn = new SqlCeConnection();
conn.ConnectionString = "Data Source=todos.sdf;Persist Security Info=False";
SqlCeCommand comm = new SqlCeCommand("SELECT OfficialRegNo, TransponderID FROM BasicAnimal UNION SELECT  CAST(FilterObjectType AS nchar), Caption FROM Filter", conn);
conn.Open();
SqlCeDataReader myReader = comm.ExecuteReader();

do
{
Console.WriteLine("\t{0}\t{1}", myReader.GetName(0), myReader.GetName(1));
               
while (myReader.Read())    //<--- SqlCeException here
Console.WriteLine("\t{0}\t{1}", myReader.GetString(0), myReader.GetString(1));

} while (myReader.NextResult());

myReader.Close();
conn.Close();

riddick_86 wrote:

Please read carefully