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

[p51x]

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

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

Я не так просил...

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

Выполнение программы повлекло

И? Появились мысли почему и откуда?

[p51x]

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

Переписал вот так

Добавить еще кучу проверок вместо исправления условия цикла - оригинально и показательно...

[Sinli]

переделал еще раз:

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

Но этот код уже не работает. Опять выводит ошибку сегментации. Почему, не могу понять, вроде все логично выглядит.

Извините, если раздражаю своей писаниной.

[Sinli]

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

И? Появились мысли почему и откуда?

Я тут провел пару небольших экспериментов, чтобы понять почему.
Сначала предположил, что это происходит, когда мы пытаемся вывести переменную, у которой нет значения:

#include <stdio.h>
int number(void);
int main(void)
{
        int x;
        x = number(5);
        printf("%d\n", x);
        return 0;
}
int number(void)
{
        int y;
        if (n > 10)
                y = 5;
        return y;
}

Но программа вывела значение 0.

Тогда я сделал так:

#include <stdio.h>
int *number(int);
int main(void)
{
        int *x;
        x = number(5);
        printf("%d\n", *x);
        return 0;
}
int *number(int n)
{
        int *y;
        if (n > 10)
                *y = 5;
        return y;
}

И выпала ошибка сегментации.
Выходит подобная ошибка выпадает, когда мы пытаемся разыменовать указатель, которому не назначили никакого адреса.

В итоге это тоже дает ту же ошибку:

#include <stdio.h>
int main(void)
{
        int *x;
        printf("%d\n", *x);
        return 0;
}

[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;
        int i = 0;
        while (st[i] != cr && st[i] != '\0' && st != NULL)
                i++;
        if (st[i] == '\0' || st == NULL)
                needtofind = NULL;
        else
                needtofind = &st[i];
        return needtofind;
}

Вы ведь про цикл while говорите?

[p51x]

Ну оно же все равно упадет...

[Sinli]

Почему?!
Если передается нулевой указатель, то благодаря st != NULL цикл while должен прекратиться. Далее условие st == NULL, присваивает needtofind значение нулевого указателя.
Что я не вижу? Я что-то неправильно сделал с логическими операциями?

[p51x]

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

Если передается нулевой указатель, то благодаря st != NULL цикл while должен прекратиться.

Но до этого вы же его фактически разыменуете... Аж дважды...

[Sinli]

Блин... st[i]...
Т.е. все, что надо было это поставить проверку на NULL первой?
В таком случае до второй и третьей проверок дело не доходит.

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

Нашел это в книге.
Скриншот 2018-10-11 13_26_18.png

И еще, насчет этого NULL
Когда мы пишем функцию, которая принимает и/или возвращает указатель, то мы всегда должны предусматривать вариант, когда она получает/возвращает нулевой указатель?