El programa dejó de funcionar al dar una respuesta? (RESUELTO)

Iniciado por mcamara46, 27 Diciembre 2013, 22:06 PM

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

mcamara46

Hola, este es mi primer post en este foro. Me dirijo a vosotros porque tengo un problema en un codigo muy sencillo de c++.

#include <iostream>

using namespace std;

int main()
{
   char* a;
   char* b = "Hola soy Maestro";
   cout << "Bienvenido, soy programa1, usted quien es?" << endl;
   cin >> a;//almaceno la respuesta del usuario
   if(a == b)
   {
        cout << "Hola maestro" << endl;    
   }
   else
   {
       cout << "Tu no quieres nada." << endl;
   }
   system("pause");
   return 0;
}


El problema real que tengo esque al  darle una respuesta deja de funcionar.

Quisiera saber cual es el problema, porque si quito los char* y lo substituyo por char y solo doy una sola letra si me admite el código y el programa funciona perfecto. Intuyo que el problema esta al comparar o en la misma seqüencia de caracteres char*.

Porfavor si alguien puede poner el resultado e indicarme cual ha sido mi error estaria muy agradecido.

Gracias de antemano :D

ivancea96

#1
Bueno, en primer lugar:

Código (cpp) [Seleccionar]
if(a == b)

Eso nunca será verdadero. Ahí estás comparando las DIRECCIONES de memoria que guardan las variables (punteros). Las direcciones nunca serán iguales.

En segundo lugar:

la variable 'a' es un puntero a una dirección. ¿Qué dirección? Ninguna, ya que no está inicializada. En cada inicialización del programa, 'a' puede tener un valor distinto. Por tanto, lo que dbees hacer es inicializarla.

No soy experto en C, soy más de C++, pero si no me equivoco, sería así:
Código (cpp) [Seleccionar]
char* a = malloc(sizeOf(char)*N); //N es el número de casillas que tendrá. Pon 100 por ejemplo, para asegurarte que no haya problemas

Bueno, creo que así debería ir.

EDITO: Perdona, pensaba que era C. Ahora vi el include iostream jeje

En C++, para guardar memoria, haz así:

Código (cpp) [Seleccionar]
char* a = new char[N]; /+/Donde N es el numero de celdas, caracteres en este caso.

mcamara46

Cita de: ivancea96 en 27 Diciembre 2013, 22:12 PM
Bueno, en primer lugar:

Código (cpp) [Seleccionar]
if(a == b)

Eso nunca será verdadero. Ahí estás comparando las DIRECCIONES de memoria que guardan las variables (punteros). Las direcciones nunca serán iguales.

En segundo lugar:

la variable 'a' es un puntero a una dirección. ¿Qué dirección? Ninguna, ya que no está inicializada. En cada inicialización del programa, 'a' puede tener un valor distinto. Por tanto, lo que dbees hacer es inicializarla.

No soy experto en C, soy más de C++, pero si no me equivoco, sería así:
Código (cpp) [Seleccionar]
char* a = malloc(sizeOf(char)*N); //N es el número de casillas que tendrá. Pon 100 por ejemplo, para asegurarte que no haya problemas

Bueno, creo que así debería ir.

EDITO: Perdona, pensaba que era C. Ahora vi el include iostream jeje

En C++, para guardar memoria, haz así:

Código (cpp) [Seleccionar]
char* a = new char[N]; /+/Donde N es el numero de celdas, caracteres en este caso.

Muchas gracias, ahora ya no se cierra, pero si miras el código, lo que pretendo hacer es que se comparen dos frases, y que si son iguales le vaya a preguntar otra cosa, sino salte otra frase.
Ahora el problema que tengo esque salta el else ponga lo que ponga.

using namespace std;

int main()
{
    char* a = new char[100];
    char* b = "Hola soy Maestro";
    cout << "Bienvenido, soy programa1, usted quien es?" << endl;
    cin >> a;//almaceno la respuesta del usuario
    if(a == b)
    {
         cout << "Hola maestro" << endl;   
    }
    else
    {
        cout << "Tu no quieres nada." << endl;
    }
    system("pause");
    return 0;
}

ivancea96

Cita de: ivancea96 en 27 Diciembre 2013, 22:12 PM
Bueno, en primer lugar:

Código (cpp) [Seleccionar]
if(a == b)

Eso nunca será verdadero. Ahí estás comparando las DIRECCIONES de memoria que guardan las variables (punteros). Las direcciones nunca serán iguales.

Como dije antes xD así solo comparas las direcciones. Tú debes comparar letra a letra (con un for, por ejemplo) cada una, y ver si son iguales. También puedes usar la librería string, pero eso ya como veas.


leosansan

Cita de: mcamara46 en 27 Diciembre 2013, 22:35 PM
Gracias, mirare como usar la libreria :D

Otra forma es usar el tipo string que permite cosas como:

Código (cpp) [Seleccionar]

if(a == b)


Código (cpp) [Seleccionar]

#include <iostream>
#include <cstdlib>

using namespace std;

int main(){
    string b;
    string a ="leosan san";
    cout << "Bienvenido, soy programa1, usted quien es?" << endl;
    getline (cin,b) ;//permite espacios en blanco a diferencia de cin
    if(a == b)
        cout << "Hola maestro  " << a <<endl;
    else
        cout << "Tu no quieres nada." << endl;
    system("pause");
    return 0;
}


;-)  ;-) Felices Navidades y Próspero Año Nuevo.  ;-)  ;-)

¡¡¡¡ Saluditos! ..... !!!!