Перегрузка функции

[Alexandr-]

Определить функцию, находящую максимальный элемент массива. Выполнить перегрузку функции для следующих типов параметров:
a. Одномерный массив типа int размерностью N.
b. Одномерный массив типа float размерностью N.
c. Одномерный массив типа double размерностью N.

В общем функция должна быть 1 имени в 3 случаях и выполнять все 3 пункта, но со 2 пункта начинаются проблемы. Помогите правильно составить функцию для 2 пункта (код ниже как пытался).

#include <iostream>
#include <math.h>
#include <time.h>
using namespace std;
int massiv (int N, int max, int A[100]) {
	int i;
	for (i=0;i<=N;i++) {
		if (A[i]>max) max=A[i]; } cout<<"\nmax int "<<max; return max; }

float massiv (float N, float max1, float A[100]) {
	int i;
	for (i=0;i<=N;i++) {
		if (A[i]>max1) max1=A[i]; } cout<<"\nmax float "<<max1; return max1; }

 main () {
	int N,A[100],i,max; float max1; srand(time(NULL));
	cout<<"vvedite razmer massiva\n";
	cin>>N;
	for (i=0;i<=N;i++) {
		A[i]=rand()%N; 
		cout<<A[i]; }

massiv(N,max,A);
massiv(N,max1,A);
system("pause"); return 0; }

[BDA]

#include <iostream>
#include <math.h>
#include <time.h>

using namespace std;

int
max(int A[], int N)
{
    int m = A[0];
    for (int i = 1; i < N; ++i)
        if (A[i] > m)
            m = A[i];
    return m;
}

float
max(float A[], int N)
{
    float m = A[0];
    for (int i = 1; i < N; ++i)
        if (A[i] > m)
            m = A[i];
    return m;
}

double
max(double A[], int N)
{
    double m = A[0];
    for (int i = 1; i < N; ++i)
        if (A[i] > m)
            m = A[i];
    return m;
}

int
main ()
{
    int N;
    srand(time(NULL));
    cin >> N;
    int A[N];
    float B[N];
    double C[N];
    for (int i = 0; i < N; ++i) {
        A[i] = rand() % 101 - 50;
        cout << A[i] << ' '; 
    }
    cout << endl << "max: " << max(A, N) << endl;
    for (int i = 0; i < N; ++i) {
        B[i] = rand() % 101 - 50;
        cout << B[i] << ' '; 
    }
    cout << endl << "max: " << max(B, N) << endl;
    for (int i = 0; i < N; ++i) {
        C[i] = rand() % 101 - 50;
        cout << C[i] << ' '; 
    }
    cout << endl << "max: " << max(C, N) << endl;
    system("pause");
}

[8Observer8]

Alexandr-, у Вас куча ошибок. Вот несколько:

Зачем Вы передаёте в функцию int max?

Массив вы задали в 100 элементов. А если пользователь захочет больше и введёт 200?

Замените Ваши функции по следующему образцу:

int maxElement(int a[], int n) {
    int max = a[0];

    for (int i = 0; i < n; i++) {
        if (a[i] > max) {
            max = a[i];
        }
    }
    
    return max;
}

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

В этом примере программа запрашивает у пользователя размер массива, заполняет массив случайными числами и выводит массив на экран:

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;

void fillArray(int arr[], int n) {
    srand (time(NULL));
    for (int i = 0; i < n; i++) {
        arr[i] = rand() % 10;
    }
}

int main(int argc, char** argv) {
    int n;
    int *array;    
    
    cout << "Enter the size of the array: ";
    cin >> n;
    array = new int[n];

    fillArray(array, n);
    
    // ...
    
    for (int i = 0; i < n; ++i)
        cout << array[i] << ' ';

    cout << endl;

    delete[] array;
    return 0;
}

[8Observer8]

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

    cin >> N;
    int A[N];
    float B[N];
    double C[N];

Ого! Я и не думал, что размер статического массива можно в время исполнения задавать. Я думал, что он во время компиляции должен быть задан константой. Попробовал - работает. Только stdlib.h нужно добавить для srand, rand и system.

[BDA]

Сообщение от 8Observer8

Попробовал - работает.

Работает-то, работает, но насколько допустим стандартом такой код, не могу сказать
http://stackoverflow.com/questions/1...th-arrays-in-c

UPD
Пожалуйста

[8Observer8]

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

Работает-то, работает, но насколько допустим стандартом такой код, не могу сказать
http://stackoverflow.com/questions/1...th-arrays-in-c

Спасибо за ссылку и инфу

[Alexandr-]

Как работает перегрузка понял спасибо. Только 1 момент не совсем понятен, про который выше писали.

 int A[N];
float B[N];
double C[N];

Если сделать например так:

 int A[]={2,3,6,5,8,6,5};
    float B[]={1.2,8.5,3.4,5.5,6.4,3.1,4.4};
    double C[]={6.6,-4.5,-3.2,9.8,6.5,3.7,3.8};

То все отлично будет работать(если не считать, что в задании надо вводить размерность), но с 1 вариантом выдаются ошибки(хотя с виду все логично):
error C2057: expected constant expression
error C2466: cannot allocate an array of constant size 0
error C2133: 'A' : unknown size

Из-за этого не получается создать 3 массива с разным типом переменных...

[8Observer8]

Судя по представленным ошибкам, не все компиляторы могут такое проделывать:

    cin >> N;
    int A[N];
    float B[N];
    double C[N];

Вот так нужно:

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;

void fillArray(int arr[], int n) {
    srand (time(NULL));
    for (int i = 0; i < n; i++) {
        arr[i] = rand() % 10;
    }
}

int main(int argc, char** argv) {
    int n;
    int *array;    
    
    cout << "Enter the size of the array: ";
    cin >> n;
    array = new int[n];

    fillArray(array, n);
    
    // ...
    
    for (int i = 0; i < n; ++i)
        cout << array[i] << ' ';

    cout << endl;

    delete[] array;
    return 0;
}

[Alexandr-]

Ну так это пример, как вывести 1 массив целых чисел. У меня проблема вывести другие 2 с другим типом данных.(или я неправильно понимаю что-то)

[8Observer8]

Просто добавьте ещё массив double. Всё по тому же принципу, что и выше для int.

Будет две функции перед main(). Одна будет принимать массив целых чисел, другая - массив вещественных чисел.

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;

void fillArray(int arr[], int n) {
    srand (time(NULL));
    for (int i = 0; i < n; i++) {
        arr[i] = rand() % 10;
    }
}

void fillArray(double arr[], int n) {
    srand (time(NULL));
    for (int i = 0; i < n; i++) {
        arr[i] = rand() % 10;
    }
}