[SOLUCIONADO] operator<<

[Pernoctador]

Buenas.
Vuelvo al ruedo con esto de C++, y me encuentro con que no se hacer correctamente el operator<<. Fui a algunos foros y copie y pegue explicaciones, pero no anda. Que no estoy viendo?

Código:

Racional.hpp:14:9: error: ‘ostream’ in namespace ‘std’ does not name a type
  friend std::ostream& operator<< (std::ostream& stream, const Racional& elem);
         ^

.h

Código C++:

/**
  * Racional.hpp
*/
#ifndef __RACIONAL__
#define __RACIONAL__
 
using namespace std;
 
class Racional
{
public:
    Racional();
    Racional(long numerador, long denominador);
    Racional(const Racional& copia);
    
    friend std::ostream& operator<< (std::ostream& stream, const Racional& elem);
 
[...]
 
private:
    
    long Num;
    long Den;
    
    long mcd(long a, long b);
};
 
#endif
 

.cpp

Código C++:

/**
   * Racional.cpp
*/
 
#include <iostream>
#include <cmath>
#include "Racional.hpp"
 
using namespace std;
 
[...]
 
std::ostream& operator<< (std::ostream& stream, const Racional& elem) 
{
    stream << elem.Num << "/" << elem.Den; //necesito friend para usar Num y Den q son privados
    return stream;
}
 

Lo mas triste es que he c&p de gente q dice que esta es la posta. No se :(

[rflorez]

Prueba con #include <ostream>

[Pernoctador]

Iniciado por rflorez

Prueba con #include <ostream>

No, iostream incluye a ostream, istream y otros.

De todas formas lo probé :P y no lo soluciona.



Como nunca entendí bien el tema tengo la sospecha que podría tener que ver con el namespace. Algún alma caritativa y conocedora del tema que me ilumine? :)

[dehm]

Creo que el error se refiere a que en el fichero *.hpp no se define el tipo ostream, porque no has incluido allí iostream

El hecho de que lo acote al namespace std es porque estas definiendo el tipo con el operador de ámbito:

Código C++:

std::ostream,
 

por lo que el compilador asume que ese tipo está definido en el namespace std, y puede tal vez hacer pensar que el error es algo relacionado con namespace

Si incluyes la librería iostream en el fichero *.h deberías no tener ese problema. De paso la puedes quitar del fichero *.cpp.

Y por último, si defines que estas en el namespace std, es redundante definir los tipos pertenecientes con el operador de ámbito...vamos, que te puedes evitar anteceder el std:: antes de los tipos. (o dejarlo así y no definir el namespace std)

Saludos y espero no haber metido la pata con la respuesta

[Pernoctador]

Iniciado por dehm

Creo que el error se refiere a que en el fichero *.hpp no se define el tipo ostream, porque no has incluido allí iostream

[...]

Si incluyes la librería iostream en el fichero *.h deberías no tener ese problema. De paso la puedes quitar del fichero *.cpp.

[...]

Veamos veamos... lo que decis suena a ser la solucion.. lastima que no es tan asi xD

Código:

In file included from Racional.cpp:3:0:
Racional.hpp: In function ‘std::ostream& operator<<(std::ostream&, const Racional&)’:
Racional.hpp:69:10: error: ‘long int Racional::Num’ is private
     long Num;
          ^
Racional.cpp:395:17: error: within this context
  stream << elem.Num << "/" << elem.Den;
                 ^
In file included from Racional.cpp:3:0:
Racional.hpp:70:10: error: ‘long int Racional::Den’ is private
     long Den;
          ^
Racional.cpp:395:36: error: within this context
  stream << elem.Num << "/" << elem.Den;
                                    ^

Para mi tiene sentido lo que me dijiste, pero ahora me sale este error :(
Intenté pasar Num y Den con funciones pero tampoco anda

Código C++:

long num(){return Num;}
long den(){return Den;}
 

y

Código C++:

const long num(){return Num;}
const long den(){return Den;}
 

En esos casos lo que me dice es que se viola el const Racional que paso como parámetro a la función -.-

[dehm]

Hola:

Parece que el error se refiera a que la función que sobrecarga << no tenga acceso a los miembros privados de la clase.¿Puede ser que le quitaras la declaración de friend?

Te pongo un ejemplo que debería de funcionarte, a ver si puedes detectar el error

Fichero h:

Código C++:

#ifndef RACIONAL_H
#define RACIONAL_H
 
#include <iostream>
 
using namespace std;
 
class Racional
{
public:
    Racional():Num(1),Den(1) {}
    Racional(long numerador, long denominador):Num(numerador),Den(denominador) {}
    Racional(const Racional& copia);
    long int LeeNum()const{return Num;}
    long int LeeDen()const{return Den;}
 
 
    friend ostream& operator<< (ostream& stream, const Racional& elem);
 
private:
 
    long int Num;
    long int Den;
    long int mcd(long a, long b);
};
 
 
#endif // RACIONAL_H
 

Fichero cpp:

Código C++:

#include "../include/Racional.h"
 
ostream& operator<< (ostream& stream, const Racional& elem)
{
    stream << elem.Num << "/" << elem.Den; //necesito friend para usar Num y Den q son privados
    return stream;
}
 

Un main para ver si funciona la cosa

Código C++:

#include "./include/Racional.h"
 
int main()
{
    Racional valor;
    cout<<valor.LeeNum()<<endl;
    cout<<valor.LeeDen()<<endl;
    cout<<valor<<endl;
    Racional valor2(10,20);
    cout<<valor2<<endl;
    return 0;
}
 

Respecto a la cláusula const, si defines la función como:

Código C++:

const long num(){return Num;}
 

Lo que le estás diciendo es que retorne un const long. Como Num es del tipo long, pues se quejará. (no hay que confundir long con const long). C/C++ es un lenguaje fuertemente tipado y no pasa ni una )

Para asegurarse de que la función no modifica el valor que lee, y que sólo lo devuelve, has de declararla así:

Código C++:

long int LeeNum()const{return Num;}
 

(bueno, verás que el tipo es long int y que le he cambiado el nombre, pero igualmente:

Código C++:

long num()const{return Num;}
 

Saludos

[Pernoctador]

Iniciado por dehm

Hola:


[...]


Respecto a la cláusula const, si defines la función como:

Código C++:

const long num(){return Num;}
 

Lo que le estás diciendo es que retorne un const long. Como Num es del tipo long, pues se quejará. (no hay que confundir long con const long). C/C++ es un lenguaje fuertemente tipado y no pasa ni una )

Para asegurarse de que la función no modifica el valor que lee, y que sólo lo devuelve, has de declararla así:

Código C++:

long int LeeNum()const{return Num;}
 

(bueno, verás que el tipo es long int y que le he cambiado el nombre, pero igualmente:

Código C++:

long num()const{return Num;}
 

Saludos

Gracias por la aclaracion del const, siempre me confundo con eso :)
[U]

Pude solucionarlo! :D

Les cuento que en algún momento, luego de la ultima corrección a mi codigo me quedo: friend ostream& operator<<(ostream& stream, const Racional& elem);

Si no lo ven, me faltaba un espacio -.-


Muchisimas gracias a todos los que contestaron y se metieron a leer este tema :D.