Why are there pointer errors in this code?

Question

I am receiving the following errors for the following lines:

randmst.c:42: warning: assignment makes integer from pointer without a cast

randmst.c:43: error: incompatible types in assignment

randmst.c:44: warning: assignment makes integer from pointer without a cast

randmst.c:50: error: invalid type argument of ‘unary *’

My code:

#include <stdio.h>
    #include <stdlib.h>
    #include <time.h>


    //function generates a random float in [0,1]
    float rand_float();

    //all info for a vertex
    typedef struct{
        int key;
        int prev;
        float loc;
    } Vertex;

    //using the pointer
    typedef Vertex *VertexPointer;

    int main(int argc, char **argv){

        //command line arguments
        int test = atoi(argv[1]);
        int numpoints = atoi(argv[2]);
        int numtrials = atoi(argv[3]);
        int dimension = atoi(argv[4]);

        //seed the psuedo-random number generator
        srand(time(NULL));

        //declare an array for the vertices
        int nodes[numpoints];

        //create the vertices in the array
        int x;
        for(x = 0; x < numpoints; x++){
            //create the vertex
            VertexPointer v;
            v = (VertexPointer)malloc(sizeof(Vertex));
            (*v).key = 100;
            (*v).prev = NULL;
            (*v).loc = rand_float;
            nodes[x] = v;
        }

        //testing
        int y;
        for(y = 0; y < numpoints; y++){
            printf("%f \n", (*nodes[y]).loc);
        }

    }


    //generate a psuedo random float in [0,1]
    float
    rand_float(){
        return (float)rand()/(RAND_MAX);
    }

nodes[x] has type int, while v is some type of pointer, you can't do nodes[x] = v;. — Yu Hao
– Yu Hao, Commented Mar 6, 2014 at 6:02
BTW, you shouldn't cast the result of malloc in C. stackoverflow.com/questions/605845/… — Barmar
– Barmar, Commented Mar 6, 2014 at 6:02
Yes.as barmar said, you should not use casting for malloc. Also I would suggest you to use a condition to check return value of malloc so that if the memory is not allocated, then such operation should not take place — user3256147
– user3256147, Commented Mar 6, 2014 at 6:08

Sakthi Kumar · Accepted Answer · 2014-03-06 06:10:07Z

4

//declare an array for the vertices
        int nodes[numpoints];

and

44 nodes[x] = v;

but v is of type VertexPointer. The nodes array has to be an array of VertexPointer

//declare an array for the vertices
VertexPointer nodes[numpoints];

This would fix the error on line 50 as well. Also on other lines,

42           (*v).prev = NULL;

prev is an int, but u assign NULL which is a pointer. You can change prev to void * or NULL to 0

43            (*v).loc = rand_float;

rand_float is a function name, which decays to a pointer. You can change loc to void * or rand_float to rand_float() <-- see the difference here. rand_float is the pointer, but rand_float() is a function call which returns float

edited Mar 6, 2014 at 6:10

answered Mar 6, 2014 at 6:04

Sakthi Kumar

3,04517 silver badges28 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Barmar · Accepted Answer · 2014-03-06 06:05:25Z

1

This is causing the error about unary *:

        printf("%f \n", (*nodes[y]).loc);

nodes[y] is an int, but * is used to dereference a pointer.

answered Mar 6, 2014 at 6:05

Barmar

789k57 gold badges555 silver badges669 bronze badges

Comments

0xF1 · Accepted Answer · 2014-03-06 06:14:33Z

0

You cannot assign integers to pointers and vice-versa without explicit typecast.

All the error statements are:

(*v).prev = NULL;      // 'prev' is 'int', 'NULL' is 'void*' type pointer
(*v).loc = rand_float; // 'loc' is 'float', 'rand_float' is 'float*' type pointer
nodes[x] = v;          // 'nodes[x]' is 'int', 'v' is 'struct Vertex *' type pointer

and

(*nodes[y]).loc        // 'nodes[y]' is already an integer and you are dereferencing it

To correct these errors, declare the variables, to which are assigning pointers, as pointer to correct type.

Example: loc should be declared as float (*loc)(); and int nodes[numpoints] should be declared as VertexPointer nodes[numpoints];

edited Mar 6, 2014 at 6:14

answered Mar 6, 2014 at 6:02

0xF1

6,1563 gold badges31 silver badges51 bronze badges

Collectives™ on Stack Overflow

Why are there pointer errors in this code?

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related