C++ - Spojový seznam - nefunkční Find() a problém s výjimkami

V tom find ziaden problem neni, ked ti to tam priradi NULL tak si si to blbo naspojkoval pri vytvarani. Pouzi debugger daj si tam breakpoint a pozri si tie datove struktury ze co v nich je.

fprintf(stderr, error) vypíše hlášku, ale za ni ještě počet vytištěných znaků

coze? Mozes sem dat nejaky konkretny priklad kodu?

Co sa tyka vynimiek tak to musis najprv trochu pochopit ze co to je a preco to vobec existuje.
Priklad: mam program
UholVystrelu = NajdiPaducha();
Strielaj(UholVystrelu);

V takom programe ak napr. fcia NajdiPaducha() nenajdi nikoho tak sa proste nesmie vratit inac by program strielal nieklam doprdele kde vobec strielat nema. T.j. program NEMOZE pokracovat. Samozrejme si si mohol tam dat navratovu hodnotu a testovat ju, ale to je nepohodlne ak je vela volani ktore mozu vyvolat stejnu vynimku, napr. pri diskovych operaciach apod. nebudem predsa za kazdou operaciou testovat nejake hodnoty. Takze si to try-catch urobim tam kde volam tych vela funkcii a akonahle nastane vynimka tak sa to NEvrati z funkcie ale skace rovno na catch. Ak neexistuje catch pred volanim fcie tak to ukonci program (nemozes predsa strielat a volanie funkcie NajdiPaducha() neni uzavrete v try-catch takze nema kam inam skocit, len ukoncit program.
//edit: deletol som zbytocny prikald

T.j. inymi slovami napisane - ak vynimku generujes vo funkcii Neco(), tak try-catch ma byt tam kde sa ta funkcia Neco() vola.

Ale kukam ze ty to try - catch mas dobre (vyvolanie exception je medzi try a catch, nemusi to byt nutne vo funkcii takze to mas OK) takze mas najskor blbo to catch nadefinovane (nechyta to tu konkretnu exception - to sa da selektovat ze co ma catch chytat a co ne.)

P.S. aha uz to vidim. Skoci to to do catch a potom to pokracuje dalej - a jaku hodnotu potom vratis? Nejake null alebo nahodne a potom jasne ze program ma problem. Normalne sa to robi tak jak som pisal v prvom prispevku, try-catch sa robi u volania funkcie, aby si po vyvolani vynimky nemusel vracat ziadnu hodnotu a aby to skocilo tam na catch. ked to urobis len vo fcii tak si chytil vynimku uz vo fcii (skocilo ti to na catch) ale potom to normalne pokracuje (vynimku si osetril v catch) takze stale musis vratit nejaku hodnotu z tej funkcie.

Co sa tyka toho spojkovania (Add_on_First) tam vidim zatial problem v tomto
if(last == NULL){
last = first;
last->next = first->next;
last->prev = first->prev;
}
konkretne to
last->next = first->next;
last->prev = first->prev;
je nezmysel pretoze last a first ukazuje na stejnu adresu takze robis vlastne neco ako X=X a Y=Y. Ale neni to chyba, len zbytocnost.

A v tom else je blbo:
first->prev = first;
ma byt
first->prev = node;
ale to by tiez nemalo robit problem ptz "prev" nikde nepouzivas.

Problem tam nikde nevidim, musi to fungovat, jak presne ta funkce Find "nefunguje"? Kde vypise co zle?

Ok, Add_First jsem přepsal takhle:

void List::Add_on_first(int data){
    item *node = new item; //vytvorim objekt

    //inicializuji objekt
    node->data = data;
    node->next = NULL;
    node->prev = NULL;

    if(first == NULL){
        first = node;
        first->next = NULL;
        first->prev = NULL;

        if(last == NULL){
            last = first;
        }

    }else{
        item *temp; //pro provazani noveho a byvaleho "first"
        temp = first;

        first->prev = node;

        first = first->prev;
        first->next = temp;
        first->prev = NULL;

    }
}

Vypadá to, že Find už funguje - jdu mrknout na try-catch :D

Edit: Kontroluješ rychleji, než já stíhám přemýšlet :D

Este tam vidim jeden potencialny problem a to ze ak pri mazani prvku vymazes posledny prvok tak nenastavis last na NULL. Mal by si testovat pri mazani na konci fcie ze ci sa first nastavil na NULL a ak ano tak last nastavit tiez na NULL.

Tak, last jsem ošetřil, teď koukám na ten try-catch. Povedlo se mi to vyřešit takto:

Definice fce:

class ListEmptyError{
public:
    ListEmptyError(){
        cout << "LEE!";
    }
};

int List::First(){
    if(first){
        return first->data;
    }

    throw ListEmptyError();
};

Volání fce v mainu:

    try{
        cout << a.First() << endl; //LEE
    }catch(...){
        cout << endl;
    }

Takhle to funguje, jak bych chtěl, ale napadá mě, jestli by to nešlo řešit nějak elegantněji, ideálně už v rámci funkce?

A co ma ta funkce "elegantene" urobit ked je list prazdny? Ma ti zatelefonovat ze kam ma skocit? Tym catch urcis ze kam sa ma skocit. Ked chces skocit mimo fiunkci tak catch musi byt mimo funkci.

Ked ma fcia vracat hodnotu tak musi vratit hodnotu, ked ziadnu nema tak musi skocit mimo funkci.
Inac sa to da len tak ze si zadefinujes funkci tak ze nevraca hodotu ale nejaky status a hodotu bude plnit do nejakej adresy, napr.

BOOL DajData(int* pInt)
{
 if(mam_data)
 {
  *pInt=data;
  return TRUE
 }
 else
  return FALSE;
}

main()
{
 int i;
 if(DajData(&i))
   cout << i;
 else
   cout << "nemam data"
}

k tomu ale potom nepotrebujes ziadne exceptions, a je to komplikovane testovat to furt po kazdom volani fcie, viz moja uplne prva odpoved. Do try-catch bloku mozes uzavret aj 10volani fcie a nasledne urobit osetrenie chyby len na jednom mieste, tym padom zdrojak neni preplneny ifmi a program je prehladnejsi.
P.S> nemaju vas to nahodou ucit v skole? :D Zdravim neschopnych pedagogov :)

No, objevil se další problém, který jsem nečekal ani já :D Vytvořil jsem funkci Del_last, která by měla mazat poslední prvek seznamu, a tedy i fungovat hodně podobně jako Del_first... Problém je, že pokud ji zavolám, a seznam je pak prázdný, následující find při "throw" nahlásí Segmentation fault. Fakt už nevím proč, kód Del_last vypadá takto:

void List::Del_last(){
    if(last == NULL){
        cout << "Seznam je prazdny!" << endl;
    }else{
        item *del;

        del = last;

        last = last->prev;
        last->next = NULL;

        delete del;

        if(last == NULL){
            first = NULL;
        }
    }
}

Celý kód pak vypadá takhle: ktblte55-19005

Opět moc díky za rady, je to nějak náročné :D

necital som cely program ale ked urobis
last = last->prev;
tak nemozes vediet ze ci ten last teraz neni NULL takze nemozes na tu adresu na ktoru ukazuje last pristupovat len tak
last->next = NULL;
ale musis najprv otestovat ci last neni null a pristupovat na tu adresu mozes len ak last neni NULL. Ak je null a chces na tu adresu cosi zapisovat tak "program proved neplatnou operaci". T.j. nemozes menit next u neexistujuceho prvku.
A neni to narocne :)