Помогите разобраться с указателями на символьные строки.

[Sinli]

Добрый день. Есть такое задание:
Скриншот 2018-10-10 14_21_10.png
Вот мой код:

#include <stdio.h>
#include <string.h>
char * charfind (char *st, char cr);
int main (void)
{
        char string[81] = "bombom h bombom";
        char ch = 'h';
        char *x, *y;
        x = charfind(string, ch);
        y = strchr(string, 'b');
        if (x && y)
        {
                printf("%p и %p\n", x, y);
                printf("%c и %c\n", *x, *y);
                *x = 'g';
                *y = 'k';
                printf("%s\n", string);
        }
        return 0;
}
char * charfind (char *st, char cr)
{
        char * needtofind = NULL;
        int i = 0;
        while (st[i] != cr)
                i++;
        needtofind = st[i];
        return needtofind;
}

Вот такое предупреждение выдает компилятор. Я не понимаю, что оно означает.
Скриншот 2018-10-10 14_23_55.png
И вот такой получается вывод:
Скриншот 2018-10-10 14_24_32.png

Помогите, пожалуйста с, поиском и устранением ошибки. Из того, что я вижу: стандартная функция strchr() работает, так как "char *y" получает адрес символа. Очевидно, проблема в моей самодельной функции charfind - *x получает какое-то странное 0х68, ну и компилятор как бы намекает. Тогда я не могу понять, почему нельзя присвоить needtofind адрес st[i]. Разве я тут что-то нарушаю?

Заранее спасибо.

[p51x]

Сообщение от Sinli

Я не понимаю, что оно означает.

Что вы указателю присваиваете символ.

Сообщение от Sinli

И вот такой получается вывод:

Ну так видно же, что вместо указателя код символа. Естественно на разыменовании падает.

Сообщение от Sinli

Тогда я не могу понять, почему нельзя присвоить needtofind адрес st[i]. Разве я тут что-то нарушаю?

Можно. Но вы ведь не адрес присваиваете... в том и проблема.

П.С. И поиск надо делать до конца строки, а не пока куда-нибудь не попадем.
П.С.С. Если строку объявили в коде, то не стоит ее пытаться менять.

[Sinli]

Понял!!
Сделал вот так:

needtofind = &st[i];

В итоге компилятор не ругался. Вывод получился такой, как я и хотел:
Скриншот 2018-10-10 14_54_53.png
Только вопрос, зачем делать поиск до конца строки, если нам нужно найти всего лишь первое вхождение символа?

[p51x]

А если символа не будет, когда и где остановитесь?

[Sinli]

И правда, цикл, выходит, не прервется. переделал:

char * charfind (char *st, char cr)
{
        char * needtofind;
        int i = 0;
        while (st[i] != cr && st[i] != '\0')
                i++;
        if (st[i] == '\0')
                needtofind = NULL;
        else
                needtofind = &st[i];
        return needtofind;
}

забавно, что сначала я сделал так:

char * charfind (char *st, char cr)
{
        char * needtofind = NULL;
        int i = 0;
        while (st[i] != cr && st[i] != '\0')
                i++;
        needtofind = &st[i];
        return needtofind;
}

И думал, что если, у needtofind будет адрес '\0', то функция вернет значение NULL, и программа завершится. Однако вместо этого программа вошла в if и заменила символ конца строки на символ *x. И получилось:

0x7ffff6286c3f и 0x7ffff6286c30
и b
kombom h bombomg

Спасибо большое за помощь!

[p51x]

А теперь передайте пустую строку...

[Sinli]

Эм...
Объявлял
так

char string[81] = "\n";

так

char string[81] = "";

и так

char string[81];

Программа просто завершается, как и должна.
Или, может быть, я не так вас понял?

[p51x]

Ну это не совсем то... Объявите

char* string = '\0';

или вызовите свою функцию с nullptr/NULL.

[Sinli]

char string[81] = "\0";

это компилятор мне не позволил сделать.

x = charfind(NULL, ch);

это он скомпилировал.
Выполнение программы повлекло:
Segmentation fault (core dumped)

[Sinli]

Переписал вот так, программа стала нормально завершаться, если передается нулевой указатель:

char * charfind (char *st, char cr)
{
        char * needtofind;
        int i = 0;
        if (st != NULL)
        {
        while (st[i] != cr && st[i] != '\0')
                i++;
        if (st[i] == '\0')
                needtofind = NULL;
        else
                needtofind = &st[i];
        }
        else
                needtofind = NULL;
        return needtofind;
}