No me compila una clase

[aguml]

Lo he solucionado con esta linea de codigo:

Código PHP:

final.append(&cadena.at(i),1); 


Eso funciona correctamente aunque quizas no sea la mejor manera y me gustaria saber como hacerlo usando append y de la manera mas correcta posible.

[aguml]

Ahora he modificado el codigo un poco y sobre todo he añadido mas caracteres a la mascara de caracteres válidos y ya no funciona correctamente. Tengo esto:
Main.cpp:

Código PHP:

//---------------------------------------------------------------------------
#include <iostream>
#include "Unit1.h"
//---------------------------------------------------------------------------

int main()
{
   string cadena = "Hola Mundo, como te va.";
   string semilla = "aguml";

   Vigenere *vig = new Vigenere;
   string cifrada = vig->cifrar(cadena, semilla);
   std::cout << cifrada << std::endl;

   string descifrada = vig->descifrar(cifrada, semilla);
   std::cout << descifrada << std::endl;
   delete vig;
   std::cin.get();

   return 0;
} 


Unit1.cpp:

Código PHP:

/************************************************
*  Nombre : Vigenere.cpp                        *
*                                               *
*  Cifrado de cesar de sustitución multiple     *
*  por medio de sumas y restas del alfabeto     *
*                                               *
*  Fulapol                                      *
*  [email protected]                            *
*  my.opera.com/fulapol                         *
*                                               *
************************************************/

/************************************************
*                                               *
*  El Constructor que debe recibir la cadena a  *
*  tratar y la cadena semilla                   *
*                                               *
************************************************/
#include "Unit1.h"

Vigenere::Vigenere(){
   letras = "abcdefghijklmnñopqrstuvwxyzABCDEFGHIJKLMNÑOPQRSTUVWXYZáéíóúÁÉÍÓÚüÜ1234567890¿?¡!,;.:-_ \"<>()=+*/&%$#@";
}

Vigenere::~Vigenere(){

}
/************************************************
*                                               *
*  El metodo cifrar, que regresa una cadena de  *
*  texto que solo despues de ser cifrada        *
*                                               *
************************************************/

string Vigenere::cifrar(string orig, string sem){
   string final;
   int i, movimiento, rotacion, total;
   size_t pos, pos2;
   //Las variables serviran para registrar la posicion
   //de las letras y calcular la semilla

   this->texto = limpiar(orig);
   this->semilla = limpiar(sem);

   for(i=0, rotacion=0; i < texto.length(); i++, rotacion++){
      pos = letras.find(texto.at(i));
      pos2 = letras.find(semilla.at(rotacion));
      total = ((int)(pos+pos2))%letras.length();
      //El total que marca la posicion de la letra y la
      //semilla sumadas

      if(total > letras.length())
         movimiento = total - letras.length();
      else
         movimiento = total;
      //Para asegurarnos que esta dentro del rango

      final.append(letras, movimiento, 1);
      if(rotacion == semilla.length() - 1)
         rotacion = 0;
      //Si se acaba la semilla, regresar al principio
   }
   return final;
}

/************************************************
*                                               *
*  El metodo descifrar, que regresa una cadena  *
*  de texto que solo despues de ser descifrada  *
*                                               *
************************************************/

string Vigenere::descifrar(string orig, string sem){
   //La funcion es exactamente la misma que en el metodo
   //cifrar, solo que aqui las posiciones se restan para
   //eliminar la semilla del texto

   string final;
   int i, movimiento, rotacion, total;
   size_t pos, pos2;

   this->texto = limpiar(orig);
   this->semilla = limpiar(sem);
   
   for(i=0, rotacion=0; i<texto.length(); i++, rotacion++){
      pos = letras.find(texto.at(i));
      pos2 = letras.find(semilla.at(rotacion));
      total = ((int)(pos - pos2)) % letras.length();
      if(total < 0)
         movimiento = total + letras.length();
      else
         movimiento = total;
      final.append(letras, movimiento, 1);
      if(rotacion == semilla.length() - 1)
         rotacion = 0;
   }
   return final;
}

/************************************************
*                                               *
*  El metodo limpiar, que regresa una cadena de *
*  texto que solo contiene los caracteres       *
*  soportados por la clase                      *
*                                               *
************************************************/

string Vigenere::limpiar(string cadena){
   int i;
   string final;
   for(i=0; i < cadena.length(); i++){
      if(letras.find(cadena.at(i)) != -1)
         final.append(&cadena.at(i),1);
      //Agregamos a la cadena final el caracter
      //si se encuentra en la variable letras
   }
   return final;
} 


Unit1.h:

Código PHP:

//---------------------------------------------------------------------------
#ifndef Unit1H
#define Unit1H
//---------------------------------------------------------------------------
#include <cstring.h>

class Vigenere{
   private:
      string texto,semilla,letras;
      string limpiar(string cadena);
      //Las variables son privadas, el metodo limpiar
      //es privada ya que solo lo utilizan los otros
      //metodos para limpiar la entrada

   public:
      Vigenere();
      ~Vigenere();
      string cifrar(string orig, string sem);
      string descifrar(string orig, string sem);
      //Esto es publico porque es la unica forma de
      //accesar a los datos de la clase
};
//---------------------------------------------------------------------------
#endif 


¿Como la pongo correctamente la máscara?

[[escafandra]]

Prueba así:

Código PHP:

string Vigenere::cifrar(string orig, string sem){
   string final;
   int i, rotacion, total;
   int pos, pos2;
   //Las variables serviran para registrar la posicion
   //de las letras y calcular la semilla

   this->texto = limpiar(orig);
   this->semilla = limpiar(sem);

   for(i=0; i < texto.length(); i++){
      rotacion = i%semilla.length();
      pos = letras.find(texto.at(i));
      pos2 = letras.find(semilla.at(rotacion));
      total = (pos+pos2)%letras.length();
      //El total que marca la posicion de la letra y la
      //semilla sumadas

      final.append(letras, total, 1);
   }
   return final;
}


string Vigenere::descifrar(string orig, string sem){
   //La funcion es exactamente la misma que en el metodo
   //cifrar, solo que aqui las posiciones se restan para
   //eliminar la semilla del texto

   string final;
   int i, rotacion, total;
   int pos, pos2;

   this->texto = limpiar(orig);
   this->semilla = limpiar(sem);

   for(i=0; i<texto.length(); i++){
      rotacion = i%semilla.length();
      pos = letras.find(texto.at(i));
      pos2 = letras.find(semilla.at(rotacion));
      total = (letras.length() + pos - pos2) % letras.length();

      final.append(letras, total, 1);
   }
   return final;
} 


Saludos.

PD/ Cuidado con la salida por consola, algunos caracteres pueden salir cambiados.

[aguml]

Si si, lo de los caracteres lo se pero es simplemente para hacer una prueba. Si fuese a leer o escribir en un archivo usando fread y fwrite no pasaria.
La verdad es que estudiando tu codigo y el que yo tenia veo que el que yo tenia tenia condicionales que eran redundantes pero no veo cual era el problema ya que haciendo las cuentas en ambos creo que funcionan igual. ¿me puedes explicar donde estaba el fallo exactamente para que diera resultados no validos?

[[escafandra]]

Empezado por aguml

¿me puedes explicar donde estaba el fallo exactamente para que diera resultados no validos?

La codificación era correcta pero no la descodificación. El problema estaba aquí:

Código PHP:

total = (pos - pos2) % letras.length(); 


Si se sobrepasó la longitud de letras al codificar, (pos-pos2) no elimina la semilla correctamente y la resta será negativa. Dado que pos1+pos2 nunca puede ser mayor que 2*letras.length(), la solución es esta:

Código PHP:

total = (letras.length() + pos - pos2) % letras.length(); 


Saludos.

[aguml]

Ahora estoy intentando hacer lo mismo con un archivo de texto plano y hago esto:

Código PHP:

//---------------------------------------------------------------------------
#include <iostream>
#include <fstream.h>
#include "Vigenere.h"
//---------------------------------------------------------------------------

using namespace std;

int main()
{
   ifstream origen;
   ofstream salida;
   string semilla,cifrada,descifrada;
   char *cadena;

   semilla = "aguml";
   
   //Parte que encripta el archivo
   origen.open("Main.h");

   if(origen.good()){
      origen.seekg(0,origen.end);
      int length = origen.tellg();
      origen.seekg(0,origen.beg);
      cadena = new char[length+1];
      Vigenere *vig = new Vigenere;
      salida.open("Encriptado.txt");

      if(salida.good()){
         origen.read(cadena,length);
         cadena[length]='\0';
         salida.write(vig->cifrar(cadena, semilla).c_str(),length);
         salida.close();
      }
      delete vig;
      delete(cadena);
      origen.close();
   }

   //Parte que desencripta el archivo
   origen.open("Encriptado.txt");

   if(origen.good()){
      origen.seekg(0,origen.end);
      int length = origen.tellg();
      origen.seekg(0,origen.beg);
      cadena = new char[length+1];
      Vigenere *vig = new Vigenere;
      salida.open("Desencriptado.txt");

      if(salida.good()){
         origen.read(cadena,length);
         cadena[length]='\0';
         salida.write(vig->descifrar(cadena, semilla).c_str(),length);
         salida.close();
      }
      delete vig;
      delete(cadena);
      origen.close();
   }
   return 0;
} 


Tengo dos problemas.
1. El archivo se abre la primera vez pero despues de cerrarlo intento abrirlo una segunda vez y parece que hay algun problema porque no entra en la parte de desencriptar.
2. Me añade basura al final del archivo y eso que pongo el caracter de fin de cadena al final de la cadena.
¿puedes ayudarme?

[aguml]

Pues al final lo he solucionado abriendolo de forma binaria. No se porque si no es asi falla y me gustaria saberlo.