Skip to main content
correccion de codigo
Origen Enlace
Min
  • 121
  • 4

El programa no debería ni si quiera compilarte, ya que estás intentando crear un array dinámico "int digitos[z]". Esa expresión no tiene sentido en C/C++ si z es una variable (no const). Tienes que reservar memoria dinámica para ese array (como se ha dicho en otras respuestas) de la siguiente manera:

int* digitos = new int[z];

Acuerdate luego de liberar la memoria:

delete[] digitos;

Otra manera un poco más elegante de hacer esto sería usando un contenedor como vectorvector<int> que crece dinámicamente según lo vayas necesitando, pero eso es otra historia.

También he notado que te falta inicializar la variable suma a 0 dentro de la función promedio(). El codigo final te quedaría así:

#include <iostream>
#include <stdlib.h>

using namespace std;

int z = 0;

int promedio(int digitos[])
{
    int suma = 0;
    for (int i = 0; i < z; i++)
        suma = suma + digitos[i];

    return suma / z;
}

int main()
{
    cout << "Introduce la cantidad de numeros a recibir" << endl;
    cin >> z;
    int* digitos = new int[z];
    for (int i = 0; i < z; i++)
        cin >> digitos[i];

    cout << "Promedio:" << promedio(digitos) << endl;

    delete[] digitos;
}

El programa no debería ni si quiera compilarte, ya que estás intentando crear un array dinámico "int digitos[z]". Esa expresión no tiene sentido en C/C++ si z es una variable (no const). Tienes que reservar memoria dinámica para ese array (como se ha dicho en otras respuestas) de la siguiente manera:

int* digitos = new int[z];

Acuerdate luego de liberar la memoria:

delete[] digitos;

Otra manera un poco más elegante de hacer esto sería usando un contenedor como vector que crece dinámicamente según lo vayas necesitando, pero eso es otra historia.

También he notado que te falta inicializar la variable suma a 0 dentro de la función promedio(). El codigo final te quedaría así:

#include <iostream>
#include <stdlib.h>

using namespace std;

int z = 0;

int promedio(int digitos[])
{
    int suma = 0;
    for (int i = 0; i < z; i++)
        suma = suma + digitos[i];

    return suma / z;
}

int main()
{
    cout << "Introduce la cantidad de numeros a recibir" << endl;
    cin >> z;
    int* digitos = new int[z];
    for (int i = 0; i < z; i++)
        cin >> digitos[i];

    cout << "Promedio:" << promedio(digitos) << endl;

    delete[] digitos;
}

El programa no debería ni si quiera compilarte, ya que estás intentando crear un array dinámico "int digitos[z]". Esa expresión no tiene sentido en C/C++ si z es una variable (no const). Tienes que reservar memoria dinámica para ese array (como se ha dicho en otras respuestas) de la siguiente manera:

int* digitos = new int[z];

Acuerdate luego de liberar la memoria:

delete[] digitos;

Otra manera un poco más elegante de hacer esto sería usando un contenedor como vector<int> que crece dinámicamente según lo vayas necesitando, pero eso es otra historia.

También he notado que te falta inicializar la variable suma a 0 dentro de la función promedio(). El codigo final te quedaría así:

#include <iostream>
#include <stdlib.h>

using namespace std;

int z = 0;

int promedio(int digitos[])
{
    int suma = 0;
    for (int i = 0; i < z; i++)
        suma = suma + digitos[i];

    return suma / z;
}

int main()
{
    cout << "Introduce la cantidad de numeros a recibir" << endl;
    cin >> z;
    int* digitos = new int[z];
    for (int i = 0; i < z; i++)
        cin >> digitos[i];

    cout << "Promedio:" << promedio(digitos) << endl;

    delete[] digitos;
}
Origen Enlace
Min
  • 121
  • 4

El programa no debería ni si quiera compilarte, ya que estás intentando crear un array dinámico "int digitos[z]". Esa expresión no tiene sentido en C/C++ si z es una variable (no const). Tienes que reservar memoria dinámica para ese array (como se ha dicho en otras respuestas) de la siguiente manera:

int* digitos = new int[z];

Acuerdate luego de liberar la memoria:

delete[] digitos;

Otra manera un poco más elegante de hacer esto sería usando un contenedor como vector que crece dinámicamente según lo vayas necesitando, pero eso es otra historia.

También he notado que te falta inicializar la variable suma a 0 dentro de la función promedio(). El codigo final te quedaría así:

#include <iostream>
#include <stdlib.h>

using namespace std;

int z = 0;

int promedio(int digitos[])
{
    int suma = 0;
    for (int i = 0; i < z; i++)
        suma = suma + digitos[i];

    return suma / z;
}

int main()
{
    cout << "Introduce la cantidad de numeros a recibir" << endl;
    cin >> z;
    int* digitos = new int[z];
    for (int i = 0; i < z; i++)
        cin >> digitos[i];

    cout << "Promedio:" << promedio(digitos) << endl;

    delete[] digitos;
}