Почему не переворачивает строку?

Question

[Pinkman]

Добрый вечер! Помогите понять в чём ошибка...
Есть такой код:

#include <stdio.h>
#include <string.h>

char	*ft_strrev(char *str)
{
	char tmp;
	int start_len;
	int end_len;

	start_len = 0;
	end_len = 0;
	while (str[end_len])
		end_len++;
	end_len -= 1;
	while (start_len <= end_len)
	{
		tmp = str[end_len];
		str[end_len] = str[start_len];
		str[start_len] = tmp;
		start_len++;
		end_len--;
	}
	return (str);
}

int		main(void)
{
	char *str = "hello";
	ft_strrev(str);
	printf("%s", str);
	return (0);
}

В итоге получаю ошибку SIGSEGV
gdb говорит что ошибка вылетает при первой итерации на строке:
str[end_len] = str[start_len];
Что не так?

Answer 1

[jcmvbkbc]

Что не так?
char *str = "hello";

Вот эта строчка не так. То, что компилятор разрешает так делать -- это анахронизм и отсутствие мало-мальской диагностики. "hello" может находиться (и в твоём случае, похоже, и находится) в области памяти доступной только для чтения, изменять эту строку нельзя. Правильно было бы написать так:
char str[] = "hello";
Если сделать эту замену, то код будет выделять место для массива str на стеке, менять такой массив можно.

Comments

[Pinkman]

спасиб! А можно поподробнее почему так получается(для меня это равнозначно(до этого момента)) что *str и str[] это разные вещи? Я ведь в функцию отправляю указатель всё равно..

[jcmvbkbc]

*str и str[] это разные вещи?

Одно -- указатель, второе -- массив. Но дело не в этом. Можно и с указателем написать так, что будет работать.

Я ведь в функцию отправляю указатель всё равно.

Важно на что он указывает. Если он указывает на строковый литерал или константные данные, то менять его нельзя. Если это массив в области данных, на стеке или в куче, то менять его можно.

[Pinkman]

jcmvbkbc, короче, я так понимаю это отдельный вопрос: в чём разница между char *var = "Word" и char var[] = "Word"?
Тогда есть разница в такой записи:
int main(int argc, char **argv) и int main(int argc, char[][]) и .. *argv[]) ?
т.е. в каком то случае я смогу изменять массив, а в каком то нет?

[jcmvbkbc]

я так понимаю это отдельный вопрос: в чём разница между char *var = "Word" и char var[] = "Word"?

Твой SIGSEGV сводится ровно к этому вопросу. Короткий ответ: char *var = "Word" -- нельзя менять, потому что указатель указывает на строковый литерал. char var[] = "Word" -- можно менять, потому что это массив на стеке или в области данных.

есть разница в такой записи:
int main(int argc, char **argv) и int main(int argc, char[][]) и .. *argv[]) ?

int main(int argc, char[][]) так написать нельзя. Между двумя другими вариантами разницы нет.

[Pinkman]

jcmvbkbc, спасибо!

Answer 2

[Евгений Залецкий]

#include <iostream>
#include <string>
#include <algorithm>
 
int main()
{
   std::string str = "test string";
   
   std::reverse(str.begin(), str.end());
   
   std::cout << str << std::endl;
 
   return 0;
}

Comments

[Pinkman]

Я на си пишу, и нужно саму функцию написать)

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

Илья, а зачем ты тогда в тегах C++ указываешь?