Chateitor

[tonilope]

Hola. No sé vosotros, pero una de las cosas que más me gustan a parte de programar es que mis programitas sean probados y recibir opiniones, críticas, etc...

Pues bien, os dejo aquí mi último pasatiempo. Se trata de un CHAT P2P con cifrado RSA (hasta 2048 bits) (Windows). En el .zip incluyo el código fuente (en C por supuesto ) por si a alguien le sirve para algo.

Sería agradable que al menos lo probarais una vez y comentaseis qué os parece.

Salu2

[javi_rubio]

Solo de ver el codigo ya me he acojonado....
Esto queda muy por encima de mis posibilidades, y por lo tanto no puedo opinar, pero parece una currada...

Saludos!!

[tonilope]

[cheroky]

Esta claro que te lo has currado pero voy hacer unas criticas sin animo de desmerecer el trabajo que has realizado que tiene su merito y habiendolo mirado por encima.

El paso de argumentos tan extenso se podria haber implementado con algo como getopt (POSIX) o similares.

Puesto que usas librerias de terceros no estaria de mas que distribuyeras
un makefile o archivo de proyecto con alguna indicacion de las dependencias que tuviere.

Te faltan algunos inclusiones como winsock.h o winsock2.h en algunos compiladores podria no compilar solo con windows.h.

Fflush no esta definido para la entrada estandar.

Gran parte del codigo se limita a la representacion de datos y a cosas tan triviales como a colorear esta, se podria haber modularizado aparte separando la paja del grano

No entiendo por que usas memoria dinamica como si fuera estatica, es decir allocas memoria con una costante y este bloque es fijo durante toda la vida del programa, es mas eficiente y no hay problemas de posibles memory leaks usando memoria automatica, tampoco usas la logica para comprobar esos mallocs, hay programadores mas o menos noveles que piensan que usando malloc o familia son mas 'geeks' (no digo que sea tu caso).

El programa es bastante inseguro en general, por ejemplo hecha un vistazo a la funcion analiza_mensaje, un potencial overflow, si ademas le añades que no compruebas malloc es una bomba.

S2.
_________________
- Explícito es mejor que implícito.
- Simple es mejor que complejo.
- Complejo es mejor que complicado.
- Plano es mejor que anidado.
- Ralo es mejor que denso.

[kanijo]

En general estoy de acuerdo con cheroky, creo que abusas de malloc y que haces muy pocas comprobaciones de error con lo que el programa se vuelve una potencial bomba.

Otro dato que creo que puede ayudar a clarificar el código es que muchas funciones como carga_comando, saca_comando etc... usan dos parámetros, uno de entrada y otro exclusivamente de salida. Creo que sería más claro si la función retornase el valor en lugar de pasarlo como parámetro por referencia.

Un saludo
_________________
Hazlo, o no lo hagas, pero no lo intentes.

[tonilope]

[tonilope]

[cheroky]