#include <stdio.h>
FILE *fp1, *fp2;
int cript (char *nocrypt, char *encrypt, int value)
{
char car;
fp1
= fopen (nocrypt
, "r+"); fp2
= fopen (encrypt
, "w+");
{
car += value;
}
return 0;
}
int main (int argc, char *argv [])
{
printf ("Krypter 0.1 : Sagrini 2011 : elhacker.net\n");
if (argc != 4)
{
printf ("[!!!] Use: %s <file-to-crypt> <dest-file> <pass-value>\n\n", argv
[0]); return 1;
}
if (atoi (argv
[3]) > 32 || atoi (argv
[3]) < 0) {
printf ("[!!!] Nice try, but Pass-Value will be a correct number (<32 && >0).\n\n"); return 1;
}
cript
(argv
[1], argv
[2], atoi (argv
[3])); printf ("[+] Crypt finished with no errors.\n\n"); return 0;
}
1) No es recomendable usar variables globales si no hace falta. No te acostumbres. Tampoco es bueno usar mucho DEFINE.
2) Tu código puede simplificarse mucho. Tienes líneas de más que hacen el texto más confuso.
Tu código tiene muchísimas cosas que mejorar. Te cedo el mío (de ahora mismo
), ahora añádele funciones de comprobación de errores y arregla el código para que la salida sea legible; yo mismo te concederé un pin
jeje. Si no te importa déjame algo de créditos
PD: No, no me he puesto a pensar cómo hacer salida legible...