C++ specjalizacja zwracająca adres

Question

Hej, napisałem specjalizację, która zwraca najdłuższy napis. Chciałbym ją zmodyfikować żeby zwracała adres tego napisu, a nie tekst.

template <> char *maxn<char*>(char *tab[], int tabSize)
{
    char *p = tab[0];
    for(int i = 1; i < tabSize; i++)
    {
        if(strlen(tab[i]) > strlen(p))
            p = tab[i];
    }

    return p;
}

W mainie:

char *tab3[] = {"Korea", "Panie Janie", "Rano wstan", "ding dang dong", "PA"};
    cout << maxn(tab3, 5);

Obecnie po odpaleniu programu mam na wyjściu napis: "ding dang dong", chodzi mi o to, w jaki sposób zmodyfikować funkcję, żeby zwracała adres do tego napisu?

Answer 1

Teraz zwraca adres do napisu. Tyle, że operator<< cout-a działa w taki sposób, że jeśli dostanie wskaźnik na char to próbuje wypisać napis (tzn. kolejne bajty - litery - od podanego adresu kończąc gdy natrafi na znak '\0'). Żeby wypisać adres, po prostu przekonwertuj wartość zwróconą przez funkcje na void*.

cout << static_cast<void *>(maxn(tab3, 5));

Druga sprawa :

char *tab3[] = {"Korea", "Panie Janie", "Rano wstan", "ding dang dong", "PA"};

Czegoś takiego nie rób. Łańcuchy znaków przypisane w taki sposób do wskaźnika :

char * str = "slfhksdhf";

są zapisywane w pamięci tylko do odczytu i próba ich modyfikowania to UB. 
http://stackoverflow.com/questions/5343721/cant-modify-char-memory-access-violation

Dlatego albo dodaj const do typu tablicy (żebyś przypadkiem nie próbował modyfikować tych stringów), albo zastosuj sie do tego co napisał w odpowiedzi gość w zalinkowanym temacie na SO.

Comments

Dzięki za odpowiedź. Robię zadanie z książki Stephena Prata, czegoś takiego jak przekonwertowanie tej wartości na void jeszcze nie było, więc może dobrze zrobiłem to zadanie. Zostawię to jak jest. 

Jeśli chodzi o ten błędny zapis to masz rację powinno być:

const char *tab3[] = {"Korea", "Panie Janie", "Rano wstan", "ding dang dong", "PA"};

... i wtedy w funkcji też wszędzie const char* zamiast char*. Dzięki za zwrócenie uwagi.