Nemožné problémy s delete[] – C / C++ – Fórum – Programujte.com

Dobrý den, vymyslel jsem metodu na překlad řetězce na celé číslo, ale nefunguje to.
Když smažu z kódu delete[] c; ,všechno funguje. Přikládám obrázek.

static int StringToInt(string s, int & ii)
  {
   ii=0;
   char * c = new char[s.length()];
   strcpy(c,s.c_str());
   for(int i=0;i<s.length();i++)
   {
    cout<<endl<<c<<endl;
    if(c[i] == '0') i=i*10+0;
    else if (c[i] == '1') ii=ii*10+1;
    else if (c[i] == '2') ii=ii*10+2;
    else if (c[i] == '3') ii=ii*10+3;
    else if (c[i] == '4') ii=ii*10+4;
    else if (c[i] == '5') ii=ii*10+5;
    else if (c[i] == '6') ii=ii*10+6;
    else if (c[i] == '7') ii=ii*10+7;
    else if (c[i] == '8') ii=ii*10+8;
    else if (c[i] == '9') ii=ii*10+9;
    else ii+=0;   
   }

   delete[] c;
   return ii;    
  }

jenom delete c;

Přepsal jsem to na delete c; ale problém je pořád stejný.

no delete [] c je spravne...

problem je v tom ze zapominas na jeden znak co jaksi ukoncuje standardni cstring ... zapisujes pak ZA pole

#1 Cpp_Amater
KIIV už tu chybu vypíchl.

Jinak (mimo dotaz) - proč vůbec pořád dokola kopíruješ ten řetězec? První kopie se udělá při volání funkce StringToInt. Ta je sice obvykle rychlá, protože se obvykle jenom inkrementuje reference counter, ale i tak je zbytečná. Stačila by zde reference na konstantní string.

Dále: pak ten řetězec překopíruješ do extra alokovaného bufferu c. Proč? Vždyť totéž se dá číst i přímo z s předaného jako parametr funkci. Mimochodem - toto je typická konstrukce ortodoxních zastánců názoru, že implementace s klasickým céčkovým stringem je vždy rychlejší než std::string.

A na závěr bych zvážil použití konstrukce switch místo série if ... else if ... Výsledná binárka asi bude úplně stejná, ale subjektivně si myslím, že je ten program mnohem čitelnější, protože se mi někde uprostřed switche nemůže schovat žádná odlišná podmínka.

#5 ondra.holub
tak tak... ke kodu sem se radeji ani nevyjadroval :D nechtelo se mi psat takovej dlouhej CodeReview, kdyz se ptal jen proc se to deje a ze je to pry nemozne :D