[SOLUCIONADO] Segmentation Fault.

[PercevalCiro]

Hola,
Desde ya muchas gracias por su atención. Estoy empezando en C++, ya que nunca había programado en este lenguaje.

cree una clase, que la resumo, pero es algo así:

Código C++:

#ifndef INIDICOM_H
#define INIDICOM_H
 
 
using namespace std;
#include <string>
using std::string;
#include <cstring>
 
class iniDicom
{
 
 
    public:
        iniDicom();
        virtual ~iniDicom();
 
        char* getPreamble();
        char* getPrefix();
        char* getGroup_number();
 
         bool setString(char* value_var);
         bool isDicom();
         int  countLength();
 
private:
        char* preamble[128];
        char* prefix[4];     
        char* group_number[2];
 
 
          void asingValues(char* value_char, int iniNumber, char* value_var);
};
#endif // INIDICOM_H
 

Y tengo la implementación:

Código C++:

iniDicom::iniDicom(){
   }
 
iniDicom::~iniDicom(){
}
/*getPreamble() : string
*/
char* iniDicom::getPreamble(){
        return *preamble;
}
/*getPrefix() : string
*/
char* iniDicom::getPrefix(){
        return *prefix;
}
 
 
 char* iniDicom::getGroup_number(){
        return *group_number;
  }
 
 
 
bool iniDicom::setString(char* value_var){
    asingValues(*preamble, 0, value_var);
 
    return true;
}
 
 
void iniDicom::asingValues(char* value_char, int iniNumber, char* value_var){
    int i, largo = strlen(value_char);
 
    for(i=0;i<largo;i++){
        value_char[i] = value_var[iniNumber+i] ;
    }
 
}
 
 
 
 
 
 
int iniDicom::countLength(){
 
    //preamble es igual a 128---> se reemplaza.
    int tmp = 128 + sizeof(*prefix) + sizeof(*group_number)+ sizeof(*element_number);
    tmp += sizeof(*value_representation) + sizeof(*value_length) + sizeof(*value);
    return tmp;
 
    }
 
 
bool iniDicom::isDicom(){
   if(((string)*prefix)=="DICM"){
        return true;
   }else{
        return false;
   }
 
}
 

el problema es que cuando corro me sale "Segmentation Fault". La verdad es que no se como asignar, mi idea era, obtener de un archivo una cantidad de caracteres, meterlos en un char y después dividirlo en cada uno de los atributos. Cual creen que es la mejor forma de hacerlo?. Muchas Gracias y Saludos.

[Malenko]

Tengo una duda... porque programando con C++ (y no con C) estás usando char* como cadenas en lugar de usar el objeto string?

Aparte de esa pregunta... en las cadenas que usas, has reservado la memoria antes de acceder a sus posiciones?

[PercevalCiro]

En principio muchas gracias por tu respuesta Malenko.

En realidad dude si hacerlo con string o con char, al principio quería usar string pero después considere que quedaba mas prolijo trabajar con espacios de memoria reservado y pensé que seria un poco mas eficiente trabajar de esa manera; la verdad todavía lo estoy dudando.

Te consulto, ¿la asignación de los espacios de memoria los debo hacer en forma posterior?

Saludos.

[carbon]

La asignación de memoria la debes hacer antes de asignarle un valor, usando malloc(). Pero en C++ directamente usa la clase string y te olvidas de la memoria!

[Malenko]

Como te han comentado, no tiene sentido usar C++ solo para tener clases y no usar el resto de potencial.

Yo optaría por usar la clase string, pero además has de vigilar los bugs al acceder a posiciones de array sin antes saber si existen:

Código C++:

void iniDicom::asingValues(char* value_char, int iniNumber, char* value_var){
    int i, largo = strlen(value_char);
 
    for(i=0;i<largo;i++){
        value_char[i] = value_var[iniNumber+i] ;
    }
 
}
 

Esta función acepta 2 arrays que pueden ser de diferente tamaño y un entero. En el bucle for solo controlar por la longitud de la cadena "value_char" pero si "iniNumber+1" es superior a "strlen(value_var)" te dará un error por "Segmentation fault" o similares.

Por otro lado, en caso de querer continuar usando punteros y cadenas mediante char*, recuerda que tendrías que reservar el tamaño (con malloc) en el constructor de iniDicom

[PercevalCiro]

carbon y Malenko,
Entendí perfectamente. Muchas Gracias.
Salduos.