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
    el 31 mar. 2018 a las 1:47
  • string result(_message.size() + 1, 0) ya funciona gracias.
    – Jorge Luis
    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? 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
    el 31 mar. 2018 a las 2:03
  • @JorgeLuis si esta respuesta soluciona tu problema márcala como solución. Gracias
    – eferion
    el 2 abr. 2018 a las 6:16

Tu Respuesta

Al pulsar en “Publica tu respuesta”, muestras tu consentimiento a nuestros términos de servicio, política de privacidad y política de cookies

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