Thread: Change value in function

I have two code, one is true, and other is wrong. (the purpose of this work is test, how to change an array in function.
First is:

Code:

#include <stdio.h>
int array(int a,int b,int *x);
int main(){
    int *a;
    array(5,6,a);
    printf("%d %d\n",a[1],a[2]);
}
int array(int a, int b,int *x){
    x[1]=a;
    x[2]=b;
}

and second is:

Code:

#include <stdio.h>
int array(int a,int b,int *x);
int main(){
    int a;
    array(5,6,&a);
    printf("%d %d\n",a[1],a[2]);
}
int array(int a, int b,int *x){
    x[1]=a;
    x[2]=b;
}

First code is true (it will print 5 6), but second code is wrong. It's wrong at line

Code:

 Printf("%d %d",a[1],a[2]);
// It has an error name:invalid type int[int] for array subscript

I have google but I don't know so much about this problem.
So, who can help me, why It's wrong, please.

thanks so much

> I have two code, one is true, and other is wrong.
Well the first only works because of pure dumb luck, since your pointer is uninitialised.

If you had
int *a = NULL;

it would probably blow up.

Further, you seem to be implying that arrays start at [1], when in fact they start at 0.

The minimal you need to do to make this work is either
int a[3];

or
int *a = malloc( 3 * sizeof(*a) );

@Salem: thanks a lot, I know where my mistake, your advice is very helpful (It can run smooth now). But I have one point that I don't know: why in my old first code, why I have "pure dumb LUCK" and why when I had: int *a=NULL, my program will be blow up.
thanks again

On most machines, dereferencing NULL is a fatal error.

The pure dumb luck is like this

Code:

int age;
printf("You are %d years old\n", age );

It's easy to see that this is wrong.

But with your uninitialised pointer, all that you've really got to go on is whether it crashes or not. Any random address is good enough (for you), so long as it doesn't crash.

Oh, I luckily read a paper that say about Pointer. After that, I see some above questions of mine is really silly A "bad pointer" is reference to random address. It hard to see problem when debug small problem, but it will be really an awful event if run this module in big project. (Am I thinking right :"> )

I think you need to go through whatever C book or reference you have, page by page, problem by problem. You have some serious level of knowledge problems.