Título: Encriptador en C++ Publicado por: mastersuv en 22 Agosto 2010, 09:26 am Después de un largo rato de pensarle y buscar conceptos T_T
he logrado crear un sencillo sistema de cifrado. Bueno... la cifrado es algo loca, pero la pueden modificar a su gusto. Me gustaría que me dieran consejos de como podría mejor este código. Que cosas hize de más, o que le falta. ¿Estoy usando mucho código innecesario? PD: Solo cifra palabras en minusculas, y solo acepta espacios como separador. Aquí les dejo el codigo: Código
Título: Re: Encriptador en C++ Publicado por: [L]ord [R]NA en 22 Agosto 2010, 15:59 pm el codigo se puede mejorar bastante... los if,else if, else no necesitan break al terminar. los goto son una mala practica en programacion, no importa el lenguaje... la encriptacion se puede mejorar bastante, se ve demasiado poco estetica. en mi S.O. no se ejecutara correctamente debido a los llamados a system(), que tambien son una mala practica. Todos los miembros de la clase son publicos, tambien es una mala practica.
consejos: 1) Busca remplazos a los system. 2) Declara el string dec fuera del while, asi evitas tener que usar el goto. 3) Decide si utilizaras Apis o no, eso le quita portabilidad al ejecutable. 4) Es bueno que las funciones tengan un valor de retorno, asi sabes si ocurrio algun error. 5) Quita "int argc" y "char *argv[]" si no los vas a utilizar. Título: Re: Encriptador en C++ Publicado por: leogtz en 22 Agosto 2010, 17:28 pm Lord R.N.A, los break' s los está usando porque están en el ámbito de un ciclo FOR, los usa para parar el ciclo.
No es que esté mal usar solo miembros públicos, pero se pierde la abstracción. Lo que se ve feo es el agregado de las palabras al vector. No sé si estás liberando bien la memoria con delete. Saludos. Título: Re: Encriptador en C++ Publicado por: [L]ord [R]NA en 22 Agosto 2010, 18:32 pm Lord R.N.A, los break' s los está usando porque están en el ámbito de un ciclo FOR, los usa para parar el ciclo. No es que esté mal usar solo miembros públicos, pero se pierde la abstracción. Lo que se ve feo es el agregado de las palabras al vector. No sé si estás liberando bien la memoria con delete. Saludos. yo veo un problema con los break; nunca pueden ser tomados, debido a que siempre salta a "r" antes de llegar. Título: Re: Encriptador en C++ Publicado por: mastersuv en 22 Agosto 2010, 18:38 pm Exactamente, eso es por lo cual use break;
Y con lo del agregado de palabras, te refieres a las que van a sustituir cierto? Yo también se que eso esta feo pero quize dramatizarlo jaja para ver como se veía. Título: Re: Encriptador en C++ Publicado por: [L]ord [R]NA en 22 Agosto 2010, 19:03 pm Exactamente, eso es por lo cual use break; Y con lo del agregado de palabras, te refieres a las que van a sustituir cierto? Yo también se que eso esta feo pero quize dramatizarlo jaja para ver como se veía. Hiciste un codigo tan feo porque lo querias hacer asi :huh: no se pero no me parece Título: Re: Encriptador en C++ Publicado por: mastersuv en 22 Agosto 2010, 19:06 pm Es mi primer programa usando vectores de esta forma.
Nunca había usado tantos for y todavía soy nuevo en C++... Este es uno de mis primeros programas pero ya veo que son basura. :( Título: Re: Encriptador en C++ Publicado por: [L]ord [R]NA en 22 Agosto 2010, 20:34 pm :¬¬ No se dijo que fuera basura... se dijo que se pueden mejorar bastante.
Título: Re: Encriptador en C++ Publicado por: do-while en 26 Agosto 2010, 03:01 am ¡Buenas!
Creo que te vendria bien algo de infromacion de las chinchetas del subforo de criptografia, en el foro de seguridad informatica. Como bien te han indicado, los metodos criptograficos son independientes del lenguaje de programacion que utilices, y conocerlos te ayudara a ordenar las ideas. Y sobre todo, sigue asi y sigue intentandolo, que todos hemos hemos empezado por codigos mas o menos "guarros", y con el tiempo y la practica, vas consiquiendo un codigo mas claro y conciso. ¡Saludos! |