Question

EDIT: solved it, turns out I should use %c not %s because foodSelect and foodSize are characters not strings 😛 thanks

I’m trying to pass 3 values to the function output: foodChoice, foodSelect, foodSize (and foodOrderNum, and foodSubtotal but I haven’t gotten around to that yet).

However, when I try to printf foodSelect, I get a segmentation fault, but when I try to print foodChoice, I don’t. When I try to printf foodSize it just shows nothing.

source:

#include <stdio.h>
#include <string.h>

void question (char choice[]);
int output(char *foodChoice, char *foodSelect, char *foodSize);


void question (char choice[]) {

    char choiceYesNo;
    char *foodOptions;
    char *foodChoice;
    char *foodSelect;
    char *foodSize;
    int foodOrderNum = 0;
    float foodSubtotal = 0;

    switch (choice[0]) {
        case 'f':
            foodChoice = "Fish";
            foodOptions = "(K- Haddock, T- Halibut)";
            break;
        case 'c':
            foodChoice = "Chips";
            foodOptions = "(C- Cut, R- Ring)";
            break;
        case 'd':
            foodChoice = "Drinks";
            foodOptions = "(S- Softdrink, C- Coffee, T- Tea)";
            break;
    }

    printf("Do you order %s? (Y/N): ", foodChoice);
        scanf("%c", &choiceYesNo);
    printf("%s choice %s: ", foodChoice, foodOptions);
        scanf("%s", &foodSelect);
    printf("What size (L - Large, M - Medium, S - Small): ");
        scanf("%s", &foodSize);
    printf("How many orders do you want? (>=0): ");
        scanf("%d", &foodOrderNum);
    output(foodChoice, foodSelect, foodSize);
}

int output(char *foodChoice, char *foodSelect, char *foodSize) {

    // printf("You ordered %s: %s - SIZE: %s   amount ordered: , subtotal price: \n", 
    // foodChoice, foodSelect, foodSize);

    printf("\n\n%s\n", foodSelect);
    // printf("\n\n%s\n", foodSelect);

}

int main() {

    question("chips");


}

Answer 1

You haven’t allocated memory for:

char *foodOptions;
char *foodChoice;
char *foodSelect;
char *foodSize;

Do malloc and allocate memory. Note that:

char *foodChoice="Fish";

And

char *foodChoice;
foodChoice="Fish";

are not the same.