Foro de elhacker.net

Buenas a ver si alguien me puede ayudar, en esta función para resumir simplemente lo que hace es abrir el ficheroIpOK.txt donde contienen las ips y las muestra por pantalla, ahora tengo una segunda función que se llama pingMediaDNS, está función esta preparada para la inserción de una ip dns por teclado (scanf) pero ahora necesito que la función ipContestan le envie el valor que lee del fichero para que haga la media del ping, no se si me explico bien.

Muchas gracias.
saludos.

void ipContestan() {
 
	char lectura[800];
 
	char ruta[] = "ficheroIpOK.txt";
 
 
	FILE* ficheroIpOK;										// Declaramos el nuevo fichero
	ficheroIpOK = fopen(ruta, "r");			// Abrimos el fichero en el modo de añadir
	if (ficheroIpOK == NULL) {
		printf("Error al abrir el fichero ficheroIpOK");
	}
	else
	{
		printf("*********************************Lectura de fichero %s********************************\n",ruta);
 
		while (fgets(lectura, 100, ficheroIpOK)!=NULL) {
			printf("%s", lectura);
 
 
		}
 
	}
	fclose(ficheroIpOK);
 
}
 
int pingMediaDNS(char* dns, const char* rutaFichero) {
 
	char command[100];
	char dnsRuta[100];
	char lectura[100];
	char mediaMs[100];
	bool pingOK = false;
	int length, newlength, j = 0, media = -1;
 
 
	/* Construcción del comando ping para los dns introducidos*/
 
	strcpy(command, "ping -n 10 ");														// Copiamos ping -n 10 a command <<El parametro de -n es para el número de pings totales>>
	strcat(command, dns);																// concatenamos ping -n 10 + dns
	strcat(command, " > ");																// concatenamos > a ping -n 10 + dns
	strcpy(dnsRuta, rutaFichero);														// Copiamos la ruta del fichero dns1.txt o dns2,txt a dnsruta
	strcat(command, dnsRuta);															// concatenamos dnsruta a ping -n 10 + dns + > + dnsruta
	system(command);																	// Ejecutamos el comando ya construido
 
	FILE* ficheroDns;
	ficheroDns = fopen(rutaFichero, "r");
	if (ficheroDns == NULL) {
		printf("El fichero %s no existe o no se puede abrir", rutaFichero);
	}
	else {
		while (fscanf(ficheroDns, "%s", lectura) != EOF)
		{
			if (strstr(lectura, "Media") != NULL) {
				pingOK = true;
			}
		}
		length = strlen(lectura);
		newlength = length - 2;
		for (int i = 0; i < newlength; i++)
		{
			mediaMs[j++] = lectura[i];
		}
		mediaMs[j] = '\0';
		media = -1;
		media = atoi(mediaMs);
 
		if (pingOK) {
			printf("\nPing correcto %s \n", dns);
		}
		else
		{
			printf("\nPing incorrecto %s \n", dns);
		}
	}
	return(media);
}
 
 
 

No sé muy bien lo que quieres hacer con la segunda función pero me parece que le estás dando muchas vueltas a los mismos datos de manera innecesaria.
De la línea 59 a la 67 se puede resumir en:

lectura[strlen(lectura)-2] = '\0';
media = atoi(lectura); 

Además dices que la función pingMediaDNS() está preparada para una entrada por teclado y no es cierto. La única entrada que recibe esa función es desde fichero en la línea 53.

Por lo tanto, no se muy bien ni lo que quieres hacer ni dónde tienes el problema.

Buenas K-YreX, te agradezco una vez más por ayudarme, la cuestión es que me piden que el programa sea modular, estás funciones la función pingMediaDNS la hice en la pasada práctica para que se introducirán las DNS con un scanf (creo que me he explicado mal) con la entrada manual toda va bien, pero ahora me piden que no sea una entrada por teclado si no que se leerá desde el fichero, claro de alguna manera ahora yo tengo que pasar el valor que está leyendo la función ipContestan para pasarla a pingMediaDNS que será el valor leído del fichero, de esta manera no debería tocar ni la función ni nada ya que de la lectura del archivo devolvería el dato que le tengo que pasar a la función pingMediaDNS que es el Char*DNS, tal vez la función está mal planteada para el propósito de modulación o mi pensamiento es erróneo al hacerlo de esta manera, ando bastante perdido y seguro que me estoy complicando la vida.

Muchas gracias
Saludos