Как написать функцию поиска пробела в строке?

Question

[dandropov95]

Вроде написал, вроде как работает. Но кажется что это такое себе решение. Подскажите как решить данную задачу более правильней. Нужно вернуть указатель на первый найденный пробел, иначе если не найден пробел вернуть NULL.

char * find_space(char * string)
{
	char ch;

	while (ch = *string++)
	{	
		if (ch == ' ')
		    return &ch;
	}

	return NULL;
}

Или вот так. Но все равно как то такое себе кажется.

const char * find_space(const char * string)
{
	int i = 0;

	while (string[i] && string[i] != ' ')
		i++;

	if (string[i] != '\0')
		return string;

	return NULL;
}

Comments

[Евгений Шатунов]

Ты сейчас в своей функции огребаешь, не стоит возвращать &ch.
Лучше for.

[dandropov95]

Евгений Шатунов, аа ch же локальная переменная лол.
А если так. Норм?

const char * find_space(const char * str)
{
	for (int i = 0; str[i]; i++)
		if (str[i] == ' ')
			return &str[i];

	return NULL;
}

[Евгений Шатунов]

dandropov95 , на самом деле тебе и int i = 0 тоже не нужен. Ты можешь спокойно итерироваться по указателю на память в строке.
А return str; сейчас делает не то, что тебе хотелось бы.

[dandropov95]

Евгений Шатунов, return &str[i]; вот так надо да?

const char * find_space(const char * string)
{
	while (*string)
		if (*string++ == ' ')
			return string;

	return NULL;
}

PS. хотя не. Опять не то.

[dandropov95]

Евгений Шатунов,
Вот так вроде то:

const char * find_space(const char * string)
{
	while (*string && *string != ' ')
		string++;

	if (*string != '\0')
		return string;

	return NULL;
}

[Евгений Шатунов]

dandropov95 , а ты не думал над тем, что const char* тоже можно использовать чтобы итерироваться по строке?
Вообще говоря, твоя последняя реализация уже, кажется, делает то, что тебе хочется. Но реализация понимается еще хуже чем изначальный вариант. На одном for с одним возвратом результата код будет пониматься легче.
const char* - это хорошо. Твоя функция должна быть чистой, т.к. ее ожидаемая суть заключается только в поиске.

А что будет если функцию вызвать так: find_space( nullptr );?
Что будет если функцию вызвать так: find_space( "" );?

[dandropov95]

Евгений Шатунов,
Единственное что смущает, так это то что явный return только при каком либо условии. Вроде как это не лучшая практика.

char * find_space(const char * string)
{
	if (string == nullptr || strlen(string) == 0) return nullptr;

	while (*string && *string != ' ')
		string++;

	if (*string != '\0')
		return (char *) string;
}

[Евгений Шатунов]

dandropov95 , а касты к char* для чего? Это же явно нарушает принцип чистоты функции.
Увы, тут видно что ты стоишь на месте. При чем с самого первого варианта. Значит надо подкинуть тебе идею.
Посмотри вот на такой вариант.

for( const char* cursor = input_string; *cursor; ++cursor )
{
   ...
}

cursor сразу дает всею полезную информацию о состоянии цикла. Из него, при должном обращении, и индекс символа можно вынуть.
Попробуй плясать от такого подхода.

[dandropov95]

Евгений Шатунов, каст потому что взглянул на функцию strchr и у нее определение имеет вид: char * strchr( const char * string, int symbol);
следовательно и кастинг к неконстантному значению

Answer 1

[al_sh]

char *findSpace(char *value)
{
    for(;value && *value && *value != ' '; ++value);
    return value && *value != '\0' ? value : nullptr;
}