Не могу вернуть указатель из функции

[giran]

задача- в функции есть массив (7 ячеек от DS1307). Хочу вернуть указатель из этой функции в основную программу (вроде как объявлять глобальные переменные- дурной тон).
Вернее указатель-то возвращается, и первый (в смысле нулевой)элемент массива выводится корректно, а вот начиная со второго выводится мусор. Что не так делаю????
Запускал на Ардуино, на Dev-C++ - один и тот же результат, то есть ошибка не в компиляторе. В примере (для Dev-C++) я поубирал всё лишнее, только массивы-указатели-функция. Где грабли, подскажите, плиз.

#include ‹iostream›
#include ‹cctype›
using namespace std;
    int *readDateTime();
int main()
{
    int *DatTim;
    DatTim= readDateTime();
for (int j=0; j‹8; j++)
{   cout ‹‹ *DatTim ‹‹ " ";
    DatTim++;
}
         system("pause"); 
    return 0;
}

int *readDateTime() {
int DateTime[7];

for (int i=0; i‹8; i++)
 {   DateTime[i]=i+10;
    cout ‹‹ *(DateTime+i) ‹‹' ';
    }
    cout‹‹ "\n";
            return DateTime;    
            } 

[akegor]

Где грабли, подскажите, плиз.

Компилятору, видимо, надо знать, сколько места от указателя нельзя использовать по своему усмотрению. Пока это один "инт", что Вы и видите.

[realid]

Потому что массив в функции существует до тех пор пока выполняется функция. Его надо объявить static или сделать глобальным.

[pambaru]

У вас возвращается указатель на локальную переменную внутри функции. А эта переменная размещается во время вызова функции в стеке данных.
После выхода из функции данное место в стеке будет использоваться под другие цели (другими функциями). Т.е. сохранность данных по данному адресу после выхода из функции не гарантируется совсем. Т.е. Ваш указатель по сути указывает на уже несуществующую DateTime, отсюда и мусор.
По хорошему, передавайте данной функции указатель в параметрах, функция должна его только заполнить нужными данными. Ну и указатель должен быть на "валидную" область памяти - т.е., либо специально динамически выделенную из кучи, либо на статический массив.

[giran]

Я взял за основу пример Г.Шилдта, где он без всяких заморочек передаёт из функции указатель и получает оставшуюся часть подстроки -

#include ‹iostream›
#include ‹cctype›
using namespace std;
               char *get_substr(char *sub, char *str);
int main()
{
    char *substr;
    substr = get_substr("three", "one two three four");
    cout ‹‹ "Substring ---›" ‹‹ substr ‹‹ "\n";
         system("pause"); 
    return 0;
}

char *get_substr(char *sub, char *str) {
int t;
char *p, *p2, *start;
for (t=0; str[t]; t++) {
  p= &str[t];
  start = p;
  p2= sub;
  while (*p2 && *p2==*p) {
        p++;
        p2++;
        }
  if(!*p2)
          return start;
}
return 0;
} 

и пример замечательно компилирутся и выдаёт что надо без всяких типов static и резервирования места. У меня примерно тоже самое, только работаю не со строкой а с массивом, а результата нет.
ЗЫ. А объявление static помогло, спасибо.

[Boba_spb]

int DateTime[7];

for (int i=0; i‹8; i++)
{ DateTime[i]=i+10;

Вас не смущет, что в массив из 7 элементов засаживаете 8 элементов?
Посмотрите в отладчике что там портится а стеке за массивом

[pambaru]

Я взял за основу пример Г.Шилдта, где он без всяких заморочек передаёт из функции указатель

У него этот указатель получается из внешнего, передаваемого в функцию, т.е. указывает на "внешний", реально существующий адрес (внутри строки, адрес которой передается в функцию). У Вас же - указатель указывает на адрес локальной переменной данной функции, которая перестает существовать по выходу из функции.

[akegor]

А объявление static помогло,

как наждачная бумага отучила кота задницу об ковер вытирать.
Очень ли корректно залазить внутрь функции извне? Чем это отличается от глобальной переменной? Только извращенным обращением/хранением?
Не корректней ли глобальный массив для подобных переменных?

[Yurkin2014]

Если много подобных функций, то можно использовать одни и те же имена для объявления локальных переменных. Например, Data [] в каждой функции, и конец мучениям в придумывании названий переменных. Области видимости у них не будут пересекаться.

[eddy]

как наждачная бумага отучила кота задницу об ковер вытирать.

C++ тоже как-то с микроконтроллерами ну никак не вяжется!