Thread: Passing array back to main function

Hello people i'm having problems with my program, what its supposed to do is ask the user how many integers to store, then for every integer it asks the user to store a number at the integer location, for example:

User enters 3 integers to input
- User enters number 4 and is stored in the first integer location
- User enters number 78 and is stored in the second integer location
- User enters number 12 and is stored in the third integer location

Then the program displays the users integers in thier spectified locations example:
4 in integer 1
78 in integer 2
12 in integer 3

However after i put my integers in the program goes crazy and generates a crazy amount of integers before the program seg faults. and i'm not quite sure to to correct this error, any ideas would be helpful.

Code:

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


void getInts(int * integersArray, int numInput);

int main() {

  int * integersArray;
  int numInput;
  int i;

  getInts(integersArray, numInput);

  for (i = 0; i < numInput; i++) {
    printf("%d ",integersArray[i]);
  }
  
  return 0;
}


void getInts(int * integersArray, int numInput) {
  int i;

  printf("Please enter the number of integers you want to input\n");
  scanf("%d", &numInput);

  integersArray = (int *) malloc(sizeof(int) * numInput);

  for (i = 0; i < numInput; i++) {
    printf("please enter integer %d: ", i+1);
    scanf("%d", &(integersArray[i]));
  }
}

Things are passed by value in C, so the value of integersArray cannot be changed by the getInts function, despite your attempt to do so with the line

Code:

integersArray = (int *) malloc(sizeof(int) * numInput);

Also the value of numInput cannot be changed, despite the scanf line.

If you want those values to change, you must pass the address of those variables to your function instead.

well i tried the above code, but this time i get an infinite loop of zeros if i enter 1 integer and store number 1 into the integer, however if i select 2 integers and put numbers within them i get a seg fault?, lol too confusing for me

I feel you might need a pointer to a pointer here in the function prototype...

Thus, have a pointer, pointing, to the address of the 1-D array, hence, something like:

Code:

getInts(int *(*integersArray), int numInput)

though idk, im in the same boat and working on it right now :-/

Code:

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


void getInts(int *integersArray, int numInput);

int main() {

  int * integersArray;
  int numInput;
  int i;

  printf("Please enter the number of integers you want to input\n");
  scanf("%d", &numInput);

/* If you get the numInput here, then you can malloc() your array, before
you call getInts(). That gets it all set with a good address. 
*/


  integersArray = (int *) malloc(sizeof(int) * numInput);
  getInts(integersArray, numInput);

/* when the program returns, the address you received before from malloc(), is still 
OK.

The cast to int* is not necessary, but is not a compiler error.
*/

  for (i = 0; i < numInput; i++) {
    printf("%d ",integersArray[i]);
  }
  free(integersArray);
  getch();
  return 0;
}


void getInts(int *integersArray, int numInput) {
  int i;

  for (i = 0; i < numInput; i++) {
    printf("please enter integer %d: ", i+1);
    scanf("%d", &(integersArray[i]));
  }
}

@ adak your above solution does not even output anything, hmm i tried messing with the program, but still i seg fault, at one point it ran an infinte loop. when it was generating the integers

Originally Posted by greg677

@ adak your above solution does not even output anything, hmm i tried messing with the program, but still i seg fault, at one point it ran an infinte loop. when it was generating the integers

Why not post the code you currently have?

Originally Posted by greg677

@ adak your above solution does not even output anything, hmm i tried messing with the program, but still i seg fault, at one point it ran an infinte loop. when it was generating the integers

Change

getch();

to

getchar();

in the code.

Then, put another getchar(); right after your scanf()'s

(yes, every one of them).

That will pull the remaining newline off, and if you accidentally input a char, it will stop it from looping forever, if the code is inside a loop .

Of course, I ran this code several times before I posted it, and it ran flawlessly - but getch() works for me, automatically.

Like the following?

Code:

int main()
{
   int *integersArray;
   int numInput;
   int i;

   getInts(integersArray, numInput);

   for (i = 0; i < numInput; i++)
   {
      printf("%d ",integersArray[i]);
   }
   free(integersArray);
   getchar();
   return 0;
}

void getInts(int *integersArray, int numInput)
{
   int i;

   printf("Please enter the number of integers you want to input\n");
   scanf("%d", &numInput);
   getchar();

   integersArray = (int *) malloc(sizeof(int) * numInput);

   for (i =0; i < numInput; i++)
   {
      printf("Please enter an integer %d: ", i+1);
      scanf("%d", &(integersArray[i]));
      getchar();
   }
}

if that is correct it still does not work for me i am using a program called PUTTY to compile this, this may be the problem.

Putty isn't a compiler.

But in any event, this:

Code:

getInts(integersArray, numInput);

just hurts. integersArray can not not not not not NOT be changed by passing it to a function. If you want it to change, you must pass a "reference" to it, i.e., it's address:

Code:

getInts(&integersArray, numInput);

And of course make the appropriate changes throughout getInts.