Функция...

[frem-dag]

Здравствуйте, уважаемые форумчане. Тут проблема возникла с функцией... Я конечно понимаю, что за задача наверно из разряда элементарных, но всё же, не могу додуматься...
Массив 3x7x2. Функция вывода на экран элемента, если он не выходит за пределы массива.
Здесь на сколько я понимаю, основное в этой функции будет сначало проверка, а потом вывод...
Сначало объявляем функцию, например int output();
Потом в main заполняем массив.
И дальше сама функцию...

int output()   
{
    cin>>i;
    cin>>j;
    cin>>k;
    if(a[i][j][k])
      return cout<<a[i][j][k];
}

Может не совсем логично я тут делаю... Подскажите пожалуйста!

[Darth.Vader]

Во-первых, если ты делаешь функцию вывода, то зачем в ней делать и ввод? Ну, это идеологическая ошибка
А по поводу проверок:
Если ты делаешь так, как написал, то ты на истинность проверяешь элемент, с индексами i,j, и k. И, даже если ты выйдешь за пределы массива, там может оказаться ненулевой мусор... а вот внутри массива может быть и ноль, который не выведется....
Надо проверять каждый индекс по отдельности.

int output([какой-то тип] a[], int i, int j, int k)
{
    if(i>=IMAX){return -1;}
    if(j>=JMAX){return -1;}
    if(k>=KMAX){return -1;}
    cout << a[i][j][k];
    return 0;
}

Не тестил, из головы написал

[frem-dag]

Да, действительно, ввод в функции вывода это глупо, не подумал
Вот только ещё вопрос по поводу возвращаемого значения... Почему в условиях возвращается -1? И разве не нужно заносить Kmax, Imax, Jmax в параметры функции? Или если мы их в main'е объявляем, то в этой функции уже не нужно?

[RoS]

Сообщение от frem-dag

Да, действительно, ввод в функции вывода это глупо, не подумал
Вот только ещё вопрос по поводу возвращаемого значения... Почему в условиях возвращается -1? И разве не нужно заносить Kmax, Imax, Jmax в параметры функции? Или если мы их в main'е объявляем, то в этой функции уже не нужно?

-1 классически считается ошибочным завершением
Не ну ты конечно можешь окружить блок try - catch и выкидывать исключения, но это уж слишком для простенькой программки
Так что минус один - в самый раз! т.е. по завершению работы функции ты проверяешь - если она вернула -1 , то произошла ошибка, возможно вызов надо повторить с другими параметрами

Kmax, Imax, Jmax - скорее всего подразумевается, что это глобальные константы...
Если ты их объявишь в мейне - функция их не увидит

[frem-dag]

Т.е. если я правильно понял, описание этих глобальных переменных потребуется в списке аргументов функции вывода?))

int output(const int Imax, const int Jmax, const int Kmax, int i, int j, int k)

[ROD]

Т.е. если я правильно понял, описание этих глобальных переменных потребуется в списке аргументов функции вывода?))

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


Что бы переменная была глобальной, ее надо объявить в самом верху программы, до всех функций.

[Lunex.08]

Сообщение от frem-dag

Т.е. если я правильно понял, описание этих глобальных переменных потребуется в списке аргументов функции вывода?))

int output(const int Imax, const int Jmax, const int Kmax, int i, int j, int k)

Нет,глобальные функции объявляются вне какой-либо функции.Но это как я считаю нежелательно использовать,т.к.нарушается пространство имен.Вот пример глобальной функции:

#include <>
int global; //Глобальная переменная.
int main ()
{
return 0;
}

Эх,припоздал немного))

[RoS]

Сообщение от frem-dag

Т.е. если я правильно понял, описание этих глобальных переменных потребуется в списке аргументов функции вывода?))

int output(const int Imax, const int Jmax, const int Kmax, int i, int j, int k)

Если ты так сделаешь, их тоже придется вводить, притом каждый раз,
Если не хочешь просто ВНЕ мейна и любой другой функции написать эти константы, т осделай их параметрами по умолчанию

int output(int i, int j, int k, int Imax=7, int Jmax=7, int Kmax=2 ) //или какие там у тебя цифры должны быть

тогда вызов может быть обычным, а размерности будут видны функции (естественно, это же ее параметры )

Вызов :

output(3,2,4); //например

[frem-dag]

Ясно, спасибо большое, с этим разобрался
А если оформить это не в виде отдельной функции, тогда код примерно таким должен быть?

#include <iostream.h>

using namespace std;

int main()
{
    const Imax=3, Jmax=7, Kmax=2;
    int i,j,k;
    int a[Imax][Jmax][Kmax];
    for(i=0;i<Imax;i++) //вводим трёхмерный массив
      for(j=0;j<Jmax;j++)
        for(k=0;k<Kmax;k++)
          cin>>a[i][j];
    cin>>i; //пользователь вводит координаты нужного элемента
    cin>>j;
    cin>>k;
    if(i>=Imax) //проверка принадлежности к нужному массиву
      return -1;
    if(j>=Jmax)
      return -1;
    if(k>=Kmax)
      return -1;
    cout << a[i][j][k];
    return 0;
}

[Darth.Vader]

А не легче не мудрить с глобальными \локальными переменными, а сделать всё, к примеру, так:

#define IMAX 5
#define JMAX 5
#define KMAX 5