programa en c (Devc++)

[darkchaos]

Tengo una duda con este codigo

Código:

int main()
{
    int n,i;
    double vmenor;
    double *elemento[50];
    puts("Cuantos elementos ");
    scanf("%d",&n);
    for(i=0;i<n;i++)
    {
                    printf("Elemento [%d]",i+1);
                    scanf("%d",(elemento+i));
    }
    vmenor = menor(n,*elemento);
    getch();
    printf("El menor es %d \n\n",vmenor);
    system("pause");
    return 0;
}
double menor(int n,double *elemento)
{
       double men;
       int j,m;
       men = elemento[0];
       for(j=1;j<n;j++)
       {
                       if(elemento[j]<men)
                            men=elemento[j];
       }
       return men;
}

lo que quiero que haga el programa es leer n numeros e imprimir el menor pero lo tengo q hacer con punteros y no se utilizarlos muy bien como habran notado cuando lo compilo no marca errores pero cuando lo corro con debug me parace un erro que dice "An access violation(Segmentation Fault) raised in your program"
espero que alguien pueda ayudarme con este programa

[Eternal Idol]

Esto son 50 punteros a double:
double *elemento[50];

Esto querrias vos:
double elemento[50];

Despues tenes que pasarlo correctamente:
vmenor = menor(n, elemento);

En menor inverti la condicion:
if(men < elemento[j])

[darkchaos]

lo que debe hacer el programa en si es tomar n numeros y solamente decir cual es el menor pero debo utilizar punteros para hacerlo por eso hice el
[CODE]double *elemento[50][\CODE]
pero no se utilizar bien los punteros ademas el maestro me dijo que cuando llamo a la funcion con
[CODE] menor(n,*elemento)[\CODE]
estoy pasando unicamente UN valor de elemento, y en un libro de programacion en c, entendi que asi estoy pasando la referencia al vector y me confundio aun mas
aunq la verdad el error que comente ocurre cuando hago un debug, antes de pedir el 2 elemento

[Eternal Idol]

Proba lo que te dije ...

[darkchaos]

ahora lo tengo de esta manera

Código:

#include<stdio.h>
#include<stdlib.h>
#include<conio.h>
double menor(int n,double elemento);
double men;
int main()
{
    int n,i;
    double vmenor;
    double elemento[50];
    puts("Cuantos elementos ");
    scanf("%d",&n);
    for(i=0;i<n;i++)
    {
                    printf("Elemento [%d]",i+1);
                    scanf("%d",&elemento[i]);
                    fflush(stdin);
    }
    vmenor = menor(n,elemento);//paso por referencia
    getch();
    printf("El menor es %d \n\n",men);
    system("pause");
    return 0;
}
double menor(int n,double elemento)
{
       //double *men;
       int j,m;
       men = elemento[0];
       for(j=1;j<n;j++)
       {
                       if(elemento[j] < men)
                            men = elemento[j];
       }
       return (men);
}

pero de esta manera no estoy usando punteros y me sale incompatible type for argument 2 of `menor' en vmenor = menor(n,elemento)

en las asignaciones y comparacion de men arroja un error de 29 E:\programacion\examen_1.c subscripted value is neither array nor pointer

pero de esta manera no uso los apuntadores y eso es lo q necesito hacer

[Eternal Idol]

Yo no te dije que no usaras punteros para la funcion menor, sino que no declares 50 punteros cuando lo que queres es un array de 50 elementos (que se pasa siempre como puntero, tal como leiste, al pasarse por referencia), la funcion menor no tiene que cambiar su signature (el segundo parametro tiene que seguir siendo double *).

[darkchaos]

ahora lo tengo asi

Código f:

#include<stdio.h>
#include<stdlib.h>
#include<conio.h>
double *menor(int n,double *elemento);
double men;
int main()
{
    int n,i;
    double *vmenor;
    double *elemento[50];
    puts("Cuantos elementos ");
    scanf("%d",&n);
    for(i=0;i<n;i++)
    {
                    printf("Elemento [%d]",i+1);
                    scanf("%d",&elemento[i]);
    }
    menor(n,*elemento);
    getch();
    printf("El menor es %d \n\n",men);
    system("pause");
    return 0;
}
double *menor(int n,double *elemento)
{
       //double *men;
       int j,m;
       men = *elemento;
       for(j=1;j<n;j++)
       {
                       if(elemento[j] < men)
                            men = *elemento;
       }
       return 0;
}
 

y el error de la violacion de acceso aprace en la linea

Código:

 men = *elemento;

tambien intente con
men = elemento[0]; // el primer elemento pero sigue teniendo el mismo error

alguien conoce alguna solucion ?

[Eternal Idol]

La solucion es pasar el array correctamente, mira mi primer mensaje, tenes que pasarlo asi:

Código:

menor(n, elemento);

Ademas hay mas problemas, tenes que recibir lo que sea doble con %lf y no con %d. Y la condicion orignal estaba bien, mal por mi:

Código:

if(men > elemento[j])

Asi funciona bien:

Código:

double menor(int n,double *elemento);

int main()
{
    int n,i;
    double vmenor;

    double elemento[50];
    puts("Cuantos elementos ");
    scanf("%d",&n);
    for(i=0;i<n;i++)
    {
       printf("Elemento [%d]",i+1);
       scanf("%lf",&elemento[i]);
    }

    vmenor = menor(n, elemento);
    getchar();
    printf("El menor es %lf \n\n",vmenor);
    system("pause");
    return 0;
}
double menor(int n,double *elemento)
{
       double men;
       int j,m;
       men = elemento[0];
       for(j=1;j<n;j++)
       {
         if(men > elemento[j])
           men=elemento[j];
       }
       return men;
}

[jairGrunge]

darkchaos, aqui te dejo el programa con punteros.

Código:

#include <stdio.h>
#include <stdlib.h>

double menor(int n, int *ptrElemento)
{
       int men;
       int j;       
       for(j=0;j<n;j++)
       {
                       if(*(ptrElemento+j)<men){ //*(ptrElemento+j) = elemento[j]
                            men= *(ptrElemento+j);                            
                       }
       }
       return men;
}

int main()

{
    int i, n=10;
    int vmenor;
    int elemento[n];
    int *ptrInt = 0; //Inicializamos el puntero

    for(i=0;i<n;i++) {
         printf("Elemento [%d]: ",i+1);
         scanf("%i", &elemento[i]);		 

    }
	
    ptrInt = elemento; //apuntamos al primer elemento del arreglo elemento[50]
    vmenor = menor(n, ptrInt);
    //getch();
    printf("El menor es %i \n\n",vmenor);

    //system("pause");
    return 0;
}

[Eternal Idol]

jairGrunge excepto que usas inten lugar de double no hay diferencia, un array se pasa por referencia asi que tu programa lo unico que hace es introducir una redundancia, un puntero al array.

[jairGrunge]

Iniciado por Eternal Idol

jairGrunge excepto que usas inten lugar de double no hay diferencia, un array se pasa por referencia asi que tu programa lo unico que hace es introducir una redundancia, un puntero al array.

darkchaos dijo:

lo que quiero que haga el programa es leer n numeros e imprimir el menor pero lo tengo q hacer con punteros

Utilice un apuntador para saber cual es el menor de los datos en el arreglo elemento.

[Eternal Idol]

Si, aunque es irrelevante debido a lo anteriormente explicado que pareces no comprender, lo que no dijo fue que queria trabajar con enteros y devolver una variable de tipo int en una funcion que deberia devolver double.

[jairGrunge]

Iniciado por Eternal Idol

Si, aunque es irrelevante debido a lo anteriormente explicado que pareces no comprender, lo que no dijo fue que queria trabajar con enteros y devolver una variable de tipo int en una funcion que deberia devolver double.

Esto(tipo int) es solo un pequeño detalle, lo que el postulante indico esta reflejado.

[darkchaos]

Gracias Eternal Idol y jairGrunge por la ayuda estoy revisando los dos aunque el primero si tengo duda de los punteros en el segundo lo compile con dev c++ en vista y corrio sin problemas cuando lo compile en dev c++ en xp siempre no importando cuantos numeros lea el resultado era -1
muchas gracias a los dos
saludos

[Eternal Idol]

De nada, lee todo y recorda lo que ya encontraste un array se pasa por referencia.