[SOLVED] segmentation fault w c++ z gnu compiler

[kolrek]

Robie program który porównuje dwa ciągi znaków i ustawia je alfabetycznie ale jak mi się wydaje że jest już dobrze to wywala mi segmentation fault. Nie zwracajcie uwagi na komentarze

Kod: Zaznacz cały

#include <iostream>
#include <stdio.h>
using namespace std;
char *text()
{
 static char *str; //deklaruje napis
 cin>>str;
 return str;
}

char *wskaznik (char tr){
    char *xtr;
    xtr=&tr;
    return xtr;

}

int strcmp (char *xtr, char *ytr){
    if(*xtr==*ytr)
        if(*xtr=='\0')
            return 0;
        else
            return strcmp(xtr+1, ytr+1);
    else
        return *xtr - *ytr;
}


int main()
{
    char *str1, *str2, *str3, *str4;
    str1=str3=text();
    str2=str4=text();
    str1=wskaznik(*str1);
    str1=wskaznik(*str2);
    /*
    cin>>str1;
    printf("oto %s\n",str1);
    cin>>str2;
    printf("oto %s\n", str2);*/
    //str1=&stri1[1];
    //str2=&stri2[1];
    if(strcmp(str1,str2)==0)
        cout<< "The words are the same"<<endl;
    else
    if(strcmp(str1,str2)<0)
        cout<<str3<<" is greater than "<<str4<<endl;
    else
        cout<<str4<<" is greater than "<<str3<<endl;
        //cout<<"mirek"<<endl;*/
    //else
    //cout<<"mireo"<<endl;
    cout << "Hello world!" << endl;
    return 0;
}

[dawwin]

Nie mam pojęcia, po co stworzyłeś własną funkcję strcmp() skoro taka już istnieje w nagłówku

Kod: Zaznacz cały

#include <cstring>

Poza tym w C++ o wiele wygodniej używać klasy string. Prosty przykład

Kod: Zaznacz cały

string a;
string b;
a = "abc";
b = "def";

if (a.compare(b) == 0) {
   cout << "a i b sa takie same" << endl;
} else {
   cout << "a i b sa inne" << endl;
}

[kolrek]

oto chodzi że miałem stworzyć własna taką funkcje i to o typie char, i jak teraz zmienie na string to będą w skakiwały błędy z konwersją na char w funkcji strcmp

[dawwin]

To nazwij ją chociaż inaczej.
I nie potrzeba Ci wywołań rekurencyjnych (czyli wywoływania strcmp() w strcmp()). Wystarczy jedna pętla while.

Kod: Zaznacz cały

int my_strcmp(const char *a, const char *b)
{
    while (*a != '\0' && *b != '\0') {
        if (*a != *b) {
            return ((*a)-(*b));
        }

        a++;
        b++;
    }

    if (*a == *b) {
        return 0;
    } else {
        return ((*a)-(*b));
    }
}

I jeszcze jedna uwaga - nie używamy w c++ printf() i scanf()