[?] cambiar una comparacion de const char * a metodo string.compare

Iniciado por Drakomorh, 24 Mayo 2011, 23:35 PM

0 Miembros y 1 Visitante están viendo este tema.

Drakomorh

Hola amigos:

Como puedo pasar de cosnt char* string y poder hacer la siguiente parte del codigo:

Original:
Código (cpp) [Seleccionar]

// Mostramos en que canal esta el ap:
const char * macs = ssh.getReceivedText(channelNum,"ansi");
   if (macs == 0 ) {
parametros->error=1;
parametros->errormsg = const_cast<char*> (ssh.lastErrorText());
        pthread_exit(NULL);
return NULL;
   }


y ahora quiero usar la clase string tal que...
Código (cpp) [Seleccionar]
// Mostramos en que canal esta el ap:
string macs = ssh.getReceivedText(channelNum,"ansi");
   if (macs.compare("0")) {
parametros->error=1;
parametros->errormsg = const_cast<char*> (ssh.lastErrorText());
        pthread_exit(NULL);
return NULL;
   }


la aplicacion sigue funcionando, pero siempre entra en el if.

A alguien se le ocurro como tengo que hacer la comparacion¿?

Gracias y un saludo.

Drakomorh

Hola amigos:

He probado a usar :

Código (cpp) [Seleccionar]

// Mostramos en que canal esta el ap:
string macs = ssh.getReceivedText(channelNum,"ansi");
    if (const_cast<char*>(macs.c_str()) ==0) {
parametros->error=1;
parametros->errormsg = const_cast<char*> (ssh.lastErrorText());
        pthread_exit(NULL);
return NULL;
    }



Ahora ya no entra siempre en el if, pero no se me ocurre como probar si llegado el momento entrara...

Gracias y  un saludo.

Littlehorse

Ten en cuenta que no es lo mismo comparar si el objeto contiene una cadena "0" que si el contenido es 0. Puedes chequear el ejemplo de cplusplus sobre el método compare:

Código (cpp) [Seleccionar]
// comparing apples with apples
#include <iostream>
#include <string>
using namespace std;

int main ()
{
  string str1 ("green apple");
  string str2 ("red apple");

  if (str1.compare(str2) != 0)
    cout << str1 << " is not " << str2 << "\n";

  if (str1.compare(6,5,"apple") == 0)
    cout << "still, " << str1 << " is an apple\n";

  if (str2.compare(str2.size()-5,5,"apple") == 0)
    cout << "and " << str2 << " is also an apple\n";

  if (str1.compare(6,5,str2,4,5) == 0)
    cout << "therefore, both are apples\n";

  return 0;
}


Primero tenes que decidir si estas tratando de comparar si la cadena es o no NULL o si estas tratando de comparar si el contenido de la cadena es o no 0. Definido eso puedes proceder con la forma que corresponda.
A simple vista no es necesario usar c_str en el ejemplo que expones, pero si lo fuese, lo estas haciendo mal por lo anteriormente mencionado.

Cuidado con meter casteos "para ver si funciona" sin saber realmente el objetivo del casteo, porque a veces el asunto podria parecer que funciona pero no.

Por lo pronto si te quieres sacar la duda puedes poner un breakpoint en el if e iniciar la depuración (debug) para comprobar que contiene "macs" y que te devuelve el método compare y demás. La mayoría de los IDEs tienen opciones para poder visualizar el contenido de las variables en forma individual durante la ejecución del programa, así que busca cual es la forma de hacerlo en tu IDE. Generalmente encontraras esa opción como "Expressions" o "Watch" o algo similar.

Saludos

PD: He visto que haces mucho doble post y hasta a veces triple post. Trata de usar el botón modificar.
An expert is a man who has made all the mistakes which can be made, in a very narrow field.