C volanie funkcii

xyzz · Napísal **xyzz**: 15.10.2011 22:23

Prosim niekoho o radu... Jedna vec mi nefunguje a neviem ako na nu... Davam tu taku zjednodusenu verziu :

Kód:

int nacitanie()
{
  int i = 5;
  return i;
}

int sekundy(int argc, char *argv[])
{
  if (argc == 2) if (strcmp("-s", argv[1]) == 0)
  {
  int sek;
  nacitanie();
  sek = nacitanie (); 
  return 0;
}
}

No, toto je vlastne prvy problem, mozem to sek = nacitanie(); spravit? Asi nie, lebo to nefunguje... Potom teda ako do premennej dostanem to co funkcia nacitanie vyreturnovala?

Dakujem pekne

nBXXL · Napísal **nBXXL**: 15.10.2011 23:43

1. medzi nacitanie a (); mas medzeru, preto ti to nejde
2. riadok nacitanie() je zbytocny, takto sa ti funkcia nacitanie(); zavola dvakrat
3. môžeš spraviť aj niečo ako "int sek = nacitanie();" a skrátiť si program
4. keď už skracuješ program, keďže funkcia sekundy nič nevracia (resp. vracia za každých okolností 0), kľudne môže byť typu void a nemusíš tam mať riadok "return 0;"

street_punk · Napísal **street_punk**: 16.10.2011 1:11

Takže to že za nacitanie a () nemá byť medzera je blbosť. Práveže sa doporučuje, aby tam vždy bola medzera za priehľadnosť programu. Funckia načítanie je ti úplne zbytočná, keďže v nej len priradíš jednu hodnotu, čo môže kľudne urobiť v hlavnej funkcii (neni to zle čo si napísal, ale je to úplne zbytočné ). Ale čo je problém, voláš dva krát if za sebou. Ono by to ani nebol ani taký problém, ale ten druhý if je zahrnutý v tom prvom, takže za ten prvý if musíš dať zátvorku {. To načo si sa pýtal (na to sek = nacitanie (); ), tak funkcia musí mať parametre, ktoré do nej dávaš. Takže v tvojem prípade, tvoja funkcia nevie, čo je to to nacitanie ()
A nBXXL má pravdu, že si tú funkciu zavoláš 2 krát, takže to prvé nacitanie () vyhodz.

chrono · Napísal **chrono**: 16.10.2011 10:20

street_punk píše:

Takže to že za nacitanie a () nemá byť medzera je blbosť. Práveže sa doporučuje, aby tam vždy bola medzera za priehľadnosť programu.

Kto to doporučuje? Napr. v K&R "štýle" sa nič také neodporúča a podľa mňa je to extrémne neprehľadné.

nBXXL · Napísal **nBXXL**: 16.10.2011 11:27

@street_punk: tak to sorry, som nevedel, že tam tá medzera môže byť, mňa tak učili že to musí byť bez medzery.. (a chrono má pravdu že je to prehľadnejšie, keď tam tá medzera nie je...)
ale zase to, čo ty píšeš sú tiež blbosti:
1. pokiaľ v ife máš len jeden príkaz (v tomto prípade ďalší if) tak tam zátvorky byť nemusia.. (predpokladám, že autorovým cieľom bolo to, aby sa príkazy spĺňali iba vtedy, pokiaľ sú splnené obe podmienky, a aby sa netestovalo na druhú, pokiaľ nie je splnená prvá)..
2. žiadna funkcia nemusí mať parametre, táto ani nemá, takže funkcia vie, čo je to nacitanie()....

teda program ako tento:

Kód:

#include <cstdio>

int blbost()
{
    int i = 5;
    return i;
}

int main()
{
    if (true)
        if (true)
        {
            int sek = blbost ();
            printf("%d\n",sek);
            return 0;
        }
}

skutočne vypíše 5...

xyzz · Napísal autor témy **xyzz**: 16.10.2011 11:30

Tu medzeru som do programu nenapisala, to sa mi nejako tu az podarilo.. V tej funkcii nacitanie nie je iba to priradenie, len som to chcela zjednodusit nech tu nevypisujem milion riadkov, princip je ten ale ten isty- funkcia returnuje nejaky int (ziadne parametre nema). Dakujem vam za rady, dnes vecer budem s programom pokracovat, keby mi nieco nejde, urcite sa zasa ozvem

nBXXL, ano presne tak je mysleny ten if a aj funguje

street_punk · Napísal **street_punk**: 16.10.2011 15:34

Tak k tým medzerám, napríklad Pavol Herout uvádza, že je lepšie nechávať medzery. Tak isto aj v škole nás tak učia. A je to aj môj osobný názor, že je tak kód priehľadnejší. Ale to že či tam tú medzeru dá, alebo nedá, programu nijak neublíži.
A stým if, je lepšie tam dať tie zátvorky hneď po napísaní if (urobiť si šablónu) a potom mať istotu, že sa nepomýli.

xyzz · Napísal autor témy **xyzz**: 17.10.2011 15:16

Tie veci, co mi nesli som uz vyriesila, ale zase je nieco dalsie... Nechcem tu zbytocne zakladat niekolko tem, tak to dam sem. Toto nacitanie zatial robi to, ze pokial clovek zada nieco ine ako cislo (napriklad "dfsahfdas" alebo "53153fdsfds13") tak program vypise to "musis zadat kladne cislo".
Problem je, ze do toho neviem zakomponovat 2 veci, vcera a dneska sa o to pokusam a vzdy mi to niekde zlyhalo :roll:

V pripade prazdneho vstupu musi byt vystup tiez vygeneruje prazdny vystup. Teraz mi tam pri zadani enteru vypise 0 sekund. Druhy problem je, ze neviem, ako zistit, ci je cislo vacsie ako ULONG_MAX...lebo samotna ta podmienka je blbost, cislo nemoze byt vacsie.. No proste neviem sa pohnut dalej... Keby mi niekto vedel s tymto poradit, bola by som moc vdacna..

Kód:

long int nacitanie() 
{
  long int cislo=0;
  char c;
  while ((c = getchar()) != '\n')
    { 
      if (c >= ascii0 && c <= ascii9)
      {
      cislo = cislo * 10;
      cislo = cislo + (c-ascii0);
      }
      else if (c != '\n')
      {
   cislo = -1;
   break;
      }
  }
  if (cislo == -1) { printf("musis zadat kladne cislo\n");}
  return cislo;
}

Fico · Napísal **Fico**: 17.10.2011 16:47

K prvej otazke, cize osetrenie prazdneho vstupu, si mozes pridat pomocnu premennu uchovavajucu stav, ci bolo nieco zadane alebo nie (dalo by sa to aj elegantnejsie, ale to by si musela prerabat asi celu funkciu):

Kód:

long int nacitanie()
{
  
  long int cislo=0;
  int vstup = 0;    /* v tejto premennej bude nula pokial sa nezada cokolvek okrem enteru */
  char c;
  while ((c = getchar()) != '\n') { 
      vstup = 1;        /* getchar() nenacital enter, takze zmenime premennu na jednotku */
      if ( c >= 0x30 && c <= 0x39 ) {      
          cislo = cislo * 10;   
          cislo = cislo + ( c - 0x30 );
      } else if (c != '\n') { 
          cislo = -1;
          break;
      }
  }
  if (cislo == -1) 
      printf("musis zadat kladne cislo\n");
  if (vstup == 0)      /* a tu uz iba staci porovnat, ci je vstup nula */
      printf("prazdny vstup\n");

  return cislo;

}

A druha otazka sa riesila prednedavnom tu --> http://www.pcforum.sk/zistenie-ci-je-cislo-vacsie-ako-ulong-max-vt122291.html
Cize mozes to vyriesit ako tam, ze bud pouzijes pretypovanie s pouzitim vacsieho datoveho typu:

Kód:

if ( (double)( cislo * 10 ) > ULONG_MAX )

alebo vydelenim ULONG_MAX desiatimi:

Kód:

if ( cislo > ( ULONG_MAX / 10 ) )

xyzz · Napísal autor témy **xyzz**: 22.10.2011 13:08

Dakujem, viem, ze moje riesenie asi nebude extra elegantne, bohuzial som zaciatocnicka :oops:

Preto mam aj dalsiu otazku... Nejako mi to nefunguje s tym ULONG_MAX. Ked zadam ozaj prilis velke cislo, tak mi to normalne vypise chybovu hlasku, ale ked zadam napriklad tu hodnotu ULONG_MAX alebo o nieco mensiu, tak mi to nevypise vobec nic

Kód:

long int nacitanie()
{
 
  long int cislo=0;
  int vstup = 0;    /* v tejto premennej bude nula pokial sa nezada cokolvek okrem enteru */
  char c;
  while ((c = getchar()) != '\n') {
      vstup = 1;        /* getchar() nenacital enter, takze zmenime premennu na jednotku */
      if ( c >= 0x30 && c <= 0x39 ) {    
          if ( cislo > ( ULONG_MAX / 10 ) ) {  pom = -2; break;} 
          cislo = cislo * 10;   
          cislo = cislo + ( c - 0x30 );
      } else if (c != '\n') {
          cislo = -1;
          break;
      }
  }
  if (cislo == -1)
      printf("musis zadat kladne cislo\n");
  if (vstup == 0)      /* a tu uz iba staci porovnat, ci je vstup nula */
      printf("prazdny vstup\n");

  return cislo;

}

Asi mam tu podmienku na zlom mieste? alebo kde je chyba?

nBXXL · Napísal **nBXXL**: 22.10.2011 14:35

ta podmienka by mala byt medzi

Kód:

cislo = cislo * 10;

a

Kód:

cislo = cislo + (c - 0x30)

chrono · Napísal **chrono**: 22.10.2011 14:45

Inak pomocou tak jednoduchej podmienky sa, pravdepodobne (nechce sa mi to skúšať), nepodarí dať do tej premennej presne ULONG_MAX, pretože je tam zaokrúhľovanie pri delení (takže teoreticky by asi bolo možné najskôr tú novú cifru odpočítať od ULONG_MAX a až potom to deliť, prípadne pri cislo == ULONG_MAX/10 kontrolovať, či je c <= ULONG_MAX % 10).

nBXXL · Napísal **nBXXL**: 22.10.2011 15:43

ignoruj môj predchádzajúci príspevok, zle som pochopil ten kód

tú podmienku máš na dobrom mieste.. chrono dobre vysvetlil, prečo ti tam nejde zadať číslo ULONG_MAX alebo trocha menšie/väčšie čísla, pokiaľ je ULONG_MAX napr. 9005 a chceš tam zadať 9000-9009, tak v tvojej podmienke vyjde 900 == 900, čo podmienku spĺňa (lebo tam máš iba >), teda ošetri ešte prípad, že cislo == ULONG_MAX/10, pridaním podmienky, či (c-0x30) <= (ULONG_MAX % 10)

C volanie funkcii

Podobné témy