Kodkritik - (auto)snake

Japp, det funkar för mig! Önskar man kunde tillräckligt med C för att kunna säga vad som var bra /dåligt

Jodå, det blev tillåtet med C99.

Jag tycker det ser bra ut, inget negativt att kommentera på. Du har väldigt tight kodstil och försvinner inte iväg i massa fluff och abstraktion. Väldigt rakt på sak och löser de problem som behöver lösas.

Detta inlägget gäller snake, autosnake läste jag inte.

Det ser väldigt bra ut. Jag gillar ditt sätt att kommentera på, som en liten rubrik för varje kodstycke.

En sak som jag funderade på var valet av unsigned int på flera ställen. Om det inte finns någon anledning att ha det tycker jag det är snyggare att bara köra på vanliga int:s även om variablen bara får positiva värden. Sen tycker jag det är nästan lite "stroppigt" att skiva "signed int" Jag blir iallafall provocerad

Det verkar som du missat return 0 på slutet... eller är jag blind?
Varför kör du "return INT_MIN" vid fel? Antar att det är smaksak med jag tror dom flesta kör return -1.

På rad 75 har du

if(mvlt == 0 && event.type == SDL_KEYDOWN) {

Jag hade bytt det till

if(mvlt != 0 || event.type != SDL_KEYDOWN)
    continue;

Bara för att slippa den extra indenteringen.

Som sagt allt detta är ju bara smaksaker men det kan ju vara intressant att höra hur andra tänker...

Hur allvarlig är läckan?

if(mvlt != 0 || event.type != SDL_KEYDOWN)
    continue;

Det här kan dock vara känsligt; ungefär som att rekommendera "goto".

Under körningen används dock realloc, så det borde inte gå åt mer minne än det hade gjort annars (och så många byte rör det sig inte om antar jag).

Om jag inte missminner mig så måste du själv frigöra allokerat minne för surfaces, fonts, ljus osv. SDL_Quit() frigör enbart minnet för den surface som används som huvudfönster.

Sen om jag ska påpeka något på koden så tycker jag om att ha lite tomrum mellan slingor, kontrollstrukturer, funktioner osv. Men det är ju en smaksak.

- rader som inte får plats ens på 1920 pixlar
- 8 space för indentering (autosnake)
- är inte en inline vara snyggare än #define för avstånds-funktionen?
- när samma siffra används flera gånger, vore det inte smidigare o ha den i en konstant? typ 16 för allokeringen.
- samla spellogik-variablar på ett ställe (snake speed, start length osv)
- finns en enum för direction, man skulle kunna tänka sig en enum/define för X=0,Y=1 oxå, att använda i tex applepos[X]

jag brukar inte koda C, och vissa av mina kommentarer är mer för diskussion om hur man bör göra än min fasta övertygelse

struct coord {int x, y};

annars.

Skriver han endast int så blir det väl ändå en signed int per default?
Då blir det inte bara positiva värden, utan det är unsigned int som endast ger positiva värden.

Utan att titta i koden så menar väl jopsan att han tycker att det är onödigt att skriva signed int eftersom, precis som du säger, så blir det ju samma sak som bara int.
Sen med unsigned tyckte han att det var onödigt att skriva det då en vanlig int kan användas även om variabeln enbart har positiva värden. Så ni är ju egentligen överens

EDIT: håller dock inte med jopsan. Det finns föredelar med att skriva en int som signed. Då blir det tydligare för den som läser att särskilja. Sen att man skriver unsigned är ju inte onödigt enligt mig. En int som man bara vill ska ha positiva värden ska ju aldrig kunna gå under 0 och om den gör det är det ju fel och då ska programmet signalera det. Annars kan man få mer felsökta problem när konstiga saker man inte tänkt sig händer eftersom att integern är under 0

Det stämmer. Tilläggas kan dock att unsigned är ekvivalent med unsigned int (så man slipper skriva int om man så vill). Vidare anser jag att det är tydligare att använda unsigned när variabeln bara ska anta positiva värden.

Gällande snake. Nu är ju inte det så mycket kod det handlar om, men jag tycker att det är tydligare med mer uppdelning i funktioner. Så istället för vissa kommentarer du skrivit så kan skapa en funktion. main-metoden skulle då kunna se ut i stil med:

int main() {
   init();
   
   bool running = false;
   while (running) {
       running = key_handler();
       if (running) {
            game_logic();
            draw();
       }
   }

   return 0;
}

Fördelen jag ser med en sådan struktur är att man lätt i main-metoden ser flödet i programmet.