Thread: Returning a vaule loses the last one

You do not need to use strcpy if you are only using a char. It is only for strings.

Just do this:

Code:

ReturnedName = Name[x];

I think this is what you are trying to do.

It should not overwrite it like you are saying unless you are using pointers and only passing the address. Can you be more specific and post your code using code tags.

- Sean

Originally posted by RatBoy
I tried making returnedName an array but that just kept giving me a NULL value

Also Im not using pointers because i cant.

The way sean told you to do it should work. Remember a function can NEVER return an array. It can return lots of other things but not array's. Just return the single character like you had but just don't use strcpy because it is unnecessary. Just use a simple assignment.

See if you can work with this, though I'm not sure it's what you need.

Code:

#include <stdio.h>

char getChoice(void);


int main(void)
{
    char letter;
    char buffer[10];
    int count;
	
    count= 0;

    do
    {
	  
       letter = getChoice();
       buffer[count++] = letter;
	
    }while(count != 10);

    buffer[count] = '\0';

    for(count = 0; buffer[count] != '\0'; count++)
    {
	letter = buffer[count];
	switch(letter)
	{
            case '1': /* do something with '1' */ break;
            case '2': /* do something with '2' */ break;
            case '3': /* do something with '3' */ break;
            case 'A': /* do something with 'A' */ break;
           case 'B': /* do something with 'B' */ break;
          default: 
        /* output error message or
	    do some action         */ break;
       };
    }

    printf("The choices made were: %s\n\n", buffer);

    return 0;
}

char getChoice(void)
{
    char choice, trash;
    static char *menu = "1> item 1\n"
                        "2> item 2\n"
                        "3> item 3\n"
                        "A> item A\n"
                        "B> item B\n"
                        "\n\n-> ";

    printf("%s", menu);
    scanf("%c%c", &choice, &trash);
    printf("\n");

    return choice;
}

I've typed too fast and now my fingers are bleeding. :P

What is the Name variable for? Can you show a bit more code so I can see what your trying to do with it.

Also, you're returning a single char, so when you print it, you need to do so like this:
>printf("%c\n", returnedValue);
There are other ways, but basically, I'm showing you that using %s will not work. Again, I need to understand your code better before advising further.

>does anyone know why the int value is being stored properly and why the string value is getting over written
Because you're storing a pointer to the data, you won't get a unique string with every iteration, you'll store a new pointer to the same string. The end result is that every element of the NameArray points to the same string. To avoid this, make a copy:

Code:

while ( fscanf ( ratesFile, "%f%s", &Rate, Name ) != EOF )
{
  RateArray[i] = Rate;
  NameArray[i] = malloc ( strlen ( Name ) + 1 );
  strcpy ( NameArray[i], Name );
  printf("\n%s", Name);
  printf("\n%s", NameArray[i]);
  i++;
}

-Prelude