Mensajes

Tienes que diferenciar entre un error y un warning. Un error aborta la compilación. Un warning es cuando el compilador te avisa de "algo" que puede resultar ambiguo, extraño o que puede causar problemas, pero que puede seguir compilando sin problemas.

¿De que tipo es la variable palabras?

[y 6 posibilidades]

Con 4 cifras y 6 posibilidades, son 360. 6*5*4*3.

Huy es verdad, menudo despiste

Ten en cuenta que hoy en día importa más la eficiencia que la memoria, hoy en día nadie se preocupa por 6-7 bytes de más pero sí un progama tarda menos o no. El realloc es una operación costosa (creas un nuevo vector, copias el contenido al nuevo vector byte a byte y borras el antiguo).

No obstante, te presento una versión con realloc "amortiguado":

Código (c) [Seleccionar] Expandir

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void LeerLinea(char** Linea)
{
   const int Incremento = 5;
   int tam = 0;
   int max = Incremento;
   int ch;

   (*Linea) = (char*) calloc(max,sizeof(char));

   while((ch = getchar()) != '\n' && ch != EOF)
   {
       (*Linea)[tam++] = ch;

       if (tam == max)
       {
           max += Incremento;
           (*Linea) = (char*)realloc (*Linea,max);

           if (*Linea == NULL)
           {
                printf("\n\t  Error de asignación de memoria, adios!!");
                exit(-1);
           }
       }
   }

   (*Linea)[tam] = 0; // caracter nulo
}

int main()
{
   char **cadena;
   int num_cadenas;
   int contador=0;
   int i,j;

   do
   {
       printf("Digite el numero de cadenas:  ");
       scanf("%d", &num_cadenas);
   }
   while(num_cadenas < 1);

   getchar();

   cadena = (char**) calloc(num_cadenas, sizeof(char*));

   if(cadena == NULL)
   {
       printf("\n\t  Error de asignación de memoria, adios!!");
       exit(-1);
   }

   // leyendo cada linea en memoria
   for(i=0; i<num_cadenas; i++)
   {
       LeerLinea(&cadena[i]);
   }

   for(i=0; i<num_cadenas; i++)
   {
       for(j=0;cadena[i]; j++ )
       {
           if(cadena[i][j] == ' ')
           {
               cadena[i][j] = cadena[i][j+1];
           }
       }
   }


   for (i = 0; i < num_cadenas; i++)
       printf("cadena # %2d: %s\n", i + 1, cadena[i]);


   /* Primero se libera la memoria de cada linea ... */
   for (i = 0; i < num_cadenas; i++)
       free(cadena[i]);

   /* ... a continuacion el bloque de memoria principal */
   free(cadena);
   return 0;
}

Es "realloc amortiguado" porque aumento la memoria de 5 en 5 en vez de 1 en 1. Así hago mucho menos reallocs y como mucho desperdicio 4 bytes.

Cuando veas un código con un # es una directiva de compilador. Eso no se verá en el código final, por tanto el código compilado no es portable.

Lo que uso se llama "compilación condicional", si estoy compilando en windows windows hará esa línea y si estoy en linux compilará la otra. Si no estoy en ninguno de los dos muestra un error de compilación.

No solo es para los sistemas operativos, puede usarlo para las cosas que quieras. Por ejemplo, puedes definir cosas que se muestren solo en "modo depuración":

Código (cpp) [Seleccionar] Expandir

#define DEBUG

//... codigo

void funciona()
{
   #ifdef DEBUG
            ofstream escritura("debug.txt");

            //...

           // Imprimir el valor de las variables internas en un archivo...
   #endif
}

Si yo quito el #define DEBUG de ahí arriba, entonces esa función no hará nada.

Tengo xp, y haciendo el include de windows.h si que me va.

No te digo que no. Pero una de dos, o windows.h tiene una función system propia o hace un include de la stdlib.

¿cstdlib es estandar? O sea a ver si me explico: ¿si lo llevo a linux sirve con el system()?

Sí, otra cosa son los comandos que le pases.

Por ejemplo, mira esta función génerica para limpiar la pantalla:

Código (cpp) [Seleccionar] Expandir

void limpiarPantalla()
{
   #ifdef _WIN32
       system("cls");
   #elif __gnu_linux__
       system("clear");
   #else
       #ERROR La funcion LimpiarPantalla no está definida para tu OS.
   #endif
}

En C, system es de la librería <stdlib.h>. En C++ es <cstdlib>

Otra cosa esque windows.h incluya esa librería por su cuenta.

Son 360 posibilidades. Mejor haz una funcion que lo verifique.

Con 4 cifras son 24 posibilidades (4 x 3 x 2 x 1).

En cuanto al tema en cuestión te dejo otra propuesta más generica:

Código (cpp) [Seleccionar] Expandir

// Prototipos...

bool comprobarCoincidencias(int a,int array[],unsigned int tam);
void generarNumeros(int Vector[],unsigned int tam, int max = RAND_MAX,int min = 0);

// Otro codigo...

// Declaracion de las funciones

void generarNumeros(int Vector[],unsigned int tam, int max,int min)
{
  // Se presupone que Vector tendrá al menos el tamaño de tam.
 
  if (tam == 0) return;
 
  Vector[0] = rand();

  for (unsigned int i = 1; i < tam;i++)
  {
         do
         {
             Vector[i] = (rand()%max)+min;
         } while (comprobarCoincidencias(Vector[i],Vector,i));
  }
}

bool comprobarCoincidencias(int a,int array[],unsigned int tam)
{
   for (unsigned int j = 0; j < tam;j++)
   {
         if (array[j] == a) return true;
   }

   return false;
}

No lo he probado pero debería funcionar. Tienes que pasarle el vector que quieres que te rellene, el numero de cifras y el máximo/menor (no inclusivo) valor que puede coger cada cifra. En este caso el maximo sería 10 porque quieres números del 0-9 supongo.

Por cierto:

Código (cpp) [Seleccionar] Expandir

void generarNumeros(int Vector[],unsigned int tam, int max = RAND_MAX,int min = 0);

Los "=" son son argumentos por defecto. Si no le dices nada el compilador te los rellena automaticamente:

Código (cpp) [Seleccionar] Expandir

generarNumeros(vector,tam);      // max = RAND_MAX y min = 0
generarNumeros(vector,tam,10);   // max = 10 y min = 0
generarNumeros(vector,tam,10,1); // max = 10 y min = 1

¿Estas usando el compilador de C?

que el compilador no soporte

Código [Seleccionar] Expandir

system("pause");

system es la librería estándar.

Yo lo he interpetrado como n caraceteres 

Por cierto:

Código (cpp) [Seleccionar] Expandir

while ((ch = getchar()) != '\n') {
    Cadena[i++] = ch;
           //printf("\ni=%d\n",i);
}

¡Peligro de desbordamiento! Si le meto más caracteres de la cuenta (5) estaré desbordando el array.

Ah leñe, lo que tengo que hacer ahí es cambiar este printf:

Código (cpp) [Seleccionar] Expandir

printf("\nIntroduce un numero de 4 digitos: \n");

Por este más genérico:

Código (cpp) [Seleccionar] Expandir

printf("\nIntroduce un numero de %d digitos: \n",MAX);

Ala, problema resuelto