Question

Why does the following code throw an exception when getting to the second scanf_s after entering an number to put into the struct.

This by no means represents a complete linked list implementation.

Not sure how to get onto the next scanf_s when having entered the value? Any ideas?

EDIT: Updated code with suggested solution, but still get an AccessViolationException after first scanf_s

Code:

struct node
{
    char name[20];
    int age;
    float height;
    node *nxt;
};

int FillInLinkedList(node* temp)
{

int result;
temp = new node;

printf("Please enter name of the person");
result = scanf_s("%s", temp->name);

printf("Please enter persons age");
result = scanf_s("%d", &temp->age); // Exception here...

printf("Please enter persons height");
result = scanf_s("%f", &temp->height);

temp->nxt = NULL;
if (result >0)
    return  1;
 else return 0;
}

// calling code

int main(array<System::String ^> ^args)
{
  node temp;

  FillInLinkedList(&temp);

...

Answer 1

You need

result = scanf_s("%d", &temp->age);

and

result = scanf_s("%f", &temp->height);

Reason is that sscanf (and friends) requires a pointer to the output variable so it can store the result there.

BTW, you have a similar problem with the parameter temp of your function. Since you’re changing the pointer (and not just the contents of what it points to), you need to pass a double pointer so that the changes will be visible outside your function:

int FillInLinkedList(node** temp)

And then of course you’ll have to make the necessary changes inside the function.