Hola! no te quiero arruinar el regalo pero tenes varios errores conceptuales. Lo mas raro es que digas "recien estoy aprendiendo vectores", cuando ya estas queriendo manejar archivos... Te marco lo errores que vi:

1º Tenes que revisar como se usa la funcion fwrite(). No podes utilizar sizeof(int) en todos los casos, estas guardando mal.

2º Lo mismo fread

3º En el main cuando elegis la opcion: "1 - Alta de un articulo" ó "3 - Clientes" cerras los archivos(con fclose). El problema es: ¿que pasa si el usuario vuelve a elegir esas opciones? los archivos van a estar cerrados por lo que vas a tener un error cuando quieras volver a guardar algo. Tenes que abrir los archivos antes de elegir esas opciones

4º Con este ciclo: while(nombre != NULL) estas generando un bucle infinito. No tiene sentido hacer esa comparacion ¿que es lo que querias hacer?

5º En la funcion STOCK() el principal error esta en el uso de fread. Ya que no sabes manejar estructuras, lo que tenes que hacer para cada ciclo de lectura es:

fread(&x,sizeof(int),1,stock1);
fread(descripcion,sizeof(descripcion),1,stock1);
fread(&cantidad,sizeof(int),1,stock1);
fread(&precio,sizeof(float),1,stock1);

Hay muchas cosas que se pueden mejorar aun. Mi consejo es que antes de seguir con esto practiques un poco mas con manejo de archivos, y con estructuras


Saludos

Eso es lo que quiero, muchas gracias por hacer criticas, si bien es cierto lo del progrma, es el primer proyecto por asi decir en el que me meto de lleno, lo de los vectores es cierto, archivos los vi bien y el problema es que como bien lo marcas la falla en el ciclo para volver a leer, no lo puedo solucionar, ya que el problema está en donde lo ubico a la apertura del archivo para que cuando se seleccione esa opción se lo pueda leer correctamente, más preciso yo queria poner los datos en un vector, para no usar tantas variables, y comparar con un for, pero me es imposible, los archivos ademas son como bd en el programa, gracias de verdad por las criticas.

Buenas, estoy de nuevo, lo modifique al programa un cachito y segui el consejo y lo separe en funciones, lo muestro para que lo vean y me den una mano para continuar.

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
FILE *ventas;
FILE *stock;
FILE *clientes;
 
float VENTAS_CONTADO(FILE *ventas1, FILE *stock1, FILE *clientes1);
float VENTAS_CTACTE(FILE *ventas1, FILE *stock1, FILE *clientes1);
void ALTAS_ARTICULOS(FILE *stock1);
void CONSULTA_ARTICULOS(FILE *stock1);
void ALTAS_CLIENTES(FILE *clientes1);
void CONSULTA_CLIENTES(FILE *clientes1);
 
main()
{
	/*op,v,cl,st son variables para la eleccion de la opreacio*/
	int op,v,cl,st;
	float cta,cont;
	printf("\nBienvenido/a: ");
	printf("\nIngrese la opcion: ");
	printf("\n1 - VENTAS ");
	printf("\n2 - STOCK");
	printf("\n3 - CLIENTES");
	printf("\n0 - SALIR: ");
	scanf("%d",&op);
	while(op != 0)
	{
		switch(op)
		{
			case 0:
				return 0;
			case 1:
				if(op == 1)
				{
					printf("\n___________________________");
					printf("\nBienvenido al modulo de ventas: ");
					printf("\nEscoja la opcion: ");
					printf("\n1 - Ventas Contado");
					printf("\n2 - Ventas Cuenta Corriente");
					printf("\n0 - Salir: ");
					scanf("%d",&v);
					printf("\n___________________________\n");
					switch(v)
					{
						case 0:
							return 0;
						case 1:
							if(v == 1)
								cont = VENTAS_CONTADO(ventas,clientes,stock);
							break;
						case 2:
							if(v == 2)
								cta = VENTAS_CTACTE(ventas,clientes,stock);
							break;
						default:
							printf("\nSu opcion no es correcta\n");
							break;
					}
				}
				break;
			case 2:
				if(op == 2)
				{
					printf("\n____________________________");
					printf("\nBienvenidoal modulo de stock: ");
					printf("\nEscoja la opcion: ");
					printf("\n1 - Alta de un articulo");
					printf("\n2 - Consulta de un articulo ");
					printf("\n0 - Salir: ");
					scanf("%d",&st);
					printf("\n___________________________\n");
					switch(st)
					{
						case 0:
							return 0;
							break;
						case 1:
							if(st == 1)
								ALTAS_ARTICULOS(stock);
							break;
						case 2:
							if(st == 2)
								CONSULTA_ARTICULOS(stock);
							break;
						default:
							printf("\nLa opcion no es correcta ");
							break;
					}					
					break;
				 }
			case 3:
				if(op == 3)
				{
					printf("\n___________________________");
					printf("\nBienvenido al modulo de clientes");
					printf("\nEscoja la opcion: ");
					printf("\n1 - Alta de un cliente");
					printf("\n2 - Consulta de un cliente");
					printf("\n0 - Salir: ");
					scanf("%d",&cl);
					printf("\n____________________________\n");
					switch(cl)
					{
						case 0:
							return 0;
							break;
						case 1:
							if(cl == 1)
							{
								ALTAS_CLIENTES(clientes);
							}
							break;
						case 2:
							if(cl == 2)
								CONSULTA_CLIENTES(clientes);
							break;
						default:
							printf("\nLa opcion no es correcta ");
							break;
					}
				}
				break;
			default:
				printf("\nLa opcion ingresada no es correcta");
				break;
		}
		printf("\n_____________________");
		printf("\nBienvenido/a: ");
		printf("\nIngrese la opcion: ");
		printf("\n1 - VENTAS ");
		printf("\n2 - STOCK");
		printf("\n3 - CLIENTES");
		printf("\n0 - SALIR: ");
		scanf("%d",&op);
		printf("\n_______________________\n");
	}
}								
 
void ALTAS_ARTICULOS(FILE *stock1)
{
	int cantidad,codigo;
	float precio;
	char art[50];
	cantidad = 0;codigo = 0; precio = 0;
	stock1 = fopen("Stock.dat","ab");//Solo escribir el archivo, en caso de existir escribe al final
	printf("\nIngrese el codigo del articulo: ");
	scanf("%d",&codigo);
	while(codigo != 0)
	{
		printf("\nIngresar el nombre del articulo y una breve descripcion: ");
		scanf("%s",art);
		printf("\nIngrese la cantidad: ");
		scanf("%d",&cantidad);
		printf("\nIngrese el precio: ");
		scanf("%f",&precio);
		fwrite(&codigo,sizeof(int),1,stock1);
		fwrite(&art,sizeof(art),1,stock1);
		fwrite(&cantidad,sizeof(int),1,stock1);
		fwrite(&precio,sizeof(float),1,stock1);
		printf("\nIngresar el codigo del articulo: ");
		scanf("%d",&codigo);
	}
	fclose(stock1);
	printf("\nEl/los articulos fueron guardados con exito\n");
}
 
void CONSULTA_ARTICULOS(FILE *stock1)
{/*Funcion que hace lo mismo que CONSULTA_CLIENTES */
	int codigo,cantidad,x;
	float precio;
	char art[50];
	stock1 = fopen("Stock.dat","rb");//rb para solo lectura ya que el archivo existe
	printf("\nIngrese el codigo del articulo: ");
	scanf("%d",&codigo);
	fread(&x,sizeof(int),1,stock1);
	fread(&precio,sizeof(float),1,stock1);
	fread(&cantidad,sizeof(int),1,stock1);
	fread(&art,sizeof(art),1,stock1);
	while(!feof(stock1))
	{
		if(codigo == x)
		{
			printf("\n____________________");
			printf("\nCodigo: %d",codigo);
			printf("\nDescripcion: %s",art);
			printf("\nCantidad: %d",cantidad);
			printf("\nPrecio: %.2f",precio);
			printf("\n____________________\n");
			fseek(stock1,2,SEEK_END);
		}
		fread(&x,sizeof(int),1,stock1);
		fread(&precio,sizeof(float),1,stock1);
		fread(&cantidad,sizeof(int),1,stock1);
		fread(&art,sizeof(art),1,stock1);
	}
	fclose(stock1);
}
 
void ALTAS_CLIENTES(FILE *clientes1)
{
	char nombre[50];
	int cod;
	float saldo;
	clientes1 = fopen("Clientes.dat","ab");//Solo escribir el archivo, en caso de existir escribe al final
	printf("\nIngrese el codigo de identificacion del cliente: ");
	scanf("%d",&cod);
	while(cod != 0)
	{
		printf("\nIngrese el nombre del cliente: ");
		scanf("%s",nombre);
		printf("\nIngrese el saldo de tener alguno: ");
		scanf("%f",&saldo);
		fwrite(&saldo,sizeof(float),1,clientes1);
		fwrite(&cod,sizeof(int),1,clientes1);
		fwrite(&nombre,sizeof(nombre),1,clientes1);
		printf("\nIngrese el codigo: ");
		scanf("%d",&cod);
	}
	fclose(clientes1); 
}
 
void CONSULTA_CLIENTES(FILE *clientes1)
{
/*Funcion que busca por codigo y si encuentra imprime codigo,nombre y
saldo y posiciona la lectura al final del archivo */
	int cod,x;
	float saldo;
	char nombre[50];
	saldo = 0;
	clientes1 = fopen("Clientes.dat","rb");//rb para solo lectura ya que el archivo existe
	printf("\nIngrese el codigo de idectificacion del cliente: ");
	scanf("%d",&cod);
	fread(&saldo,sizeof(float),1,clientes1);
	fread(&x,sizeof(int),1,clientes1);
	fread(&nombre,sizeof(nombre),1,clientes1);
	while(!feof(clientes1))
	{
		if(cod == x)
		{
			printf("\n________________");
			printf("\nCodigo: %d",cod);
			printf("\nNombre %s",nombre);
			printf("\nSaldo: %.2f",saldo);
			printf("\n__________________\n");
			fseek(clientes1,2,SEEK_END);
		}
		fread(&saldo,sizeof(float),1,clientes1);
		fread(&x,sizeof(int),1,clientes1);
		fread(&nombre,sizeof(nombre),1,clientes1);
	}
	fclose(clientes1);
}
 
float VENTAS_CONTADO(FILE *ventas1, FILE *stock1, FILE *clientes1)
{
}
 
float VENTAS_CTACTE(FILE *ventas1, FILE *stock1, FILE *clientes1)
{
}
 

Como veran, esta un poco mejor y estoy manejando las exepciones, pero yo de la unica forma de leer y escribir un archivo es esa, no se de estructuras, ya lo voy a mejorar, pero sigo con el problema de la lectura, me esta costando y todavia faltan las funciones mas importantes com las ventas.Gracias a todos por ayudarme.

Bueno, ya logre que funcione el modulo de ventas de contado, funciona casi todo, pero quiero hacer que se descuente el stock por medio de una variable que paso por la funcion, les muestro el codigo ai lo pispean y me dicen. Yo creo que el problema está en la lectura del archivo, por eso no me descuentea.

void DESCONTAR_STOCK(FILE *stock1, int diferencia1, int comp1)
{
	int cod,cantidad,ns;//nc: nueva cantidad del articulo
	float precio;
	char art[50];
	stock1 = fopen("Stock.dat","r+b");
	fread(&cod,sizeof(int),1,stock1);
	fread(&art,sizeof(art),1,stock1);
	fread(&cantidad,sizeof(int),1,stock1);
	fread(&precio,sizeof(float),1,stock1);
	while(!feof(stock1))
	{
		if(*comp1 == cod)
		{
			ns = cantidad - *diferencia1;
			fwrite(&cod,sizeof(int),1,stock1);
			fwrite(&art,sizeof(art),1,stock1);
			fwrite(&ns,sizeof(int),1,stock1);
			fwrite(&precio,sizeof(float),1,stock1);
			fseek(stock1,2,SEEK_END);
		}
		fread(&cod,sizeof(int),1,stock1);
		fread(&art,sizeof(art),1,stock1);
		fread(&cantidad,sizeof(int),1,stock1);
		fread(&precio,sizeof(float),1,stock1);
	}
	fclose(stock1);
}
 

diferencia y comp, son variables que saque de una función para poder realizar la resta, diferencia contiene el valor de la cantidad que vendo - la cantidad que tengo y comp es el codigo del articulo que vendi para poder saber a cual descontar.

yo aun me lio con algunas cosas de apuntadores, pero a ver si consigo explicarme bien:

primero, en vez de declarar un puntero como int *a intenta declararlo como int* a. parece una tonteria, y para el compilador es lo mismo, pero de la segunda manera se ve claramente que a es una variable de tipo puntero a int. en cambio de la primera se puede pensar que *a es un puntero a int.

segundo, imaginate a la variable tipo puntero a int como una variable que almacena 2 datos, una direccion de memoria a la que apunta y el valor que hay en esa direccion. si haces un printf("%d", a), el resultado sera un numero largo que indica la direccion de memoria a la que apunta. en cambio, *a indica el valor almacenado en esa direccion de memoria.

las diferencias entre, por ejemplo, una variable int a y una variable int* b son que:
1. el valor almacenado se saca con: a y con *b
2. la direccion de memoria donde esta almacenada la propia variable se saca con: &a y &b
3. el valor "extra" que indica donde apunta una variable de tipo puntero se saca con: b

quizas te preguntes entonces que para que sirven los punteros ya que aparentemente hacen lo mismo pero de una manera distinta y mas complicada. pues el principal uso de los punteros es poder gestionar la memoria de manera dinamica (poder asignar y liberar memoria despues de haber compilado). algo que, por ejemplo, te permitiria hacer un programa que preguntase al usuario cuantos datos quiere ingresar y entonces tu podrias usar la memoria justa sin riesgo a usar demasiada ni a quedarte corto.

te dejo un pequeño codigo tipico de aprender apuntadores (da algunos warnings al compilar pero corre perfectamente):

#include <stdio.h>
 
int main()
{
	int a=12, *b;
	printf("\nPARA a:\ndireccion propia: &a= %d\nvalor de a: a= %d\n\n", &a, a);
	b=&a; //hacemos que el puntero b apunte a la variable a
	printf("PARA b:\napunta a la direccion: b= %d\nel valor almacenado ahi es: *b= %d\ndireccion propia: &b= %d\n\n", b, *b, &b);
	return 0;
}
 

su salida:


y ultimo, cuando hagas alguna funcion con parametros tienes que preguntarte si modificaras el valor de algun parametro. si no te interesa hacerlo, puedes pasar el parametro como una "variable normal" y usarla como tal. si necesitas modificar el valor de la variable, entonces le tienes que pasar a la funcion la direccion de memoria donde esta almacenada la variable, para que asi digamos que pueda escribir sobre ella. y para pasar la direccion de memoria entonces tienes que usar punteros.

ultimo codigo de ejemplo:

#include <stdio.h>
 
void funcion(int n, int* p);
 
int main() {
	int a=1, b=5;
	printf("VARIABLES ANTES DE LA FUNCION:\na= %d\nb= %d\n", a, b);
	funcion(a, &b);
	printf("VARIABLES DESPUES DE LA FUNCION:\na= %d\nb= %d\n", a, b);
	return 0;
}
 
void funcion(int n, int* p) {
// n es una variable de tipo int "normal". n contiene el valor de la variable a. PASO POR VALOR (no se puede modificar)
// p es una variable de tipo puntero a int. p contiende la direccion de memoria de b. PASO POR REFERENCIA (se puede modificar)
	printf("VALORES EN LA FUNCION:\nn= %d\n*p= %d\n", n, *p);
        // intentamos modificar las variables
	(*p)=3;
	n=2;
}

y su salida, donde se ve que 1 variable se ha modificado y la otra no:


espero que sirva de ayuda... si tienes alguna duda no tienes mas que consultar. un saludo!