Помогите, нужно исправить код программы на СИ!!!

[Vitami]

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

#include <stdio.h> 
#include <iostream> 

void konvert(char a[], char b[])
{
	int j = 0;
	for (int i = 0; a[i] != '\0'; i++)
	{
		if(a[i]!=' ')
		{
			for (i; a[i] != ' '; i++)
			{
				b[j] = b[j] + a[i];
			}//for
			j++;
		}//if
	}//for
        for (int i = 0; i < j; i++)
	{
		printf("%c", b[i]);
		printf(" ");
	}//for
}//konvert

int main()
{
	char stroka[100] = {' '};
	char strSl[100] = { ' ' };
	printf("Vvedite stroku:");
	scanf("%[^\n]s", &stroka);
	konvert(stroka, strSl);
	free(strSl);
	system("pause");
	return 0;
}//main

Можно пользоваться только стандартными библиотеками!

[New man]

Так ты же не в массив строк переводишь, а в строку.

И да, не нужно освобождать переменные на стеке.

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

free(strSl);

А так, наверно, стоит сделать так:

inline BOOL is_delimeter(const char c){
   return c==' ' || c == '\t' || c == '\n';
}
char*[] split(const char* text, size_t& result_size)
{
    // Выделяем память под промежуточный результат
    // размер - длина строки (очевидно, что слов в строке меньше, чем длина строки)
    // В принципе, размер можно делить смело на два, но я этого не сделал.
    // строки - это указатели на первый символ строки, поэтому умножаем на размер указателя
    char*[] temporary = (char*[])malloc(strlen(text)*sizeof(void*));
    size_t res_size = 0; // реальный размер итогового массива
    // Берём указатель на самый левый символ
    char* left = text;
    char* right;
    // Все строки заканчиваются нулём, поэтому пробегаем по строке, пока не натыкаемся на символ, равный нулю
    while(*left){
        // Здесь пропускаем все пробельные символы
        // ++ к указателю переходит на следующий символ
        while(*left && is_delimeter(*left)){
           left++;
        }
        // Мы нашли первый символ слова или же конец.
        // теперь точно так же ищем уже новый пробел
        right = left;
        while(*right && !is_delimeter(*right)){
            right++;
        }
        // Если левый указатель показывает на символ, а не на конец строки
        if(*left){
             // Выделяем память - длина слова + 1 на символ окончания строки
             temporary[res_size] = (char*)malloc(rigth-left+1);
             // копируем память начиная с указателя left, размером с длину слова
             memcpy(temporary[res_size], left, right - left);
             // добавляем в конец новой строки символ окончания строки
             temporary[res_size][rigth-left] = 0;
             // Запоминаем, что размер увеличился
             res_size++;
        }
        // Ищем следующее слово - перемещаем левую границу к правой
        left = right;
    }
    // мы знаем, сколько нам на самом деле нужно памяти для результата
    // поэтому копируем данные в новый массив
    char*[] result =  (char*[])malloc(res_size*sizeof(void*));
    memcpy(result, temporary, res_size);
    result_size = res_size; // запоминаем размер в параметре, переданном по ссылке
    free(temporary); // временный массив чистим
    return result;
}

Никогда не писал на C, поэтому не уверен, что правильно, код не запускал, могут быть ошибки.

split возвращает массив строк, в аргументе result_size размер получившегося массива.

[Vitami]

Сообщение от New man

Так ты же не в массив строк переводишь, а в строку.

И да, не нужно освобождать переменные на стеке.


А так, наверно, стоит сделать так:

inline BOOL is_delimeter(const char c){
   return c==' ' || c == '\t' || c == '\n';
}
char*[] split(const char* text, size_t& result_size)
{
    char*[] temporary = (char*[])malloc(strlen(text)*sizeof(void*));
    size_t res_size = 0;
    char* left = text;
    char* right;
    while(left){
        while(left && is_delimeter(*left)){
           left++;
        }
        right = left;
        while(right && !is_delimeter(*right)){
            right++;
        }
        if(*left){
             temporary[res_size] = (char*)malloc(rigth-left+1);
             memcpy(temporary[res_size], left, right - left);
             temporary[res_size][rigth-left] = 0;
             res_size++;
        }
        left = right;
    }
    char*[] result =  (char*[])malloc(res_size*sizeof(void*));
    memcpy(result, temporary, res_size);
    result_size = res_size;
    free(temporary);
    return result;
}

Никогда не писал на C, поэтому не уверен, что правильно, код не запускал, могут быть ошибки.

split возвращает массив строк, в аргументе result_size размер получившегося массива.

Если честно, то я практически ничего не понимаю, из того что тут написано((((((( Пока только осваиваю СИ

[New man]

Добавил комментарии.

Использовать надо так:

   size_t length;
   char*[] words = split(str, length);
   ...
   for(size_t i = 0; i< length; i++){
     free(words[i]);
   }
   free(words);

В words будут слова, length - длина массива words