Foro de elhacker.net

Hola como bien dice el titulo, tengo un archivo .txt así:
45,678
67,890

y yo quiero que queden así:
45.678
67.890

mi idea es, aprovechando que se que el caracter 8 en ascii es retroceso , el 44 la coma y el 46 el punto hacer esta función:

el problema es que el archivo no se modifica, si alguien sabe que es lo que esta mal en el código y me lo dice me serviría de mucho, saludos.

En vez de usar números mágicos utiliza directamente los caracteres entre comillas simples.
Por otra parte, en vez de intentar ir a atrás insertando el carácter de control '\b' directamente atrasa una posición el puntero del archivo.

Con todo esto el código queda como sigue:

if (c==',') {
    fseek(archivo, -1, SEEK_CUR);
    fputc('.',archivo);
}

No logro poder modificarlo.

¿Cual se supone que es la diferencia despues de modificar el archivo?

cambiar la coma por el punto.

Buenas, si he entendido bien lo que quieres es leer el contenido del archivo y sustituir su contenido por el mismo pero cambiando solo las comas por puntos.

Con el código de MAFUS debería bastar, porque el fputc() sobreescribe el valor que se encuentra en esa posición. Cada fgetc() mueve el puntero una posición adelante para continuar leyendo, cuando encuentras una ',' usas fseek(archivo, -1, SEEK_CUR) para que vuelva atrás y no pase al siguiente caracter y entonces con fputc('.',archivo), escribes '.' en la posición de la ',' por lo que quedaría sobreescrita.

Si es un fichero de texto entonces no lo abras como binario usa "rt" en vez de "r+".

Un saludo.

#include <stdio.h>
#include <stdlib.h>
 
void main (void) {
FILE *archivo;
int c;
 
archivo=fopen("varx.txt","rt");
c=fgetc(archivo);
while (c != EOF) {
	if (c==',') {
               fseek(archivo, -1, SEEK_CUR);
               fputc('.', archivo);
	}
	c=fgetc(archivo);
}
fclose(archivo);
}
 

Muchas gracias, creo que el error entonces seria que use el "r+". saludos.

---

tampoco me anda   :( :( :( :(

Haz el favor, prueba el código que puse y dime si anda.

Postea el nuevo código, NO solo cambia el "r+" sino más cosas.

En caso de que no funcione dinos cual es el mensaje de error y si no le hay verifica que el archivo se abre correctamente.

Saludos.

no me tira ningún error, es como que se cuelga y tengo que cerrar el programa con ctrl+c.

acá el código:

Coloca fflush después de una operación de escritura si luego vas a hacer otra lectura. Además, es preferible que utilices r+, pues la 't' no siempre stá soportdada, y en este caso, realmente no va a cambiar nada.

#include <stdio.h>
#include <stdlib.h>
 
int main() {
	FILE *archivo;
	int c;
 
	archivo=fopen("asd.txt","r+");
	if (archivo == NULL){
		printf("\nError de apertura del archivo. \n\n");
	}
	else {		
		c=fgetc(archivo);
		while (c != EOF) {
			if (c==',') {
				fseek(archivo, -1, SEEK_CUR);
				fputc('.',archivo);
				fflush(archivo);
			}
			c=fgetc(archivo);
		}
	}
	fclose(archivo);
}

muchas gracias, me funciono perfecto.

¿sabes porque es necesario colocar fflush? estuve viendo y esta función lo que hace es borrar el buffer creo.

¿si la lectura y escritura es sobre una pantalla también se debe hacer?

 y la ultima, ¿porque, en el ejemplo que vimos  hay que retroceder un byte de la posición actual? si técnicamente el puntero esta apuntando al caracter que queremos cambiar.

saludos.

otra duda, al final el código lo tuve que cambiar, porque solo en la primer linea me imprimía bien, quedaba así:
12,345
23.,456
56.,678



igualmente tampoco anda a la perfección, los últimos números tampoco los imprime bien, no se si sera porque el archivo es muy grande o que (el archivo es de 1600 lineas aprox)

fflush vacía el buffer, que vienen siendo 2 pasos: Primero, enviar los datos a su destino (el archivo en este caso), y segundo, limpiarlo. Para evitar que vaya lento, cada operación como fputc en evz de escribir al archivo, lo va almacanando en el buffer. Con fflush, grabamos lo almacenado.

Si es sobre pantalla también se debe? Pues según. Si vas a pedir entrada al usuario, generalmente el fflush no es necesario, se hace automáticamente. Pero si vas a tener a tu programa trabajando continuamente (Por ejemplo, listando todos lso archivos de una carpeta), entonces sí es preferible ahcer fflush. Pero no cadaescritura, sino cada "bloque" de texto qeu quieras que salga junto a la pantalla. Pero por lo general, no será necesario. Eso sí, si en algún momento ves que no sale texto por pantalla que debería salir, prueba poniendo flush.

Tanto fputc como fgetc avanza 1 posición en el archivo. Cuando fgetc() retorna ',', ya no está en ese caracter, sino en el siguiente. Entonces, retrocedes uno, y lo reescribes con fputc.

Con respecto a lo que acabas de comentar, ¿qué te genera el código que puse antes?

con tu codigo
esta así:
0.0000
77.725
146.327
199.039
230.767
235.482
215.951
171.428
108.204
34.306
-44.094
-118.699
-178.576


queda así:
0,0000
77.,725
146.,327
199.,039
230.,767
235.,482
215.,951
171.,428
108.,204
34.,306
-44.,094
-118.,699
-178.,576

y en las ultimas lineas simplemente no modifica nada

tengo que aclarar que el archivo no es .txt sino que es un archivo de texto que no tiene especificado el tipo de archivo que es. obviamente no le puse "asd.txt" sino simplemente "asd" (aunque no se si esta bien).

Extraño. ¿Usas Windows, Linux?

windows uso

Utiliza este código:

#include <stdio.h>
#include <stdlib.h>
 
int main() {
	FILE *archivo;
	int c;
 
	archivo=fopen("asd.txt","r+");
	if (archivo == NULL){
		printf("\nError de apertura del archivo. \n\n");
	}
	else {		
		c=fgetc(archivo);
		while (c != EOF) {
			if (c==',') {
				printf("%i ", ftell(archivo));
				fseek(archivo, -1, SEEK_CUR);
				printf("%i ", ftell(archivo));
				fputc('.',archivo);
				fflush(archivo);
				printf("%i\n", ftell(archivo));
			}
			c=fgetc(archivo);
		}
	}
	fclose(archivo);
}

Le añadí unos printf para comprobar que la posición del cursor varíe correctamente.
Te debería salir por pantalla los valores:
"N N-1 N"

pone números como vos decís:" N N-1 N ", pero no se detiene nunca.
empieza con "11 10 11"