Dotaz: Konverze string -> integer

[Milan63]

Resim problem konverze stringu na integer, pouzivam tuto funkci:

int str2int(string value)
{
    int number;
    
    if(sscanf(value.c_str(), "%d", &number) == EOF)
    {
        cerr << "Input file is not in correct format" << endl;
        exit(EXIT_FAILURE);
    }
          
    return number;
}

Ta funguje v poradku, pokud prevadim napr. "123", pokud do teto funkce ale poslu napriklad retezec "xxx", vrati mi nejake zaporne cislo (misto toho, aby skoncila chybou). Jak tedy mohu tuto funkci opravit, aby mi v pripade nevalidniho vstupu tuto skutecnost opravdu zahlasila?

---

---

Odpovědi

[rastos]

Nainicializuj number na hodnotu, ktorú to má vrátiť, keď sa vstup na číslo skonvertovať nedá. Alebo výsledok sscanf() kontroluj nielen na EOF ale aj na nulu.

[Kit]

Vypiš si návratovou hodnotu funkce sscanf() a bude ti to jasné hned. Zřejmě je chybou ji testovat na konstantu EOF, když na konec souboru nenarazila.

V jakém je to vlastně jazyku?

[Šangala]

Já tomu říkám „C+“ .

[Kit]

Zkusil jsem si to po úpravách v C (protože v C a C++ skoro nic nedělám) a vyšlo mi, že kdž v podmínce místo ==EOF bude <=0, tak by to mělo fungovat.

Na příkladu je vidět, jak nebezpečné může být používání neinicializovaných proměnných.

[l4m4]

Pouze v případě, že se bude převádět právě jedna hodnota. Viz můj dva dny starý příspěvek...

[Kit]

Jasně. Aby to bylo správně i logicky, mělo by být spíš <1, pokud očekávám jednu hodnotu, <3, pokud očekávám 3 hodnoty apod.

[Šangala]

No to bych raději volil != 1, to je přesnější.

Ale jinak pokud by to mělo být C++, ne jen C+, tak by to mohlo být třeba takto:

int str2int(std::string value)
{
  std::istringstream iss(value);
  int number;
  if((iss >> number).fail())
  {
    std::cerr << "The input number is in a invalid format" << std::endl;
    exit(3);//or throw a exception…
  }
  return number;
}

[l4m4]

Tak funkce to samozřejmě hlásí, a to tak, že vrátí nulu, protože proběhlo celkem nula úspěšných konverzí. Viz dokumentaci. EOF vrací funkce z rodiny scanf() pouze v případě, kdy nejde číst vstup, což zrovna u sscanf() moc nenastává.

Jinak tenhle prapodivný mix C a C++ taky nechápu.

[petrz]

ja by ch na to pouzil funkci atoi. (pokud ovsem neni cilem "udelej si sam")

[l4m4]

S atoi() je kontrola chyb konverze opravdu jednoduchá, jelikož žádnou kontrolu nedělá ;)

Šlo by ale samozřejmě použít strtol() a podobné.