Apua c-ohjelman kanssa!

KOODI:

#include
#include
#include

struct henkilo {
char enimi[21];
char snimi[21];
char puh[21];
};

const char rekisteri[] = "luettelo.txt";

char* TiedonKopiointi(char*out, char*in, unsigned koko)
{
char* pc, *pc2;
unsigned j;
memset(out, 0, koko);
pc = in;
pc2 = out;
for(j = koko; j && *pc != ' ' && *pc != '\n'; --j){
*pc2 = *pc;
pc2++;
pc++;
}
return ++pc;
}

struct henkilo* TaytaTietue(char*buffer, struct henkilo*phenk)
{
char* pc;
pc = buffer;
pc = TiedonKopiointi(phenk->enimi, pc, 21);
pc = TiedonKopiointi(phenk->snimi, pc, 21);
pc = TiedonKopiointi(phenk->puh, pc, 21);
return ++phenk;
}

unsigned LueTiedostoMuistiin(struct henkilo*henkilo_lista)
{
char buffer[sizeof(struct henkilo)];
FILE *tiedosto;
int hlomaara = 0;
struct henkilo* phenk;
long i;
if ((tiedosto = fopen(rekisteri, "r")) != NULL) {
fgets(buffer, 4, tiedosto);
hlomaara = atoi(buffer);
if (hlomaara >= 50) {
printf("Korkeintaan %d tietuetta", 50);
fclose(tiedosto);
return -1;
}
for(i = 1, phenk = henkilo_lista; i i)
return phenk;
else if(poistettava != i){ /*Viimeistä poistetaan*/
phenk1 = henkilo_lista;
phenk1 += (poistettava - 1); /*Poistettavan kohdalle*/
phenk2 = phenk1;
phenk2++; /*Ensimmäisen siirrettävän kohdalle*/
/*Siirretään loppuja pykälällä*/
pc = (char*)(void*)phenk2;
pc2 = (char*)(void*)phenk;
memcpy(phenk1, phenk2, pc - pc2);
}
*hlomaara = i - 1;
return --phenk;
}

unsigned TallennaTiedostoon(struct henkilo* phenk, unsigned maara)
{
FILE *tiedosto;
unsigned i;
if ((tiedosto = fopen(rekisteri, "w")) == NULL) { /*Uuden tiedoston luonti*/
printf("Virhe tiedoston avaamisessa.");
return 1;
}
fprintf(tiedosto, "%d\n", maara); /*Kirjoitettavien määrä tiedoston alkuun*/
for(i = maara; i; --i){
fprintf(tiedosto,"%s %s %s\n", phenk->enimi, phenk->snimi, phenk->puh);
phenk++;
}
fclose(tiedosto);
return 0;
}

int main(void)
{
struct henkilo henkilo_lista[50];
unsigned hlomaara, poistettava;
struct henkilo* phenk;
unsigned i, j;
struct henkilo etsittavahlo;

hlomaara = LueTiedostoMuistiin(henkilo_lista);
if(hlomaara > (unsigned)-1)
return 1;

printf("Anna etunimi:");
scanf("%s", etsittavahlo.enimi);
printf("Anna sukunimi:");
scanf("%s", etsittavahlo.snimi);

for(i = 0, j = 0, phenk = henkilo_lista; i < hlomaara ; phenk++, ++i){
if(strcmp(phenk->enimi, etsittavahlo.enimi) == 0) {
if(strcmp(phenk->snimi, etsittavahlo.snimi) == 0) {
PoistaHenkilo(henkilo_lista, phenk, &hlomaara, i + 1);
printf("Tiedot poistettu luettelosta.");
j = 1;
break;
}
}
}
if(!j)
printf("Etsimääsi henkilöä ei löytynyt luettelosta.");

/*Tallennetaan lista tiedostoon*/
TallennaTiedostoon(henkilo_lista, hlomaara);

return 0;
}

tuttujuttu, et käytä IBM ASCII-järjestelmää, jossa on kaikki ääkköset ja öökköset, vaan uutta "muka uutta (25v vanhaa)", tai toinen vaihtoehto on nämä "whitespacet", eli.. joka tapauksessa kirjaimet toimivat, mutta huono uusi näppäinjärjestelmä.

Tuon voi kyllä kiertää jos osaa, laitat vaan käyttöjärjestelmään että IBM 256-letter ASCII.

Ei ole vaan täysin oikein :)
ei oteta vastaan mitään, eikä palauteta mitään koko ohjelmasta :)

Muuten näyttää tuo hyvälle ihan kun tarkemmin katoin, mutta miksi palautat koko ohjelmasta mitään, kun kerran jo tallennat tulokset tiedostoon???

siitä tämä void-juttu.

Tuo puuttuu sulla tiedostoon tulostamisesta: "\r \n", kumpikin pitää olla peräkkäin..

Myöskin jos tuo on C/C kääntäjä mitä käytät, niin lisätä noiden tyhjien include-tiedostojen tilalle: "using namespace std;"

void main(void), sitten return (void); täysin oikein.

Eiköhän opiskelun tarkoitus ole itse oppia näin yksinkertainen ohjelma, eikä yrittää saada joku netistä korjaamaan ohjelmaasi. Vilkaisepa sitä toisen kerran paremmin. Kyllä se syy selkeästi on koodissasi.

Se mitä standardia main function kanssa käytetään on täysin jonni joutavaa kiistelyä, joka viittaa siihen, että kiistelijä on joko kaikkea vihaava ideologi tai melko kokematon ohjelmoija. Ei ole merkitystä, millä standardilla se on kirjoitettu, kunhan se toimii oikein.