Проблемы с фукнцией atoi?

Question

[Maqsat Batyrqul]

char *temp, s[100];
	int i,len1,pos;
	int a[100]];

 	std::cin>>s;
 	len1 = strlen(s);
 	i = len1-1;

 	while(i>=0)
 	{
 		*temp = s[i];
 		a[pos] = atoi[temp];
 		pos++;--i;
 	}

Тут компилятор выводить ошибку
Segmentation fault (core dumped)
По мне ошибка здесь > *temp = s[i];
Почему то я не могу писать так > a[pos] = atoi[s[i]];
Компилятор g++, версия 5.4

Comments

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

Maqsat Batyrqul , а почему скобки квадратные? atoi(temp) жеж.

[jcmvbkbc]

> *temp = s[i];
Это явная ошибка. Непонятно что именно вы пытаетесь сделать, возможно здесь должно быть temp = s + i;
Тогда вообще эту строчку можно выкинуть и написать a[pos] = atoi(s + i);

[Maqsat Batyrqul]

jcmvbkbc: да точно я с вчера не смог это решить, но почему вы написали s + i? мне же надо элемент из строки

[jcmvbkbc]

Maqsat Batyrqul: Потому что по коду непонятно, что вы пытаетесь сделать. Моя интерпретация: вы пытаетесь пройти по введённой строке начиная с последнего символа и заполнить массив a числовыми значениями текущего видимого хвоста строки. Т.е. для строки "123" массив a будет {3, 23, 123}. И, кстати, pos не инициализирован в коде...

[Maqsat Batyrqul]

jcmvbkbc: ваша интерпретация верна только массив должен быть не {3, 23, 123} а {3, 2, 1}, вот этого не могу сделать

[jcmvbkbc]

Maqsat Batyrqul: ок. чтобы работало так нужно заменить char *temp на char temp[2] = {0, 0}. Тогда *temp = s[i] скопирует только один символ в строку temp, а вторым символом там будет символ с кодом 0, т.е. конец строки. Понятнее было бы temp[0] = s[i].

[Maqsat Batyrqul]

jcmvbkbc: Сработало спасибо большое, но суть я не смог понять можете объяснить?

[jcmvbkbc]

Maqsat Batyrqul: напишу уже большой ответ сейчас.

Answer 1

[jcmvbkbc]

Maqsat Batyrqul: atoi -- функция из стандартной библиотеки С, она работает с С-строками. С-строка это последовательность символов в памяти, заканчивающаяся символом конца строки -- символом с кодом 0.
Написав *temp = s[i], или что то же самое temp[0] = s[i] вы скопировали i-й символ из строки s на первое место в строке temp. Проблема изначального кода в том, что указатель temp не инициализирован, а значит может указывать в любое место в памяти. Т.е. вы скопировали символ в любое место в памяти. Мест в памяти в которые можно записывать существенно меньше, чем мест, в которые записывать нельзя. По счастливому стечению обстоятельств temp в вашей программе указывал в место, в которое записывать нельзя, в результате чего вы видели Segmentation fault при попытке записать туда.
В отличие от char *temp, запись char temp[2] определяет не указатель, а массив. Массив -- это непрерывный участок памяти. Если массив не константный, в него можно писать. Имя массива в большинстве случаев ведёт себя как указатель на первый элемент массива. Инициализировав массив нулями вы получили два нулевых символа в памяти, в которые можно писать. В цикле вы заменяете первый символ в этом массиве на i-й символ из массива s, второй символ остаётся неизменным -- символом с кодом 0, т.е. символом конца строки. Т.е. массив temp в цикле представляет собой С-строку единичной длины, которую можно передавать на вход atoi.

Answer 2

[Wexter]

а зачем вы к ней обращаетесь как к массиву?

Comments

[Maqsat Batyrqul]

Спасибо, затупил :)