строки

[Алексей Денисов]

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

реальный результат такой:
Адель
Адель
Адель
Адель
Адель

подскажите пожалуйста, в чем тут дело


#include <iostream>
#include <clocale>
#include <cstring>
#include <conio.h>

using namespace std;

int main()
{
setlocale(LC_ALL, "Russian_Russia.1251");

char *dict[] = { "Адам/K", "Адамович/K", "Адамовичем", "Адамовна/I", "Адель/H" };
char *form[10];
char *s;
char c[20];
int i, j, n, l;

n = sizeof(dict)/sizeof(dict[0]);

for(j = 0; j < n; j++) {

s = dict[j];
l = strlen(s);

for(i = 0; i < l; i++) {
if(s[i] == '/') break;
else c[i] = s[i];
}

c[i] = '\0';

*(form + j) = c;
}

for(i = 0; i < j; i++)
cout << *(form + i) << "\n";

_getch();
return 0;
}

[EUGY]

char c[20];
...
*(form + j) = c; // это ведь у тебя не копирование строки а присвоение указателя.
Адрес массива с[] постоянный и все присвоения этого указателя левому выражению *(form + j) будут ссылаться разумеется на этот адрес.
А в нем сохранена последняя в цикле строка.

[Алексей Денисов]

спасибо, я догадывался об этом )). только не могу сообразить, как правильно собрать в массив строк переделанные строки из массива *dict[]

[Алексей Денисов]

решил:
вместо *(form + j) = c;
сделал sscanf(c, "%s", form[j]);

в смысле это работает. только скажите, плз, это изврат? есть способ более канонический?

[EUGY]

...
    c[i] = '\0';
    //*(form + j) = c;
    *(form + j)=new char[i+1];
    strcpy(*(form + j), c);
} 

for(i = 0; i < j; i++)
{
    cout << *(form + i) << "\n";
    delete [] *(form + i) ;
}

[Алексей Денисов]

спасибо большое за внимание к моему тривиальному вопросу. скажите пожалуйста, а sscanf(c, "%s", form[j]); сильно не правильно?

[Алексей Денисов]

а на *(form + j)=new char[i+1]; компилятор ругается, что невозможно преобразовать "char *" в "char [20]"

[EUGY]

Оно было бы правильным, если аргумент form[j] был бы адресом буфера выделенной памяти.
char *form[10]; - объявлен массив указателей под будущие строки, но память под сами строки не выделена.

[EUGY]

Сообщение от Алексей Денисов

а на *(form + j)=new char[i+1]; компилятор ругается, что невозможно преобразовать "char *" в "char [20]"

char *form[10]; - у тебя так объявлено?

[Алексей Денисов]

не, прошу прощения, уже так: char form[10][20];