Hola de nuevo, pues ahora hice un programa para practicar las operaciones aritméticas básicas (suma, resta, multiplicación y división). Solamente me gustaría su opinión sobre el mismo, es decir, que les parece la estructura del código. Hice un programa anteriormente (distinto a este) y me hicieron la recomendación de no comentar tanto el código por que se veia muy adornado y pues puse en practica ese consejo, esta comentado pero solo lo necesario. Me gustaría saber en que aspectos podría mejorar mi programa, me refiero a la estructura del código, hago este tipo de preguntas por que quiero aprender a programar limpiamente y hacer que mi código sea fácil de comprender. Agradecimientos por adelantado.

/* Este programa sirve para practicar sumas, restas, multiplicaciones y divisiones */
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
/* Prototipos de funciones */
float generarNumero( int nivelDificultad );
float operacion( float numero1, float numero2, int operacionElegida );
void respuestaCorrecta( void );
void respuestaIncorrecta( void );
void impresionTipoDeOperacion( int tipoDeOperacion );
 
/* Inicia la ejecucion del programa */
int main()
{
	int i, correcta, nivel, operando, operando2;
	float num1, num2, resultado, intento;
	correcta = 0;
 
	srand( time( NULL ) ); /* Modificacion de la semilla de los numeros pseudoaleatorios */
 
	printf( "Bienvenido al sistema de aprendizaje asistido por computadora, se te preguntaran\n" );
	printf( "10 operaciones y luego se te mencionara si necesitas ayuda tutorial.\n" );
 
	printf( "Por favor ingresa el nivel que quieres practicar ( 1 al 3 )..." );
 
	do {		
		scanf( "%d", &nivel );
 
		if ( nivel < 1 || nivel > 3 ) {
			printf( "Nivel no valido, intenta otravez..." );
		}
	} while ( nivel < 1 || nivel > 3 );
 
	printf( "Que tipo de operaciones quiere resolver:\n" );
	printf( "1. Sumas\n2. Restas\n3. Multiplicaciones\n4. Divisiones\n" );
	printf( "5. Operaciones Aleatorias\n" );
	printf( "Recuerda que no es obligatorio hacer las operaciones mentalmente, puedes\n" );
	printf( "apoyarte de lapiz y papel para realizarlas. ATENCION! No se permite el\n" );
	printf( "uso de calculadora.\n..." );
 
	do {
		scanf( "%d", &operando );
 
		if ( operando < 1 || operando > 5 ) {
			printf( "Opcion no valida, intenta de nuevo..." );
		}
	} while ( operando < 1 || operando > 5 );
 
	operando2 = operando; /* Esta variable guarda el valor de operando, esto es para tener intacta nuestra variable y modificar la copia */
 
	for ( i = 1; i <= 10; i++ ) {
		/* Llamamos a las funciones generadoras de numeros aleatorios */
		num1 = generarNumero( nivel );
		num2 = generarNumero( nivel );
 
		if ( operando == 5 ) { /* Y por este motivo es por el que debemos mantener intacta nuestra variable operando */
			operando2 = 1 + rand() % 4; /* Genera una operacion aleatoria */
		}
 
		resultado = operacion( num1, num2, operando2 );	/* Llamamos a la funcion que resuelve la operacion elegida ( o generada ) */
 
		printf( "Cuanto es %.0f ", num1 );
		impresionTipoDeOperacion( operando2 );
		printf( " %.0f?...", num2 );
 
		do {
			scanf( "%f", &intento );
 
			if ( intento != resultado ) {
				respuestaIncorrecta(); /* Llamada a la funcion que genera los mensajes aleatorios cuando se responde incorrectamente */
				correcta--;	/* Se decrementa en uno cada vez que se ingresa una respuesta correcta */
			}		
		} while ( intento != resultado );
 
		/* Cuando se ingresa la respuesta correcta, se llama a a funcion que genera el mensaje aleatorio y se incrementa correcta */
		respuestaCorrecta();
		correcta++;
 
	} /* Fin de for */ 
 
	if ( correcta < 7 ) { /* Si hubo mas de 3 respuestas incorrectas, se imprime este mensaje */
		printf( "Cometiste varios errores, por favor pide ayuda adicional a tu profesor. Hasta la proxima!\n" );
	}
 
	return 0;
} /* Fin de main */
 
/* Definicion de funciones */
float generarNumero( int nivelDificultad )
{
	switch ( nivelDificultad ) { /* Para generar numeros de 1, 2 y 3 digitos en los niveles 1, 2 y 3 respectivamente */
		case 1:
			return 1 + rand() % 9;
			break;
 
		case 2:
			return 10 + rand() % 90;
			break;
 
		case 3:
			return 100 + rand() % 900;
			break;
	} /* Fin de switch */
} /* Fin de funcion generaNumero */
 
float operacion( float numero1, float numero2, int operacionElegida ) 
{
	switch ( operacionElegida ) { /* Para la resolucion de nuestra operacion, esto es para compararlo posteriormente con la respuesta del
	 																user*/
		case 1:
			return numero1 + numero2;
			break;
 
		case 2:
			return numero1 - numero2;
			break;
 
		case 3:
			return numero1 * numero2;
			break;
 
		case 4: 
			return numero1 / numero2;
			break;
	}
} /* Fin de funcion operacion */
 
void respuestaCorrecta( void )
{
	switch ( 1 + rand() % 4 ) {
		case 1:
			printf( "Muy bien!!\n" );
		break;
 
		case 2:
			printf( "Excelente!!\n" );
		break;
 
		case 3:
			printf( "Manten ese buen rendimiento!!\n" );
		break;
 
		case 4:
			printf( "Buen trabajo!!\n" );
		break;
	} /* Fin de switch */
} /* Fin de funcion respuestaCorrecta */
 
void respuestaIncorrecta( void )
{
	switch ( 1 + rand() % 4 ) {
		case 1:
			printf( "No. Por favor intenta de nuevo...\n" );
		break;
 
		case 2:
			printf( "Incorrecto. Trata una vez mas\n" );
		break;
 
		case 3:
			printf( "No te rindas!!\n" );
		break;
 
		case 4:
			printf( "No. Sigue intentando\n" );
		break;
	} /* Fin de switch */
} /* Fin de funcion respuestaIncorrecta */
 
void impresionTipoDeOperacion( int tipoDeOperacion )
{
	switch ( tipoDeOperacion ) {
		case 1:
			printf( "mas" );
			break;
 
		case 2:
			printf( "menos" );
			break;
 
		case 3:
			printf( "por" );
			break;
 
		case 4:
			printf( "entre" );
			break;
	}
} /* Fin de funcion impresionTipoDeOperacion */

Que tal capsulasinformaticas! ¿Entonces no hay necesidad de modificarle nada respecto a la estructuración de código? ¿Alguna recomendación o algo?
   

Ok capsulasinformaticas muchas gracias por tu opinion
 Respecto a lo de la compilacion, creo que es tu compilador por que a mi me compila sin ningun problema.

Ok tomare en cuenta tu consejo amchacon! Muchas gracias!

No!! es mejor dejarlo así, a veces es muy bueno indicar en que momento se acaba el while, pues dependiendo del ide o el editor puede que muestre o no esa ubicación, por eso muy buena idea colocar esas referencias.

Y lo de los if, es solo una forma de validar el dato, hay otras formas, pero esa esta bien, es solo una tecnica, cada quien tiene su propio estilo.

Muy bueno el código, no lo compile, así que no puedo decirte que tal corre, pero mirandolo por encima no le veo problema, es la organización lo que me sorprende, me siento un poco avergonzado al darme cuenta lo mal que documento (o hago poco o nada, o a veces ocupo muchisimas lineas comentando) y creo que es buena idea hablar de este punto.

Lo que acaba de ocurrirseme es diferenciar un poco sobre comentar de una forma la estructura del codigo (fin do-while, fin funcion...) y comentar de otra el funcionamiento del programa (por ejm: Esta variable guarda el valor de operando...) creo que se podria empezar con un signo como % cuando es de estructura y con $ cuando de es de funcionamiento. desde ahora, todo el código que realice, lo hare de esta forma. Gracias muchachos me inspiraron a trabajar mejor, sea el lenguaje que sea.