Ошибка int32 как исправить

I am trying to print out my array of numbers that I have assigned to a particular array. My algorithm for choosing numbers consists of choosing a random number that is not a duplicate and storing it inside the array.

Pretty simple really, but I have no idea as to why it is printing out this error.

int[] ticket1 = new int[4];
for (int i = 0; i < 4; i++)
{
    int temp = rand.Next(43);
    while (ticket1.Contains(temp)) 
    {
        temp = rand.Next(43);
    }
    ticket1[i] = temp;
}
Console.WriteLine("{0}{1}", item.PadRight(20), ticket1.ToString());//ticket1 produces System.Int32[] instead of 4 numbers.
//I have changed this line to:
//Console.WriteLine("{0}{1}", item.PadRight(20), string.Join(",", ticket1));
//And it still doesn't work. the error remains. (System.Int32[])

My question is, how can I print out my 4 numbers (beside each other) in string format.

//EDIT: I’ve found my problem. I am putting my ticket1 inside a foreach loop, it’s somehow not reaching out to the array values and it therefore prints out System.Int32[] instead.

All fixed.

asked Jun 13, 2015 at 12:06

ticket1.ToString() does not print the content of the array, only its type, because this is the way the ToString() method is implemented on arrays.

You can fix this in several ways — for example, by using string.Join method:

Console.WriteLine("{0}{1}", item.PadRight(20), string.Join(",", ticket1));

answered Jun 13, 2015 at 12:08

Because when you call .ToString() on an object you get the type of that object. For basic primitive (value) types this behavior is overridden to output the value. But for something like an array there’s no «default» string representation, so the behavior you’re seeing is the default.

You could wrap your data in an object and override .ToString() on that object. If you have to output the values in many places in the code that would be the way to go so you only have to write the logic once. («Smart data structures and dumb code works a lot better than the other way around.» — Eric Raymond)

But if you only need to do it here then you can just output the values directly. Basically join the values as a string in whatever representation you want. For example, if they should be comma-separated:

Console.WriteLine(
  "{0}{1}",
  item.PadRight(20),
  string.Join(",", ticket1));

answered Jun 13, 2015 at 12:11

I am trying to print out my array of numbers that I have assigned to a particular array. My algorithm for choosing numbers consists of choosing a random number that is not a duplicate and storing it inside the array.

Pretty simple really, but I have no idea as to why it is printing out this error.

int[] ticket1 = new int[4];
for (int i = 0; i < 4; i++)
{
    int temp = rand.Next(43);
    while (ticket1.Contains(temp)) 
    {
        temp = rand.Next(43);
    }
    ticket1[i] = temp;
}
Console.WriteLine("{0}{1}", item.PadRight(20), ticket1.ToString());//ticket1 produces System.Int32[] instead of 4 numbers.
//I have changed this line to:
//Console.WriteLine("{0}{1}", item.PadRight(20), string.Join(",", ticket1));
//And it still doesn't work. the error remains. (System.Int32[])

My question is, how can I print out my 4 numbers (beside each other) in string format.

//EDIT: I’ve found my problem. I am putting my ticket1 inside a foreach loop, it’s somehow not reaching out to the array values and it therefore prints out System.Int32[] instead.

All fixed.

asked Jun 13, 2015 at 12:06

ticket1.ToString() does not print the content of the array, only its type, because this is the way the ToString() method is implemented on arrays.

You can fix this in several ways — for example, by using string.Join method:

Console.WriteLine("{0}{1}", item.PadRight(20), string.Join(",", ticket1));

answered Jun 13, 2015 at 12:08

Because when you call .ToString() on an object you get the type of that object. For basic primitive (value) types this behavior is overridden to output the value. But for something like an array there’s no «default» string representation, so the behavior you’re seeing is the default.

You could wrap your data in an object and override .ToString() on that object. If you have to output the values in many places in the code that would be the way to go so you only have to write the logic once. («Smart data structures and dumb code works a lot better than the other way around.» — Eric Raymond)

But if you only need to do it here then you can just output the values directly. Basically join the values as a string in whatever representation you want. For example, if they should be comma-separated:

Console.WriteLine(
  "{0}{1}",
  item.PadRight(20),
  string.Join(",", ticket1));

answered Jun 13, 2015 at 12:11