Странная проблема со строками(подробности внутри)

[Ksardas13]

Сори, за странное название темы, другого придумать не смог(((

Есть такой вот код(убрал всё лишнее):

void logging(char *text) //процедура вывода на экран строки
{std::cout<<text;} //проста до неприличия

... //ниже текст программы
char *s;
s="12345678910";
logging("\n"); logging(s);logging("\n");

int i=5;
char s2[i];
for (int j=0;j<i;j++) {s2[j]=s[j];
logging("\n");logging(s2);logging(" \n");}
logging("\n\n"); logging(s2);logging("\n");
...//уровень детского сада для умственно отсталых -_-

Что получаем на экране:

12345678910

1Aжv?@

12жv?@

123v?@

1234?@

12345@


12345@

АААААА!!!! Откуда эти два лишних символа в конце строки?!?!?! О_О
Причём прикол ещё в том, что если написать вот так(т.е. просто вывод на экран сделать не отдельной процедурой, а сразу выводить... см.ниже):

...
char *s;
s="12345678910";
std::cout<<"\n";std::cout<<s;std::c out<<"\n";

int i=5;
char s2[i];
for (int j=0;j<i;j++) {s2[j]=s[j];
std::cout<<"\n\n"; std::cout<<s2;std::cout<<"\n";}
std::cout<<"\n\n"; std::cout<<s2;std::cout<<"\n";
...

Всё выводиться на экран нормально! Всё как надо!

Вопрос - что не так? Ошибка в передаче строки в процедуру вывода?

[netrino]

Нет, ошибка в том, что нет завершающего нуля в конце строки. Можете сделать так:

char s2[i + 1] = { 0 }; // Инициализация всего массива нулём

Должно нормально выводить. Только следует учитывать, что массивы переменной длины в C++ - это расширение компилятора и может не скомпилироваться на других компиляторах.

[onewho]

лучше int i=strlen(s)+1; или типа того

[Ksardas13]

netrino, на строчку "char s2[i + 1] = { 0 };" компилятор ругается(пишу в Dev-Cpp) ;( :

variable-sized object `s2' may not be initialized

Пока гуглю ответ.
*Ответ найден экспериментально - если написать "char s2[6] = { 0 };" всё пашет, если "char s2[i + 1] = { 0 };" нет. Я в печали. Что делать?
*Пока продолжаю эксперименты. Но всё таки что делать если бы было необходимо именно задать размер массива через i?

onewho, цель взять именно 5 символов(а не все) из строки S и создать из них строку S2.))

[Ksardas13]

Вообщем, задачу поставленную перед собой решил. Спасибо netrino.)
Мучаюсь с приложениями сервер-клиент. Там всёравно стоит огранчиение на перекидываемые между сервером и клиентом строки, так что строчка "char s2[4096 + 1] = { 0 };" спасла мои нервы и мозг.))

[netrino]

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

netrino, на строчку "char s2[i + 1] = { 0 };" компилятор ругается(пишу в Dev-Cpp) ;( :

Подозревал что так будет, но лень было проверять )
Заполнить массив нулями можно также пройдясь по нему в цикле:

for (int j = 0; j < i + 1; j++)
   s2[j] = 0;

Или воспользовавшись функцией memset

[profi]

int i=5;
char s2[i];

Размер массива должен быть константой. В Си такое еще можно (я про выше приведенный код), но в С++ нет.

[onewho]

так в конце кода надож было добавить s2[j]=0; вродь, и всё? (ну и размер строки разумеется 6)