Regne med struct i C

Der er flere tilgange man kan bruge.

Du kan køre alle kampene igennem for hvert hold og så lægge til 1 til en variabel, 'udekampe_vundet', hvis de vandt en udekamp, og trække 1 fra hvis de vandt en hjemmekamp. Hvis variablen er positiv til sidst har holdt vundet flere udekampe end hjemmekampe.

En anden måde er at lave en struktur som indeholder statistikker for hvert hold:

 struct hold
 {
   char navn[MAX_NAVN];
   int total_hjemme_maal;
   int total_ude_maal;
   int total_maal_scoret_mod; /* Hvor mange mål er gået forbi dette holds målmand */
   int kampe_tabt;
   int kampe_vundet;
   int kampe_lige;
 }
 alle_hold[ANTAL_HOLD];

For hver kamp kan du så opdatere statistikkerne for de hold der spillede. Efter du er gået igennem alle kampe har du nu en masse info der kan hjælpe med at besvare andre spørgsmål.

Hvorfor har du to felter til antallet af tilskuere?

Istedet for at have to felter til dato/tid vil det være enklere at konvertere dato og tid til et timestamp, som er ét tal.

Hvis du har et array af dine strukturer vil det være ret enkelt at rende dem igennem og finde alle holdnavne. Derefter kan du lave et gennemløb for alle hold og finde antallet af vundne kampe, hvor holdet var på udebane.
Betingelsen der vil være: udebanemål > hjemmebanemål && strcmp(hold, udehold) == 0

Se her:

 #include <stdio.h>
 
 char * strings[] = {
     "10.352",
     "10352",
     "7",
     "875",
     NULL
 };
 
 unsigned int convert_to_int(char * str) {
     unsigned int n = 0;
     while (*str) {
         if (*str >= '0' && *str <= '9') {
             n *= 10;
             n += *str - '0';
         }
         str++;
     }
     return n;
 }
 
 int main(int argc, char const *argv[]) {
     int i;
     for (i = 0; strings[i]; i++) {
         printf("%s konverteret til %d\n", strings[i], convert_to_int(strings[i]));
     }
     return 0;
 }

Output:

$ ./t
10.352 konverteret til 10352
10352 konverteret til 10352
7 konverteret til 7
875 konverteret til 875

Det er altid værd at gøre sine strukturer så simple som muligt.

Tusind tak for forslagene Søren!

Umiddelbart virker din første løsning som den mest enkle, og den jeg har mest lyst til, men får måske brug for et stuct af holdene alligevel i en senere opgave, så kunne være jeg bare skulle bide det i mig, og lave den løsning!

Robert, det er fordi tekst filen er skrevet med punktum således 10.352, 9.232 osv - Min første tanke var at indlæse som double og så gange med tusind, men det er ikke alle kampe som har over 1.000 hvilket resulterede i at nogle kampe eksempelvis fik 875000 tilskuere ;-) Denne måde var det eneste jeg kunne komme i tanke om.

Hmm, det var faktisk smart Robert
Det ville da også være lidt nemmere end først at gange den ene med 1000, og så addere den anden. Tak!

Nu har jeg siddet og rodet lidt med opgaven, efter Sørens råd, men det virker stadig ikke helt. Nogle forslag til hvad der er galt?

Mine 2 structs:

 struct kamp
 {
   int runde;
   char dato[MAX_NAVN];
   char klokkeslet[MAX_NAVN];
   char hjemmehold[MAX_NAVN];
   char udehold[MAX_NAVN];
   int hjemmebane_maal;
   int udebane_maal;
   int antal_tilskuere_tusind;
   int antal_tilskuere;
 }
 alle_kampe[ANTAL_KAMPE];
 
 struct team
 {
   char hold_navn[MAX_NAVN];
   int  udekampe_vundet;
   int  hjemmebane_sejre;
   int  tilskuere;
 }alle_hold[ANTAL_HOLD];

min funktion lav_hold():

 void lav_hold()
 {
   strcpy(alle_hold[0].hold_navn,"AAB"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[1].hold_navn,"ACH"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[2].hold_navn,"AGF"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[3].hold_navn,"BIF"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[4].hold_navn,"EFB"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[5].hold_navn,"FCK"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[6].hold_navn,"FCM"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[7].hold_navn,"FCN"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[8].hold_navn,"OB"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[9].hold_navn,"RFC"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[10].hold_navn,"SDR"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;
 
   strcpy(alle_hold[11].hold_navn,"SIF"); 
          alle_hold[0].udekampe_vundet = 0;
          alle_hold[0].hjemmebane_sejre = 0;
          alle_hold[0].tilskuere = 0;

Min funktion flere_udekampe_sejre()

 void flere_udekampe_sejre()
 {
   lav_hold();
   int n = 0, m = 0, k = 0, udekampe_vundet = 0;
  
   for (n = 0;
        n < ANTAL_KAMPE;
        n++)
   {
     if (alle_kampe[n].hjemmebane_maal < alle_kampe[n].udebane_maal)
       {
         alle_hold[n].udekampe_vundet += 1;
       }
    else if (alle_kampe[n].hjemmebane_maal > alle_kampe[n].udebane_maal)
       {
         alle_hold[n].udekampe_vundet -= 1;
       }
   }
     for (k = 0;
          k <= ANTAL_HOLD;
          k++)
       {
         if (alle_hold[k].udekampe_vundet > 0) 
     {
           printf("%s har vundet flere kampe paa udebane end paa hjemmebane\n", alle_hold[m].hold_navn);
         }
       }
   
 }

Lige nu skriver den dette output
AAB har vundet flere kampe paa udebane end paa hjemmebane
AAB har vundet flere kampe paa udebane end paa hjemmebane
AAB har vundet flere kampe paa udebane end paa hjemmebane
AAB har vundet flere kampe paa udebane end paa hjemmebane
AAB har vundet flere kampe paa udebane end paa hjemmebane

/Thomas

Prøv at simplificere funktionerne så de kun gør én ting.

Hav f.eks. en funktion, som tager et holdnavn som parameter og som tæller antallet af vundne udekampe.
Og en funktion som tager et holdnavn som parameter og som tæller antallet af vundne hjemmekampe.

Og så en funktion som render igennem alle hold og vha. de to førnævnte funktioner tjekker om holdet er bedst på udebane.

Der er et par problematikker med de variabler du indekserer dine arrays med. F.eks. i den første løkke hvor 'n' er en af kampene bruger den til at indeksere ind i holdene på alle_hold[n], som der kun er 'k' af. Du er 'heldig' at den ikke segfaultede.

Samme i den nederste løkke hvor du itererer over 'k' men skriver holdt som ligger på 'm' som aldrig ændrer sig fra 0. Det er os derfor den udskriver samme (det første) holdnavn.

Følg Roberts forslag med at lave flere funktioner som hver gør en bestemt ting, på den måde undgår du en stor kompleks funktion, og får flere mindre komplekse funktioner, og de er nemmere at arbejde med.