[РЕШЕНО] Функция в программе. Помогите пожалуйста найти и исправить ошибки

[Vyacheslav.]

Здравствуйте! Помогите пожалуйста найти и исправить ошибки.
Создать функцию определяющую среднее значение парных элементов матрицы и минимальное среди непарных элементов матрицы, и с ее помощью обработать три двумерных массива.

#include <iostream>// подключение заголовочного файла
#include <Windows.h>// подключение заголовочного файла
using namespace std;//подключение пространства имён
float B[100][100] = { 0 }, s, min, k, n, m, z; //обьявление массивов
int i, j;
float example() {
    int i, j;
    min = B[1][1]; //присваивание первого элемента масива как минимальное
    for (i = 0; i < ::n; i++) {  //начало цикла обработки данных
        for (j = 0; j < ::m; j++) {
            if (fmod(B[i][j], 2) == 0) { s = s + B[i][j]; k = k + 1; }  //условие с расчётом для парных элементов
            else {
                if (B[i][j] < min) { min = B[i][j]; }   //условие с расчётом для непарных элементов
            }
        }
    }return 0;
}
int main()// оглавление главной функции
{
    SetConsoleCP(1251);  //установка кодовой таблици для ввода и вывода кирилици в консоли
    SetConsoleOutputCP(1251);   
    for (z = 0; z < 3; z++) {
        cout << "vvedite kol-vo strok i stolbcov" << endl;  //ввод кол-ва строк и столбцов
        cin >> m >> n;
        k = 0; //обнуление кол-ва парных элементов
        s = 0; //обнуление суммы парных элементов
        for (i = 0; i < n; i++) {  //начало цикла ввода масива
            for (j = 0; j < m; j++) {
                cout << "vvedite element masiva [" << i + 1 << "][" << j + 1 << "]" << endl;
                cin >> B[i][j];
            }
        }
        example();
        cout << "sred. arefmet. parnih elementov =" << s / k << "\n" << "minimalnoe iz neparnih =" << min << endl;
    }
    system("pause");
}

[2Lui]

min = B[1][1]; //присваивание первого элемента масива как минимальное
    for (i = 0; i < ::n; i++) {  //начало цикла обработки данных
        for (j = 0; j < ::m; j++) {
            if (fmod(B[i][j], 2) == 0) { s = s + B[i][j]; k = k + 1; }  //условие с расчётом для парных элементов
            else {
                if (B[i][j] < min) { min = B[i][j]; }   //условие с расчётом для непарных элементов
            }
        }

Ну массив(матрица) начинается с
[0][0]->[0][1]->[0][2]->[0][...100]
[1][0]->[1][1]->[1][2]->[1][...100]
[...100][0]->[1][1]->[1][2]->[1][...100]

не используй глобальные переменные, массивы создавай в мейне и передавай в функцию,сделай еще одну функцию для заполнения массива рандом числами, можешь и в мейне сделать если функцию не можешь написать, а то в ручную каждый раз задолбаешся вводить, зачем ты создал функцию float и возвращаешь 0? сделай тогда void

[Vyacheslav.]

Спасибо. Уже сделал.

[Вадим Мошев]

Vyacheslav., Напишите, как вы это сделали. Это будет полезно другим читателям.

[jura_k]

Сообщение от 2Lui

не используй глобальные переменные, массивы создавай в мейне и передавай в функцию,

Не согласен. Глобальный массив удобнее. Придётся передавать функции ссылку на массив, если объявить его в 'main'.

[2Lui]

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

Не согласен. Глобальный массив удобнее. Придётся передавать функции ссылку на массив, если объявить его в 'main'.

Может быть удобнее в таких простых программах. А так нет.

[Julik2]

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

Не согласен. Глобальный массив удобнее. Придётся передавать функции ссылку на массив, если объявить его в 'main'.

Сообщение от 2Lui

Может быть удобнее в таких простых программах. А так нет.

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

[2Lui]

А зачем по значению передавать? указатель передай, где написано что массив не должен передаваться в функцию? Что хочешь то и передаешь,
мне интересно как пишут функции сортировки массива, или что то подобное без передачи массива)

[Julik2]

Сообщение от 2Lui

где написано что массив не должен передаваться в функцию? Что хочешь то и передаешь.

Программист - desireless сущность, хотелками не должен руководствоваться.

А массив передавать в функцию не стоит по причине отсутствия логики. Массив в программе является уникальным объектом, и следовательно, обработчик тоже будет уникальным, и выделять его в функцию нецелесообразно.

Функции для массивов с передачей параметров на языке C рациональны только для математических программ или учебных, просто для разделения последовательности операций - ввод, обработка, печать массива.

[2Lui]

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