Program odwracający porządek łańcucha -źle działa Je

Question

Witam

Ostatnio na laboratoriach z programowania dostałem następujące zadanie:

Napisac program obracajacy porzadek znakow lancucha, np: "abcd" zamieniamy na "dcba". 

Napisałem program , który odwraca mi porządek łańcucha.Problem polega na tym, że jak  na przykład  chce odwrócić porządek  łańcucha  abcd  to wpisuje abcd i wyświetla mi się dcba ale dodatkowo wyświetlają mi się jakieś dziwne  znaki i nie wiem dla czego?.Dodatkowo w linijce 11 mam warninga  o treści implicit declaration of function 'strlen' [-Wimplicit-function-declaration]|  również w tej same linijce mam 

jeszcze jednego warninga o treści  incompatible implicit declaration of built-in function 'strlen'|.

Czy ktoś może mi pomóc w rozwiązaniu tego problemu?

Dodam jeszcze , że program ma być w języku C.Z góry dziękuję

#include <stdio.h>
#include <stdlib.h>
int main(void)
{
    char napis[100],odwrocony[100];
    int i;
    int dlugosc;
    printf("Podaj tekst , ktory ma zostac odwrocony: \n");
    gets(napis);

    dlugosc=strlen(napis);
    for(i=0; i<dlugosc;i++)
    {
        odwrocony[i]=napis[dlugosc-i-1];
    }
    printf("odwrucony tekst to:%s  \n",odwrocony);
    system("pause");
    return (0);
}

 

Answer 1

Pamiętaj, że napis kończy się znakeim NULL (a jego nie przepisałeś z tablicy napis do tablicy odwrocony). 

Można by zrobić tak:

for(i=0; i<=dlugosc; i++)

albo pod pętlą dodać NULL w dobrym miejscu tablicy:

odwrocony[dlugosc]=NULL;

Kiedy printf napotka NULL, to raczej nie wypisze już niczego bzdurnego.

 

Comments

Zamiast NULL lepiej dać 0 lub '\0'. NULLa lepiej stosować do wskaźników, coby kod był czytelniejszy.

---

ok teraz wszystko działa dziękuję za pomoc a dlaczego mam jeszcze te waringi? w linijce 11?

---

Załącz <string.h>.

---

Zamiast NULL lepiej dać 0 lub '\0'. 

Też tak mysle.  Ciąg znaków zawsze musi się kończyć znakiem o wartości 0, a NULL niekoniecznie musi być zerem (standard tego nie definiuje, chociaż jeszcze nie widziałem platformy na której byłoby inaczej).

---

ok działa dziękuję

---

Przyrównanie NULL do zera znajdziemy w stdio.h -> http://www.cplusplus.com/reference/cstdio/NULL/  ale jeśli ktoś nie chce używać NULL, to imo '\0' będzie najlepsze do terminacji łańcucha, 0 to jednak kod ASCII 0x30, nie 0x00. W praktyce i tak wiadomo, że printf we wszystkich trzech przypadkach zakończy wypisywanie tablicy na znaku o indeksie dlugosc.

---

Zero to zero, a nie znak ASCII ('0') ;)

---

Racja, nie ma tu przecież apostrofów, przepraszam

---

odwrocony[dlugosc]=NULL;

To jest błąd typu. Kompilator języka C nie ma obowiązku czegoś takiego skompilować.

---

http://www.cplusplus.com/reference/cstdio/NULL/