Black Jack C-programmering

Tjenare alla kära medlemmar, snart kommer någonting gå sönder, känner mig så totalt jävla usel att det är helt otroligt....

Sitter med ett sista projekt i skolan nu i kursen Inledande programmering C. Vi ska koda i olika dokument, alltså ett dokument för funktionsdeklarationer, ett dokument för funktionsdefinitoner och ett med själva programmet så att säga.

Det jag gjort än så länge är att skapa en struct för kort (enum color och enum value) hela kortleken, och en funktion som blandar hela kortleken. Jag vill nu på något sett ge ut 2 kort var till båda spelare och sedan göra en funktion som fortsätter ge ut ett kort. Jag tänker att eftersom att kortleken är blandad, så kanske man kan göra en loop som bara ger nästa kort som ligger i leken, så tar man bort chansen att man ger ut samma kort 2 gånger.

Det största problemet för mig egentligen är hur man får över färgen och värdet på kortet över från funktionsdefinitionen till själva programfilen.

giveCardToPlayer(color, rank); {
printf("Your starting cards is %s of %s and %s of %s", rank1, color1, rank2, color2);
}

Tänker mig att något i stil med koden ovan i programfilen ska göra att vid start av program står det "Your starting cards is Four of Diamonds and Six of Hearts".

Har förstått att man inte kan returnera 2 värden, men kan jag returnera en struct? För Struct Card består ju av just Rank och Color, så det vore väl perfekt?

Tack för svar, tänker du att jag ska göra i en array? förstår inte riktigt hur det ska fungera...

enum rank {
    ACE,
    TWO,
    ...
    QUEEN,
    KING,
    NUM_RANKS
};

enum color {
    CLUBS,
    DIAMONDS,
    HEARTS,
    SPADES,
    NUM_COLORS
};

struct card {
    enum color;
    enum rank;
};

struct card_deck {
    int top;  /* index to top of deck */
    struct card cards[NUM_RANKS * NUM_COLORS]; /* deck of card, bottom card at index 0 */
};

...
struct card draw_top(struct card_deck *deck) {
    if (deck->top == 0) {
       fail("tried to draw from empty deck");
    } 
    return deck->cards[--deck->top];
}

Ja du, förstår vad koden gör men inte hur den fungerar... Blir så sjukt besviken på att jag inte fattar...

struct Card* createDeck() {
struct Card* deck = NULL;
deck = (struct Card*)malloc(sizeof(struct Card) * 52);

int currentColor = 0;
for (int i = 0; i < 52; i++) {
deck[i].rank = (enum value)(i % 13);

//färgbyte
if (i != 0 && i % 13 == 0) {
currentColor++;
}

deck[i].color = (enum suite)
currentColor;
}
return deck;

Så ser createDeck ut just nu, fick hjälp av läraren för att få ihop det där. Men jag förstår mig inte på "enum"

int cardIndex = 0;
for(int c = 0; c < 4; c++)
{
  for(int r = 0; r < 13; r++, cardIndex++)
  {
    deck[cardIndex].rank = (enum value)r;
    deck[cardIndex].color = (enum suite)c;
  }
}

struct Card giefcardplx(struct Card* deck, int index) {
  return deck[index];
}

Haha, tack så mycket! Har gjort något i stil med det där, gjorde så en funktion för att ge startkorten till player1 och en för player2 sen tänkte jag göra en gemensam funktion som kan ge 1 kort åt gången. Jag satte även printfunktionen i funktionsdefinitionen och i och med att det är 13 värden och 4 färger blev det 17 if-satser 3 gånger, haha. Ser väldigt ineffektivt ut, men fungerar för det jag vill ÄN sålänge.

Funktionsdefinition
void giveStartingCardsToPlayer1(struct Card deck[], int nrOfCards) {
for (int i = 0; i < 2; i++) {
if (deck[i].rank == ACE)
printf("Ace");
else if (deck[i].rank == TWO)
printf("Two");
else if (deck[i].rank == THREE)
printf("Three");
else if (deck[i].rank == FOUR)
printf("Four");
else if (deck[i].rank == FIVE)
printf("Five");
else if (deck[i].rank == SIX)
printf("Six");
else if (deck[i].rank == SEVEN)
printf("Seven");
else if (deck[i].rank == EIGHT)
printf("Eight");
else if (deck[i].rank == NINE)
printf("Nine");
else if (deck[i].rank == TEN)
printf("Ten");
else if (deck[i].rank == JACK)
printf("Jack");
else if (deck[i].rank == QUEEN)
printf("Queen");
else
printf("King");

printf(" of ");

if (deck[i].color == HEARTS)
printf("Hearts");
else if (deck[i].color == SPADES)
printf("Spades");
else if (deck[i].color == CLUBS)
printf("Clubs");
else
printf("Diamonds");
printf("\n");
}
}

Blackjack.C
printf("%s, these are your starting cards, would you like another? (Yes or no)\n", player1Name);

giveStartingCardsToPlayer1(deck, 2);

scanf_s("%s", answer, sizeof(answer));
if (answer == "yes") {
giveCardToPlayer();
}
else if (answer == "no") {
return 0;
}

Och nu kommer ju funktionsdefinitionen för "giveCardToPlayer();" bli lika stor som "giveStartingCardsToPlayer1", haha. Bara att den startar på ett annat index. Känns detta som en vettig lösning?

enum suite {
    CLUBS,
    DIAMONDS,
    HEARTS,
    SPADES
};

if (card->suite == HEARTS) {
    ...
}

if (card->suite == 2) {
    ...
}

Det där var pedagogiskt, tack så mycket! :). Vad tror du om lösningen jag gjorde ovan? Jag tänker mig att jag nu kommer behöva göra funktioner som håller koll på Player1CurrentValue och Player2CurrentValue dessutom göra en playerHouse, eller vad man kallar dealern :D.
Sen en funktion som berättar vem som vinner. Är Enum ett effektivt sätt att göra detta på? Min programmeringskunniga vän, sa att han tyckte det var mycket lättare med arrayer för kortlekarna, då kunde han använda en för att ta bort kort som han ger ut.

Gör gärna programmet min egen väg, även om det är oeffektivt så tror jag att jag lär mig bäst så, se kanske reflektera över koden i efterhand, men som jag nämnde tidigare, tror du att min lösning i kommentaren ovan fungerar?

if (deck[i].rank == ACE)
printf("Ace");
else if (deck[i].rank == TWO)
printf("Two");
else if (deck[i].rank == THREE)
printf("Three");
else if (deck[i].rank == FOUR)
printf("Four");
else if (deck[i].rank == FIVE)
printf("Five");
else if (deck[i].rank == SIX)
printf("Six");
else if (deck[i].rank == SEVEN)
printf("Seven");
else if (deck[i].rank == EIGHT)
printf("Eight");
else if (deck[i].rank == NINE)
printf("Nine");
else if (deck[i].rank == TEN)
printf("Ten");
else if (deck[i].rank == JACK)
printf("Jack");
else if (deck[i].rank == QUEEN)
printf("Queen");
else
printf("King");

const char * rankToStr (enum Rank rank)
{
    switch (rank)
    {
    case ACE: return "Ace";
    case TWO: return "Two";
    case THREE: return "Three";
    case FOUR: return "Four";
    case FIVE: return "Five";
    case SIX: return "Six";
    case SEVEN: return "Seven";
    case EIGTH: return "Eigth";
    case NINE: return "Nine";
    case TEN: return "Ten";
    case JACK: return "Jack";
    case QUEEN: return "Queen";
    default: return "King";
    }
}
...
printf("%s", rankToStr(deck[i].rank));

Oj, dem där funktionerna har jag aldrig hört talas om! Kraven i uppgiften är att använda Struct, allokeringar och inte ha NÅGRA memorylekages. Sen får inte heller några globala variabler finnas med.

Skriver ju i visualstudio. Prövade köra programmet, det som händer än så länge är att den frågar person 1 och 2 om namn, sedan printas de 2 första korten i kortleken ut och den frågar spelare1 om den vill ha fler kort, har inte hunnit göra en funktion som printar ut ett kort åt gången, men om svaret blir något annat än "ja", så hoppar den till spelare 2 och gör samma grej där. tanken är väl att jag ska ha ännu en funktion som printar ut totalpoängen innan den frågar om man vill ha ett till kort och sen hur jag ska göra med dealern har jag ännu ingen aning om

Ursäkta att jag citerar dig, men hoppas på proffshjälp igen, Haha.

har fastnat när jag ska göra en if-loop med en char. I detta fall ser det ut såhär

char answer1[16];

printf("%s, would you like another card? (yes or no) \n", player1Name);
scanf_s("%s", answer1, sizeof(answer1));

printf("%s", answer1); //bara här för att kontrollera om scanf tog ut rätt svar (vilket den gör)

if (answer1 == "yes") {
printf("\n%s, This is your new card.\n", player1Name); //För att kontrollera om programmet går in i loopen, vilket den inte gör...
}

Okej, lyckades lösa det problemet, genom att köra strcmp. Men nu har ett nytt problem dykt upp.

Denna funktion

void giveCardToPlayer(struct Card deck[], int nrOfCards) {
for (int i = 6; i < 52; i++) {
alla kort
}

gör att när jag kallar på den genom dessa kodrader

printf("Okey, this is your new card.\n", player1Name);
giveCardToPlayer(deck, 1);

printar ut alla kort i hela kortleken som finns kvar, jag vill ju bara att funktionen ska printa ut ett kort :/

Har försökt tänka så, jag tycker bara att det är sjukt komplicerat att komma på själva koden. Jag tycker det verkar vettigt att blanda kortleken till att börja med och sen bara börja på index 0 och ta det som i verkligheten att man tar kortet högst upp i leken, sen går man steg för steg neråt och med det, ökar man indexen för varje nytt kort som tas. Problemet för mig är att jag inte har någon aning om hur man kan koda det i C.

Vi ska ju dessutom använda oss av Structs. Det jag gjort är så här

void giveStartingCardsToPlayer1(struct Card deck[], int nrOfCards) {
for (int i = 0; i < 52; i++) {
if (deck[i].rank == ACE)
printf("Ace");
else if (deck[i].rank == TWO)
printf("Two");
else if (deck[i].rank == THREE)
printf("Three");
else if (deck[i].rank == FOUR)
printf("Four");
else if (deck[i].rank == FIVE)
printf("Five");
else if (deck[i].rank == SIX)
printf("Six");
else if (deck[i].rank == SEVEN)
printf("Seven");
else if (deck[i].rank == EIGHT)
printf("Eight");
else if (deck[i].rank == NINE)
printf("Nine");
else if (deck[i].rank == TEN)
printf("Ten");
else if (deck[i].rank == JACK)
printf("Jack");
else if (deck[i].rank == QUEEN)
printf("Queen");
else
printf("King");

printf(" of ");

if (deck[i].color == HEARTS)
printf("Hearts");
else if (deck[i].color == SPADES)
printf("Spades");
else if (deck[i].color == CLUBS)
printf("Clubs");
else
printf("Diamonds");
printf("\n");
}
}

Problemet med denna, är att om jag inte begränsar forloopen till "int i = 0; i < 1; i++", så kommer den att fortsätta rulla på tills alla kort upp till 51 är utrprintade. Jag vet inte hur jag ska lyckas med det, jag kan ju sätta break i slutet av loopen och på så vis endast få den att printa ut ett kort, men nästa gång jag kallar på funktionen kommer den fortfarande kasta fram samma kort som förra gången...

Därav har jag gjort en forloop som har "int i = 0; i < 2; i++" för att kasta ut spelare 1's första kort och en som är "int i = 2; i < 4; i++", sen har jag en till som kastar ut "int i = 4; i < 5; i++", alltså ett kort. sen har jag gjort 45 sådana här loopar till typ. Tills jag kommer upp till "int i = 51; i < 52; i++". Detta blir ungefär 2100 rader kod, vilket ser väldigt dåligt ut och icke effektivt. Jag vill verkligen kunna koda bra, men det känns inte som att jag fått tillräckliga kunskaper för det. Jag har sökt som en galning efter olika funktioner på internet också, men blir inte jättemycket vettigare för det, då jag verkligen inte vill kopiera rakt av heller.

Förstår inte riktigt hur det fungerar med structs. Jag har ju t.ex

enum suit {
HEARTS, SPADES, CLUBS, DIAMONDS
};

enum value {
ACE, TWO, THREE, FOUR, FIVE, SIX, SEVEN, EIGHT, NINE, TEN, JACK, QUEEN, KING
};

// deklarerar structs i headerfilen
struct Card {
enum suite color;
enum value rank;
};

Om jag nu vill printa ut color och rank på kortet på index 1 t.ex, hur går jag tillväga?

void giveCard1(struct Card deck[], int nrOfCards)
{
int i = 1;
printf("Your card is %s of %s", deck[i].rank, deck[i].color);
}
Denna funktion printar ju ut värdena som finns på index1, både färg och rank, men dessa returneras ju som int. Alltså blir det typ något i stil med "your card is 1 of 9", vilket inte är så förståeligt för den som spelar. Hur får jag över dessa till char utan att behöva göra sådär sjukt lång kod som jag skickat i kommentaren ovanför? För det känns så sjukt oeffektivt

const char * suiteToStr[] = {
  "Hearts", "Spades", "Clubs", "Diamonds"
};
...
printf("Suite is %s", suiteToStr[deck[i].color]);

Så, du tycker inte att det verkar dumt som nybörjare att ha 2100 rader kod som i stort sätt repeterar sig själv för en funktion som egentligen kan förkortas mycket? Funderar på om jag bara ska låta det vara såhär, då det faktiskt fungerar. Med tanke på att vi ska presentera uppgiften muntligt för läraren, så kan jag ju berätta hur jag försökt på andra sätt men den där var den enda jag behärskade ordentligt.

Jo, det som är samma är ju att man vill printa ut kortet på indexplatsen x. (0<x<52)
Jag har funderat på att göra en funktion för att printa ut kortleken utan att ha en for-loop, dvs en funktion som skulle returnera kortets värde på indexplatsen. Problemet är ju att vi inte får ha globala variabler och vi måste använda struct. Då måste man ju använda pekare och annat lullull som jag inte heller behärskar än. Lust att hjälpa?

Tänker just nu något i stil med

void giveCard(struct Card deck[], int nrOfCards) {
i=0
if (deck[i].rank == ACE)
return Ace;
else if (deck[i].rank == TWO)
return Two;
osvosvosv
i++
}
Problemet här är ju att den kommer sätta index till 0 varje gång och kommer därför skicka ut samma kort varje gång?

Sen tänkte jag mig som sagt denna funktion
void giveCard1(struct Card deck[])
{
int i = 1;
printf("Your card is %d of %d", deck[i].rank, deck[i].color);
}

Denna funktion skulle ju kräva att jag gör 52 stycken "void giveCardsX" (0<x<52)
Men här returnar den ju som sagt ett INT-värde, men jag vill ha en textrepresentation (char) på detta.

Kan även tillägga att det vore trevligt att göra på det sättet med 52 upprepelser, då jag i stort sätt gjort hela spelets grund på det sättet nu och skulle behöva skriva om main() helt annars, haha.