Hola, me estoy iniciando en la programacion de C++ estoy compilando una tarea para la Universidad, pero me arroja dos warnings:
C:\Users\Har\Desktop\Taller.cpp||In constructor 'calle::calle(int, int)':|
C:\Users\Har\Desktop\Talle2.cpp|23|warning: 'calle::nro' should be initialized in the member initialization list|
C:\Users\Har\Desktop\Taller2.cpp|23|warning: 'calle::sentido' should be initialized in the member initialization list|
||=== Build finished: 0 errors, 2 warnings ===|
el programa funciona bien, pero me causa duda esos warnings, tambien trate poniendole inline pero me arrojaba el mismo warning en la otra linea xD
le pregunte a mi profesor y me dijo que lo ignorara D: que no es muy poco etico si quiero programar bien.
el warning lo manda en calle(int a,int b){nro=a;sentido=b;}
class calle
{
int nro,sentido;
public:
calle(int a,int b){nro=a;sentido=b;}
};
desde ya muchas gracias por todo :D
Fijate de ponerlo asi...
class calle
{
private:
int nro, sentido;
public:
calle( int a, int b ){ this->nro = a; this->sentido = b; }
};
Otra forma
class calle
{
private:
int nro, sentido;
public:
calle( int a, int b ) : nro( a ), sentido( b ) { }
};
Salu10.
¿Qué compilador?
A mi me compila bien así:
leo@lein:~/Escritorio/c_proyect$ cat code.cpp
#include <iostream>
class calle
{
public:
int nro,sentido;
public:
calle(int a,int b)
{
nro = a;
sentido = b;
}
};
int main()
{
return 0;
}
leo@lein:~/Escritorio/c_proyect$ g++ code.cpp -o code
leo@lein:~/Escritorio/c_proyect$
Cita de: Leo Gutiérrez.
class calle
{
public:
int nro,sentido;
public:
calle(int a,int b)
{
nro = a;
sentido = b;
}
};
Una pregunta, no seria redundante poner 2 veces public? O te confundiste y quisiste poner private?
Salu10.
Cita de: nicolas_cof en 27 Junio 2010, 03:29 AM
Cita de: Leo Gutiérrez.
class calle
{
public:
int nro,sentido;
public:
calle(int a,int b)
{
nro = a;
sentido = b;
}
};
Una pregunta, no seria redundante poner 2 veces public? O te confundiste y quisiste poner private?
Salu10.
No, quise ponerlas, me gusta ser lo más explícito posible.
@nicolas_cof, ¿crees que está incorrecto lo que hice? :-\
Leo, es redundante. Pones un solo public arriba como bien dice Nico y ya esta. En el caso que sean muchos miembros en todo caso podría ser discutible pero en este es redundante. Igualmente los miembros están declarados private por algo en el código inicial. Los miembros de una clase por defecto son de carácter privado.
En cuanto al warning, hazle caso o simplemente déjalo pasar. En este caso no tiene relevancia alguna y lo puedes solucionar simplemente inicializando las variables con un valor neutro o dejandolo pasar en caso que dichas variables tengan garantizado un valor útil.
Saludos
muchas gracias a todos por las repuestas
el unico que no mando warnig fue este:
class calle
{
private:
int nro, sentido;
public:
calle( int a, int b ) : nro( a ), sentido( b ) { }
};
los otros de igual forma funcionan, pero mandan advertencia al igual que el mio..
otra vez muchas gracias :D
a por cierto, uso codeblocks y tanto en win como ubuntu mandaba el mismo
eso si si alguien pudiera explicar a que se debe el warnings seria genial ;D
thank you!!! ;)
misterharry, de aca puedes leer algo... http://www.cprogramming.com/tutorial/initialization-lists-c++.html
Leo Gutiérrez., no creo que sea incorrecto, simplemente me parecio redundante!
De ultima nos podemos juntar con el amigo Bjarne y lo conversamos ;D
Salu10.
CitarSi no fuera por C, programaríamos en BASI, PASAL y OBOL
jajajjajjajajaa... q risa!!!
gracias
nicolas_cof le echare un ojo ;)
y en mi opinion lo de Leo no es relevante, creo que se dio cuenta del detalle.
Pero bueno, es solo una linea mas de codigo, no afecta en nada o si?
No, no afecta, pero tienen razón, las variables miembro son declaradas por defecto a private, yo las cambié a mi gusto a public, haciendo mal, no entendí el contexto de tu código.
Saludos.