Autor Tema: Rellenar vector sin repetir (Leído 14358 veces)

Gussr78 · « **en:** Sábado 1 de Enero de 2011, 19:56 »

Hola, el caso es que estoy intentando enteder los vectores, y quiero rellenar manualmente un vector introduciendo números y comprobando que no se repitan.

Lo que tengo y que no me funciona es esto. A ver si alguien me puede ayudar y explicar se lo agradezco.

Código: C

 
#include <stdio.h>
#include <stdlib.h>
 
int main(){
 
  typedef int numeros[10];
  int num,temp;
  numeros misnumeros;
 
 
  for (int i=0;i<11;i++){
    misnumeros[i]=0;
  }
  num=0;
 
  while (num<10){ /*comienzo a introducir números*/
  printf("introduzca un numero: ");
  scanf("%d",&temp); /*Lo guardo en temp para comparar con los datos del vector*/
     for (int i=0;i<=num;i++){ /*Comprobar si ya existe*/
     while ((temp==misnumeros[i])!=0){
    printf ("El numero existe, introduzca otro numero: ");
    scanf ("%d",&temp);
    printf ("El  numero temp vale %2d.n",temp);
  }
}
  misnumeros[num]=temp; /*Si no existe pasamos el valor al vector*/
  num++;
}
for (int i=0;i<10;i++){
    printf ("los numero son: %2d.n",misnumeros[i]);
  }
  system("pause");
}
 
 

m0skit0

Siempre es aconsejable que indiques concretamente qué problema te está dando.

Para comprobar, es así:

Código: C

while (temp==misnumeros[i])
 

Aparte, fíjate que cuando encuentras un número repetido, al pedir otra vez el número, no vuelves a comprobar que no esté ya en el vector. Sólo compruebas esa misma posición.

Código: C

while (temp==misnumeros[i]){
    printf ("El numero existe, introduzca otro numero: ");
    scanf ("%d",&temp);
    printf ("El  numero temp vale %2d.n",temp);
}
 

Además, deberías estructurar más tu código para que sea más legible. Tal y como lo tienes es un caos. ¿Has utilizado funciones alguna vez?

Saludos

Gussr78

Hola, gracias por responder. La verdad es que llevo muy poco tiempo aprendiendo y además de forma autodidacta. En cuanto a las funciones si las he usado, pero en este caso solo estoy intentando entender para luego desarrollarlo mas estructurado y seguro que con funciones como túdices.

Ya veo lo que me comentas de no comparar nuevamente cuando se introduce un nuevo valor, pero no se me ocurre como hacerlo sin repetir todo desde el for, a lo mejor ahí es donde dices de poner funciones??, si me puedes dar una idea te lo agradezco.

Saludos.

m0skit0

Ahí va un ejemplo:

Código: C

#include <stdio.h>
#include <stdlib.h>
 
// Devuelve la posición del número en el vector
// -1 si no se encuentra
// num: número a buscar
// misnumeros: vector en el que buscar
// tam_vector: tamaño del vector a buscar
int posicion_numero_vector(const int num, const int vec[], const unsigned int tam_vector)
{
    int pos = -1;
    int i = 0;
    while(i < tam_vector)
    {
        if (vec[i] == num)
        {
            pos = i;
            i = tam_vector;
        }
        else
        {
            i++;
        }
    }
 
    return pos;
}
 
int main()
{ 
    int misnumeros[10];
    int i, num, temp;
 
    for (i=0;i<10;i++){
        misnumeros[i]=0;
    }
 
    num=0;
 
    while (num<10){
        printf("introduzca un numero: ");
        scanf("%d",&temp); /*Lo guardo en temp para comparar con los datos del vector*/
        for (i=0;i<=num;i++){ /*Comprobar si ya existe*/
            while (posicion_numero_vector(temp, misnumeros, sizeof(misnumeros)/sizeof(int)) >= 0){
            //while (temp==misnumeros[i]){
                printf ("El numero existe, introduzca otro numero: ");
                scanf ("%d",&temp);
                printf ("El  numero temp vale %2d.n",temp);
            }
        }
        misnumeros[num]=temp; /*Si no existe pasamos el valor al vector*/
        num++;
    }
 
    for (i=0;i<10;i++){
        printf ("los numero son: %2d.n",misnumeros[i]);
    }
 
    system("pause");
}
 

LapizTolita

Una recomendación usar getchar(); en vez de system("pause"); por que? simplemente porque llamar al sistema es costoso tener que guardar en contexto hacer el pedido, si trabajas en otro sistema operativo puede que no funcione y aparte es inseguro.

Gussr78

Gracias por vuestras respuestas.

En cuanto a usar getchar(), no se por que puede ser que no me lo ejecuta, se cierra el programa y ya está. No se si es que me falta algo mas que poner.

En fin, gracias.

LapizTolita

Quiza te queda lleno el buffer te doy 2 opciones

Alternativa windows

Código: C++

#include <windows.h>
#define PAUSA printf("nPulse enter para continuar");
    getch();
int getch(void)
{
  int caracter;
  DWORD leidos, modo;
 
  GetConsoleMode(GetStdHandle(STD_INPUT_HANDLE), &modo);
  SetConsoleMode(GetStdHandle(STD_INPUT_HANDLE), modo & !ENABLE_ECHO_INPUT & !ENABLE_PROCESSED_INPUT);
  ReadConsole(GetStdHandle(STD_INPUT_HANDLE), &caracter, 1, &leidos, NULL);
  SetConsoleMode(GetStdHandle(STD_INPUT_HANDLE), modo);
  return caracter;
}
 
 

O podes usar la misma macro pero llamando a esta función para limpiar el buffer que es mas portable, además te va a servir para otros momentos que necesites vaciar el buffer del teclado.

Código: C++

 
#include <unistd.h>
#include <errno.h>
#include <fcntl.h>
#define PAUSA printf("nPulse enter para continuar");
     stdin_flush();
     getchar();
int stdin_flush(){
  char buf[64];
  unsigned int bytes_read;
  char flag;
 flag = fcntl(0, F_SETFL, O_NONBLOCK);
 if(flag == -1){
   return flag;
 }
 while(1){
    bytes_read = read(0, buf, sizeof(buf));
    if(bytes_read == 0){
     flag = fcntl(0, F_SETFL, O_SYNC);
    return flag;
    }
    if(bytes_read == -1){
      if(errno = EWOULDBLOCK){
        fcntl(0, F_SETFL, O_SYNC);
        return flag;
      }
      write(2, "Error #%d while reading stdinn", errno);
      flag = fcntl(0, F_SETFL, O_SYNC);
      return -1;
    }
  }
}
 
 

Podrías usar fflush(stdin); pero también podrías freír huevos con aceite del auto

m0skit0

Le complicas la vida demasiado al chaval sólo por algo insignificante como que no se cierre la ventanita

Y encima le dices que no es portable, pero le plantas un getchar() con WinAPIs

Eso de cerrarse la ventana es simplemente porque lo lanzas desde el IDE en una ventana externa. Si abres un terminal Windows y lanzas el programa verás que no se cierra (o si usas otro IDE que tenga consola integrada, por ejemplo Eclipse, tampoco lo necesitas). Es algo que no tiene relevancia ni importancia, realmente, como para estar dándole tantas vueltas.

Saludos

LapizTolita

Bueno veo que te molesto que un random como yo te de sugerencias sobre como programar, te juro que cuando postie lo de la portabilidad era para darle al pibe ese una sugerencia que a mi me habían dado (si soy algo nuevo en esto de la programación)

Otra cosa no sé porque decís que

Cita de: "m0skit0"

Y encima le dices que no es portable, pero le plantas un getchar() con WinAPIs

Siendo que ahí aclaro que esa es la alternativa Windows y por eso le doy 2 alternativas. Capaz que si hubieras leido por lo menos lo que escribí antes de responderme te hubieras dado cuenta de que la función que esta implementada con WinAPIs es getch() que por cierto en el único lugar que es "nativa" es en borland, y que puse:

Cita de: "lapiztolita"

O podes usar la misma macro pero llamando a esta función para limpiar el buffer que es mas portable, además te va a servir para otros momentos que necesites vaciar el buffer del teclado.

Ahí claramente expreso que podes usar la misma macro, oh wait!! si era "mas portable" como usar el getch() implementado con WinAPIs ? claro está que es un getchar(); pero bueno de esos detalles se dan cuenta los que leen por lo menos una vez el post.

Además yo pruebo todo lo que le digo no escribo porque si nada más, si dije que era más portable fue porque lo probé en ubuntu también.

m0skit0

Cita de: "LapizTolita"

Bueno veo que te molesto que un random como yo te de sugerencias sobre como programar

SoloCodigo

Autor Tema: Rellenar vector sin repetir (Leído 14358 veces)

Gussr78

Rellenar vector sin repetir

m0skit0

Re: Rellenar vector sin repetir

Gussr78

Re: Rellenar vector sin repetir

m0skit0

Re: Rellenar vector sin repetir

LapizTolita

Re: Rellenar vector sin repetir

Gussr78

Re: Rellenar vector sin repetir

LapizTolita

Re: Rellenar vector sin repetir

m0skit0

Re: Rellenar vector sin repetir

LapizTolita

Re: Rellenar vector sin repetir

m0skit0

Re: Rellenar vector sin repetir