Imprimir Página - El for no me hace su funcion? (Solucionado)

Título: El for no me hace su funcion? (Solucionado)
Publicado por: nolasco281 en 4 Febrero 2014, 21:57 pm

Hola.

Espero que me pueda ayudar con mi duda estoy tratando de hacer métodos de administración, pero al momento de mostrar datos en pantalla no sé qué estoy haciendo mal.

No es un error de sintaxis creo que es mas de lógica pero no entiendo que parte es la que trabaja mal.

El problema que detecto es en el archivo de implementación de Camion.cpp en la función Camion::flistaDeCamiones() en el for cuando intento acceder a la a listaDeCamiones->imprimir(); el primer registro si lo introduce bien pero cuando le agrego otro, el programa se cierra y parece, que borrara el primer registro.

Y eso no me deja seguir con los demás métodos.

he sacado la funcion imprimir fuera del for por solo me registra una ves los datos, y no una secuencia, si lo pongo antes de empezar el for me hace la secuencia pero me repite los ultimos datos las veces que haya registrado un camion.

Lamento preguntar tanto.

//Camion.cpp

Código

#include "Camion.h"
#include <iostream>
#include <iomanip>
#include <string>
 
using namespace std;
 
Camion::Camion (int idCam, int cilindrajeCam, int puertasCam, int anioCam, 
                double precioCam, string marcaCam, string modeloCam, 
                string colorCam)
                {
                   idCamion = idCam;
                   cilindrajeCamion = cilindrajeCam;
                   nPuertasCamion = puertasCam;
                   anioCamion = anioCam;
                   precioCamion = precioCam;
                   marcaCamion = marcaCam;
                   modeloCamion = modeloCam;
                   colorCamion = colorCam;
 
                   Maximo =100;
                   cantidadDeCamiones =0;
                }
//Funciones set
void Camion::setIdCamion(int idCam)
{
     idCamion = idCam;
}
 
void Camion::setCilindrajeCamion(int cilindrajeCam)
{
     cilindrajeCamion = cilindrajeCam;
}
 
void Camion::setNPuertasCamion(int puertasCam)
{
     nPuertasCamion = puertasCam;
}
 
void Camion::setanioCamion(int anioCam)
{
     anioCamion = anioCam;
}
 
void Camion::setPrecioCamion(double precioCam)
{
     precioCamion = precioCam;
}
 
void Camion::setMarcaCamion(string marcaCam)
{
     marcaCamion = marcaCam;
}
 
void Camion::setModeloCamion(string modeloCam)
{
     modeloCamion = modeloCam;
}
 
void Camion::setColorCamion(string colorCam)
{
     colorCamion = colorCam;
}
 
//Funciones get
int Camion::getIdCamion() const
{
    return idCamion;
}
 
int Camion::getCilindrajeCamion() const
{
    return cilindrajeCamion;
}
 
int Camion::getnPuertasCamion() const
{
    return nPuertasCamion;
}
 
int Camion::getanioCamion()const
{
    return anioCamion;
}
 
double Camion::getPrecioCamion() const
{
       return precioCamion;
}
 
string Camion::getMarcaCamion() const
{
       return marcaCamion;
}
 
string Camion::getModeloCamion() const
{
       return modeloCamion;
}
 
string Camion::getColorCamion() const
{
       return colorCamion;
}
 
//Ingresa un nuevo camion
 
void Camion::ingresarCamion()
{
   listaDeCamiones = new Camion*[Maximo];// arreglo de 100
 
   if (cantidadDeCamiones < Maximo)
   { 
     system("cls");
     cout << "Ingrese los datos del camion\n";
     cout << "Ingrese el ID del camion: ";
     cin  >> idCamion;
 
     cout << "Ingrese la Marca del Camion: ";
     cin  >> marcaCamion;
 
     cout << "Ingrese el Modelo del camion: ";
     cin  >> modeloCamion;
 
     cout << "Ingrese el año del Camion: ";
     cin  >> anioCamion;
 
     cout << "Ingrese el cilindraje del camion: ";
     cin  >> cilindrajeCamion;
 
     cout << "Ingrese el numero de puertas: ";
     cin  >>  nPuertasCamion;
 
     cout << "Ingrese el color del camion: ";
     cin  >> colorCamion;
 
     cout << "Ingrese el presio del camion: ";
     cin  >> precioCamion;
 
     listaDeCamiones[cantidadDeCamiones] = 
     new Camion(idCamion, cilindrajeCamion, 
         nPuertasCamion, anioCamion, 
         precioCamion, marcaCamion,
         modeloCamion,colorCamion);
 
     cantidadDeCamiones++;
 
     cout << endl <<"Los datos del camion fueron ingresados correctamente\n";
   }
   else
   {
       system("cls");
       cout << "Ya no hay espacio disponible para mas camiones\n"; 
   }
}
 
//Muestra los datos del camion
void Camion::flistaDeCamiones()
{
     if (cantidadDeCamiones==0)
     {
       system("cls");
       cout<<"No se a registrado ninguna Camion todavía"<<endl;
     }
       else
       {
           cout<<"..:::Lista de Camiones:::.."<<endl<<endl;
           cout << setw(10)<<"Id Camion"<<setw(15)<<"Modelo"<<setw(15)<<"marca"<<endl;
           cout<<"================================================="<<endl;
 
           //Imprimmir todos los camiones almacenadas en el arreglo
           for (int i=0; i<cantidadDeCamiones; i++)
           {
             listaDeCamiones[i]->imprimir(); //PROBLEMA Y NO LO IDENTIFICO
           }
           cout<<endl<<"*******Fin de la lista **********"<<endl<<endl;
       }
}
//Busca ese camion
int Camion::buscarCamion(int codigo)
{
    for (int i=0; i < cantidadDeCamiones; i++)
    {
        if(listaDeCamiones[i]->getIdCamion() == codigo)
        {
          return i;
        }
    }
    return -1;
}
 
//Modifica ese Camion.
void Camion::modificarCamion()
{
     if (cantidadDeCamiones == 0)
     {
       system("cls");
       cout << "No se ha registrado ningun camion tadavia\n";
     }
     else
     {
         cout << "Modificar Camion\n\n";
         int codigoCamion;
         cout << "Ingrese el codigo del camion: ";
         cin >> codigoCamion;
 
         int ubicacion = buscarCamion(codigoCamion);
         if (ubicacion == -1)
         {
           cout << "Lo sentimos ese codigo del camion no existe\n";
         }
         else
         {
             cout << "El codigo se encontro\n\n";
             cout << setw(10)<<"Id"<<setw(15)<<"Modelo"<<setw(15)<<"marca"<<endl;
             cout<<"================================================="<<endl;
 
             listaDeCamiones[ubicacion]->imprimir();
             cout << "\n\n";
             cout  << "Ingrese los nuevos datos del Camion\n\n";
 
             cout <<  "Marca";
             cin  >> marcaCamion;
 
             cout << "Modelo del camion: ";
             cin  >> modeloCamion;
 
             cout << "Año del Camion: ";
             cin  >> anioCamion;
 
             cout << "Cilindraje del camion: ";
             cin  >> cilindrajeCamion;
 
             cout << "numero de puertas: ";
             cin  >>  nPuertasCamion;
 
             cout << "Color del camion: ";
             cin  >> colorCamion;
 
             cout << "Presio del camion: ";
             cin  >> precioCamion;
 
             listaDeCamiones[ubicacion]->setMarcaCamion(marcaCamion);
             listaDeCamiones[ubicacion]->setModeloCamion(modeloCamion);
             listaDeCamiones[ubicacion]->setanioCamion(anioCamion);
             listaDeCamiones[ubicacion]->setCilindrajeCamion(cilindrajeCamion);
             listaDeCamiones[ubicacion]->setNPuertasCamion(nPuertasCamion);
             listaDeCamiones[ubicacion]->setColorCamion(colorCamion);
             listaDeCamiones[ubicacion]->setPrecioCamion(precioCamion);
 
             cout << "El camion fue modificado Exitosamente.\n";
         }
     }
}
 
void Camion::eliminarCamion()
{
     if (cantidadDeCamiones == 0)
     {
       system("cls");
       cout << "No se ha registrado ningun camion tadavia\n";
     }
     else
     {
         cout << "Eliminar Camion\n\n";
         int codigoCamion;
         cout << "Ingrese el codigo a eliminar: ";
         cin >> codigoCamion;
 
         int ubicacion = buscarCamion(codigoCamion);
         if (ubicacion == -1)
         {
           cout << "Lo sentimos ese codigo del camion no existe\n";
         }
         else
         {
             cout << "El codigo se encontro\n\n";
             cout << setw(10)<<"Id"<<setw(15)<<"Modelo"<<setw(15)<<"marca"<<endl;
             cout<<"================================================="<<endl;
 
             listaDeCamiones[ubicacion]->imprimir();
             cout << "\n\n";
 
             cout << "Presione enter para eliminar ese camion\n\n";
             system("pause");
 
             //Pasa a eliminar usando una copia del arreglo
             Camion**temporalListaDeCamiones = new Camion*[Maximo];
             int temporalCantidadDeCamiones = 0;
 
             //Se copia el arreglo en otro arreglo exepto el que se quiere eliminar
             for(int i =0; i < cantidadDeCamiones; i++)
             {
               if(i!=ubicacion)
               {
                 temporalListaDeCamiones[temporalCantidadDeCamiones] = new Camion(listaDeCamiones[i]->getIdCamion(),
                                                                                  listaDeCamiones[i]->getCilindrajeCamion(),
                                                                                  listaDeCamiones[i]->getnPuertasCamion(),
                                                                                  listaDeCamiones[i]->getanioCamion(),
                                                                                  listaDeCamiones[i]->getPrecioCamion(),
                                                                                  listaDeCamiones[i]->getMarcaCamion(),
                                                                                  listaDeCamiones[i]->getModeloCamion(),
                                                                                  listaDeCamiones[i]->getColorCamion());
                 temporalCantidadDeCamiones++;
               }//Fin if
             }//Fin for
 
             //Limpio la memoria usada por el arreglo principal de camiones
             for(int i=0; i<cantidadDeCamiones; i++)
             {
               listaDeCamiones[i]->~Camion();
             }
 
             //copia el arreglo temporal al principio
             cantidadDeCamiones = 0;
             for(int i =0; i<temporalCantidadDeCamiones; i++)
             {
                listaDeCamiones[cantidadDeCamiones] = new Camion(temporalListaDeCamiones[i]->getIdCamion(),
                                                                 temporalListaDeCamiones[i]->getCilindrajeCamion(),
                                                                 temporalListaDeCamiones[i]->getnPuertasCamion(),
                                                                 temporalListaDeCamiones[i]->getanioCamion(),
                                                                 temporalListaDeCamiones[i]->getPrecioCamion(),
                                                                 temporalListaDeCamiones[i]->getMarcaCamion(),
                                                                 temporalListaDeCamiones[i]->getModeloCamion(),
                                                                 temporalListaDeCamiones[i]->getColorCamion());
                cantidadDeCamiones++;
             }//Fin for
             cout << "El camion fue eliminado exitosamente\n\n";
         }//fin else
     }
 
}
 
//Imprime los datos del Camion
void Camion::imprimir()
{
     cout<<setw(10)<<idCamion<<setw(15)<<modeloCamion<<setw(15)<<marcaCamion<<endl;
}

//.h

Código

#ifndef CAMION_H
#define CAMION_H
#include <iostream>
using namespace std;
// Variables globales
class Camion
{
      private:
      int idCamion;
      int cilindrajeCamion;
      int nPuertasCamion;
      int anioCamion;
      double precioCamion;
      string marcaCamion, modeloCamion, colorCamion;
 
      public:
      Camion(int =0, int=0, int=0, int=0, double = 0.0, string = "", string = "", string = "");
 
      void setIdCamion(int);
      void setCilindrajeCamion(int);
      void setNPuertasCamion(int);
      void setanioCamion(int);
      void setPrecioCamion(double);
      void setMarcaCamion(string);
      void setModeloCamion(string);
      void setColorCamion(string);
 
      int getIdCamion() const;
      int getCilindrajeCamion() const;
      int getnPuertasCamion() const;
      int getanioCamion() const;
      string getMarcaCamion() const;
      string getModeloCamion() const;
      string getColorCamion() const;
      double getPrecioCamion() const;
 
      //Otros Metodos
      void ingresarCamion();
      void flistaDeCamiones();
      void imprimir();
      int buscarCamion(int);
      void modificarCamion();
      void eliminarCamion();
 
      private:
      Camion**listaDeCamiones;
      int Maximo;
      int cantidadDeCamiones;
 
};
#endif

//principal

Código

#include <cstdlib>
#include <iostream>
#include "Camion.h"
 
using namespace std;
 
void menuCamiones();
 
int main(int argc, char *argv[])
{    
     menuCamiones();
    system("PAUSE");
    return EXIT_SUCCESS;
}
 
void menuCamiones()
{
     Camion camionOb;
     int opcion;
     do
     {
         system("cls");
         cout << "Menu\n"
              << "1.Desea ingresar un nuevo camion a la empresa\n"
              << "2. mostrar lista de Camiones\n";
         cout << "Elija una opcion: ";
         cin >> opcion;
 
         switch(opcion)
         {
         case 1:
              system("cls");
              camionOb.ingresarCamion();
              system("pause");
              break;
 
          case 2:
               system("cls");
               camionOb.flistaDeCamiones();
               system("pause");
          break;         
 
          case 3:
          break; 
 
          default:
                  cout << "Esa opcion no existe.\n\n";
                  break;
         }
 
 
     }while(opcion != 3);
}

Gracias.

Título: Re: El for no me hace su funcion?
Publicado por: Yoel Alejandro en 5 Febrero 2014, 00:02 am

Hola nolasco, acabo de revisar tu código. En rasgos generales, pero tienes como un pequeño problema de organización. Fíjate que estás guardando la cantidad de camiones como un miembro de cada objeto (o "instancia") de la clase Camion. Eso es absurdo. Imagínate que creas un primer objeto Camion, entonces su campo cantidadDeCamiones tiene el valor de 1, porque hasta el momento has creado un solito objeto de dicha clase.

Ahora creas otro camión. El primero tiene el campo cantidadDeCamiones = 1, el segundo tiene cantidadDeCamiones = 2, porque (a menos que yo haya leído mal el código) tú no actualizas el campo cantidadDeCamiones en cada uno de los objetos anteriormente creados de la clase Camion (¿verdad?). Y creo que precisamente por eso es que te imprime uno solo, porque la primera instancia creada tiene el valor cantidad de camiones en "1".

En todo caso, hacer esto de esa manera tampoco sería lo ideal. Yo te recomendaría definir una clase aparte listaCamiones con dos únicos campos: Uno un arreglo de camiones y otro un entero indicando la cantidad de camiones creados hasta el momento:

Código:

class listaCamiones {
public:
    Camion Camiones[ MAXIMO ];
    int N_Camiones;
}

La constante MAXIMO (que tú quieres sea 100) yo te recomiendo la definas como una macro de cabecera:

Código:

#define MAXIMO 100

ya que es un valor que se supone debe permanecer constante durante todo el programa (si la pones como variable int, ¿te imaginas qué pasaría si por error cambias su valor durante el programa?)

Los métodos ingresarCamion(), flistaDeCamiones(), imprimir(), buscarCamion(int), modificarCamion(), eliminarCamion() serían de la clase listaCamiones, y no de la clase individual Camion (¿vas captando la idea?). Al añadir un nuevo camión puedes hacer algo como esto:

Código:

void listaCamiones :: ingresarCamion( ) {
    
    /* aquí pides los datos para el nuevo camión */

    /* creas una instancia de Camion */
    Camion new_Camion = new Camion( idCamion, cilindrajeCamion,
        nPuertasCamion, anioCamion,
        precioCamion, marcaCamion,
        modeloCamion,colorCamion);

    /* y la añades al arreglo, actualizando el contador */
    Camiones[ N_Camiones++ ] = new_Camion;
}

Ahora debes definir la función de imprimir lista como un método de la clase listaCamiones, y ya debería imprimir todos los camiones.
En fin, sólo hace falta una pequeña reingeniería del software (mover unas cosas de aquí para allá) y debería estar funcionando bien.

Saludos, y cualquier cosa comenta, .... :) Yoel.

Título: Re: El for no me hace su funcion?
Publicado por: nolasco281 en 5 Febrero 2014, 02:31 am

Hola yoel_alejandro gracias por contestar ahora lo leeo

Título: Re: El for no me hace su funcion?
Publicado por: nolasco281 en 5 Febrero 2014, 03:59 am

Hola ya cree la clase y le paso los datos pero ahora me dice.
lamento preguntar tanto. pero me quiero arrancar la cabeza.

Hola pense que era por que no habia includio la libreria de string o el usign namespace std; pero no.

y se que hace referencia al constructor de la clase camion pero veo que le mando los parametros o argumentos correctos.

Ya note donde está el problema es que no estoy apuntando a nada. Según leí Cuando se usa el operador new se tiene que usar apuntadores a fuerza. Y ese es el problema espero resolverlo pronto.

(http://1.bp.blogspot.com/-ruTXNqfWrV0/UvGp9MKSWCI/AAAAAAAAAfM/uGZT2wzLyWY/s1600/sdf.png)

Código

#ifndef LISTACAMIONES_H
#define LISTACAMIONES_H
#include "Camion.h"
#define Maximo 100
 
class listaCamiones
{
      Camion camiones[Maximo];
      int nCamiones;
 
      //Ingresa un nuevo camion
void listaCamiones::ingresarCamion()
{
     int idCamion, anioCamion, cilindrajeCamion, nPuertasCamion;
     double precioCamion;
     string colorCamion, marcaCamion, modeloCamion; 
    //listaDeCamiones = new Camion*[Maximo];// arreglo de 100
 
   if (nCamiones < Maximo)
   { 
     system("cls");
     cout << "Ingrese los datos del camion\n";
     cout << "Ingrese el ID del camion: ";
     cin  >> idCamion;
 
     cout << "Ingrese la Marca del Camion: ";
     cin  >> marcaCamion;
 
     cout << "Ingrese el Modelo del camion: ";
     cin  >> modeloCamion;
 
     cout << "Ingrese el año del Camion: ";
     cin  >> anioCamion;
 
     cout << "Ingrese el cilindraje del camion: ";
     cin  >> cilindrajeCamion;
 
     cout << "Ingrese el numero de puertas: ";
     cin  >>  nPuertasCamion;
 
     cout << "Ingrese el color del camion: ";
     cin  >> colorCamion;
 
     cout << "Ingrese el presio del camion: ";
     cin  >> precioCamion;
 
     Camion nuevoCamion = new Camion(idCamion, cilindrajeCamion, nPuertasCamion, 
                                     anioCamion, precioCamion, marcaCamion, modeloCamion,
                                     colorCamion);
 
     camiones [nCamiones++] = nuevoCamion;
 
     cout << endl <<"Los datos del camion fueron ingresados correctamente\n";
   }
   else
   {
       system("cls");
       cout << "Ya no hay espacio disponible para mas camiones\n"; 
   }
}
 
};
 
#endif

Título: Re: El for no me hace su funcion?
Publicado por: nolasco281 en 5 Febrero 2014, 07:44 am

Código

 Camion new_Camion = new Camion( idCamion, cilindrajeCamion,
        nPuertasCamion, anioCamion,
        precioCamion, marcaCamion,
        modeloCamion,colorCamion);
 
    /* y la añades al arreglo, actualizando el contador */
    Camiones[ N_Camiones++ ] = new_Camion;

no se, si se le olvido a yoel_alejandro asignar el apuntador hice esto.
pero el arreglo a la hora ingresar un registro del camion entra de un solo al else y me dice que ya no hay espacio para mas camiones :-(

Código

 Camion *nuevoCamion = new Camion(idCam, cilindrajeCam,   
                             nPuertasCam,  anioCam,  
                             precioCam, marcaCam, 
                             modeloCam, colorCam);
 
Camiones[nCamiones++] = *nuevoCamion;

Título: Re: El for no me hace su funcion?
Publicado por: eferion en 5 Febrero 2014, 08:53 am

Código

 Camion(int =0, int=0, int=0, int=0, double = 0.0, string = "", string = "", string = "");

Esta declaración será broma, no???

No te recomiendo crear, de buenas a primeras, funciones que usen más de 4 argumentos porque es poco usable... si cometes el error de invertir dos argumentos te vas a dar de cabezazos hasta que des con el problema.

Además es altamente recomendable que en la declaración de las funciones aparezcan los nombres de los argumentos... normalmente cuando estés programando mirarás el archivo de cabecera para recordar el uso de las funciones, ya que suelen ser más sencillos de consultar... si tienen 20 argumentos y encima no están etiquetados te quedas sin esta opción.

Por otro lado, otra sugerencia, deberías eliminar el using namespace std de los archivos de cabecera. Usar espacios de nombres en los archivos de cabecera es una mala práctica porque obliga a arrastrar ese uso a todos los archivos que incluyan esa cabecera... y en algunos casos puede ser contraproducente... por ejemplo si dos funciones o clases se llaman igual y están en espacios de nombres diferentes.

Código

namespace N1
{
  void func( );
}
 
namespace N2
{
  void func( );
}
 
using namespace N1;
using namespace N2;
 
void main( )
{
  // a que func estamos llamando??
  func( );
}

Más cosillas, si estás usando C++... por qué no usas los contenedores para almacenar la lista de camiones?? Son más potentes, versátiles y seguros que los arreglos.

* Potentes porque ofrecen una interfaz... los arreglos no.

* Versátiles porque se adaptan perfectamente a las necesidades de cada programa, solo hay que elegir correctamente el contenedor a utilizar.

* Seguros porque previenen desbordamientos y otros fallos típicos del uso de memoria dinámica.

Y además son muy sencillos de usar:

Código

std::vector< Camion* > Camiones;
Camiones.push_back( new_camion );
 
Camion* camion = Camiones[ 0 ];

Cita de: nolasco281 en 5 Febrero 2014, 07:44 am

Código

 Camion *nuevoCamion = new Camion(idCam, cilindrajeCam,   
                             nPuertasCam,  anioCam,  
                             precioCam, marcaCam, 
                             modeloCam, colorCam);
 
Camiones[nCamiones++] = *nuevoCamion;

Si "Camiones" es un arreglo de punteros... "*nuevoCamion" debería ser "nuevoCamion", ya que con el asterisco pasas a usar una variable estática, no un puntero.

Además no has inicializado la variable nCamiones... por lo que ésta pasa a tener un valor aleatorio y, tirando un poco de estadística... si un int ocupa 32 bits y lo llenamos con basura... ¿que posibilidades hay de que el número resultante esté entre cero y 100?? hago constar que con 32 bits el rango de valores va de –2.147.483.648 a 2.147.483.647.

Sin embargo, sacando a relucir el tema del contenedor, el código podría quedar tal que

Código

std::vector< Camion* > camiones;
 
// ...
 
if ( camiones.size( ) < Maximo )
{
  // ...
 
  camiones.push_back( new_camion );
}
else
{
  // ...
}

Título: Re: El for no me hace su funcion?
Publicado por: nolasco281 en 5 Febrero 2014, 09:19 am

Gracias primero que nada

Cita de: eferion en 5 Febrero 2014, 08:53 am

Además es altamente recomendable que en la declaración de las funciones aparezcan los nombres de los argumentos.

tipo nomFuncion(int); //Yo.
tipo nomFuncion(int argumento) //Si no me equivoco esto es lo que me decís pueda que se la costumbre y por eso no lo hago. Pero lo adoptare. Gracias por eso también.

Cita de: eferion en 5 Febrero 2014, 08:53 am

Además es altamente recomendable que en la declaración de las funciones aparezcan los nombres de los argumentos... normalmente cuando estés programando mirarás el archivo de cabecera para recordar el uso de las funciones, ya que suelen ser más sencillos de consultar... si tienen 20 argumentos y encima no están etiquetados te quedas sin esta opción.

Gracias por este consejo siempre etiqueto los métodos lo hago al final pero creo que tienes razón es mejor hacerlo desde un principio.

Cita de: eferion en 5 Febrero 2014, 08:53 am

Por otro lado, otra sugerencia, deberías eliminar el using namespace std de los archivos de cabecera

Totalmente de acuerdo solo que a veces no se cual el problema y me pongo tonto.

Ahorita pruebo he intento todo lo que me has comentado.

Se agradece mucho a todos los que comentan y reitero que se toman su tiempo para ayudarme y además te aconsejan en la buena práctica.

Probando y comento como me va si lo logro.

PD:Me regresaste a la vida loco estaba por, darme, por vencido.

Título: Re: El for no me hace su funcion?
Publicado por: nolasco281 en 5 Febrero 2014, 11:06 am

Hola Muchas gracias a todos, me da hasta un poco de pena molestarlos.
Y antes que nada entiendo si ya no quieren contestar.
Pero como pueden ver no soy muy bueno en C++ y quiero aprenderlo.

Creeo que esta es la parte fea de C++ ;D

Tengo una duda no se si es por que no he dormido

En Camion.cpp tengo el maximo ya lo quite ya que lo defini en la clase listasCamiones.cpp.

Código

Maximo =100;
cantidadDeCamiones =0;

Y en Camion.h eso está bien como lo he modificado tanto que así lo tenía cuando funcionaba pero no de la forma correcta pero ahora que estoy hice clase listaCamiones no sé si sean necesarias.

Código

private:
     Camion**listaDeCamiones;
     //int Maximo;
     int cantidadDeCamiones;

Si se fijan estoy probando tanto vectores me señalo eferion y con arreglos por que se, de que de las dos formas es impórtate hacerlo.

Y este es el error con arreglo de punteros. Que es el que mencionaba de la forma de yoel_alejandro.

(http://2.bp.blogspot.com/-vVsPB85wCIk/UvIOY_Ewa7I/AAAAAAAAAfw/HYn6qs2XbNo/s1600/Arreglos1.png)

Y este es la forma en que me señala eferion.
(http://3.bp.blogspot.com/-oGvU850SAoA/UvIKZu1DErI/AAAAAAAAAfo/a3VOYoVFbgU/s1600/vector.png)

Título: Re: El for no me hace su funcion?
Publicado por: eferion en 5 Febrero 2014, 11:30 am

Cita de: nolasco281 en 5 Febrero 2014, 11:06 am

En Camion.cpp tengo el maximo ya lo quite ya que lo defini en la clase listasCamiones.cpp.

Código

Maximo =100;
cantidadDeCamiones =0;

Y en Camion.h eso está bien como lo he modificado tanto que así lo tenía cuando funcionaba pero no de la forma correcta pero ahora que estoy hice clase listaCamiones no sé si sean necesarias.

Código

private:
     Camion**listaDeCamiones;
     //int Maximo;
     int cantidadDeCamiones;

Prueba a poner la declaración completa de las clases... con lo que has puesto no me hago una idea de cómo ha quedado.

Cita de: nolasco281 en 5 Febrero 2014, 11:06 am

Y este es el error con arreglo de punteros. Que es el que mencionaba de la forma de yoel_alejandro.

...

Y este es la forma en que me señala eferion.

...

Código

Camion nuevoCamion = new Camion( ... );

Ese código está mal. new sirve para crear objetos dinámicos; y los objetos dinámicos tienen que gestionarse usando punteros:

Código

Camion *nuevoCamion = new Camion( ... );

Además, estás redefiniendo la misma variable varias veces

Código

Camion nuevoCamion = new Camion( ... );
Camion nuevoCamion = new Camion( ... );
Camion nuevoCamion = new Camion( ... );

Cada una de esas variables tiene que tener un nombre diferente, al menos si están en el mismo ámbito, lo que sucede en tu caso:

Código

Camion *nuevoCamion1 = new Camion( ... );
Camion *nuevoCamion2 = new Camion( ... );
Camion *nuevoCamion3 = new Camion( ... );

Y tienes que aprender a leer e interpretar los mensajes de error... el último te está diciendo que la varible "nuevoCamion" no existe... si te fijas, en la línea anterior a la instancia de la clase la has llamado "cammion", no "nuevoCamion".

Nota final... ten siempre presente que en tu código ha de haber un delete por cada new que escribas... si no vas a acabar con lagunas de memoria... la memoria reservada por un new no se libera por sí misma.

Título: Re: El for no me hace su funcion?
Publicado por: nolasco281 en 5 Febrero 2014, 11:52 am

El de vectores ya esta, si funciono y me confundi en esta parte por eso no sabia donde declarar esa varible Gracias.

Cita de: eferion en 5 Febrero 2014, 08:53 am

std::vector< Camion* > Camiones;
Camiones.push_back( new_camion );

Camion* camion = Camiones[ 0 ]; //1.esta parte

Cita de: eferion en 5 Febrero 2014, 08:53 am

std::vector< Camion* > camiones;

   // ...

   if ( camiones.size( ) < Maximo )
   {
   // ...

   camiones.push_back( new_camion ); //2.estaparte
   }
   else
   {
   // ...
   }

//Codigo arreglo de punteros
//Camiones.h

Código

#ifndef CAMION_H
#define CAMION_H
#include <iostream>
using namespace std;
// Variables globales si la hay
 
class Camion
{
      //Variables Privadas
      private:
      int idCamion;
      int cilindrajeCamion;
      int nPuertasCamion;
      int anioCamion;
      double precioCamion;
      string marcaCamion, modeloCamion, colorCamion;
 
 
      public:
 
      //Constructores.
      Camion(int =0, int=0, int=0, int=0);
      Camion(double =0.0);
      Camion(string = "", string = "", string = "");
 
      //Funciones o metodos publicos 
      void setIdCamion(int);         //Devuelve el Id del Camion.
      void setCilindrajeCamion(int); //Devuelve el cilindrage del camion
      void setNPuertasCamion(int);   //Devuelve el numero de puertas del camion
      void setanioCamion(int);       //Devuelve el año del camion
      void setPrecioCamion(double);  //Devuelve el precio del camion
      void setMarcaCamion(string);   //Devuelve la marca del camion
      void setModeloCamion(string);  //Devuelve el modelo del camion
      void setColorCamion(string);   //Devuelve el color del camion
 
      int getIdCamion() const;         //Obtiene Id del Camion.
      int getCilindrajeCamion() const; //Obtiene el cilindrage del camion
      int getnPuertasCamion() const;   //Obtiene numero de puertas del camion
      int getanioCamion() const;       //Obtiene el año del camion
      double getPrecioCamion() const;  //Obtiene el precio del camion
      string getMarcaCamion() const;   //Obtiene la marca del camion
      string getModeloCamion() const;  //Obtiene el modelo del camion
      string getColorCamion() const;   //Obtiene el color del camion
 
 
      //Otros Metodos
      void ingresarCamion();
      void flistaDeCamiones();
      void imprimir();
      int buscarCamion(int);
      void modificarCamion();
      void eliminarCamion();
 
      private:
      Camion**listaDeCamiones;
      int cantidadDeCamiones;
 
};
#endif

//Camion.cpp

Código

#include "Camion.h"
#include <iostream>
#include <iomanip>
#include <string>
 
//Constructores valores enteros
Camion::Camion (int idCam, int cilindrajeCam, int puertasCam, int anioCam)
{
  idCamion = idCam;
  cilindrajeCamion = cilindrajeCam;
  nPuertasCamion = puertasCam;
  anioCamion = anioCam;
  cantidadDeCamiones =0;
}
 
//Constructores valores double
Camion::Camion(double precioCam)
{
  precioCamion = precioCam;
}
 
//Constructores valores string
Camion::Camion(string marcaCam, string modeloCam,string colorCam)
{
   marcaCamion = marcaCam;
   modeloCamion = modeloCam;
   colorCamion = colorCam;
}
 
//Funciones set
void Camion::setIdCamion(int idCam)
{
     idCamion = idCam;
}
 
void Camion::setCilindrajeCamion(int cilindrajeCam)
{
     cilindrajeCamion = cilindrajeCam;
}
 
void Camion::setNPuertasCamion(int puertasCam)
{
     nPuertasCamion = puertasCam;
}
 
void Camion::setanioCamion(int anioCam)
{
     anioCamion = anioCam;
}
 
void Camion::setPrecioCamion(double precioCam)
{
     precioCamion = precioCam;
}
 
void Camion::setMarcaCamion(string marcaCam)
{
     marcaCamion = marcaCam;
}
 
void Camion::setModeloCamion(string modeloCam)
{
     modeloCamion = modeloCam;
}
 
void Camion::setColorCamion(string colorCam)
{
     colorCamion = colorCam;
}
 
//Funciones get
int Camion::getIdCamion() const
{
    return idCamion;
}
 
int Camion::getCilindrajeCamion() const
{
    return cilindrajeCamion;
}
 
int Camion::getnPuertasCamion() const
{
    return nPuertasCamion;
}
 
int Camion::getanioCamion()const
{
    return anioCamion;
}
 
double Camion::getPrecioCamion() const
{
       return precioCamion;
}
 
string Camion::getMarcaCamion() const
{
       return marcaCamion;
}
 
string Camion::getModeloCamion() const
{
       return modeloCamion;
}
 
string Camion::getColorCamion() const
{
       return colorCamion;
}

//ListaCamiones.cpp

Código

#ifndef LISTADECAMIONES_H 
#define LISTADECAMIONES_H
#include "Camion.h"
#define MAXIMO 100
 
class listaDeCamiones
{
      //Variables para esta clase
      Camion Camiones[MAXIMO];
      int nCamiones;
 
      void listaDeCamiones::ingresarCamion()
      {
           int idCam, cilindrajeCam, nPuertasCam, anioCam;
           string marcaCam, modeloCam, colorCam;
           double precioCam;
 
           nCamiones =0; //Inicializo variable nCamiones.
 
           //If No sobre pase la cantidad de camiones 
           if (nCamiones < MAXIMO)
           {
            system("cls");
            cout << "Ingrese los datos del camion\n";
            cout << "Ingrese el ID del camion: ";
            cin  >> idCam;
 
            cout << "Ingrese el cilindraje del camion: ";
            cin  >> cilindrajeCam;
 
            cout << "Ingrese el numero de puertas: ";
            cin  >>  nPuertasCam;
 
            cout << "Ingrese el año del Camion: ";
            cin  >> anioCam;
 
            cout << "Ingrese el presio del camion: ";
            cin  >> precioCam;
 
            cout << "Ingrese la Marca del Camion: ";
            cin  >> marcaCam;
 
            cout << "Ingrese el Modelo del camion: ";
            cin  >> modeloCam;
 
            cout << "Ingrese el color del camion: ";
            cin  >> colorCam;
 
            //Instancia de camion y separado por cada constructo     
            Camion *nuevoCamion1 = new Camion(idCam, cilindrajeCam, nPuertasCam, anioCam);
            Camion *nuevoCamion2 = new Camion(precioCam);
            Camion *nuevoCamion3 = new Camion(marcaCam, modeloCam, colorCam);
 
            //Añade al arreglo y lo actualiza                                         
            Camiones[nCamiones++] = new_nuevoCamion;
            cout << endl <<"Los datos del camion fueron ingresados correctamente\n";
            }
 
        else
        {
         system("cls");
         cout << "Ya no hay espacio disponible para mas camiones\n";
        }
     }
};
 
#endif

//Esta parte ya la había probado y estas en lo correcto no tira error de esa forma.

Código

 Camion *nuevoCamion1 = new Camion(idCam, cilindrajeCam, nPuertasCam, anioCam);
            Camion *nuevoCamion2 = new Camion(precioCam);
            Camion *nuevoCamion3 = new Camion(marcaCam, modeloCam, colorCam);

//Ahora el error es acá. Entonces eso quiere decir que ahora tendré que crear un arreglo y que lo actualice de acuerdo a sus argumentos.

Código

 Camiones[nCamiones++] = new_nuevoCamion1;
Camiones[nCamiones++] = new_nuevoCamion2;
Camiones[nCamiones++] = new_nuevoCamion3;

//En esta parte tambien pense lo mismo y era lo que preguntaba mas arriba que si //era asi que es lo que tu mecomentas.

Código

Camion nuevoCamion* = new Camion(idCam, cilindrajeCam, nPuertasCam, anioCam);

En cuanto a la liberación de memoria tengo estas funciones y hay una donde lo hace lo tengo en mente pero no puedo implementarlas si no puedo hacer la primera función que es la de ingresarlos porque estas irían en listaCamiones.Y tengo claro que cambiaran por supuesto y que tengo que arreglarlos también.

Código

//Muestra los datos del camion
void Camion::flistaDeCamiones()
{
     if (cantidadDeCamiones==0)
     {
       system("cls");
       cout<<"No se a registrado ninguna Camion todavía"<<endl;
     }
       else
       {
           cout<<"..:::Lista de Camiones:::.."<<endl<<endl;
           cout << setw(10)<<"Id Camion"<<setw(15)<<"Modelo"<<setw(15)<<"marca"<<endl;
           cout<<"================================================="<<endl;
 
           //Imprimmir todos los camiones almacenadas en el arreglo
           for (int i=0; i<cantidadDeCamiones; i++)
           {
             listaDeCamiones[i]->imprimir(); //PROBLEMA Y NO LO IDENTIFICO
           }
           cout<<endl<<"*******Fin de la lista **********"<<endl<<endl;
       }
}
//Busca ese camion
int Camion::buscarCamion(int codigo)
{
    for (int i=0; i < cantidadDeCamiones; i++)
    {
        if(listaDeCamiones[i]->getIdCamion() == codigo)
        {
          return i;
        }
    }
    return -1;
}
 
//Modifica ese Camion.
void Camion::modificarCamion()
{
     if (cantidadDeCamiones == 0)
     {
       system("cls");
       cout << "No se ha registrado ningun camion tadavia\n";
     }
     else
     {
         cout << "Modificar Camion\n\n";
         int codigoCamion;
         cout << "Ingrese el codigo del camion: ";
         cin >> codigoCamion;
 
         int ubicacion = buscarCamion(codigoCamion);
         if (ubicacion == -1)
         {
           cout << "Lo sentimos ese codigo del camion no existe\n";
         }
         else
         {
             cout << "El codigo se encontro\n\n";
             cout << setw(10)<<"Id"<<setw(15)<<"Modelo"<<setw(15)<<"marca"<<endl;
             cout<<"================================================="<<endl;
 
             listaDeCamiones[ubicacion]->imprimir();
             cout << "\n\n";
             cout  << "Ingrese los nuevos datos del Camion\n\n";
 
             cout <<  "Marca";
             cin  >> marcaCamion;
 
             cout << "Modelo del camion: ";
             cin  >> modeloCamion;
 
             cout << "Año del Camion: ";
             cin  >> anioCamion;
 
             cout << "Cilindraje del camion: ";
             cin  >> cilindrajeCamion;
 
             cout << "numero de puertas: ";
             cin  >>  nPuertasCamion;
 
             cout << "Color del camion: ";
             cin  >> colorCamion;
 
             cout << "Presio del camion: ";
             cin  >> precioCamion;
 
             listaDeCamiones[ubicacion]->setMarcaCamion(marcaCamion);
             listaDeCamiones[ubicacion]->setModeloCamion(modeloCamion);
             listaDeCamiones[ubicacion]->setanioCamion(anioCamion);
             listaDeCamiones[ubicacion]->setCilindrajeCamion(cilindrajeCamion);
             listaDeCamiones[ubicacion]->setNPuertasCamion(nPuertasCamion);
             listaDeCamiones[ubicacion]->setColorCamion(colorCamion);
             listaDeCamiones[ubicacion]->setPrecioCamion(precioCamion);
 
             cout << "El camion fue modificado Exitosamente.\n";
         }
     }
}
 
void Camion::eliminarCamion()
{
     if (cantidadDeCamiones == 0)
     {
       system("cls");
       cout << "No se ha registrado ningun camion tadavia\n";
     }
     else
     {
         cout << "Eliminar Camion\n\n";
         int codigoCamion;
         cout << "Ingrese el codigo a eliminar: ";
         cin >> codigoCamion;
 
         int ubicacion = buscarCamion(codigoCamion);
         if (ubicacion == -1)
         {
           cout << "Lo sentimos ese codigo del camion no existe\n";
         }
         else
         {
             cout << "El codigo se encontro\n\n";
             cout << setw(10)<<"Id"<<setw(15)<<"Modelo"<<setw(15)<<"marca"<<endl;
             cout<<"================================================="<<endl;
 
             listaDeCamiones[ubicacion]->imprimir();
             cout << "\n\n";
 
             cout << "Presione enter para eliminar ese camion\n\n";
             system("pause");
 
             //Pasa a eliminar usando una copia del arreglo
             Camion**temporalListaDeCamiones = new Camion*[Maximo];
             int temporalCantidadDeCamiones = 0;
 
             //Se copia el arreglo en otro arreglo exepto el que se quiere eliminar
             for(int i =0; i < cantidadDeCamiones; i++)
             {
               if(i!=ubicacion)
               {
                 temporalListaDeCamiones[temporalCantidadDeCamiones] = new Camion(listaDeCamiones[i]->getIdCamion(),
                                                                                  listaDeCamiones[i]->getCilindrajeCamion(),
                                                                                  listaDeCamiones[i]->getnPuertasCamion(),
                                                                                  listaDeCamiones[i]->getanioCamion(),
                                                                                  listaDeCamiones[i]->getPrecioCamion(),
                                                                                  listaDeCamiones[i]->getMarcaCamion(),
                                                                                  listaDeCamiones[i]->getModeloCamion(),
                                                                                  listaDeCamiones[i]->getColorCamion());
                 temporalCantidadDeCamiones++;
               }//Fin if
             }//Fin for
 
             //Limpio la memoria usada por el arreglo principal de camiones
             for(int i=0; i<cantidadDeCamiones; i++)
             {
               listaDeCamiones[i]->~Camion();
             }
 
             //copia el arreglo temporal al principio
             cantidadDeCamiones = 0;
             for(int i =0; i<temporalCantidadDeCamiones; i++)
             {
                listaDeCamiones[cantidadDeCamiones] = new Camion(temporalListaDeCamiones[i]->getIdCamion(),
                                                                 temporalListaDeCamiones[i]->getCilindrajeCamion(),
                                                                 temporalListaDeCamiones[i]->getnPuertasCamion(),
                                                                 temporalListaDeCamiones[i]->getanioCamion(),
                                                                 temporalListaDeCamiones[i]->getPrecioCamion(),
                                                                 temporalListaDeCamiones[i]->getMarcaCamion(),
                                                                 temporalListaDeCamiones[i]->getModeloCamion(),
                                                                 temporalListaDeCamiones[i]->getColorCamion());
                cantidadDeCamiones++;
             }//Fin for
             cout << "El camion fue eliminado exitosamente\n\n";
         }//fin else
     }
 
}
 
//Imprime los datos del Camion
void Camion::imprimir()
{
     cout<<setw(10)<<idCamion<<setw(15)<<modeloCamion<<setw(15)<<marcaCamion<<endl;
}

Título: Re: El for no me hace su funcion?
Publicado por: eferion en 5 Febrero 2014, 13:00 pm

Veo que te bailan un poco los conceptos. vamos por partes.

1. Constructores (parte1)

Al igual que los seres vivos ( con perdón de las reencarnaciones ) solo nacen una vez y en ese proceso ya nacen con todo, los objetos sólo se pueden crear una vez.

Cada vez que tu llamas a un constructor estás creando un objeto nuevo y diferente de los ya existentes.

El siguiente código crea tres camiones diferentes:

Código

//Instancia de camion y separado por cada constructo     
            Camion *nuevoCamion1 = new Camion(idCam, cilindrajeCam, nPuertasCam, anioCam);
            Camion *nuevoCamion2 = new Camion(precioCam);
            Camion *nuevoCamion3 = new Camion(marcaCam, modeloCam, colorCam);

Imagino que tú ahí estás intentando asignar todas las propiedades al mismo elemento... eso se hace con los getters y los setters que has implementado.

Código

Camion* nuevoCamion = new Camion( );
nuevoCamion->setId( idCam );
nuevoCamion->setCilindraje( cilindrajeCam );
nuevoCamion->setNPuertas( nPuertasCam );
// ...

Fíjate de paso que no he dicho "setIdCamion" ni "setCilindrajeCamion"... si la clase se llama camión es redundante y totalmente innecesario que pongas el sufijo "Camion" a todos los miembros de la clase... pero eso lo tratamos más adelante para no desviarnos.

El caso es que quizás es una buena práctica que el constructor de una clase reciba únicamente aquella información que es necesaria desde un primer momento para el buen funcionamiento del código. En tu caso, la clase Camion puede funcionar perfectamente si todos sus elementos tienen valores por defecto, luego con gestionar el constructor por defecto te sobra y te basta.

No tengas miedo a que esta forma de pensar te obligue a tener 10 líneas más de código... créeme si te digo que hay malas prácticas que generan muchas más líneas de código que además suele ser bastante ilegible.

Además, no he podido evitar fijarme en la siguiente línea:

Código

temporalListaDeCamiones[temporalCantidadDeCamiones] = new Camion(listaDeCamiones[i]->getIdCamion(),
                                                                                  listaDeCamiones[i]->getCilindrajeCamion(),
                                                                                  listaDeCamiones[i]->getnPuertasCamion(),
                                                                                  listaDeCamiones[i]->getanioCamion(),
                                                                                  listaDeCamiones[i]->getPrecioCamion(),
                                                                                  listaDeCamiones[i]->getMarcaCamion(),
                                                                                  listaDeCamiones[i]->getModeloCamion(),
                                                                                  listaDeCamiones[i]->getColorCamion());

Resulta que tú aquí estás intentando hacer una copia de un objeto ya existente. Los chicos que diseñaron C++ eran bastante listos y ya pensaron en esta posibilidad y la incorporaron al lenguaje... estamos hablando del constructor copia.

El constructor copia se implementa por defecto en tooooodas las clases de C++... esto quiere decir que mientras la clase en cuestión no utilice memoria dinámica en sus miembros el constructor que se genera por defecto es perfectamente válido... aunque eso no quita que te apetezca implementarlo tu explícitamente para aprender, por optimización, por seguridad o por lo que sea.

El constructor copia, si lo defines explícitamente, debería tener la siguiente firma.

Código

class Camion
{
  public:
    Camion( const Camion& );
};

Y en su implementación deberías realizar la tarea de copiar los datos de un objeto en otro:

Código

Camion::Camion( const Camion& otro )
{
  idCamion = otro.idCamion;
  cilindrajeCamion = otro.cilindrajeCamion;
  // ...
}

Con esto, tu línea puede quedar reducida a algo tan simple como:

Código

temporalListaDeCamiones[temporalCantidadDeCamiones] = new Camion( *listaDeCamiones[i] );

o incluso, si temporalListaDeCamiones fuese un vector, el código quedaría aún más simple:

Código

temporalListaDeCamiones.push_back( new Camion( *listaDeCamiones[ i ] );

Resumiendo, tu clase debería tener el constructor por defecto y, si te apetece implementarlo explícitamente, el constructor copia... el resto de constructores puedes obviarlos:

Código

class Camion
{
  public:
    Camion( );
    Camion( const Camion& otro );
};

2. Constructores (parte 2)

Cuando tú reservas memoria con un new, la memoria que el sistema te da viene "tal cual", es decir, no hay un proceso de limpieza que deje la memoria lista como si fuese a estrenar. Mas bien te encuentras con que la memoria que te ha dado el sistema tiene basura no es sino la información que otra aplicación, en su momento, escribió en esa posición y ahora ya no la usa.

Es tarea del que programa una clase inicializar toda la memoria utilizada por dicha clase. Esto quiere decir, en otras palabras, que has de dar valores por defectos a todas las variables miembros, al menos a aquellas que no sean clases.

El sitio adecuado para hacer esta tarea es en el constructor ( o constructores ). Recuerda que la llamada a un constructor cualquiera, por ejemplo al constructor copia, no implica una llamada al constructor por defecto.

Código

Camion::Camion( )
{
  idCamion = 0;
  cilindrajeCamion = 0;
  // ...
}

Con este trabajo te aseguras que todos los objetos que se creen van a tener valores válidos SIEMPRE, que es el objetivo a conseguir. Un objeto con valores no válidos se vuelve inestable y da problemas, quedas avisado.

3. Clases (parte3)

En este apartado me voy a centrar en la nomenclatura utilizada para los miembros de la clase.

Si bien es algo meramente estético no deja de ser importante para facilitar la lectura del código.

Como te comenté anteriormente no tiene sentido que un método de la clase Camion contenga el sufijo Camion. Si por ejemplo tu dices:

Código

camion->setNPuertas( X );

Se presupone que lo que vas a indicar es el número de puertas del camión... luego el sufijo "Camion" sobra.

Con esta premisa, los setters deberían tener más o menos esta forma:

Código

class Camion
{
  public:
      void setId(int);         //Devuelve el Id del Camion.
      void setCilindraje(int); //Devuelve el cilindrage del camion
      void setNPuertas(int);   //Devuelve el numero de puertas del camion
      void setanio(int);       //Devuelve el año del camion
      void setPrecio(double);  //Devuelve el precio del camion
      void setMarca(string);   //Devuelve la marca del camion
      void setModelo(string);  //Devuelve el modelo del camion
      void setColor(string);   //Devuelve el color del camion
};

Aunque se les podría dar una vuelta de tuerca más.

Si has leído todo hasta aquí ya te habrás enterado de que existe el constructor copia. En tu código actual, cuando tu haces:

Código

camion->setMarca( loquesea );

estás creando una copia de loquesea y esa copia será la que se use dentro de la función... esto supone una carga adicional y absurda, pues se crea un objeto copia con cada llamada.

Es una buena costumbre que las clases se pasen como referencias constantes. De esta forma evitas tener que crear un objeto temporal perfectamente evitable.

El cambio es relativamente sencillo:

Código

class Camion
{
  public:
      void setId(int);         //Devuelve el Id del Camion.
      void setCilindraje(int); //Devuelve el cilindrage del camion
      void setNPuertas(int);   //Devuelve el numero de puertas del camion
      void setanio(int);       //Devuelve el año del camion
      void setPrecio(double);  //Devuelve el precio del camion
      void setMarca( const string& );   //Devuelve la marca del camion
      void setModelo( const string& );  //Devuelve el modelo del camion
      void setColor( const string& );   //Devuelve el color del camion
};

Y además tiene la ventaja de que las llamadas a las funciones son exactamente iguales que antes!!!

Ponerle la etiqueta const indica que el objeto que se pasa como argumento no va a ser modificado... una referencia es similar a un puntero y aquí tienes un ejemplo:

Código

void func( int& numero )
{
  numero = 10;
}
 
void main( )
{
  int num = 0;
 
  std::cout << num << std::endl;
  func( num );
  std::cout << num << std::endl;
}

El resultado de ejecutar este programa es el siguiente:

Código:

0
10

4. Principio de responsabilidad única

Cada clase y cada método que programes debería regirse por este principio, por tu bien.

El principio se basa en que cada función debería dedicarse a una única tarea. Una clase que permite imprimir por pantalla no debería controlar el estado de la tarjeta de sonido, por ejemplo.

En tu caso, la clase Camion tiene dos tareas diferentes:

* Gestiona los datos de un camion
* Gestiona una lista de camiones

Esta parte de tu código en la clase Camion

Código

  private:
      Camion**listaDeCamiones;
      int cantidadDeCamiones;

Debes eliminarla. Si quieres gestionar una lista de camiones, estupendo... pero que no sea la clase Camion la encargada de dicha gestión. Si es necesario que de dicha gestión se encargue una clase, creas una clase nueva.

Otra forma de violar este principio lo encontramos en este otro ejemplo que te vas a encontrar más veces de las que piensas:

Código

class Persona
{
  public:
    void SetDatos( )
    {
      std::cout << "Introduce el nombre: ";
      std::cin >> nombre;
      std::cout << "Introduce su edad: ";
      std::cin >> edad;
    }
 
  private:
    std::string nombre;
    int edad;
};

En este caso, la clase Persona no solo gestiona los datos de una persona, también se encarga de comunicarse con el usuario para pedirle datos... mala solución.

Lo mismo esto te resulta familiar. Tu clase listaDeCamiones infringe este principio.

5. Contenedores

Insisto, si no es un requisito del ejercicio, usa contenedores en vez de arreglos. Salvo que quieras o tengas que practicar, no es necesario reinventar la rueda con cada programa... no es práctico.

Los contenedores te permiten almacenar listas de elementos de formas diversas: ordenados, sin ordenar, sin duplicados, con índice, ...

Además con los contenedores reduces las probabilidades de acceder a posiciones de memoria que no debes, te proporcionan información sobre el número de elementos que tienen, pueden crecer dinámicamente sin que tú tengas que preocuparte por ello...

Como norma general, todo son ventajas.

6. Otras cuestiones

Deberías leerte y reorganizar tu código. La última secuencia de código que has puesto deberían formar parte de la clase listaDeCamiones... ya que es esta clase la que gestiona la lista de camiones.

Por el momento creo que ya es bastante para asimilar, dedica el tiempo necesario a arreglar el código y no intentes ir rápido... las prisas no son buenas.

Título: Re: El for no me hace su funcion?
Publicado por: nolasco281 en 5 Febrero 2014, 18:00 pm

Entendí la mayoría de las cosas que me dijiste, ya lo he leído como 30,35 veces

Y la funcion listaDeCamiones::ingresarCamion() ya la probe y esta bien ya no tengo problema.

Código

Camion* nuevoCamion = new Camion( );
            nuevoCamion->setId(idCam);
            nuevoCamion->setCilindraje(cilindrajeCam);
            nuevoCamion->setNPuertas(nPuertasCam);
            nuevoCamion->setanio(anioCam);
            nuevoCamion->setPrecio(precioCam);
            nuevoCamion->setMarca(marcaCam);
            nuevoCamion->setModelo(modeloCam);
            nuevoCamion->setColor(colorCam);
 
            //Añadiendo los camiones a nCamiones a arreglo y lo actualiza
            //recorar nuevoCamion instancia de Camion                                        
            Camion[nCamiones++] = nuevoCamion;

Y mis ultimas dos dudas.

1. Pero para ir guardando los registros del cada camion. estaria bien asi ya que todavía no lo estoy mostrando solo guardanlos.

Código

Camion[nCamiones++] = nuevoCamion; //Tambien esta donde le mando los argumentos a los metodos para que se observe mejor.

2. Una anotación que me llamo mucho la atención es de la clase persona. Eso me hace pensar, no sé si estoy en lo correcto que debería de crear la de encabezado que gestiona los datos de una persona. y la de implementación que se encarga de comunicarse con el usuario. .

En mi caso sería una para que me gestione los datos de listaDeCamiones la .h y la otra que se comunique con el usuario .cpp

Y no sé si estoy mal lo dices para mantener la robustez de un software y la legibilidad del mismo.

Muchas gracias por su ayuda enserio mil gracias me han ayudado a mejorar en un 100% en cuanto al manejo de código y a las buenas practicas. Siempre las tomo y más cuando te ayudan a agilizar y sobretodo que otras personas lo entiendan.

agradezco a primeramente a eferion y a yoel_alejandro. por tomarce todo ese tiempo para ayudarme y con eso doy por cerrado este tema.

y me dare la tarea de los demas hacerlo yo, ya que no veo mucha dificulta haciendo el primero. tanto con vectores como con arreglo de apuntadores. para entender este tema mejor.

Y gracias de nuevo un saludo a todos.

Título: Re: El for no me hace su funcion?
Publicado por: eferion en 6 Febrero 2014, 08:22 am

Cita de: nolasco281 en 5 Febrero 2014, 18:00 pm

1. Pero para ir guardando los registros del cada camion. estaria bien asi ya que todavía no lo estoy mostrando solo guardanlos.

Código

Camion[nCamiones++] = nuevoCamion; //Tambien esta donde le mando los argumentos a los metodos para que se observe mejor.

Si tú en tu código tienes algo tal que:

Código

class Camion ...

Ni puedes ni debes usar la palabra Camion para identificar un array, vector o arreglo de camiones.

Que haya nombres que colisionen ( es decir, que sean iguales ) y sirvan para cosas diferentes causa confusión a la hora de revisar el código y de encontrar errores.

Además, lo de nCamiones++... eso solo es necesario si usas arreglos al estilo de C, con contenedores no suele ser necesaria esa variable.

Cita de: nolasco281 en 5 Febrero 2014, 18:00 pm

A lo que me refería con el ejemplo de la clase Persona es que hay que separar actividades que no tienen nada que ver entre sí; me explico:

Tu diseñas una clase, por ejemplo una llamada "Camion"... y le pones un método para crear instancias de esta clase que le pide al usuario vía consola los datos requeridos para su creación. Pasa el tiempo y te llega un encargo para que ahora pueda funcionar con una interfaz de ventanas... ¿que haces en este caso? Al final te toca modificar una clase que ya funciona debido a un mal diseño.

Código

// Diseño malo
 
class Camion
{
  public:
    Camion( );
    Camion( const Camion& );
    virtual ~Camion( );
 
    void SetId( int id );
    void SetMatricula( const std::string& matricula );
 
    int GetId( ) const;
    std::string GetMatricula( ) const;
 
   void PideDatos( ); // funcion conflictiva
};

En cambio imagínate que en vez de optar por ese diseño "rápido" y problemático, sacas de "Camion" la función que pide los datos al usuario y la dejas en una clase independiente... en este caso para cambiar la interfaz con el usuario no hace falta tocar para nada una clase que funciona como es el caso de Camion. Y no solo eso, si además has diseñado la clase con cierto cuidado puedes hasta hacer uso de polimorfismo para que tu código pueda funcionar con varias interfaces diferentes sin tener siquiera que recompilar.

Código

// Diseño bueno
 
class Camion
{
  public:
    Camion( );
    Camion( const Camion& );
    virtual ~Camion( );
 
    void SetId( int id );
    void SetMatricula( const std::string& matricula );
 
    int GetId( ) const;
    std::string GetMatricula( ) const;
};
 
class Interfaz
{
  public:
    // Funcion virtual pura, esta clase sirve para poder hacer polimorfismo
    virtual Camion* PideDatos( ) = 0;
};
 
class InterfazDOS
  : public Interfaz
{
  public:
    // Este metodo contiene la logica necesaria para comunicarse con el usuario via consola
    virtual Camion* PideDatos( ) override;
};
 
class InterfazWindows
  : public Interfaz
{
  public:
    // Este metodo contiene la logica necesaria para comunicarse con el usuario via windows
    virtual Camion* PideDatos( ) override;
};
 
Interfaz ObtenerInterfazUsuario( )
{
  // Estas directivas de precompilador se pueden cambiar por variables o lo que sea
  // a gusto del consumidor
  #ifdef _WIN32
    return new InterfazWindows( );
  #else
    return new InterfazDOS( );
  #endif
}
 
void main( )
{
  Interfaz* interfaz = ObtenerInterfazUsuario( );
 
  // Le pedimos al usuario los datos del camion y nos da igual si
  // lo hace a traves de una ventana de Windows o de una consola.
  Camion* camion = interfaz->PideDatos( );
}

Espero que con el ejemplo haya quedado este tema un poco más claro.

Un saludo.

Título: Re: El for no me hace su funcion? (Solucionado)
Publicado por: nolasco281 en 7 Febrero 2014, 02:36 am

Aclarado cuando termine el programa lo comparto para quien le sirva. Y si es un poco confuso pero hai voy saludos y gracias a todos.

Foro de elhacker.net

Programación => Programación C/C++ => Mensaje iniciado por: nolasco281 en 4 Febrero 2014, 21:57 pm