C. Ошибка при компиляции из-за строки с массивом, не отличающимся от других.

[CofeIn]

Тут проблема заключается в первой строке функции RazbrasyvaemMassiv(), т.к. когда я закомменчиваю её, всё работает. Но я никак не пойму, что с ней не так: обычное объявление массива, ничем не отличается от массива ar[] в этой же программе. Мало того, когда я переношу эту же строку в главную функцию, она работает! Вот уж этого я точно не понимаю. Вывожу соутом size - это тоже работает. У кого есть идеи?

Код:

#include <iostream>
#include <stdlib.h>
#include <time.h>
using namespace std;

void RazbrasyvaemMassiv (int ar[], long size)
{
	int arnum[size];  //Когда я закомменчиваю эту строку, всё тоже работает, но я не вижу в ней ошибку. 
//Массив, как массив, от ar[] ничем не отличается. Мало того, когда я пишу _ту_же_ строчку в главной функции, никаких ошибок нет.
	/*ar[3]=10; //Для теста - эта строка работает*/

}

void main(){
	srand(time(NULL));
	const long size=10;
	int ar[size];
	
for (int i=0; i<size; i++)
{
ar[i]=i+1;
cout<<ar[i]<<"\t";
}
	cout<<"\n\n";

RazbrasyvaemMassiv (ar, size);
for (int i=0; i<size; i++)
cout<<ar[i]<<"\t";
}

Что пишется при компиляции:

1>------ Rebuild All started: Project: home 10 3, Configuration: Debug Win32 ------
1>Deleting intermediate and output files for project 'home 10 3', configuration 'Debug|Win32'
1>Compiling...
1>home 10 3.cpp
1>c:\documents and settings\админ\мои документы\visual studio 2008\projects\home 10 3\home 10 3.cpp(8) : error C2057: expected constant expression
1>c:\documents and settings\админ\мои документы\visual studio 2008\projects\home 10 3\home 10 3.cpp(8) : error C2466: cannot allocate an array of constant size 0
1>c:\documents and settings\админ\мои документы\visual studio 2008\projects\home 10 3\home 10 3.cpp(8) : error C2133: 'arnum' : unknown size
1>c:\documents and settings\админ\мои документы\visual studio 2008\projects\home 10 3\home 10 3.cpp(14) : warning C4244: 'argument' : conversion from 'time_t' to 'unsigned int', possible loss of data
1>Build log was saved at "file://c:\Documents and Settings\Админ\Мои документы\Visual Studio 2008\Projects\home 10 3\Debug\BuildLog.htm"
1>home 10 3 - 3 error(s), 1 warning(s)
========== Rebuild All: 0 succeeded, 1 failed, 0 skipped ==========

[Obey-Kun]

Проблема в том, что твой компилятор не соответствует стандарту и не поддерживает variable length array -- то есть массивы, размер которых не является констаной. Можешь заменить long size на const long size в объявлении функции или же сменить компилятор на более адекватный.

void main()

должно быть int main(), исправь

[Stilet]

Да ладно пугать то... Не все так плохо.
Чуть-чуть поправить типа такого:

int* RazbrasyvaemMassiv (long size)
{
	int *arnum = new int[size];  
	arnum[3]=10; 
	return arnum;
}

и

	srand(time(NULL));
	const long size=10;
	int *ar=RazbrasyvaemMassiv (size);
	for (int i=0; i<size; i++)	cout<<ar[i]<<"\t";
	delete[] ar;
	cin.get();
	return 0;

[CofeIn]

Obey-Kun, да, ты прав, дело в константности сайза. Я, кстати, только что придумала довольно простое решение этой проблемулины: сделать эту переменную глобальной. До этого мне почему-то показалось, что "константность" переменной сохраняется при передаче из функции в функцию.

Stilet, по твоему коду пара вопросов появилась (просто я ещё не дошла до некоторых вещей, которые ты использовал):
- что значит '*' возле int и ar?
- new int[size] - это, похоже, извращённый способ создания массива?
- delete[] ar; - из названия вроде ясно, но всё же подстрахуюсь..
- cin.get(); - Почему после cin стоит '.', а не '<<'?
Если можно, по одному предложению, желательно на русском, а не очень уж на компьюьтерном))

[p51x]

Читать про указатели и динамические массивы - вопросы отпадут сами собо.

[Stilet]

что значит '*' возле int и ar?

Это значит что переменка будет указателем на... блок памяти

new int[size] - это, похоже, извращённый способ создания массива?

Ниразу. Самый стандартный. в С++ конечно

delete[] ar; - из названия вроде ясно, но всё же подстрахуюсь..

Освобождение всего массива

cin.get(); - Почему после cin стоит '.', а не '<<'?

Потому что я не собираюсь в переменку вводить, а просто хочу вызвать метод get() обьекта cin для задержания консоли с результатом и ожидания нажатия любой клавиши.

[onewho]

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

Obey-Kun, да, ты прав, дело в константности сайза. Я, кстати, только что придумала довольно простое решение этой проблемулины: сделать эту переменную глобальной.

просто, но очень плохо, лучше передайте как const

upd// возможно так тоже не будет работать, тогда только динамический массив вам в помощь, вообще оно и понятно, ведь размер статического массива должен быть известен при компиляции

[CofeIn]

Stilet, всё ясно, а '*' в "int* RazbrasyvaemMassiv" - это потому, что мы возвращаем переменную-указатель?

Кстати, тогда ещё вопрос: я попыталась вызвать из данной функции ещё одну функцию, и в ней создать массив, и при передаче у меня ничего не получилось.
Потом я попробовала сделать всё так, как написали вы, и всё заработало. Коды есть во вложении, там и ошибочный вариант, и рабочий. Но у меня вопрос: почему надо делать указатели, почему нельзя или не получилось просто передать массив из одной функции в другую?

onewho, почему плохо?

[onewho]

не то, чтобы глобальные данные плохи, но использовать их следует там - где это действительно необходимо, в вашем случае это излишне

[Stilet]

это потому, что мы возвращаем переменную-указатель?

Ну если не буквоедствовать то да

почему надо делать указатели, почему нельзя или не получилось просто передать массив из одной функции в другую?

Наверное потому что в Си работа с массивами идет только по указателям. Да и вообще если честно во многих языках работа с массивами только по указателю. Просто другие языки вуалируют эту работу, так как программисту удобнее, не заставляют его думать за распределение памяти. (Это Мое ИМХО...)