2

la idea es captar un mensaje y cifrarlo y retornar el string cifrado

tengo el siguiente código

string simple_crypt(string _message){
    int j=0;
    int len = alphabet.size();
    int mid = len/2+1;
    string result;

     while (_message[j] != '\0'){
        int n = alphabet.find(_message[j]);
        int aux = (-2)*((n+mid)-len)+n;
        result[j]= alphabet[aux];
        j++;

    }
   return result;
}

El problema esta que no retorna el string.

3
  • 1
    Quiere usted, por favor, inicializar la variable result antes de intentar acceder a un indice que no existe?
    – NaCl
    Commented el 31 mar. 2018 a las 1:47
  • string result(_message.size() + 1, 0) ya funciona gracias.
    – Jorge Luis
    Commented el 31 mar. 2018 a las 2:04
  • Mentira. Si retorna el string. Pero seguramente no contiene lo que esperabas ¿puedes mostrar más código? ¿cómo y dónde se usa la función simple_crypt? Commented el 31 mar. 2018 a las 9:55

1 respuesta 1

5

El problema esta que no retorna el string.

Es debido a que no ha sido inicializado y debido a eso la variable nunca ha tenido un tamaño inicial.

Entonces acordando a una de las sobrecargas dadas por el estándar:

string();

string (const string& str);

string (const string& str, size_t pos, size_t len = npos);

string (const char* s);

string (const char* s, size_t n);

string (size_t n, char c);


template <class InputIterator>
  string  (InputIterator first, InputIterator last);

Podríamos hacer algo como:

std::string simple_crypt(std::string _message)
{
    int j=0;
    int len = alphabet.size();
    int mid = len/2+1;
    std::string result(strlen(_message) + 1, 0); // reservamos memoria para el string y lo llenamos de 0's.

     while (_message[j] != '\0')
     {
        int n = alphabet.find(_message[j]);
        int aux = (-2)*((n+mid)-len)+n;
        result[j]= alphabet[aux];
        j++;

     }
   return result;
}
2
  • Excelente, gracias por la explicación ya funciona.
    – Jorge Luis
    Commented el 31 mar. 2018 a las 2:03
  • @JorgeLuis si esta respuesta soluciona tu problema márcala como solución. Gracias
    – eferion
    Commented el 2 abr. 2018 a las 6:16

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.