tomarMayor

[ignatar20]

Hola buenos días.
Estoy haciendo un programa, el cual hace lo siguiente:
El usuario mete una cierta cantidad de números y de esos números me dice cual es el mayor.
Lo estoy haciendo con arrays y funciones.
Os dejo el código para que le echéis un vistazo:

Código C++:

#include <iostream>
using namespace std;
float num[10];
float mayor;
float tomarMayor(float num[10]);
 
int main() {
    float i=0;
    for(i=0;i<=num[10];i++)
    {
        cout << "Introduzca un numero para introducir en el array:";
        cin >>num[i];
    }
    cout<<tomarMayor;
 
    return 0;
}
 
float *tomarMayor(float num)
{
    float w=0;
    for(w=0;w<=num;w++)
    {
        if(num[w]>=num[w-1])
        {
            mayor = num[w];
        }
        else
        {
            mayor = num[w-1];
        }
    }
    return mayor;
}
 

El caso es que en el if me pone error, "invalid types 'float[float]' for array subcript",
en return mayor me pone "cannot convert float to float in return", y en cin>>num[i] me pone "invalid types `float[10][float]' for array subscript".

Alguien me podría ayudar.

Muchas gracias

[Malenko]

El motivo es que en la declaración del parámetro de la función lo defines como un float, no como un array de floats:

Código C++:

float *tomarMayor(float num)
 

Por eso cuando haces:

Código C++:

if(num[w]>=num[w-1])
 

Se queja porque lo estas accediendo como un array cuando no lo has declarado como tal.

También he de decir que en tu código, en la linea 5 defines la función de una manera y en la 19 de otra...

[dehm]

Hola:

El programa tiene bastantes fallos:

Por un lado estás intentando acceder a los índices del array así:

Código C++:

cin >>num[i];
 

Cuando i es un float.
Los índices del array son int. No se puede acceder a array[4.5], por ejemplo (en rigor sí se podría, pero sobrecargando el operador y en todo caso ya estamos hablando de otra cosa)

Otro fallo:

Código C++:

for(i=0;i<=num[10];i++)
 

Estas intentando que el bucle vaya desde float = 0 (error comentado arriba) a float = num[10]. No tiene sentido, querrás que el bucle vaya hasta el índice 10 (realmente 9, ya que un array de 10 elementos va de 0 a 9), pero no hasta el valor que haya en num[10];

Por otro lado, en la declaración de la función tienes:

Código C++:

float tomarMayor(float num[10]);
 

y en la implementación:

Código C++:

float *tomarMayor(float num){}
 

La función retorna un puntero a float*, pero tú retornas mayor, que es float (no float*)

Por otro lado, el propio algoritmo de la función no te va a funcionar correctamente, porque haces comparaciones 2 a 2, por lo que realmente sólo te va a retornar el mayor de los últimos dos números comparados. Suponiendo que el blucle estuviera bien, te compararía el valor del índice 8 y del índice 9 (valores 9 y 10 del array)
Además ten cuidado porque cuando haces:

Código C++:

for(w=0;w<=num;w++)
    {
        if(num[w]>=num[w-1])
.......................
 

En la primera iteración te estás saliendo fuera del array, ya que si w=0, w-1=-1, o sea que estás intentando acceder a la posición -1 del array. Si te compilase, lo siguiente que tendrías probablemente fuera un error de segmentación.

Bueno, además, haces uso de variables globales que no tienen mucho sentido.
Con respecto a recorrer el array en la función, puedes definir el tamaño del mismo con una constante, así puedes usar esa constante para recorrer el bucle, y con sólo cambiar su valor te valdrá para cualquier tamaño de array:

Te pongo una propuesta sin los errores:

Código C++:

#include <iostream>
    using namespace std;
 
    const int tam_array=10;
    float tomarMayor(float M[tam_array]);
 
    int main() {
        float A[tam_array];
        int i=0;
        for(i=0;i<tam_array;i++)
        {
            cout << "Introduzca un numero para introducir en el array:";
            cin >>A[i];
        }
        cout<<tomarMayor(A);
 
        return 0;
    }
 
    float tomarMayor(float M[tam_array])
    {
        float mayor=0;
        for(int w=0;w<tam_array;w++)
        {
            if(M[w]>=mayor)
            {
                mayor = M[w];
            }
        }
        return mayor;
    }
 

[dehm]

hmmmm...no había visto que ya te habían respondido.
Además veo que mi código tiene un fallo

En la función mejor así:

Código C++:

float tomarMayor(float M[tam_array])
    {
        float mayor=M[0];//----esta es la variación, donde estaba el fallo
        for(int w=1;w<tam_array;w++)//---como el valor de M[0] ya lo tienes, te ahorras una iteración y empiezas el bucle en M[1]
        {
            if(M[w]>=mayor)
            {
                mayor = M[w];
            }
        }
        return mayor;
    }
 

[ignatar20]

Muchísimas gracias a ambos por ayudarme, gracias a vuestra ayuda tengo más esperanzas para aprobar jajaja