Thread: Confused myself more than ever

Hello everyone, I'm in the process of trying to code a program that catalogues baseball cards. The information for the cards is populated through user input. After 10 cards have been populated the user has the option to print the cards to the console, or quit. I've confused myself more than ever on this one now and I can't seem to get the populate function working correctly. I'm not looking for a solution, but more for someone to steer me in the right direction.

Code:

#include"stdafx.h"
#include<stdlib.h>
#include<stdio.h>
typedefstruct{

char lName[20], fName[20], bats, throws, team[30];

int weight, height, year, games, runs, hits, RBI;

double BA;

}Player;


Player p;


void populatePlayer (Player p);

void askPrint (Player p);

void printPlayer (Player p);


int_tmain(intargc, _TCHAR* argv[])

{

    populatePlayer(p);

}


void populatePlayer(Playerp){


    printf("Enter last name: ");

    scanf_s("%c", &p.lName);

    printf("Which hand does player bat? ");

    scanf_s("%c", &p.bats);


    printPlayer(p);


}


/*void askPrint(Player p){

    char choice;


    printf("Print data? (y/n) ");

    scanf_s("%c", &choice);


    switch(choice)

    {

        case 'y':

            printPlayer(p);

        case 'n':

            break;

    }

}*/


void printPlayer(Playerp){


    printf("last name is %s\n", p.lName);


if(p.bats == 'l')

        printf("player bats lefty\n");

elseif (p.bats == 'r')

        printf("player bats righty\n");

else

        printf("player uses some odd hand\n");

}

I appreciate any tips or guidance. Thanks in advance!

hk, I believe that clears things up a lot! I'm going to lunch at the moment and I'll give it a shot when I return. thanks for the quick reply

Is this looking any better?

Code:

#include"stdafx.h"

#include<stdlib.h>

#include<stdio.h>


typedefstruct{

char *lName[20], *fName[20], *bats, *throws, *team[30];

int weight, height, year, games, runs, hits, RBI;

double BA;

}Player;


Player populatePlayer(Playerp[]){


for(int i = 0; i < 10; i++){

        printf("Enter last name: ");

for(int j = 0; j < 20; j++){

            scanf_s("%s", &p[i].lName[j]);

        }

        printf("Which hand does player bat? ");

        scanf_s("%s", &p[i].bats[0]);

    }

}


void printPlayer(Playerp[]){


for(int i = 0; i < 10; i++){

for(int j = 0; j < 20; j++){

            printf("last name is %s\n", p[i].lName[j]);

        }

if(p[i].bats[0] == 'l')

            printf("Player bats lefty\n");

elseif (p[i].bats[0] == 'r')

            printf("Player bats righty\n");

else

            printf("Player uses some odd hand\n");

    }

}


void askPrint(Playerp[]){

char choice;


    printf("Print data? (y/n) ");

    scanf_s("%c", &choice);


switch(choice)

    {

case'y':

            printPlayer(p);

case'n':

break;

    }

}


int_tmain(intargc, _TCHAR* argv[])

{

Player p[10];

    

    populatePlayer(p);

    askPrint(p);

}

Also, what should populatePlayer be returning? if I try to return p, it throws an error. In the state the code is in the program crashes when I run it. Thanks again