В чем ошибка кода?

Question

[DufGut]

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

#include <iostream>
#include <math.h>
using namespace std;

int cubeI(int num) {
    if (num > 5) {
        cout << num << " ";
        return pow(num, 3);
    }
}

long cubeL(long num) {
    if (num > 5) {
        cout << num << " ";
        return pow(num, 3);
    }
}

int main() {
    setlocale(0, "");
    const int size = 10;
    int masI[size];
    long masL[size];

    cout << "Массив int: ";
    for (int i = 0; i < size; i++) {
        masI[i] = rand() % 10 + 1;
        masI[i] = cubeI(masI[i]);
    }

    cout << endl;

    cout << "Массив long: ";
    for (int i = 0; i < size; i++) {
        masL[i] = rand() % 10 + 1;
        masL[i] = cubeL(masL[i]);
    }

    cout << endl;
}

Comments

[res2001]

1. Функции cubeX не перегруженные
2. В Си нет перегрузки, такой же как в плюсах. Уберите Си из тегов (к тому же и код представлен плюсовый, а не сишный). В Си можно сделать аналог перегрузки, используя макрос _Generic. Но _Generic не поддерживается С++.
3. Ваши функции cudeX должны возвращать значение, но в случае не выполнения условия никакое значение не возвращается - это ошибка компиляции.

Answer 1

[Василий Дёмин]

int cubeI(int num) {
    if (num > 5) {
        cout << num << " ";
        return pow(num, 3);
    }
}

long cubeL(long num) {
    if (num > 5) {
        cout << num << " ";
        return pow(num, 3);
    }
}

Если элемент меньше или равен пяти, функции должны возвращать исходное значение. Нужно добавить return num после if, иначе будет возвращено то, что на момент выхода из функции было в регистре rax (если у вас amd64 архитектура).

Comments

[DufGut]

То есть добавить ?

else {
        cout << num << " ";
        return num;
    }

[Василий Дёмин]

Ну и, кстати, функции у вас не перегруженные получаются. Перегруженная функция должна так выглядеть:

int cube(int num) {
    // Ваш код для int
}

long cube(long num) {
    // Ваш код для long
}

Тогда оба цикла будут иметь вид:

for (int i = 0; i < size; i++) {
    masI[i] = rand() % 10 + 1;
    masI[i] = cube(masI[i]);
}

for (int i = 0; i < size; i++) {
    masL[i] = rand() % 10 + 1;
    masL[i] = cube(masL[i]);
}

[Василий Дёмин]

DufGut, else блок вам не нужен, так как в основной ветке делается выход из функции. Можно просто после if разместить.

[DufGut]

#include <iostream>
#include <math.h>
using namespace std;

int cube(int num) {
    if (num > 5) {
        cout << num << " ";
        return pow(num, 3);
    }
    return num;
}

long cube(long num) {
    if (num > 5) {
        cout << num << " ";
        return pow(num, 3);
    }
    return num;
}

int main() {
    setlocale(0, "");
    const int size = 10;
    int masI[size];
    long masL[size];

    cout << "Массив int: ";
    for (int i = 0; i < size; i++) {
        masI[i] = rand() % 10 + 1;
        masI[i] = cube(masI[i]);
    }

    cout << endl;

    cout << "Массив long: ";
    for (int i = 0; i < size; i++) {
        masL[i] = rand() % 10 + 1;
        masL[i] = cube(masL[i]);
    }

    cout << endl;
}

[Василий Дёмин]

DufGut, Должно работать. Только вы результирующие значения не выводите на экран

[DufGut]

Василий Дёмин,
Вроде бы так, но он выводит 5-6 числа, а не 10

#include <iostream>
#include <math.h>
using namespace std;

int cube(int num) {
    if (num > 5) {
        cout << num << " ";
        return pow(num, 3);
    }
    return num;
}

long cube(long num) {
    if (num > 5) {
        cout << num << " ";
        return pow(num, 3);
    }
    return num;
}

int main() {
    setlocale(0, "");
    const int size = 10;
    int masI[size];
    long masL[size];

    cout << "Массив int: ";
    for (int i = 0; i < size; i++) {
        masI[i] = rand() % 10 + 1;
        masI[i] = cube(masI[i]);
        cout << masI[i];
    }

    cout << endl;

    cout << "Массив long: ";
    for (int i = 0; i < size; i++) {
        masL[i] = rand() % 10 + 1;
        masL[i] = cube(masL[i]);
        cout << masL[i];
    }
    cout << endl;
}

[Василий Дёмин]

DufGut, Вам надо выводить все числа или только возведённые в куб? Если все, то надо убрать cout из функции cube, а если только возведённые, то из из main.

Answer 2

[Alex1144]

В чем ошибка? Во всем.
Во-первых вы создали функцию int cubeI(int num) для передачи переменной num типа int, и хотите в неё засунуть указатель массива
(masL[i]). Это так не работает.
Второе, я не специалист, но в литературе по С++ не виднел, что бы элементы массива передавали в функцию. Передают весь массив и там производят все нужные манипуляции.
Третье ваш rand() не будет генерировать рандомные числа, а будет выдавать одно и тоже число. Загуглите, там ничего сложного.
Для примера код как передать массив в функцию и возвратить в main:

#include
using namespace std;

int* cubeI(int *mas, int size) {
for (int i = 0; i < size; i++)
if (mas[i] > 5) {
mas[i]= 10;
}
return mas;
}
int main() {
setlocale(0, "");
const int size = 10;
int mas[size];

cout << "Массив int: ";
for (int i = 0; i < size; i++) {
mas[i] = i+3;
}
int *ptr=NULL;
ptr=cubeI(mas,size);
for (int i = 0; i < size; i++)
cout<<*(ptr+i)<<" ";
cout << endl;


}

Comments

[Василий Дёмин]

masL[i]
Это не указатель, это значение, взятое из массива по индексу i. Всё правильно.

по С++ не виднел, что бы элементы массива передавали в функцию.

То, что вы не видели, не значит, что так не делают. От ситуации зависит. Если есть функция, возводящая элемент в куб, и вам надо возвести некоторые элементы массива в куб, то в чём проблема передать нужный элемент в функцию?

Третье ваш rand() не будет генерировать рандомные числа, а будет выдавать одно и тоже число. Загуглите, там ничего сложного.

Каждый вызов rand() будет возвращать псевдослучайное число, просто на каждом запуске программы числа будут повторяться. https://en.cppreference.com/w/cpp/numeric/random/rand

[Василий Дёмин]

int* cubeI(int *mas, int size) {
Тут вы вообще дичь какую-то делаете. Зачем возвращать из функции указатель на переданный массив? Может сложиться впечатление, что функция возвращает указатель на новый массив, а не на старый. Если уже совсем по уму делать, то вместо передачи указателя и размера можно использовать std::array