Teen viopessa c-ohjelmoinnin harjoituksia ja eräs tehtävä tuottaa hieman päänvaivaa. Ensimmäiset kaksi viopen testeistä menevät läpi mutta kolmas, joka on piilotettu feilaa.. Osaisiko joku jeesata ja kertoa mitä pitäisi muuttaa, että ohjelma toimisi. En siis tiedä mitä nimiä viope syöttää ohjelmalle, mutta lopputulos on seuraava:
Tiedoston luettelo.txt sisältö:
3
Keijo Kojootti 050-3500980
Brian Kottarainen 040-3980982
Brian Kottarainen 040-3980982
Kun tiedoston pitäisi olla seuraavanlainen:
3
Keijo Kojootti 050-3500980
Brian Kottarainen 040-3980982
Brita Kottarainen 05-4567393
Eli ongelma liittyy varmaankin tohon että molemmat Brian ja Brita alkaa samoilla kirjaimilla ja sukunimi on sama. En vaan keksi mitä pitäisi muuttaa. Pistän alle tehtävänannon ja lopuksi vielä koodin.
Tehtävänanto:
Puhelinluettelo - henkilön tietojen poistaminen ohjelmointitehtävä 3/3
Tee ohjelma, jolla voidaan poistaa luettelosta tietyn henkilön tiedot. Ohjelma kysyy ensin etunimen ja tämän jälkeen sukunimen. Ohjelma tarkistaa, löytyykö henkilön tiedot. Mikäli henkilö löytyy luettelosta, tiedot poistetaan. Muuten tulostetaan virheilmoitus "Etsimääsi henkilöä ei löytynyt luettelosta.".
Vihje:
Ennen kuin ryhdyt laatimaan varsinaista koodia, sinun kannattaa suunnitella ohjelma hyvin. Rakenteiden käyttäminen voi helpottaa tehtävää. Päivitetyn listan tallentaminen mitä ilmeisimmin tuhoaa aikaisemmat tiedot tiedostosta luettelo.txt.
Nimitietojen vertailu onnistuu c-kielen funktiolla strcmp(), joka löytyy kirjastotiedostosta string.h. Voit tutustua funktion käyttöön c-manuaalissa.
Esimerkkitulostus
Anna etunimi:Timo
Anna sukunimi:Timpuri
Tiedot poistettu luettelosta.
Apua c-ohjelman kanssa!
10
2366
Vastaukset
- aloitteleva_kdr
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;
} - johanonnistuu.
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.- jf394yfgefok4
Mutta voisin mainita tuosta sinun c-koodista... näyttääpä sotkuiselta, koneella tehtyä jotain :) tuonkin voisit korjata pääfunktiossa: "void main(void)", ja palautus "return ;"
- sdfsfsdfs54353
jf394yfgefok4 kirjoitti:
Mutta voisin mainita tuosta sinun c-koodista... näyttääpä sotkuiselta, koneella tehtyä jotain :) tuonkin voisit korjata pääfunktiossa: "void main(void)", ja palautus "return ;"
Kai tuo oli vitsi.
void main(void); on totaalisen väärin.
int main(void); on oikea standardin mukainen menettely.
- idomaattinen9
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.. - tyhmättyhjätincludet
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;"
- noohhhhhflds43lf43
void main(void), sitten return (void); täysin oikein.
- sdfsdfsdfsd
Tuo on totaalisen väärin.
Ensinnäkin standardin mukaan mainin täytyy olla aina INT tyyppinen ja mainin täytyy aina palauttaa INT arvo.
Tämä siksi että käyttiksen puolella voidaan havaita mitä ohjelma palauttaa ja sen perusteella päätellä onnistuiko ohjelman suoritus ja mitä virheitä ohjelmassa tapahtui.
Tuo void main(void); käytäntö on jostian kivikaudelta kun C:n standardi ei ollut vielä kehittynyt.
Muistakaa AINA: "int main(void)" tai "int main(int argc, char *argv[])"
Nämä ovat ainoat standardinmukaiset vaihtoehdot puhtaassa C:ssä.
Toki myös on winmaineja muita entrypointteja mutta ne toteuttavat saman kaavan.
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.- tgdgdfgdfg
Ei ole jonninjoutavaa.
Ensinnäkin miksi opetalla huonoja ja VÄÄRIÄ ohjelmointitapoja kun voi samalla vaivalla opetella asian oikein.
Lisäksi nimenomaan "void main()" version kanssa tukee oikeasti ongelmiakin käyttisalustoilla. Sulautetussa void main(); voi olla ok jos alusta ei vaadi / tarkista mitä main palauttaa.
Tässä syitä käyttää standardinmukaista oikeaoppista int mainia.
1. Koska se on standardi
2. Koska ohjelman aloitus rutiinit jotka kutsuvat mainia saattavat olettaa tai olettavat että mainin palautusarvo asetetaan pinoon. Jos main funktio ei tee tätä, saattaa ohjelman lopetus rutiinin pino vikaantua / korruptoitua ja aiheuttaa määrittelemätöntä toimintaa tai kaatumisia.
3. Koska standardista poikkeava maini saattaa palauttaa aivan sattumanvaraisen arvon kutsuvalle käyttikselle. Tämän takia ei pystytä tarkistamaan esimerkiki ohjelman vikakoodia. Lisäksi makefilet ja vastaavat luottavat ohjelmien palautusarvoihin toiminnoissaan.
4. Koska mainin palauttama kokonaislukua käytetään ohjelman tilan tarkastuun (onnistunut suoritus, virhetilanne, virhekoodi).
Ketjusta on poistettu 0 sääntöjenvastaista viestiä.
Luetuimmat keskustelut
SDP jo 100 % suositumpi kuin persut
Kertoo Hesarin uusin kannatuskysely. Demareiden kannatus on miltei tuplat verrattuna persuihinl. Suomen kansa ei selväst449298- 2234433
SDP haluaa 40 000 nettomaahanmuuttajaa
SDP:n Suunnanmuutos-vaihtoehtobudjetissa, käy ilmi, että demarit itse asiassa vaativat räjähdysmäistä ”työperäisen” maah1543880Orpo: Velkajarrua vastustavaa puoluetta vaikea ajatella hallitukseen
No Minja Koskelan kommunistipuolue jäi ulos tuosta. Kaikki eduskuntapuolueet vasemmistoliittoa lukuun ottamatta sopivat1453341PS ylivoimainen nousija myös HS:n gallupissa, SDP laskee taas
https://www.verkkouutiset.fi/a/hs-gallup-sdpn-suosio-laskee-ps-nousussa/#0a7d2507 Ylivoimainen viime kuukausien nousija373119Mikä tämä henkilö mahtaa touhuta Parkanossa
Kamalaa https://www.ylasatakunta.fi/teksti/pirkanmaan-karajaoikeus-vangitsi-koiran-tappamisesta-epaillyn-6.68.127794.b58272141Ikävä sinua mies
Vuosia kuluu, mutta tunteet ei ole hävinnyt. Tasoittuneet toki, kun ei olla nähty. Järki palannut päähän kuitenkin. Se i201810Hienoa! Eduskunta luopui käteisen käytöstä
Nyt tuo sama muutos pitää saada myös muuhun yhteiskuntaan. Käteistähän ei tarvitse tänä päivänä enää kuin rikolliset.621706Sulla on avaimet ja keinot
Jos haluat jatkaa tutustumista. Itse olen niin jäässä etten pysty tekemään enää mitään. Pidempi keppi johon on helpompi251435Orpo loukkaantui fasismiin viittaavasta sanavalinnasta
Mutta miksi loukkaantui? Orpohan on tehnyt yhteistyötä fasistien kanssa jo vuonna 2019, siis jo neljä vuotta ennen loukk321389