C program - greška kad dođe do % u printf

Tom69 kaže...

Zato što računaš period prije nego što si uopće učitao podatke iz datoteke.

Također string učitaj s %39s, a ne %39c.

eee, hvala za ovo sa %39s. ja se dosad patio kako da maknem onaj višak iza imena

char *trim(char *str) {
  char *end;

  // Trim trailing space
  end = str + strlen(str) - 1;
  while(end > str && *end == ' ') end--;

  // Write new null terminator
  *(end+1) = 0;

  return str;
}

...

comet.name[39] = '\0';
trim(comet.name);

A kosu crtu možeš pronaći sa strchr pa ju zamjeniti s razmakom ili čime već. Ako hoćeš spojiti C i 2010 onda ćeš morati kopirati sve poslije kose crte za jedno mjesto ulijevo.

Proslijediš pokazivač na prvi član polja  i uz to broj elemenata polja kao drugi argument

#include<stdio.h>

void ProslijediPolje(int* polje, int brojElemenata)

{

int i;

for(i=0; i< brojElemenata; i++)

{

printf("%d ", polje[i]);

}

}

int main()

{

int polje[5] = {1, 2, 3, 4, 5};

ProslijediPolje(polje, 5);

}

Kod char nizova ne trebaš čak ni drugi argument tj. broj elemenata, broj elemenata polja odrediš tako da petlja vrti do zaključnog '\0' znaka.

@jurluk:

Nitko nije na ti sa pokazivačima.

Jel ti to vježbaš za vještine?

 Tako sam i ja radio jedan program, dok nisu kolokviji počeli.  Sada čuči na desktopu i čeka. Jbg. Ista sudba je pogodila moje učenje Linuxa i terminala.

Nisan odavno pisa nista u c-u, pa ako san sta falia zanemarite ali evo par savjeta testaj pa javi rezultat (neman vrieman da pisen cili program sad)

ugl kad pozivas funckciju dosta je prosljedit to polje npr.

ako u mainu imamo 

int polje[30];

//i funckiju

int funkcija(int *aaa)

//onda bi poziv treba izgledat ovako:

int funckija(polje)

Sta se tice samih pointera ako si napisa char *pointer to samo znaci da imas jedan pokazivac koji pokazuje na char ne i da je ono alociralo memoriju (ako se nevaram)

Znaci ako si napisa gore char *newName onda trebas napisat i na sta ce on pointat npr

int polje[50]

int *pointer

pointer = polje;

Ako pak neznas velicinu polja (sta je pretpostavljam kod tebe slucaj) onda moras memoriju dinamiski alocirat

Puno toga san ode nabacia nabrzaka, a i sa pointerima san se vise bavia u c++, iako bi sve trebalo bit slicno / isto testaj pa javi probat cu pomoc ako buden zna

Korisna vježba, ali zašto ne koristiš funkcije iz <string.h> za to, strstrza pronalazak dvostrukog razmaka ("  ") i strncpy za kopiranje prvih n znakova?

Time izbjegavaš potrebu dokumentiranja što to točno funkcija uredi_ime radi.

I da, prije kopiranja bi trebao alocirati memoriju s malloc.

BTW, u kojem okruženju radiš i kompajliraš s obzirom da ti ne baca upozorenja i exceptione za to (ili baca)?

Funkcija ti vraća char, a koliko vidim njenu povratnu vrijednost pridružuješ char*-u. I da, naravno da fali return ako fja vraća char :D

Btw opreza sa granicom kod prve petlje, ako ti je riječ dugačka 40 znakova, ovaj i+1 će otić izvan toga.

Probaj ovo:

char* uredi_ime (char *name, int duljina)
{
    int i, n=duljina;
    char *new_name;

    for (i=0; i<duljina-1; i++)
       if (name[i]==' ' && name[i+1]==' ')
         { n=i; break; }
   
    new_name = (char*) malloc (n*sizeof(char));

    for (i=0; i<n; i++)
       new_name[i]=name[i];

    new_name[n]='\0';
    free(name);
   
    return new_name;
}

poziv:

comet[i].name = uredi_ime (comet[i].name, strlen(comet[i].name));

Ok, očito ima problem sa char* i ograničenim nizom charova. Probaj ovo:

void uredi_ime (char *name, int duljina)
{
    int i, n=duljina;
    char *new_name;

    for (i=0; i<duljina-1; i++)
       if (name[i]==' ' && name[i+1]==' ')
         { n=i; break; }
   
    new_name = (char*) malloc (n*sizeof(char));

    for (i=0; i<n; i++)
       new_name[i]=name[i];

    new_name[n]='\0';
    strcpy(name,new_name);
   
    free(new_name);
}

poziv (bez pridruživanja ičega):

uredi_ime (comet[i].name, strlen(comet[i].name));

Aj daj cijeli kod. Jer meni radi na mom primjeru.

Trebaš free(new_name) u funkciji. To ti je možda greška, da ne pregledavam sad sve. I vitice kod n=i; break;

Nisu tam bezveze. Doslovce copy-pasteaj kod.

Za if moraju bit vitice

ne

for { if } nego for if {}

I makni pridruživanje u pozivu. Drito kopiraj linije kak sam ih napisao

ne nešto=uredi__ime

nego samo

uredi_ime

Nema veze. Kad staviš char s[40] to znači da će imati maximalno 39 znakova, ne točno 39. Tako da je i manji string ok.

Btw meni uz promjene koje sam napisao kod radi

Opcija koju još možeš pokušati je ovo:

void uredi_ime (char *name, int duljina) {

    int i, n=duljina;

    for (i=0; i<duljina-1; i++)
       if (name[i]==' ' && name[i+1]==' ')
         {  n=i;  break; }
   
    name[n]='\0';
}

Ovako će samo ubacit \0 tamo di treba završit riječ, i sve fje sa stringovima rade do \0 pa nema problema. Jedino što će ostat viška memorije zauzeto, ali nije tak strašno budući da su charovi mali :D

Probaj s ovakvom funkcijom, ona niš ne radi sa memorijom, možda mu je u tome bio problem.