SoloCodigo

Hola, quiero que me digan porque me sale este warning y porque deberia cambiar a const en la entrada de parámetros,es decir yo tengo el siguiente método:


su prototipo es void Color(char *);

teniendo esto me sale siempre un aviso que me dice;
aviso: conversión obsoleta de una constante de cadena a ‘char*'

me han comentado que dicho método lo tendría que poner como
void Color(const char*);

porque? esa funcion solo debe de cambiar el color de un poro(que es el objeto), y yo en mi main lo probaria de la siguiente manera:


porque debo cambiarlo a const, es por el compilador, uso la version 4.3.3 de g++.

Espero respuesta.
Gracias.

Código: C++

1. cout.setf(ios::fixed);
2. cout.precision( 2 );
3. float f = 25.57695;
4. cout << f << endl;
5. f = 25;
6. cout << f << endl;
7.  

si te fijas, la ultima impresion por pantalla de f tiene que ser 25, porque en el paso anterior asignas 25 a f, entonces, f es 25 y no se nota la precision..no?


Yo estoy en el caso de la programacion orientada a objetos, entonces estas intrucciones debo de meterla dentro del operador salida? cuando imprimo el volumen?

HOla, creo que aqui me pueden solucionar estas dudas, que me parecen esenciales para seguir programando en programacion orientada a objetos en c++, por ejemplo ,empezemos con los pasos por referencias en el constructor copia.
Yo me estoy guiando por un libro de teoria de c++ que me dice que el unico parametro que hay que pasarle al constructor copia se debe pasar por referencia.,y me dice que el constructor copia se invoca automáticamente al pasar un objeto por valor a una función.Por tanto si el objeto que recibe el constructor de copia como parámetro se pasase por valor, se invocaría automaticamente al constructor de copia para hacer una copia de ese objeto y se entraria en un bucle infinito.

mis preguntas..bucle infinito? eso no lo entiendo....

y despues es la utilizacion de const

porque en el constructor copia se pasa el parametro por const?

TPoro(const &p);

y algunos métodos fuera del método se pone const?

void MiMetodo(ixxxx) const ?

Si alguien que me diga algún enlace que yo pueda ver con ejemplos todo esto se lo agradecería.

Un saludo y espero respuesta.
Gracias.

Ya lo estas haciendo, pero mal, tiene que coincidir el tipo de reserva y de liberacion de memoria. Ahora tenes esto:

color=new char[strlen(col)+1];
delete color;

Tenes que usar delete[] para la memoria reservada con new[] (asi se libera y se llama al constructor por cada uno de los objetos, en este caso no hay por ser un tipo primitivo, pero esa es la idea y por eso se queja).

es decir tengo que hacer delete  []color? o no es eso lo que me estais diciendo?

Pero de la forma que lo hago yo está mal?
como no se quejaria valgrind?

Hola, la ejecucion de mi programa funciona bien, pero creo que no estoy haciendo bien algun delete.

Por ejemplo yo tengo mi main

int main(){

TPoro c4(4,5,20.35,"YElloW");
cout<<c4<<endl;


return 0;
}

el valgrind  me dice lo siguiente:
==14577== Mismatched free() / delete / delete []
==14577==    at 0x402599A: operator delete(void*) (vg_replace_malloc.c:342)
==14577==    by 0x8049044: TPoro::~TPoro() (TPoro.cpp:151)
==14577==    by 0x8048993: main (tad.cpp:107)
==14577==  Address 0x42c4028 is 0 bytes inside a block of size 7 alloc'd
==14577==    at 0x402630E: operator new[](unsigned int) (vg_replace_malloc.c:268)
==14577==    by 0x8048CE1: TPoro::TPoro(int, int, double, char*) (TPoro.cpp:92)
==14577==    by 0x8048960: main (tad.cpp:10)
==14577==
==14577== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 17 from 1)
==14577== malloc/free: in use at exit: 0 bytes in 0 blocks.
==14577== malloc/free: 1 allocs, 1 frees, 7 bytes allocated.
==14577== For counts of detected errors, rerun with: -v
==14577== All heap blocks were freed -- no leaks are possible.


yo creo que puede ser en uno de los siguiente metodos


No libero bien la memoria?
me pueden ayudar?

Hola, no consigo ver donde está el fallo de segmentacion que me da, me pueden ayudar?


Tengo que decir que las funciones Convertir (pasa de mayusculas a minusculas) y Mayuscula comprueba si es o no mayuscula el caracter.
y dichos metodos me los he implementado en la parte pública.

pero cuando llega fuera de Convertir..pega el cascazo....

Me ayudan?

correcto, es uno de los fallos, pero es que me sigue dando errores con la siguiente correcion, he modificado los constructores

TPoro::TPoro(int px,int py,double vol,char *col){

   int tam;


      

   x=px;
   y=py;
   volumen=vol;

   color=new char[strlen(col)+1];
   
   if(color==NULL){
      return;
   }

   //if(color !=NULL){

      strcpy(color,col);
   //}

   //else{
   //   return;
   //}

}


TPoro::TPoro(const TPoro &po){

   //int tam=strlen(po.color)+1;//saco el tamaño de la cadena de caracteres

   x=po.x;
   y=po.y;
   volumen=po.volumen;

   color=new char[strlen(po.color)+1];

   if(color ==NULL){
      return;
      //strcpy(color,po.color);
   }
      strcpy(color,po.color);
   //else{
   //   return;
   //}
}

y pasando el gdb me dice lo siguiente

Program received signal SIGSEGV, Segmentation fault.
0x08048fe1 in TPoro (this=0xbfa81a0c, px=4, py=5, vol=20.353000000000002,
    col=0x0) at lib/TPoro.cpp:41
41              color=new char[strlen(col)+1];

ese es el error que me da

Ok , lo he entendido, como el parametro le estamos pasando una referencia constante, y declarando el metodo como constante lo que hacemos es que el objeto utilizado no modificará ninguna de sus propiedades.

Una ultima cuestion

yo para probar mi programa mi estoy haciendo mi propio main, es decir yo tengo un constructor por defecto que le meto estos parametros

TPoro (int, int, double, char *);

y yo en mi main, pongo para hacerme un ejemplo

TPoro c1(5,6,20,3,"red");

pero como le meto yo en el cuarto parámetro NULL, si se lo meto asi directamente me da error, como le tengo que meter NULL, eso es para hacer comprobaciones.

Error de compilacion no me daria, pero seria correcto meterle el parametro asi?, lo digo porque luego me da fallo de segmentacion.



Espero respuesta.
Gracias.

Es decir despues de utilizarlo debo de hacer delete []v?, para borrar el vector? o tengo que hacer

?? porque estó es para liberar memoria..verdad?

lo que me dice usted es que TAfd v[100], lo haga dinámico?, como expreso eso? para hacerlo dinámico? me puede decir la definicion para hacerlo dinámico?

Hola,queria preguntarle si sabia la causa del siguiente "Fallo de segmentacion",ya que no entiendo el porque y llevo días,buscando la causa.

si yo tengo el siguiente programa:

#include<iostream>
#include<fstream>
#include<string>


const int Ksimbol=5;
const int Kmax=100;
struct TNodo{
   int est;
   int Matriz[Ksimbol][Kmax];
   char sim[Ksimbol];

};

struct TAfd{

   int estado;
   int vestados[Kmax];
   TNodo v1[Kmax];
   
};

//TAfd v[100];

using namespace std;


int main(int argc,char *argv[]){

//Talumno v[1000];
   TAfd v[100];
   cout<<argv[1]<<endl;
   


}

//Mi pregunta es porque ahora declarándome TAfd v[100] como local, me da un fallo de segmentación,pero esa misma variable declarándomela como global, no me da error,le he puesto ese programa ,para simplificar mi duda, ya que mi programa es mucho más extenso,pero es el caso,pero minimizado.

¿Me puede decir la causa de que por una parte me da fallo de segmentacion y por otra parte no?

Mi version de compilador es la 4.1.2 y tengo instalado el valdring(que creo que es el que me detecta el fallo),ya que en un compilador como un 3.1.6 no detecta el siguiente fallo,siempre en c++;



Un saludo,espero respuesta.
Gracias.

Hola, tengo que hacer una practica, y un apartado es cuando un coche se averie...que espere durante un momento(determinado) ,pero claro tiene que haber una cuenta atras..que me tiene que salir por pantalla...y no se como implementar esa cuenta atrás...

Yo he echo lo siguiente..

....

 LOOP
                                         --    exit when (Clock - Comienzo)>10.0;
                                         --    Put_line("se mete");
                                                DELAY UNTIL comienzo+1.0;
                                                Durac:=Fin-Comienzo;
                                                Jaja:=Integer(Durac);
                                                Put(jaja);
                                                Op_Reloj_Averia(Via1).Actualiza_Reloj_Averia(Jaja);--vemos el cast de convertir un tipo duration a entero
                                         --    Jaja:=Jaja-1;
                                         --    Put(jaja);
                                          end loop;

....

Es dificil entender el problema..de todas formas muchas gracias por la ayuda.
Muchas gracias.