Foro de elhacker.net

Hola soy nuevo en el foro, me mandaron a hacer un ejercicio en c que dice lo siguiente:

Una compañía disquera organiza una encuesta para determinar el éxito de sus canciones. La población encuestada es dividida en cuatro categorías de acuerdo al sexo y a la edad (Por ejemplo: en personas de 20 años o menos y mayores de 20 años). A cada persona se le piden sus cinco canciones favoritas, las canciones son identificadas por números del 1 al N. Los resultados de la encuesta son tomados de la entrada estándar. Cada elemento leído representa un encuestado y contiene su nombre, apellido, sexo, edad, cedula y sus cinco canciones favoritas en orden de preferencia. Se desea que Ud. diseñe una estructura de datos en Lenguaje C para almacenar dicha información. Además, se pide realizar un programa que imprima en pantalla la tabla con la relación de los 10 primeros encuestados.

Tengo que realizarlo por el método Burbuja, agradecería un poquito de ayuda.

Hola!

Postea lo que llevas hecho (utiliza etiquetas GeSHi para el código) o realizar una pregunta más concreta.

Saludos!

Hola compañero tengo algo listo pero no creo que esta bien hecho puedes hecharle un vistazo a ver que se le puede cambiar? aqui lo tienes tal y como lo puse, por cierto estoy usando Dev C++, el problema surgue cuando se habre la consola hecha un vistazo a ver que tal.

#include<stdio.h>
#include<string.h>
 
void lista_de_personasM();
void lista_de_personasF();
 
int main()
{
	lista_de_personasF();
}
 
 struct lista_de_personas{
char nombre,apellido;  //variables para nombre y apellido
int CI; // variable para la cedula (DNI)
int edad; //Variable para la edad
int sexo; //variable para el sexo 	
int cancion[5]; //variable para a cancion
 
 
 }lista_de_personas[10];
 
   void lista_de_personasF(){
 
 
   	struct lista_de_personas;
   	int i; //variable bucle 
   	int j; //Variable bucle
 
   		for (i = 0;i < 2; i++){
 
   	//INGRESAMOS LOS DATOS//
   	fflush(stdin);
   	//Guardamos nombre
   	printf("Personas N°[%d] \n",i+1);
   	printf("Introduce tu nombre y apellido por favor: \n");
   	printf("****Nombre**** \n");
   	gets(lista_de_personas[i].nombre);
 
   	//Guardamos apellido
   	printf("****Apellido**** \n");
   	scanf("%s",&lista_de_personas[i].apellido);
   	//Guardamos cedula
   	printf("Introduce cedula de indentidad por favor: \n");
   	scanf("%i",&lista_de_personas[i].CI);
   	//Guardamos Edad
   	printf("Introduce tu edad por favor: \n");
   	scanf("%i",&lista_de_personas[i].edad);
   	//GUardamos genero
   	printf("Introduce sexo 1 = Masculido\n 2 = Femenino \n");
   	scanf("%s",&lista_de_personas[i].sexo);
   		for(j = 0;j < 5; j++){
   		//guardamos canciones
   		printf("Intoduce tus 5 canciones favoritas: \n");
   		printf("TOP[%i] \n",j + 1);   		   		
   		scanf(" %i",&lista_de_personas[i].cancion[j]);
 
	   }
}
 
//Mostramos
for(i=0; i<2; i++){
	        fflush(stdin);
                printf("Persona numero[%d]",i+1);
				printf("Nombre: %s\n",lista_de_personas[i].nombre);
				printf("Numero de cedula: %i\n",lista_de_personas[i].CI);
				printf("edad: %i\n",lista_de_personas[i].edad);
				printf("sexo: %d\n",lista_de_personas[i].sexo);
				for(j = 0; j < 5; j++){
					printf("Tus canciones favoritas son: \n");
					printf("%i",lista_de_personas[i].cancion[j]);
				}
				printf("///////////////////////////////////////////////////////////// \n");
 
 
	   }
}
 
 
 
 
   void lista_de_personasM (){
 
 
   	int i;
   	int j;
   	struct lista_de_personas;
 
 
   	for(i = 0; i < 2; i++){
   			printf("------------------------------------------------------------ \n ");
 			printf("------------------------------------------------------------ \n");
			printf("Nombre: %s\n", lista_de_personas[i].nombre);
			printf("Numero de cedula: %i\n", lista_de_personas[i].CI);
			printf("edad: %i \n", lista_de_personas[i].edad);
			printf("sexo: %s",lista_de_personas[i].sexo);
			for(j = 0; j < 5; j++){
				printf("Tus canciones favoritas son: \n");
				printf("%i",lista_de_personas[i].cancion[j]);
				}
	   }
   } 

---

MOD: El código debe estar entre etiquetas de Código GeSHi

Bueno... A ver a ver. Parece que hay mucho trabajo por delante así que vamos a ir punto por punto:

• Las funciones no siguen las normas básicas de las funciones:
  Una función debe realizar una función completa y autónoma. Una función es un fragmento de código que puedes reutilizar muchas veces. Para ello tiene que hacer una tarea muy bien definida y genérica. Por ejemplo: calcularMedia(...), mostrarMatriz(...), buscarMaximo(...), ... Si te fijas es muy sencillo saber qué hacen esas funciones, en cambio dos funciones que se llaman lista_de_personas_M() y lista_de_personas_F() no se sabe qué hacen a simple vista. Y además no hacen una tarea concreta sino que tienen dentro todo el programa en sí.
  
  
• El nombre del struct no es del todo preciso. La estructura define una persona, no una lista de personas. Además estás definiendo en la línea 20 un array de struct global (no recomendable). Y cada vez que quieras usar el struct debes poner la palabra struct: en la línea 25 te falta el nombre de la variable. Poner: 'struct lista_de_personas;' es como poner: 'int;' Falta el nombre de la variable, sólo estás indicando el tipo que va a tener pero no cómo se llama. Seguramente este sea el primer error que te da al compilar.
  Otra cosa importante es que estás usando un char para guardar el nombre y otro para el apellido. Es decir, un único carácter para cada cosa, no una cadena.
  
  Una recomendación es usar 'typedef' para no tener que usar la palabra 'struct' todo el tiempo.
  

// Consejo EXTRA: Utiliza constantes para la longitud de los arrays estaticos
#define MAX_LONGITUD_NOMBRE 20
#define MAX_LONGITUD_APELLIDO 50
 
typedef struct {
  char nombre[MAX_LONGITUD_NOMBRE];
  char apellido[MAX_LONGITUD_APELLIDO];
  // El DNI lo guardaria como un char[] y no como un int
  //...
} Encuestado;
 
int main() {
  // Ahora para declarar un array de encuestados no hace falta hacer:
  struct Encuestado encuestados[];
  // Tan solo tienes que hacer:
  Encuestado encuestados[];
} 

• El main() de un programa de este estilo debería dar una visión general de lo que hace el programa (no cómo lo hace pero sí qué hace). En este caso si lees el main() te quedas más o menos así:  :huh:
  Yo apostaría por algo más de este estilo:

#define NUM_ENCUESTADOS 10
#define NUM_CANCIONES 5
//...
 
int main() {
  Encuestado encuestados[NUM_ENCUESTADOS];
 
  for(int i = 0; i < NUM_ENCUESTADOS; ++i) {
    printf("Introduce el nombre del encuestado %d: ", (i+1));
    // Consejo EXTRA: Las cadenas de texto se leen con fgets() no con scanf(). Volveremos a esto mas adelante
    fgets(personas[i].nombre, MAX_LONGITUD_NOMBRE, stdin);
 
    printf("Introduce la edad del encuestado %d: ", (i+1));
    scanf("%d", &personas[i].edad);
 
    //...
  }
  // Tambien puedes hacer una funcion que sea pedirDatosEncuestado(Encuestado encuestado) y dentro poner todos los mensajes pidiendo datos y guardando la entrada del teclado...
 
  // Aqui ya puedes usar las funciones que quieras en base a lo que necesites
  for(int i = 0; i < NUM_ENCUESTADOS; ++i) {
    printf("***** DATOS DEL ENCUESTADO %d *****\n", (i+1));
    // En el main() no necesito saber como va a mostrar los datos pero se lo que va a hacer. Para los detalles siempre puedes mirar dentro de la funcion
    mostrarDatosEncuestado(encuestados[i]);
    printf("\n");
  }
} 

• Entrada de datos: La función fflush() no debe usarse para limpiar el buffer de entrada (stdin). El fin de esta función (mal utilizada) es eliminar los saltos de línea '\n' que se insertan en el buffer de entrada al pulsar Enter. Aquí pueden darse 2 casos:
  
  • Lectura de cadenas de caracteres. La forma correcta/segura es mediante la función fgets(). Aquí pueden darse 2 casos también:
    
    • Se introducen más caracteres de los que se pueden almacenar en el array -> En el buffer quedarán todos los caracteres que no caben en el array y al final el salto de línea '\n'. Todo esto que queda en el buffer tiene que limpiarse.
      
    • Se introducen menos caracteres de los que se pueden almacenar en el array -> Se guardan todos los caracteres (incluido el salto de línea) y el buffer queda vacío.
      
    Para ambos casos se puede usar el siguiente código:
  Código

  char cadena[MAX_SIZE]; // La longitud maxima es indiferente
  printf("Introduce una cadena: ");
  fgets(cadena, MAX_SIZE, stdin);
  if(cadena[strlen(cadena) - 1] == '\n') // Si el ultimo caracter guardado es el salto de linea -> El buffer esta limpio
    cadena[strlen(cadena) - 1] = '\0'; // Eliminamos el salto de linea que es muy molesto
  else // En caso de que el ultimo caracter no sea un salto de linea, significa que quedan restos en el buffer de entrada -> Hay que limpiar todo
    while(getchar() != '\n'); // Elimina los caracteres que quedan en el buffer hasta encontrar el salto de linea (que es el ultimo caracter del buffer) 

  • Lectura de datos que no sean cadenas de caracteres. Para este caso la función más común a utilizar es scanf(). Esta función nunca incluye el salto de línea dentro de la variable por lo que siempre quedará en el buffer. En este caso tan solo hay que limpiar todo el buffer justo después de un scanf() para asegurarnos de que siempre estará limpio.
  Código

  printf("Introduce un numero: ");
  scanf("%d", &numero);
  while(getchar() != '\n'); // Con el bucle podremos limpiar cualquier resto si se introduce un dato no valido ademas del salto de linea 

---

Y con esto creo que tienes ya recomendaciones para rato. Cualquier duda de estos temas tienes temas en este foro más antiguos donde se explican, puedes usar el botón de "Buscar" para encontrarlos.
PD: Cuidado con los tipos de datos, tienes por ahí un %s para guardar un int...  :rolleyes: :rolleyes:
PD2: Mencionas el método burbuja pero eso es un algoritmo de ordenación y hasta donde yo he entendido no necesitas ordenar nada en este programa... :huh: