A ver, muchas cosas.
¿Estás usando C++ para compilar C?
En C no hay valores por defecto en los argumentos, así que esto
int generateRandom(int number = '1'){está mal.
C no tiene el tipo de dato bool por defecto (tampoco false ni true), hay que incluir la librería stdbool para ello.
Inicializar cadenas:
char lxr[101];
strcat(lxr, "/home/");
strcat(lxr, user);
strcat(lxr, "/");
strcat(lxr, "readme.txt");
Todo esto se puede sustituir por una familia función de la familia printf:
char lxr[101];
sprintf(lxr, "/home/%s/readme.txt);
Por otra parte
// Global
char boot[21];
// ...
// Local en una función
strcat(boot, "/");
strcat(boot, "etc");
strcat(boot, "/");
strcat(boot, "rc.d");
strcat(boot, "/");
strcat(boot, "rc.local");
Podrías cambiarlo por
strcpy(boot, "/etc/rc.d/rc.local");
Y así reducir el tamaño del código y quede mejor a la vista. Aparte de que no pasas tanto tiempo llamando a funciones. Fíjate que que he hecho strcpy y no strcat porque aquí te salvas porque las cadenas globales se inicializan a 0. Pero si fuera una local o ya tuviera algo dentro copiaría la dirección al final de lo que hubiera causando un fallo. Usando esta técnica de strcat sucesivos asegúrate que el primero sea un strcpy.
De todas formas como sólo vas a usar esa variable de forma local puedes inicializar la variable de esta forma:
char boot[21] = "/etc/rc.d/rc.local";
Aunque también usando la característica de que en definiciones C te calcula el tamaño del array
char boot[] = "/etc/rc.d/rc.local";
Y apurando un poco más, ya que estás trabajando con una cadena literal y no vas a modificarla puedes dársela directamente a un puntero
char* boot = "/etc/rc.d/rc.local";
Código redundante:
if(remove(lxr) == 0){
remove(lxr);
}En el if ya has llamado a remove y dices que si ha tenido éxito en borrar lo llame otra vez.
Arrays de punteros:
void wn(void){
char *path[432];
strcat(*path, "C:\\ProgramData\\Microsoft\\Windows\\Start Menu\\Programs\\StartUp\\");
}Así no puedes construir una cadena porque cada elemento del array son punteros a char, no son char. Esto te podría dar fallos multiorgásmicos (palabro intencionado
).
Comparar cadenas:
if (OS == "Linux"){
lx();
}
else if (OS == "Windows"){
wn();
}
else if (OS == "Mac"){
mc();
}El == no compara cadenas tipo C. Pero aquí te salvas por una característica de C: todas las cadenas literales iguales tienen una única copia en memoria y todas sus apariciones en el código apuntan al mismo sitio, así C ahorra memoria (y esta es la causa por la que una cadena literal no puede ser modificada). Por tanto si comparas dos cadenas literales, dos punteros que hayan sido definidos con la misma cadena o un de estos punteros con la misma cadena literal C devolverá cierto.
Mostrar Mensajes
