Как вернуть указатель в начало?

Question

[BuckvicK]

Имеется такой код.
text - это текст состоящий из слов, которые разделены разделителями
separator - массив разделителей
num_str количество строк.

bool checkString(char **text, char *separator, int num_str){
	int j = 0;
	for (int i = 0; i < 1; i++, j = 0){
		char *word = (char*)malloc(sizeof(char));
		while (text[i][j] != '\0'){
			if (!(strchr(separator, text[i][j]) != NULL) && !(text[i][j] == '\0')){
				word = (char*)realloc(word, (j + 1) * sizeof(char));
				word[j] = text[i][j];
				
			}
			else{
				word[j] = '\0';
				puts(word);
				word = NULL;
			}
			j++;
		}
	}
	return true;
}

Функция должна находить слова и выводить их в консоль, аналог strtok, но она первое слово определяет нормально, а остальные начинает записывать в память не с первой позиции, а с некоторой N - ой позиции, N = Длина_первого_слова + количество_разделителей. То есть, если есть строка:
Mama.Papa,Brat
В памяти будет выглядеть первое слово так:
"Mama"
Второе получится так:
"hhhhhPapa"
Где "hhhhh" это просто что то в памяти, как сделать что бы записывалось в первый символ?

Answer 1

[Rsa97]

Жуть. Три года расстрела через повешение за такой код. realloc на каждый символ, обнуление указателя без освобождения памяти, постоянное индексное обращение при переборе строки - кол с минусом.

А ошибка очень простая - запись символа производится в word[j], соответственно на втором слове j начинается не с нуля. Вам ещё повезло, что программа не пытается записать символы поверх кода или в чужую область.

bool checkString(char **text, char *separator, int num_str) {
  char *str, *cur, *word;
  int len;
    for (int i = 0; i < num_str; i++, text++) {    
        str = cur = *text;
        while (*cur) {
            if (strchr(separator, *cur)) {
                len = cur-str;
                word = malloc(len+1);
                strncpy(word, str, len);
                puts(word);
                free(word);
                str = cur;
            }
            cur++;
        }
        puts(str);
    }
    return true;
}

Comments

[BuckvicK]

Огромное спасибо!
Да, возможно код не идеален, да возможно я заслуживаю наказания, но Вы еще весь код не видели, тогда точно мне кирдык был бы)
Ведь я только учусь, первый семестр изучаю С, выкручиваюсь как могу, все начинают с глупых решений, с самых безобразных комбинаций. Вот это моё безобразное решение)

[jcmvbkbc]

> word = malloc(len+1);
> strncpy(word, str, len);
> puts(word);

И что, работает? Ничего лишнего не выводит?
И зачем так, а не просто printf("%.*s", len, str);

[Rsa97]

jcmvbkbc: А с чего оно должно что-то лишнее выводить? А написано просто для примера выделения памяти. printf, конечно же, проще.

[Rsa97]

jcmvbkbc: Хотя да, забыл
word[len+1] = 0:

[jcmvbkbc]

о, пошло дело (:
Может тогда и strncpy на memcpy заменить, всё равно длина известна заранее?

[jcmvbkbc]

Хотя не совсем пошло, word[len] = 0; же.

[Rsa97]

jcmvbkbc: Угу. Торможу капитально.

[Rsa97]

jcmvbkbc: Сколько лет уже C в руки не брал, сплошные богопротивные JS да PHP и изредка кузявый Erlang.

[BuckvicK]

Rsa97: Возможно я что то не правильно делаю, но у меня заработало только если добавить
word = (char*)malloc(len + 1);
Преподаватель называл это "Явным приведением типа", так ли это?
Не могли бы Вы посоветовать действительно хорошую литературу по указателям или где можно наглядные примеры посмотреть?

[Rsa97]

BuckvicK: C++ требует явного приведения, C не требует. А по литературе ничего посоветовать не могу, сам начинал давно и на ассемблере КР580, затем уже был C на 80286, где приходилось вникать в тонкости работы программы для экономии ресурсов. Так что при виде указателей, структур, массивов и других примитивов C просто в голове возникает представление всего этого на низком уровне.

Практически все данные в программе - набор байтов в оперативной памяти. Указатель - не что иное, как просто адрес в этой памяти. При разыменовании указателя компилятор, зная тип указателя, представляет нужное количество байт по этому адресу как переменную этого типа. Инкремент/декремент указателя - всего навсего прибавление к этому адресу размера (sizeof) типа. Ну и т.д.