Foro de elhacker.net

Hola, buenas necesito un poco de ayuda a ver si me podéis echar una mano, necesito que hacer un programa en C que  me pida que escriba un número y que me diga a que mes pertenece.
A partir de este código, tengo que hacer que me pida el número hasta que introduzca un 0, he pensado en introducir un while ( mes !=  0) arriba del while de los meses pero no me sale.
¿Algún consejo?
/*
**   Fichero:   estaciones.c
**   Autor:      Departamento de Ingenieria Telematica
**   Fecha:      24-09-2012
**
**   Descripcion:   Determina (aproximadamente) la estacion del año en
**         funcion del numero de mes que se le indique por teclado.
*/

/* Includes del sistema */
#include <stdio.h>

/* Includes de la aplicacion */

/* Constantes del fichero */
#define ERROR        0
#define INVIERNO     1
#define PRIMAVERA    2
#define VERANO       3
#define OTONO         4

#define ENERO         1
#define FEBRERO      2
#define MARZO        3
#define ABRIL        4
#define MAYO         5
#define JUNIO        6
#define JULIO        7
#define AGOSTO        8
#define SEPTIEMBRE   9
#define OCTUBRE     10
#define NOVIEMBRE   11
#define DICIEMBRE   12

/* Tipos definidos por el usuario para el fichero */
/* Prototipo de funciones locales */

int main ()
{
  int mes;
  int estacion;

  printf("Introduzca un mes: ");
  scanf(" %d", &mes);
  while ( mes > DICIEMBRE || mes < ENERO )
   {
    printf ("Introduzca un mes: ");
    scanf (" %d", &mes);
   }
      
  if(DICIEMBRE == mes || ENERO == mes || FEBRERO == mes)
  {
  estacion = INVIERNO;
  }
  else if (MARZO == mes || ABRIL == mes || MAYO == mes)
  {
  estacion = PRIMAVERA;
  }
  else if (JUNIO == mes || JULIO == mes || AGOSTO == mes)
  {
  estacion = VERANO;
  }
  else if(SEPTIEMBRE == mes || OCTUBRE == mes || NOVIEMBRE == mes)
  {
  estacion = OTONO;
  }
  else
  {
  estacion = ERROR;
  }
  printf ("El mes %d pertenece a la estacion %d\n", mes, estacion);

return 0;
}

Cuando pones:

printf("Introduzca un mes: ");
  scanf(" %d", &mes);
  while ( mes > DICIEMBRE || mes < ENERO )
   {
    printf ("Introduzca un mes: ");
    scanf (" %d", &mes);
   }

Lo haces porque necesitas que se ejecute una vez primero el mensaje no? para eso se usa Do-While, asi se ejecuta siempre una vez y luego si compara si es correcto para volver a preguntar o no. Tambien te aconsejaria que pongas el comando para limpiar la pantalla cada vez que entra al bucle Do-While.

Despues, en vez de hacer tantos "ifs" se usa "Swich" para mejor comodidad y se entendera mejor, busca para ver como aplicarlo.

Programa en papel y lapiz antes siempre asi te salen mejor las cosas.

Se espera la correcion a ver si logras mejorar algunas cosas por lo menos :P Sino preguntas de nuevo que alguien te contestara siempre, saludos!

Lo que yo he entendido, es que quieres que al introducir 0 en los meses cierre el programa, esto se hace facilmente añadiendo esto despues del scanf que pide el numero del mes:

printf("Introduzca un mes: ");
    scanf(" %d", &mes);
    if (mes == 0) break;
 

Aparte de esto, he metido todo el codigo en un bucle do - while para que al final te pida si quieres volver a hacerlo o finalizar el programa, al final el codigo queda asi:

/*
**   Fichero:   estaciones.c
**   Autor:      Departamento de Ingenieria Telematica
**   Fecha:      24-09-2012
**
**   Descripcion:   Determina (aproximadamente) la estacion del año en
**         funcion del numero de mes que se le indique por teclado.
*/
 
/* Includes del sistema */
#include <stdio.h>
 
/* Includes de la aplicacion */
 
/* Constantes del fichero */
#define ERROR        0
#define INVIERNO     1
#define PRIMAVERA    2
#define VERANO       3
#define OTONO         4
 
#define ENERO         1
#define FEBRERO      2
#define MARZO        3
#define ABRIL        4
#define MAYO         5
#define JUNIO        6
#define JULIO        7
#define AGOSTO        8
#define SEPTIEMBRE   9
#define OCTUBRE     10
#define NOVIEMBRE   11
#define DICIEMBRE   12
 
/* Tipos definidos por el usuario para el fichero */
/* Prototipo de funciones locales */
 
int main ()
{
  int mes;
  int estacion;
  int opcion;
 
  do
  {
    printf("Introduzca un mes: ");
    scanf(" %d", &mes);
    if (mes == 0) break;
 
 
    if(DICIEMBRE == mes || ENERO == mes || FEBRERO == mes)
    {
    estacion = INVIERNO;
    }
    else if (MARZO == mes || ABRIL == mes || MAYO == mes)
    {
    estacion = PRIMAVERA;
    }
    else if (JUNIO == mes || JULIO == mes || AGOSTO == mes)
    {
    estacion = VERANO;
    }
    else if(SEPTIEMBRE == mes || OCTUBRE == mes || NOVIEMBRE == mes)
    {
    estacion = OTONO;
    }
    else
    {
    estacion = ERROR;
    }
    printf ("El mes %d pertenece a la estacion %d\n", mes, estacion);
    printf("Pulse cualquier letra para volver al inicio, pulse 0 para terminar el programa\n");
    scanf("%d", &opcion);
 
  }while (opcion != 0);
 
return 0;
}

Tambien te recomendaria que cambiases todos los if por un switch, es mas como y mas facil de entender.

Saludos

P.D: Advertencia - mientras estabas escribiendo, una nueva respuesta fue publicada. Probablemente desees revisar tu mensaje.

el problema es que no puedo usar ni break ni Swich porque es para clase probare lo del do while

Perdonar que siga dando el tostón pero estoy muy jodido  :huh:   a ver ahora, utilizando tu consejo  Caster, consigo que me diga la estación a la que pertenece el mes y que si pulso 0 se cierra el programa pero ahora lo que no consigo hacer es que cuando introduzca un mes incorrecto no lo procese, y ahora me dice: " el mes x pertenece a la estación 0"
Pego aquí el codigo:
/*
**   Fichero:   estaciones.c
**   Autor:      Departamento de Ingenieria Telematica
**   Fecha:      24-09-2012
**
**   Descripcion:   Determina (aproximadamente) la estacion del año en
**         funcion del numero de mes que se le indique por teclado.
*/
 
/* Includes del sistema */
#include <stdio.h>
 
/* Includes de la aplicacion */
 
/* Constantes del fichero */
#define ERROR        0
#define INVIERNO    1
#define PRIMAVERA 2
#define VERANO       3
#define OTONO         4
 
#define ENERO         1
#define FEBRERO     2
#define MARZO         3
#define ABRIL            4
#define MAYO            5
#define JUNIO            6
#define JULIO             7
#define AGOSTO        8
#define SEPTIEMBRE   9
#define OCTUBRE     10
#define NOVIEMBRE   11
#define DICIEMBRE   12
 
/* Tipos definidos por el usuario para el fichero */
/* Prototipo de funciones locales */
 
int main ()
{
  int mes;
  int estacion;
 
  do
  {
    printf("Introduzca un mes: ");
    scanf(" %d", &mes);
    while (mes == 0)
     {
      return 0;
     }
 
 
    if(DICIEMBRE == mes || ENERO == mes || FEBRERO == mes)
    {
    estacion = INVIERNO;
    }
    else if (MARZO == mes || ABRIL == mes || MAYO == mes)
    {
    estacion = PRIMAVERA;
    }
    else if (JUNIO == mes || JULIO == mes || AGOSTO == mes)
    {
    estacion = VERANO;
    }
    else if(SEPTIEMBRE == mes || OCTUBRE == mes || NOVIEMBRE == mes)
    {
    estacion = OTONO;
    }
    else
    {
    estacion = ERROR;
    }
    printf ("El mes %d pertenece a la estacion %d\n", mes, estacion);
   
   
 
  }while (mes != 0);
 
return 0;
}

Por fiin, muchas gracias a todos los que habéis contestado por fin me ha salido no se hasta que punto es correcto el return 0 ese que he puesto por hay en medio pero funcionar el programa funciona.
Ya mañana indagare que llevo desde las 5 con el puto programa.
Os pego como lo he dejado al final.
/*
**   Fichero:   estaciones.c
**   Autor:      Departamento de Ingenieria Telematica
**   Fecha:      24-09-2012
**
**   Descripcion:   Determina (aproximadamente) la estacion del año en
**         funcion del numero de mes que se le indique por teclado.
*/
 
/* Includes del sistema */
#include <stdio.h>
 
/* Includes de la aplicacion */
 
/* Constantes del fichero */
#define ERROR        0
#define INVIERNO     1
#define PRIMAVERA    2
#define VERANO       3
#define OTONO        4
 
#define ENERO        1
#define FEBRERO      2
#define MARZO        3
#define ABRIL        4
#define MAYO         5
#define JUNIO        6
#define JULIO        7
#define AGOSTO       8
#define SEPTIEMBRE   9
#define OCTUBRE     10
#define NOVIEMBRE   11
#define DICIEMBRE   12
 
/* Tipos definidos por el usuario para el fichero */
/* Prototipo de funciones locales */
 
int main ()
{
  int mes;
  int estacion;
 
  do
  {
    printf("Introduzca un mes: ");
    scanf(" %d", &mes);
    while (mes == 0)
     {
      return 0;
     }
 
     {
      while ( mes < ENERO || mes > DICIEMBRE )
      {
       printf ("Introduzca un mes: ");
       scanf (" %d", &mes);
      }
     }
    if(DICIEMBRE == mes || ENERO == mes || FEBRERO == mes)
    {
    estacion = INVIERNO;
    }
    else if (MARZO == mes || ABRIL == mes || MAYO == mes)
    {
    estacion = PRIMAVERA;
    }
    else if (JUNIO == mes || JULIO == mes || AGOSTO == mes)
    {
    estacion = VERANO;
    }
    else if(SEPTIEMBRE == mes || OCTUBRE == mes || NOVIEMBRE == mes)
    {
    estacion = OTONO;
    }
    else
    {
    estacion = ERROR;
    }
    printf ("El mes %d pertenece a la estacion %d\n", mes, estacion);
   
   
 
  }while (mes != 0);
 
return 0;
}

Si, el codigo funciona, el uso de return 0 en medio del programa nose si es correcto, supongo que si, pero el bucle while que contiene return 0, no tiene sentido que sea un bucle, porque esa sentencia termina el programa, entonces solo se ejecutara una vez, lo mas logico seria con un if, cambia esto:

while (mes == 0)
     {
      return 0;
     }

Por esto:

if (mes == 0) return 0;

Saludos

Normalmente se devuelve 0 cuando el programa termina correctamente u otro número si hay un error. Con el return 0 funcionará, pero como técnica sería mejor poner otro número (normalmente 1 o -1).

Saludos.

No he leído el código pero además si dentro del while no tuviera una instrucción que modificara la variable de control del bucle, este se iría a infinito. Porque tiene return y termina la ejecución de la función que si no... Catapún xD

Por cierto:

if (mes == 0) return 0;

Es una redundancia, lo lógico sería escribir:

return mes

Por otra parte, dejas el return en una bifurcación, que puede ser tomada o no. Problema avistado xD

¡Saludetes!

Muchas gracias por todos vuestros consejos, me habéis salvado el culo, al final lo he dejado cambiando el while ese del principio por un if, como ha dicho Caster pero el return 0; no lo puedo quitar, si lo cambio por un 1 o -1 , como habéis dicho, cuando escribo 0 no termina y si pongo return mes tampoco.

Pero si no pones 0, el programa no termina, y lo que esta tratando de buscar eaguel es que el programa se cierre.


Sera una redundancia, pero si pones return mes el programa no se cierra, y el return 0 esta puesto para que se cierre si introduces el numero 0, lo que no entiendo es lo de la bifurcacion, el return 0 solo se ejecutara si introduces el numero 0, y despues el otro return 0 si la ejecucion del programa es correcta, no hay mas.

Saludos

Disculpa, eso pasa cuando no lees los códigos xD.

Otra cosa, el do while va a terminar cuando mes sea igual a 0... Pero dentro de ese ciclo tienes otro ciclo que verifique que introduces números entre 1 y 12 para mes... Nunca va a terminar...

while (mes < ENERO || mes > DICIEMBRE) {..}

Ahí pides un número entre 1 y 12 para mes... Por tanto al llegar al final del do while, mes nunca será 0.

Si lo colocas así ya va a funcionar como quieres:

while ((mes < ENERO-1) || mes > DICIEMBRE) {..}

Ahi si que tienes razon, si la primera vez introduces un mes incorrecto, despues ya no saldria del bucle, con tu solucion es suficiente.

Saludos

Tenéis razón si introduzco un número raro antes de uno correcto ya no se corta con el 0, lo he corregido introduciendo otra vez el if ese del principio dentro del bucle.

No tienes porque poner dos veces el if, lee la solucion que te ha dejado BatchianoISpyxolo mas arriba.

Saludos

Si, es cierto, muchas gracias a los dos por todo, estaba bastante jodido con el puto ejercicio.