Изменяю значение по ссылке, в чем моя ошибка?

Question

[Vadim kyklaed]

Добрый день. передаю значение по ссылке в функцию и меняю это значение в функции . после мне надо это значение использовать после отработки функции. в чем у меня ошибка ? я получаю значение N ==0 , следовательно цикл for не отрабатывает.

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

using namespace std;

void conct_str(char str[],int& N){
	char c;
	char b;
	cin.get(c);
	cin.get(b);
	while (c != '\n'){
		str[N]=c;
		N++;
		cin.get(c);
		}
	while (b != '\n'){
		str[N]=b;
		N++;
		cin.get(b);
		}	
		
	str[N]='\0';
	
	}
	

int main(){
	
	int N=0;
	char str[N];
	conct_str(str, N);
	for (int i=0;i<N;i++){
		cout<<str[i];
	}
	return 0;
}

Comments

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

Vadim kyklaed , а давай попробуем словами рассказать, что этот код должен делать?
Скажу наперед, то что он должен - он делать не будет, что бы он делать не должен был. Поэтому, в самом коде невозможно увидеть то, что код должен делать.

Какой процесс ты ожидаешь от кода?

[Станислав Макаров]

Есть ли уверенность в том что один из циклов в conct_str отрабатывает и что N там инкрементируется?

[Vadim kyklaed]

Евгений Шатунов, сложение двух строк, да мне уже ответили. размер массива должен быть константой

Answer 1

[res2001]

К вашему вопросу отношения не имеет, но:
В main вы выделяете память для массива из 0 элементов.
int N=0;
char str[N];
Как думаете сколько памяти выделится под этот массив?
Еще вы, видимо, считаете, что инкрементируя N вы увеличиваете размер массива? Это не так - увеличивая N вы никак не влияете на str.

PS: создается впечатление, что VLA для новичков - это зло.

Comments

[Vadim kyklaed]

Да спасибо. я так и думал. размер массива должен быть константой

[Vadim kyklaed]

что такое VLA ?

[res2001]

Vadim kyklaed, Нет, в современных стандартах размер массива может не быть константой. VLA - Variable Size Array. Возможность введенная в языки С/С++ относительно не давно.
Это как раз то что вы используете.
И хоть компилятор и проглатывает значение N=0 при определении массива, но при этом память не выделяется. У этого варианта определения массива есть свои нюансы и не всегда его нужно применять.

[Vadim kyklaed]

res2001, так все же верно в моем случае константу использовать ?, я использую компилятор года 2007 выпуска с ide dev C++ может быть он не знает о новом стандарте ?

[res2001]

Vadim kyklaed, Раз при компиляции нет ошибок, значит знает. На MSDN нашел, что в 2008 студии это есть.
Но суть то не в том знает или нет. Перед тем как выделять массив вы должны знать сколько элементов в нем будет. Это без относительно того какой у вас массив VLA/статический/динамический.
Если по условию задачи вы этого не знаете (как у вас), то обычно выделяют сразу какой-то небольшой массив фиксированного размера, если элементов оказывается больше, то выделяют массив большего размера, переносят данные из первого во второй, первый освобождают.
На самом деле, если уж вы пользуетесь С++, то есть смысл использовать и классы стандартной библиотеки для массивов, там все это уже реализовано и спрятано внутрь классов.
Но для лучшего понимания массивов и работы памяти в С/С++ реализуйте это один раз вручную, когда разберетесь можно будет использовать stdlib.