aca pongo el codigo es uno ya escrito que lo unico que quise es optimizarlo el codigo fue publicado ya pero para no mesclar las dudas cree un tema aparte
es el codigo que puse no puse todo el codigo ya que crei que no era necesario ya que por donde creo q viene el problema talvez sea por el ingreso de datos que es la informacion que quiero guardar.

#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
#include <string.h>
 
struct datos {
					char nombre [15];// nombre del usuario
					char pass [6];//password del usuario
					char acceso [21];//acceso del usuario
					char tipo [2];//tipo de acceso
					};
void main()
{ int i=1;
 
datos regin;
 
FILE *pf; // apunta al archivo
 
 
pf=fopen ("C:\\users.txt","at+");
// C:\Program Files\Counter-Strike\cstrike\addons\amxmodx\configs
if (pf==NULL)
  {
	printf ("ERROR DE LECTURA");
	getch();
	exit(1);
  }
 
  for(i=1;i<=1;i++) {
printf ("\nINGRESE NOMBRE DE USUARIO:");
gets (regin.nombre);
 
printf ("\nINGRESE PASSWORD DE USUARIO:");
gets (regin.pass);
 
printf ("\nINGRESE ACCESO:");
gets (regin.acceso);
printf ("\nINGRESE TIPO DE ACCESO:");
gets (regin.tipo);
 
fwrite (&regin,sizeof(datos),1,pf);
}
fclose (pf);
 
//re leo grabacion y compruebo
pf=fopen ("C:\\users.txt","r");
if (pf == NULL){
						printf ("ERROR DE ACCESO");
						getch();
						exit(1);
					}
	fread (&regin,sizeof (datos),1,pf);
 
while (!feof(pf)){
fread (&regin,sizeof (datos),1,pf);
printf("NOMBRE DE USUARIO:%s    PASS:%s 	ACCESO:%s 		TIPO DE ACCESO:%s",regin.nombre,regin.pass,regin.acceso,regin.tipo);
fread (&regin,sizeof (datos),1,pf);
					  }
fclose (pf);
getch();
}
 

edito: bueno ahi por lo q veo que dice littlehorse  voy a cambiar el gets pro un fgets aver que pasa

strlen: calcula la longitud de la cadena, no incluye el "\0" del final
sleep: si no me equivoco es un tiempo de espera sleep (1000); un segundo aunque la diferencia con delay (1000); no se cual es tube varios problemas una vuelta a programar en tc me andabla el delay pero en borlard o dev ninguna de las dos jaja me tiraba error de compilacion en esas lineas

EI: juntando mensajes.

Sigo con el problema ultimamente viendo, por ahi es un problema en las cadenas
aver como explicarme
en primer lugar vamos ignorar el overflow, por ahora.
bueno tengo q puedo ingresar en nombre 15 caracteres en pass 6 caracteres, ect
ahora esto esta dentro de una estructura para mi el error esta en que si ingreso en nombre 10 caracteres me estan sobrando 5 caracteres talves rellena los suigientes 5 caracteres con basura del buffer osea por lo q tenia entendido al apretar la tecla enter este actua como \0 o \n dependiendo que use

bueno ahora mi pregunta como implemento algo para q coloque el \0 al final de la cadena ingresada
en este momento estoy pensando en un strlen para q me cuente la longitud de la cadena y colocar un \0 al final de la cadena ingresada pero no se como implementarlo i si es de esta manera q deberia hacerlo.

Littlehorse te repito...no puede usar fgets porque no esta leyendo un archivo.esta leyendo desde el teclado y guardandolo en un buffer.

el code q puse yo esta bien..probalo

cambia esto:

scanf("%14s",&regin.nombre);
scanf("%5s",&regin.pass);

saludos.

No te ofendas, taul, pero, ¿así quieres crear un sistema operativo?

Es que vos en el código declaras la estructura así:

datos regin;

Y eso si no pones un typedef es incorrecto, en todo caso seria:

typedef struct data
{
//lo que sea
}datos;

De no hacer eso, al declarar la estructura tendrías que hacer:

struct data regin;

A lo que voy con fgets es que la uses para leer desde el teclado, es decir con stdin. Si utilizas fgets con fp (el puntero al archivo) lees una cadena desde el archivo que maneja ese puntero. A lo que me refería, a grandes rasgos, es que uses fgets en reemplazo de gets, no en reemplazo de fread.
También tienes algunas confusiones entre el modo binario y el modo texto en el manejo de archivos, son dos cosas distintas. Se tratan de distinta forma, y se usan distintas funciones. Busca información sobre ello para tener ciertos conceptos un poco mas sólidos.


Revisa lo que te dije del while, la lectura la haces mal, no te enteras cuando llegas al final del archivo y te salteas una lectura a la hora de imprimir (es decir, lees dos veces, imprimes una).


De nada, para eso estamos!

Saludos!

no me ofendo para nada Leo Gutiérrez...yo programo de una forma y a lo mejor vos de otra..si esta bien programado o no eso pasa por cada uno..creo q podemos opinar diferente o no? para eso estamos juntados aca en el foro..porque todos vemos las cosas desde un punto de vista diferente...quien decide cual es la mejor opinion? pues el q formulo la pregunta...yo no vine al foro a criticar los conocimientos de nadie..solo opino al igual q cualquier usuario. si vos Leo Gutiérrez pensas q no se programar...respeto tu opinion..nadie se puede ofender por la opinion de alguin no?


Saludos...Taul.