Foro de elhacker.net

Tengo este problema al comparar cadenas de caracteres. A la hora de buscar un código previamente ingresado en el arreglo, este imprime el mensaje "No existe" cuando únicamente ingreso uno de los códigos. Ejemplo:
Código ingresado en el segundo arreglo: cp020
Búsqueda: No Existe
'Nombre del producto'
No existe

#include <stdio.h>
#include <stdlib.h>
#include <stdbool.h>
#include <string.h>
 
void menu(){
	printf("\t*****CONTROL DE INVENTARIO*****\n");
	printf("1.-Listado del inventario actual\n2.-Busqueda de productos\n3.-Ingresos totales\n4.-SALIR\n");
	printf("--------------------------------\n");
}
 
void salida(){
	printf("Saliendo del programa... Hasta pronto!\n");
}
struct Inventario{
	char codigo[10];
	char nombre[25];
	int cantidad;
	float pventa;
};
int Registrar(Inventario x[], int i){
	for(i=0;i<=2;i++){
		printf("\tRegistrar productos\n");
		printf("Producto #%d \n",i);
		printf("Ingrese el codigo del producto: ");
		gets(x[i].codigo);
		fflush(stdin);
		printf("\nIngrese el nombre del producto: ");
		gets(x[i].nombre);
		fflush(stdin);
		do{
			printf("\nIngrese la cantidad: ");
			scanf("%d",&x[i].cantidad);
			fflush(stdin);
		}while(x[i].cantidad<=0);
		do{
			printf("\nIngrese el precio de venta: ");
			scanf("%f",&x[i].pventa);
			fflush(stdin); //Limpieza del buffer
		}while(x[i].pventa<=0); 
		system("cls");
	}
}
void Lista(int i, Inventario x[]){
	for(int j=0;j<=2;j++){
		printf("   %s    %s        %d      %0.2f \n",x[j].codigo, x[j].nombre, x[j].cantidad, x[j].pventa);
	}
}
char Busqueda(Inventario x[], int i){
	char cod[10];
	printf("Ingrese el codigo que desea buscar: ");
	scanf("%s",&cod);
	fflush(stdin);
	for(int j=0;j<=2;j++){
		if(strcmp(x[j].codigo, cod)==0){
			printf("\nProducto: %s \n",x[j].nombre);
			break; 
		}else{
			printf("\nNo Existe\n");
			break;
		}
	}
}
float Venta(Inventario x[], int i){
	float total;
	for(int j=0;j<=2;j++){
		total+=x[j].cantidad*x[j].pventa;  //Primero, multiplica la cantidad por el precio y al final, se suma para el total del inventario vendedido.
	}
	printf("Ingreso total: %0.2f $ \n",total);
}
int main(){
	struct Inventario prod[20];
	int i=0, op;
	bool salir = true;
	Registrar(prod,i); //Llamamos a la funcion "Registrar"
	do{
		system("cls");
		menu(); //Llamamos a la funcion "Menu"
		do{
			printf("\nSeleccione una opcion: ");
			scanf("%d",&op);
		}while(op<=0 || op>=5);
		system("cls");
		switch(op){
			case 1:{
				printf("\tINVENTARIO ACTUAL\n");
				printf("   Codigo    Producto        Cantidad      Precio de Venta\n");
				Lista(i,prod);
				system("PAUSE");
				system("cls");
				break;
			}
			case 2:{
				printf("\tBUSQUEDA\n");
				Busqueda(prod, i);
				fflush(stdin);
				system("PAUSE");
				system("cls");
				break;
			}
			case 3:{
				printf("\tVENTA TOTAL DEL INVENTARIO\n");
				Venta(prod, i);
				system("PAUSE");
				system("cls");
				break;
			}
			case 4:{
				salida();
				salir = false;
				break;
			}
		}
	}while(salir==true);
	return 0;
}
 

Primero te recomendaría que elijas solo para ingresar strings scanf+fflush o gets, si mal no recuerdo fflush no era necesario con gets, pero por otra parte estás en el scanf asignando mal la dirección de memoria del string cod, si ya de por sí es un puntero/array/etc... no hace falta asignarle el símbolo ampersand para especificarle dirección de memoria porque de por sí lo toma como dirección de memoria. En todo caso al asignarle ampersand al vector estarías especificandole que dentro de los valores basura que tiene cod dicho valor sea dirección de memoria, por lo tanto estaría guardandose el valor en cualquier lado y no en el vector real...
El hecho que no haya crasheado el programa con eso sería raro habría que depurarlo para ver bién donde quedó guardado realmente el código ingresado...


B#

El problema no está en el registro, el problema está en la búsqueda, porque si corres el programa en tu entorno, podrás observar que, al ingresar un código previamente ingresado y compararlo, el programa además de mostrar el nombre del producto al que le pertenece el código, también imprime otros dos mensajes del else

Aparte del error con el scanf que te han marcado, el uso de gets (no recomendado y que se ha eliminado en las últimas versiones de C) y el uso de fflush en stdin (la función solo sirve para flujos de salida, en los de entrada no está especificado su funcionamiento) el error que tienes en Busqueda es que no realizas una búsqueda en el array, solo miras el primer elemento.

Sigue el flujo del programa.
Entras en un for para recorrer el array y el if busca si existe el dato. Si lo encuentra devuelve el nombre y sale del for. Bien.
La cosa viene cuando no encuentra el dato: directamente, sin tener en cuenta el resto del array, indica que no ha encontrado el producto y el break hace que salga del for, sin existir posibilidad de buscar en el resto del array.

Por tanto te falta algo para, en caso de que no se encuentre pase al siguiente elemento del array y diga que no hay existencias en caso de no encontrarse.

Por cierto, te iría bien separar la búsqueda de la presentación de resultados, verás que la lógica para llegar a la solución es más clara: es decir, por una parte buscas en el array y si encuentras el elemento marcas como que lo has encontrado y detienes la búsqueda pero si no lo has encontrado indicas que no ha habido éxito. Después según el resultado lanzas un mensaje u otro.