Возврат массива
Здравствуйте довольно тривиальная задача но возникла одна проблема. Суть в чем есть две функции которые возвращают массивы символьного типа, одна функция переводи число с плавающей точкой вторая переводит число. вот коды этих функция
первая
функция возвращает все нормально
вторая функция
функция всегда возвращает массив с одним элементов значение которого равно 69, причем на этапе трассировки все расчеты идут нормально перед самим возвратом значение верные но когда происходит присвоение значения массиву присваивается символ с кодом 69 немогу понять в чем ошибка первая функция работает правильно, вторая не работает
первая
Код:
unsigned char* summtocode(double d)
{
unsigned char cod[5];
cod[0]=0;
cod[1]=0;
cod[2]=0;
cod[3]=0;
cod[4]=0;
if(d==0.0)
return cod;
...код сильно большой вкратце тут идут расчеты
return cod;
}
{
unsigned char cod[5];
cod[0]=0;
cod[1]=0;
cod[2]=0;
cod[3]=0;
cod[4]=0;
if(d==0.0)
return cod;
...код сильно большой вкратце тут идут расчеты
return cod;
}
вторая функция
Код:
unsigned char* chektocode(int i1)
{
unsigned char cod[2];
cod[0]=0;
cod[1]=0;
if(i1<=255)
{
cod[0]=i1;
}
else
{
z=i1/256;
cod[1]=z;
cod[0]=i1-(z*256);
}
return cod;
}
{
unsigned char cod[2];
cod[0]=0;
cod[1]=0;
if(i1<=255)
{
cod[0]=i1;
}
else
{
z=i1/256;
cod[1]=z;
cod[0]=i1-(z*256);
}
return cod;
}
Вашу задачу можно использовать в качестве примера как делать не надо. Причем классической - ибо настолько часто описанной, что оскома на зубах.
Ваш указатель указывает на переменную, время жизни которой уже давным давно закончилось.
Советую читать статьи в которых присутсвуют слова "область видимости переменных в С++".
Как вариант - использование ссылок.
Ваш указатель указывает на переменную, время жизни которой уже давным давно закончилось.
Советую читать статьи в которых присутсвуют слова "область видимости переменных в С++".
Как вариант - использование ссылок.
Спасибо за указание ошибки. :rolleyes: действительно сразу не обратил внимание
Создал указатель на тип char с общей областью видимости
функцию изменил таким образом:
перед возвратом через трассировку проверяю в массиве pointer правильные значения, но когда вернул его в нем по прежнему только один элемент с постоянным значением 69, пробовал после вызова функции обращаться не к массиву который вернул а напрямую к массиву pointer результат тот же
Код:
unsigned char * pointer;
функцию изменил таким образом:
Код:
unsigned char* chektocode(int i1)
{
unsigned char cod[2];
cod[0]=0;
cod[1]=0;
if(i1<=255)
{
cod[0]=i1;
}
else
{
z=i1/256;
cod[1]=z;
cod[0]=i1-(z*256);
}
pointer=cod;
return pointer;
}
{
unsigned char cod[2];
cod[0]=0;
cod[1]=0;
if(i1<=255)
{
cod[0]=i1;
}
else
{
z=i1/256;
cod[1]=z;
cod[0]=i1-(z*256);
}
pointer=cod;
return pointer;
}
перед возвратом через трассировку проверяю в массиве pointer правильные значения, но когда вернул его в нем по прежнему только один элемент с постоянным значением 69, пробовал после вызова функции обращаться не к массиву который вернул а напрямую к массиву pointer результат тот же
Вы наверное верите в симпатическую магию, иначе мне трудно понять - что все таки у вас изменилось. Правильный ответ: НИЧЕГО.
Ваш указатель по прежнему указывает на неопределенную область памяти. Это С/С++. Смиритесь и прочтите чтото для ньюбов :)
Ваш указатель по прежнему указывает на неопределенную область памяти. Это С/С++. Смиритесь и прочтите чтото для ньюбов :)
Для вас наиболее очевидным вариантом является передача в функцию указатель на массив, числового значения и границы массива (например).
Абсолютно согласен, С-шаман, С\С++ это не PHP, да и в PHP у вас наврядли такой фокус прокатит. Советую открыть Герберта Шилдта, справочник по С. Там на первых же страницах рассказывается как правильно работать с указателями.
Дам небольшую подсказку - *,& - вот про эти значки почитайте внимательно и разберитесь с ними, либо переходите на C#.
Дам небольшую подсказку - *,& - вот про эти значки почитайте внимательно и разберитесь с ними, либо переходите на C#.
Код:
unsigned char cod[5];
cod[0]=0;
cod[1]=0;
cod[2]=0;
cod[3]=0;
cod[4]=0;
cod[0]=0;
cod[1]=0;
cod[2]=0;
cod[3]=0;
cod[4]=0;
страшно подумать если массив будет, к примеру, 10к
Код:
if(d==0.0)
return cod;
...код сильно большой вкратце тут идут расчеты
return cod;
}
return cod;
...код сильно большой вкратце тут идут расчеты
return cod;
}
d == 0 - для float такое условие некорректно
Код:
if(i1<=255)
{
cod[0]=i1;
}
else
{
z=i1/256;
cod[1]=z;
cod[0]=i1-(z*256);
}
{
cod[0]=i1;
}
else
{
z=i1/256;
cod[1]=z;
cod[0]=i1-(z*256);
}
забавно...
а почему не 4 if, параметр то int ? ;)
а ведь можно без единого if...