Blackjack k-kobling

Fa via App Store Les dette innlegget i var app!

Forste Blackjack-spill i C.

Dette er mitt forste virkelige program, selv om det har gatt gjennom noen store revisjoner. Uansett er jeg sikker pa at det er mye jeg kunne ha gjort bedre, renere eller sikrere.

Kan noen se noe jeg virkelig trenger a jobbe med eller fikse?

Jeg har noen kommentarer, hvorav noen er ganske pedantiske.

Unnga hardkodede konstanter (f.eks. 10, ‘c,’ d ‘,’ A ‘etc) – Bruk definerer eller const inkonsekvent avstand rundt = etc.

Bruk av globals er uonsket med mindre det er nodvendig – her er det for det meste ikke, egentlig. Og hvis du ma bruke globals i en fil, gjor dem statiske med mindre de virkelig ma deles mellom filer. Og det er sjeldent.

Gjor lokale funksjoner statiske nar det er mulig.

Legg til band pa enkelt setning:

Variable navn x, y, z etc er ikke-beskrivende og unhelpful. Bruk av sv rt korte navn er ikke nodvendigvis darlig, men du bor bare bruke slike navn der det er tydelig hva de relaterer til. I denne koden hindrer det forstaelse (mine likevel)

bruk av kort snarere enn int er generelt ubrukelig.

Jeg foretrekker a se «handen» passert som en peker til struktur (const hvis mulig) i stedet for et handnummer a.

tally er mer nyttig hvis det bare summerer innholdet i card_value array og returnerer summen uten a skrive til card_tally. Merk at check_for_ace inneholder en sloyfe som er mye den samme som (og derfor kan bruke) tally.

I draw_card:

Denne sloyfen skal v re en funksjon. Ogsa det er ingen feilkontroll.

Den etterfolgende sloyfen genererer et tilfeldig kort som ikke allerede er brukt opp, men det burde v re et cards_used array for hver dress. Jeg vet ditt dekk * 4 skal handtere det, men hindrer det pa at det samme kortet (4 av hjerter, si) blir brukt 4 ganger (med ett dekk)? Du kan kanskje bruke en rekke 52 oppforinger som dekker alle dresser. Jeg vil sette lokken i en.

I display_hands.

Det er to nesten identiske lokker – trekk dem inn i en funksjon.

main er for stor Har du hugget ut noen argumenthandtering i main? Det du ser ser merkelig ut. Tilbakestillingssloyfen i hovedet bor ogsa trekkes ut i en egen funksjon. Variabel sloyfe i bitmap-odeleggelseslokken heter darlig END_OF_MAIN – hva er dette? – fjern det uansett.

Her er noen ting jeg merker fra skimming gjennom koden din:

Fin og ren stil, linjer er korte nok til a lette lesing Plukket gode navn for funksjoner.

Bruken av en bokstav variabel. De er ikke beskrivende. For eksempel sier void tally (int a) ikke hva det er.


Hello! Do you want to play in the most honest casino? We gathered it for you. Push here now!