Dotaz: cyklus while a podivne spravanie

[oxymoron]

mam strasne neprijemny problem ktory ma brzdi pri zadani do skoly a preto hladam pomoc...neviem akym zazrakom ale ked tento program pustim a zadam 'b' tak mi do premennej pocet ulozi 1...avsak ked ten cyklus bezi dalej tak mne z neznameho dovodu premennu pocet vynuluje.. :(

#include malloc.h
#include stdio.h
#include stdlib.h
typedef struct {
  char meno[10];
  char priez[20];
  int oscislo;
  char znamka;
} STUD;

void vlozStud(int *pocet) {
  (*pocet)++;
}
  

int main() {
  
  int pocet;
  char vyber;
  
  pocet=0;
	
  STUD *student;
  student=(STUD *) malloc(1*sizeof(student));
  
 
  printf("%d\n",pocet);
  while (vyber!='x') { 
    scanf("%s",&vyber);
    if(vyber=='b') vlozStud(&pocet);
    printf("%d",pocet);
  }
  
  printf("%d",pocet);
  
  return (0);
}

Odpovědi

[oxymoron]

zabudol som ze pracujem v Anjute pod linuxami a najhorsie je ze pod win to ide :(

[David Watzke]

Nehodí kompilátor nějakej warning? Nepíšou se includy mezi < a >?

[zabza]

no to je jasný, jenže zkus si to zadat na ABCLinuxu s těma < a > ...

[Michal Kubeček]

No a na co jsou asi nad polem pro zadávání příspěvku tlačítka '<' a '>'? Pro ty línější nabízím pomůcku, kterou jsem kdysi vyrobil kvůli starému redakčnímu systému Intervalu, ale hodí se i zde.

[oxymoron]

hmm....ospravdlnujem sa to som si nevsimol ... :)

[zabza]

hint: kterým znakem končí v c-čku stringy?

[oxymoron]

hmm...no nerozumiem resp. asi neviem na to vobec odpoved

[Martin Tůma]

Snažíš se zapsat řetězec na místo znaku. I když z klávesnice zadáš jenom "b", tak funkce scanf("%s", &vyber) na adresu &vyber uloží "b\0" (C-čkové řetězce jsou zakončeny znakem "\0"). A ten nulovej znak přepíše něco, co už nepatří k tomu charu. (V tvém případě proměnnou pocet)

To, že na windows to funguje je způsobený pravděpodobně "zarovnávánim" proměných v paměti, který provádí překladač - zapisuješ někam, kde by stejně nic nebylo.

[Michal Kubeček]

  scanf("%s",&vyber);

Tak takové věci si raději hodně rychle odvykněte. Termín buffer overflow už jste slyšel? Tohle by byla sebevražda i kdyby proměnná vyber nebyla typu char.

[oxymoron]

ok beriem a budem pamatat ale ako to ma byt spravne? resp. ako to riesit??

[David Watzke]

Jak to má být správně? Takhle to je i v učebnici... no, v jiném spojení.

[Michal Kubeček]

Máte pointer na proměnnou typu char a na místo, kam ukazuje, zkopírujete zadaný řetězec. Takže prvním znakem naplníte tu proměnnou a zbytkem přepíšete pár bytů za ním. Když budete mít štěstí, bude to pár jiných proměnných. Když ne, mohou se začít dít podivuhodné věci… V případě tazatele to zřejmě odnesla proměnná pocet, která se nacházela na zásobníku hned nad vyber.

[oxymoron]

hmm...ok tak prosim ako to zapisat spravne? ale inak len tak mimo ze to pod win ide a to ma tiez hneva lebo pod win sa mi moc ten program pisat nechce...resp. nemam to cim prelozit a v com pisat...prosim dalo by sa navrhnut nejake riesenie konkretne presny zapis? inak ked tam dam namiesto scanf("%s",&vyber) scanf("%c",&vyber) tak mi to nevynuluje premennu pocet ale ten cyklus ide vzdy dvakrat namiesto toho aby si zakazdym vypytal zadat znak...

[Michal Kubeček]

Asi byste si měl nejdřív zjistit, co vlastně děláte. Tedy nastudovat si z jakékoli učebnice jazyka C něco o tom, jaká je standardní reprezentace řetězců v jazyce C, co je to pointer a co dělá funkce scanf(). Bez toho to nepůjde a pokud si toto nastudujete, bude vám hned jasné, v čem je problém. Fórum vám může pomoci s konkrétním problémem, ale základy jazyka si musíte nastudovat sám.

[oxymoron]

no mne je jasne to ze pod windowsackym compilerom to ide tak preto ma zaujima v com je problem....mne je jasne co je pointer aj co robi scanf ale ked mi nieco nejde tak skusam a vidim ze to pod win ide :(

[Jan Martinek]

Na začátku cyklu proměnná "vyber" není inicializována, takže je celkem náhoda, že to nějak funguje. Dále, "vyber" nesmí být "char", ale "pointer na char", tedy

char *vyber;

No a když je to takhle inicializováno, tak je potřeba paměť naalokovat

int N = 20;
vyber = (char *) malloc (N * sizeof(char));

a po použití uvolnit

free(vyber);

Přičemž NIKDY nepoužívej scanf() tak jak jsi to zapsal. Když už to mermomocí musíš napsat právě takto, tak aspoň

scanf ("%19s", vyber);

Abys omezil možnost buffer-overflow.

[oxymoron]

ok dakujem velmi pekne skusim .....ale aj tak ma zaraza ze pod win to v poho funguje...a ze nie je inicializovana to som len zabudol moja chyba ale mam to tam.. :)

[Jan Martinek]

To, že to ve windows funguje, je opravdu náhoda. Každá platforma je trochu jiná. Céčko je trochu zrádné v tom, že špatně napsaný program někdy funguje, jindy zase ne, chová se "záhadně" v závislosti na ledasčem a dost špatně se hledají chyby.

Kdo s Céčkem teprve začíná, tak má dvojnásob složitou situaci. Jednak tápe, ale také používá "školské" ukázkové příklady, které jsou paměťově nenáročné a všechno se odehrává v zásobníku. Tam vznikají pravé záhady.

Zatímco když hrábneš někam daleko do paměti, která není tvoje, tak tě systém hned odstřelí a člověk hned ví, na čem je (teda někdy).

No nic, zkus tohle řešení:

#include<string.h>
#include<stdio.h>
#include<stdlib.h>

typedef struct {
  char meno[10];
  char priez[20];
  int oscislo;
  char znamka;
} STUD;

void vlozStud(int *pocet) {
  (*pocet)++;
}


int main() {
  
  int pocet, N = 20;
  char *vyber;
  STUD *student;
  
  pocet=0;
	
  student=(STUD *) malloc(1*sizeof(student));
  vyber = (char *) malloc(N*sizeof(char));
 
  printf("%d\n",pocet);
  
  while(1){ 
    scanf("%19s", vyber);
    if (!strcmp(vyber, "x")) break;
    if (!strcmp(vyber, "b")) vlozStud(&pocet);
    printf("%d",pocet);
  }
  
  printf("%d",pocet);
  free(student);
  free(vyber);
  return (0);
}

(Chtěl jsem napsat ten program slušněji, ale to už bych ho příliš měnil.)

[Martin]

Tak zkuste ten svůj původní program spustit pod windows a místo jednoho písmenka jich zadejte najednou třeba dvacet a vsadil bych se že ten program spadne i pod windows, protože si něco přepíšete.

Nebo zkuste mezi
int pocet;
char vyber;
vložit nějakou proměnou, kterou dále v programu nijak nepoužijete (doufám, že ji kompilátor bez optimalizace nevyhodí, pokud ano, tak ji nějak nesmyslně použijte, aby zůstala v tom programu zachována). A spusťte program pod linuxem a zadejte jedno písmenko a program najednou bude chodit v pořádku.

[oxymoron]

Dakujem vsetkym velmi pekne za odpovede :)