Duda en un ejercicio de programación

[eaguel]

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;
}

[$Edu$]

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 Sino preguntas de nuevo que alguien te contestara siempre, saludos!

[Caster]

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:

Código (c) [Seleccionar] Expandir

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:

Código (c) [Seleccionar] Expandir

/*
**   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.

[eaguel]

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

[eaguel]

Perdonar que siga dando el tostón pero estoy muy jodido     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;
}

[eaguel]

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;
}

[Caster]

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:

Código (c) [Seleccionar] Expandir

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

Por esto:

Código (c) [Seleccionar] Expandir

if (mes == 0) return 0;

Saludos

[DickGumshoe]

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:

Código (c) [Seleccionar] Expandir

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

Por esto:

Código (c) [Seleccionar] Expandir

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.

[BatchianoISpyxolo]

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:

Código (c) [Seleccionar] Expandir

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

Por esto:

Código (c) [Seleccionar] Expandir

if (mes == 0) return 0;

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:

Código (c) [Seleccionar] Expandir

if (mes == 0) return 0;

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

Código (c) [Seleccionar] Expandir

return mes

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

¡Saludetes!

[eaguel]

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.