I’m working on an exercise in K&R (ex. 5–9) and I was trying to convert the original program’s 2D array of
static char daytab[2][13] = {
{0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31},
{0, 31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31}
};
into using pointers to an array of 13 ints like
static char (*daytab)[13] = {
{0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31},
{0, 31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31}
};
But compiler prints warning: excess elements in scalar initializer.
Googling did not help and even K&R writes when passing the array to a function,
myFunction(int daytab[2][13]) {...}
is the same as
myFunction(int (*daytab)[13]) {...}
The excess elements in scalar initializer is an error thrown when a scalar object is assigned more than one value. Remember that scalar objects are objects that only hold one value at a time.
The initializer for a scalar (an object of integer type including booleans and enumerated types, floating type including complex and imaginary, and pointer type including pointer to function) must be a single expression, optionally enclosed in braces: = expression. (1) = { expression } (2)
But compiler prints warning: excess elements in scalar initializer. Googling did not help and even K&R writes when passing the array to a function, myFunction (int daytab [2] [13]) {...}
The difference being that: declares a two-dimensional array, which includes setting aside space for the array and ensuring that daytab references that memory. However: ...only declares a pointer. So you're trying to initialize a pointer with an array initializer, which doesn't work as expected.
Recall that the name of an array is a pointer to the first element. Now you really have an array of pointers to two arrays of 13 int s.
What happens instead is that the first number in your initializer is assigned to the pointer daytab, and the compiler generates a warning to let you know you've specified a lot of additional values that are just discarded. Since the first number in your initializer is 0, you're just setting daytab to NULL in a rather verbose way.
The two are only partly equivalent. The difference being that:
static char daytab[2][13] = {
{0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31},
{0, 31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31}
};
declares a two-dimensional array, which includes setting aside space for the array and ensuring that daytab
references that memory. However:
static char (*daytab)[13] = {
{0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31},
{0, 31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31}
};
...only declares a pointer. So you're trying to initialize a pointer with an array initializer, which doesn't work as expected. There is no array; there's no memory set aside for an array. What happens instead is that the first number in your initializer is assigned to the pointer daytab
, and the compiler generates a warning to let you know you've specified a lot of additional values that are just discarded. Since the first number in your initializer is 0
, you're just setting daytab
to NULL
in a rather verbose way.
So if you want to do this sort of initialization, use the first version -- it decays to the same pointer type that you explicitly declare in the second version, so you can use it the same way. The second version, with the array pointer, is needed when you wish to dynamically allocate the array or get a reference to another array that already exists.
So you can do this:
static char arr[2][3] = { { 1, 2, 3 }, { 4, 5, 6 } };
static char (*ptr)[3] = NULL;
ptr = arr;
...and then use ptr
and arr
interchangeably. Or this:
static char (*ptr)[3] = NULL;
ptr = malloc(2 * sizeof(*ptr));
...to get a dynamically allocated 2-dimensional array (not an array of pointers to 1D arrays, but a real 2D array). Of course, it's not initialized in that case.
The "equivalence" of the two variations just means that the 2D array, when it decays to a pointer to its first element, decays to the type of pointer declared in the second variation. Once the pointer version is actually pointed at an array, the two are equivalent. But the 2D array version sets up memory for the array, where the pointer declaration doesn't... and the pointer can be assigned a new value (pointed at a different array) where the 2D array variable cannot.
In C99 you can do this, though (if not static
at least):
char (*daytab)[13] = (char [][13]){
{0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31},
{0, 31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31}
};
@Dmitri explained it well, but I wanted to add that
static char (*daytab)[13] = { ... };
is one pointer to an array of 13 char
elements. The compiler gives you the warning because you've passed in two arrays. It's like trying to assign two addresses to one pointer char *p = {a, b}
. There are more elements than necessary per your declaration. See Geekforgeek's explanation on what an array pointer really means.
As for answering the K&R exercise, consider
Option 1:
static char *daytab[2] = {
(char []) {0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31},
(char []) {0, 31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31}
};}
or Option 2:
static char (*daytab)[13] = (char [][13]) {
{0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31},
{0, 31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31}
};}
Option 1 is an array of two char
pointers.
Option 2 is one array pointer. It points to an array of 13 char
elements. Just as you could increment a char
pointer to get the next letter in a string, you can increment this array pointer to grab the next array of 13 char
s.
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