Buenos días.

Si me aceptas la crítica:

1. La lógica de tu "programa" no debería residir en un constructor. Eso es, por decirlo de forma suave, una aberración. Es mucho mejor tener un método público en "programa" para realizar esta tarea.

2. Separas interfaz de implementación... pero ya está. Creo que el ejemplo sería más útil si separases además la lógica de presentación de la lógica de negocio.

3. Llamar al constructor de la clase desde un miembro sin crear una instancia nueva es otra aberración consecuencia directa de lo comentado en el punto 1. Otro motivo más para cambiar eso.

4. Si una función miembro de una clase no modifica el estado del objeto lo suyo es que sea etiquetada como "const". Si además resulta que dicha función no accede al estado del objeto (las variables miembro de la clase), lo lógico es que dicha función fuese etiquetada como "static". Por cierto, no puedes usar "static" y "const" a la vez en la misma función miembro (básicamente porque una función static no puede acceder, y por tanto modificar, a nada que no sea estático... de hecho en esta función no existirá el puntero implícito "this").

Te animo a seguir mejorando el ejemplo

Un saludo

Lo primero. El código tiene que ir "decorado" con la etiqueta del lenguaje correspondiente. Básicamente para que sea legible y porque lo dicen las normas. Puedes encontrar estas etiquetas en el desplegable que dice "GeSHi" en el editor de mensajes.

C++, como has visto, es un lenguaje en el que se pueden usar objetos. Además, el lenguaje dispone de una librería estándar que, si bien no te va a permitir realizar una aplicación gráfica, dispone al menos de una serie de clases muy útiles para tus programas. Los contenedores son un ejemplo de estas clases. Un contenedor te permite almacenar un número indeterminado de elementos. Esto último supone una gran ventaja sobre los arreglos típicos de C (int numeros[30] no puede almacenar más de 30 números).

Lo que te piden es que almacenes los datos en contenedores en vez de usar un archivo.

1. Variables no inicializadas

if (comprobar_repetir_vector(num_random, clave)==1)

int comprobar_repetir_vector(int numero, int *vector )
{
	int x;
	for(x=0; x<27;x++)
  {
		if(numero==vector[x])
			return 1;
	}
	return 0;
}
 

Si te das cuenta, no estás dando a "clave" unos valores iniciales... simplemente le asignas un valor y después verificas que dicho valor no esté repetido. ¿Problemas? Varios:

• "clave" tendrá inicialmente valores aleatorios. Si uno de estos valores coincide con el que intentas ingresar el algoritmo dará este nuevo valor como no válido.
• No tiene sentido que para cada valor que intentas añadir a "clave" compruebes sus 27 posiciones... lo suyo sería comprobar desde 0 hasta N-1, siendo N el índice del número actual. De esta forma únicamente comparas el número actual con los ya insertados en el arreglo. Corrigiendo este punto podrías olvidarte del problema del punto anterior
• No es buena idea tener el "27" a pelo en el código. ¿Qué sucede si luego tienes que crear una clave de 40 elementos? Te toca editar ese número en varias partes del código... es bastante recomendable tener estos valores en un #define para evitar errores

2. Cuidado con los decimales

Cuando trabajes con números decimales (float, double, ... ) tienes que tener muy en cuenta que NUNCA debes hacer una comparación directa, tipo "if( float == float )". ¿Por qué? Bueno, pues porque un número en coma flotante nunca va a ser 100% preciso. Los números en coma flotante tienen una serie de dígitos significativos, mientras que el resto pueden ser considerados "basura". Si tu fuerzas a una comparación exacta, esa basura va a hacer que la comparación sea bastante inestable. La forma de comparar si un número decimal es igual a otro pasa por asumir que dos números son iguales si el resto entre ellos es menor a X. Ejemplo:

int main( )
{
  // Un float nunca es preciso 100%.
  // Esta inicialización intenta reflejar esta situación
  float a = 5.0001, b;
  for( b=1.0; b < 10.0; b+=1.0 )
  {
    if( a == b )
    {
      printf( "Bucle 1 OK!!!\n" );
      break;
    }
  }
 
  for( b=1.0; b < 10.0; b+=1.0 )
  {
    if( fabs( a - b ) < 0.001 )
    {
      printf( "Bucle 2 OK!!!\n" );
      break;
    }
  }
}

¿Dónde realizas este tipo de comprobaciones? Por ejemplo en la función "suma", que se llama desde "arrancar_motor"... esa es, al menos, una de las razones por las que el código "tarda mucho".

Un saludo

Aparte de lo comentado, dos cositas más:

• El código lo tienes que decorar eligiendo el lenguaje correspondiente en el desplegable que dice "GeSHi". Te crea dos etiquetas, una de apertura y otra de cierre... el código lo pegas dentro.
• Editar un mensaje tiene sentido cuando nadie te ha respondido aún... si ya tienes respuesta, casi mejor añadir una respuesta al hilo para que éste tenga cierta coherencia.

Un saludo.

wxWidgets está centrado únicamente en el desarrollo de interfaces gráficas, mientras que Qt es un framework completo con miles de utilidades:

• Interfaz gráfica
• Gestión de impresoras
• Sockets
• Bases de datos
• Interacción con opengl
• ...

Bien es cierto que dominar todo este chiringuito no es algo que se vaya a conseguir en dos días... pero la verdad es que Qt te ofrece unas librerías muy cuidadas y potentes. Podría decirse que Qt es una especie de C++ orientado hacia Java y .Net.

Si bien crear un proyecto de Qt con QtCreator es de las cosas más sencillas del mundo, configurar otro IDE para que compile satisfactoriamente un proyecto de Qt puede ser un tanto desalentador. Aun así, yo creo que merece la pena probarlo.

Pues mira que bien, ya tienes trabajo para casa, y además parece interesante.

¿pensabas que te íbamos a dar la solución así por las buenas?

A ver, una cosa es ayudar y otra hacer un trabajo a medida... ahora, si quieres presupuesto para hacerte la tarea seguro que más de uno te pasa un precio.

Un saludo

Deberías buscar un manual de C por Internet.

Una cosa es tener dudas sobre cómo funciona un código y otra es preguntar cosas tan básicas que cualquier manual de C te sabe explicar.

Cuando abres un archivo, puedes usar, por ejemplo fopen... si echas un vistazo a la documentación de esta función verás que tienes que indicar la ruta del archivo a abrir además de las restricciones con las que abres el fichero (solo lectura, crear si no existe, etc).

Y esto, básicamente, creo que responde a tus dudas:

* ¿Dónde se encuentran? Donde tú hayas indicado. Si has indicado una ruta relativa tendrás que mirar cual es el directorio de trabajo y empezar a trazar la ruta relativa a partir de ahí.
* ¿Hay que crearlos para que se sobreescriban? No necesariamente... una de las opciones a la hora de abrirlo es que se borre su contenido.
* ¿Dónde los guardo? Donde te de la gana. Sin embargo, recuerda que determinadas rutas pueden tener restricciones de acceso, como determinadas carpetas del sistema. Los únicos requisitos son tener espacio en el disco y permisos de escritura en la carpeta destino y al fichero en el caso de modificaciones.

Un saludo.

Las GeSHi te explico: Habrás notado que, al escribir un mensaje, tienes una serie de botones encima del TextBox para, por ejemplo, poner el texto en negrita, cursiva o subrayado, elegir la alineación del texto, etc... uno de esos controles es un ComboBox con el texto GeSHi. Si despliegas esa lista verás que aparece el nombre de varios lenguajes de programación... elijes el que corresponda y te añadirá un par de etiquetas (una de apertura y otra de cierre) a tu mensaje... pon el código entre medias y listo. También funciona si pones el código, lo seleccionas y luego elijes el idioma en el desplegable.

Para borrar un elemento de una lista tienes 3 posibilidades:

1. El elemento a borrar es el primero. Tienes que hacer que el puntero de inicio de la lista pase al siguiente elemento:
A -> B -> C -> NULL ==> B -> C -> NULL

2. El elemento a borrar es el último. Simplemente pones a NULL el puntero "sig" del penúltimo elemento:
A -> B -> C -> NULL ==> A -> B -> NULL

3. El elemento está en el medio de la lista. En este caso tienes que hacer que el puntero "sig" del elemento anterior apunte al elemento que sigue al actual:
A -> B -> C -> NULL ==> A -> C -> NULL

En cualquiera de los tres casos no te olvides de liberar la memoria del elemento recién liberado.

Para liberar la memoria reservada por un malloc (o calloc o realloc) tienes que usar la función free:

// Reserva de memoria
int* ptr = (int*)malloc( 100 *sizeof( int ) );
 
// Se libera la memoria. Vuelve a estar disponible
free( ptr );

Un saludo

Línea 33: scanf ( "n%c", &item ); <<-- debería ser \n

Línea 136: currentPtr = currentPtr->NextPtr; <-- esa 'n' debería ir en minúsculas

  instruccions();
  scanf ( "%d", &choice );
 
  while ( choice != 3 )
  {
    // ...
 
    printf ( "?" );
    scanf( "%d", &choice );
  }
 

Por qué actualizar "choice" en dos sitios diferentes??

si usas "do-while" en vez de "while" puedes eliminar uno de los usos... además, sería recomendable poner las instrucciones siempre salvo que pretendas que el usuario se las apunte nada más arrancar el programa:

  do
  {
    instruccions();
    scanf ( "\n%d", &choice );
 
    // ...
  } while ( choice != 3 );
 

Claro, ahora tienes que añadir al "switch" un "case 3" para evitar mostrar las instrucciones al salir:

      case 3:
        break;
 
      default:         //si la opcion es invalida le vuelve a preguntar
        printf ( "invalid choice \n" );
        //instruccions(); <<-- Esto también sobra ya
        break;
 

Luego, no se si te has dado cuenta, pero los elementos los estás insertando al inicio de la lista... no al final, no se si esa parte la tienes que programar así, aunque me da que no.

y bueno, luego te toca arreglar algun detalle relacionado con la forma de presentar los mensajes... pero eso ya corre de tu cuenta.

Ah bueno, sí, se me olvidaba jejeje. No estás liberando la memoria reservada por la lista una vez finalizas el programa... deberías plantearte corregir eso también.

Un saludo