C / C++ / MFC

coder21k wrote:

What is the best way to track/troubleshoot the 'crash' errors like NULL pointers, memory crashes, etc? As the application is a huge code base, I am looking for one smart way to track the application crashes.

Chat in Europe Now with 24% more Twitter

void ftoa(char *buf, float f, int p)
{
  int i, n;
  unsigned long l
  
  for (n=0; n<p; n++)
    f *= 10;

  l = (long) (f + .5);

  for (n=0; l>0; n++, l/=10)
  {
    if (n==p)
    {
      buf[n]='.';
      n++;
    }
    buf[n]= (l % 10) + '0';
  }

  for(i=0; i<n/2;i++)
  {
    char c;
    c = buf[i];
    buf[i] = buf[n-1-i];
    buf[n-1-i]= c;
  }

  buf[n]='\0';
}

float tmpVALUE=0;
int n=0;


void setup(){
   SensorEvent.setBoardMode(SENS_ON);
 
  XBee.setMode(XBEE_ON);
  delay(50);
  USB.begin();
  USB.println("USB port started...");
  USB.close();
  
  // Powering BT
  BT.ON();
  
  if( BT.flag & BT_ERROR_ON ) USB.println("Error, 'ROK' not received");
  else USB.println("BT module ready...");
  
  // Init
  if(!BT.init()) USB.println("Init OK");
}

void loop()
{
    float tmpVALUE=0;    //necessary to place it here to get temp reading
    char dataToSend[100];
    char converttmpVALUE=0;
    
    
    
    
    
    
  // Creating a connection to a device
  if(!BT.createConnection("0003190D102D","02"))
  {
    USB.println("");
    USB.print("--- CONNECTION -- MTU: ");
    USB.print(BT.connection_mtu[0],BYTE);
    USB.print(BT.connection_mtu[1],BYTE);
    USB.println(BT.connection_mtu[2],BYTE);    
  }
  else USB.println("Connection failed");
  
  
  tmpVALUE = SensorEvent.readValue(SENS_SOCKET5);
  
  if(tmpVALUE>0.5)
  {
    tmpVALUE=((tmpVALUE-0.5)/0.01);
  }
  else
  {
    tmpVALUE=tmpVALUE/0.01;
  }
  
  delay(100);
  XBee.print("Temperature: ");
  XBee.println(tmpVALUE);
  delay(5000);
  
  
  // obtain and sending tmpVALUE
  sprintf(dataToSend,"%s",converttmpVALUE);  
   ftoa(char* buf,float f,int p);
   {
  int i, n;
  unsigned long l
  
  for (n=0; n<p; n++)
    f *= 10;

  l = (long) (f + .5);

  for (n=0; l>0; n++, l/=10)
  {
    if (n==p)
    {
      buf[n]='.';
      n++;
    }
    buf[n]= (l % 10) + '0';
  }

  for(i=0; i<n/2;i++)
  {
    char c;
    c = buf[i];
    buf[i] = buf[n-1-i];
    buf[n-1-i]= c;
  }

  buf[n]='\0';
}
  if(!BT.sendData(dataToSend)) USB.println("Data sent OK");
  else USB.println("Error while sending data");
  
  // Removing the connection previously opened
  if(!BT.removeConnection()) USB.println("Connection removed");
  else USB.println("Error while removing connection");  
  
  // Removing a trusted device
  if(!BT.removeTrustedDevice("0003190D102D")) USB.println("Trusted Device removed");
  else USB.println("Error while removing trusted device");

  delay(5000);
}

piul wrote:

my boss does not think the same. I have mentioned him several times that it is more secure to use std::string or any other, but apparently it is best to use char * and home cooked functions for the system the application will run in.

piul wrote:

char * Func (char * sT)
{
	char sRet [MAX_STR_SIZE] = "";
	strcpy (sRet, sT);
	return sRet;
}

And when compiling I get a warning returning address of local variable or temporary

piul wrote:

I think I know why it tells me this, but not completely sure...

1. allocate the output buffer out from the function
2. allocate the output buffer using new or malloc (but you should remember to deallocate it when it is no more needed)

int Func(char* sRet, const char* sT)
{
    return strcpy(sRet, sT);
}

lakshman rao wrote:

But that edit control doesn't accept text...

DavidCrow wrote:

You can't type into it? You can't paste into it? You can't set its text via code?