diganme si este programa esta bien lo hice en dev c++

[eduardo17445]

programa que implemente y utilice una funcion para determinar si un numero es positivo o negativo lee un numero por teclado e imprime por pantalla si el numero leido es positivo o negativo haciendo uso de la funcion definida



#include <iostream>
#include <conio.h>
using namespace std;
int numero(int num)
{
    int pos,ne;
    if(num>=0)
    cout<<"el numero es positivo";
    else
    if(num<=0)
    cout<<"el numero es negativo";
    return pos;
    return ne;
}
int main()
{
    int num,pos,ne;
    cout<<"introduzca un numero";
    cin>>num;
    cout<< "numero"<<numero(pos,ne)<<endl;
    system ("pause");
  return 0;
}

[leosansan]

Reiterativo el tema o te has equivocado de código:

Código (cpp) [Seleccionar] Expandir

#include <iostream>
#include <cstdlib>

using namespace std;
void numero(int num)
{
   if(num>0)
   cout<<"el numero es positivo";
   else
   if(num<0)
   cout<<"el numero es negativo";
   else
   cout<<"el numero es cero";
}
int main()
{
   int num;
   cout<<"introduzca un numero ";
   cin>>num;
   numero(num);
   cout << endl<< endl;
   system ("pause");
 return 0;
}


Saluditos!. ...

[amchacon]

Pues la verdad:

- Usar Devcpp, un IDE desactualizado y en desuso:
http://foro.elhacker.net/programacion_cc/iquestpor_que_no_deberias_usar_el_devcpp-t385150.0.html

- Has incluido la librería conio, y no la estas usando en ningún momento (ni tampoco deberías usarla, no la incluyas!).
- La función no tiene un nombre muy poco explicito (Numero??)
- La función tiene incrongruencias:

Código (cpp) [Seleccionar] Expandir

 if(num>=0)
   cout<<"el numero es positivo";
   else
   if(num<=0)
   cout<<"el numero es negativo";

Ese if sobra (si numero no es mayor/igual a cero, entonces va a ser negativo, no tienes que comprobar nada)

Código (cpp) [Seleccionar] Expandir


return pos;
return ne;

Solo se puede devolver un numero.... Por otro lado, ¿Que estas devolviendo? Pos y ne son variables que te has sacado de la manga.

EDITO: El código de Leosansan es una solución correcta y óptima.

[Luchoz95]

Empecemos por ...

Código (c) [Seleccionar] Expandir

#include <conio.h>
no es conveniente usarla y por lo que veo , en el source no llama ninguna función de esa librería ...
y bueno acá tenes la solucion que intente hacer ..

Código (c) [Seleccionar] Expandir

#include <iostream>
#include <windows.h> //Para la funcion system

using namespace std;

void numero(int num) //Use una funcion void, ya que no me devuelve nada
{
if(num>0) //Si el numero es positivo ...
{
cout<<"Es positivo"<<endl;
}
if(num<0) //Si el numero es negativo ...
{
cout<<"Es negativo"<<endl;
}
}
int main()
{
    int nume; //Variable del numero ingresado ...
    cout<<"introduzca un numero: ";
    cin>>nume;
    numero(nume); //Implementamos la funcion
    system ("pause"); //Esta funcion no es muy recomendada , fíjate en el foro hay mas opciones ...
  return 0;
}


[eduardo17445]

ah ok muchas gracias