Foro de elhacker.net

Hola que tal,
Compañeros soy nuevo en esto de la programación "c" constantemente tengo dudas, supongo que es normal. A ver si me podeis ayudar con un codigo. El codigo me lo han pasado en algorismo y lo estoy pasando a "C", tengo algun fallo.
En el codigo no me pedian la funcion main. Pero si no la incluyo me da fallo, Aun asi tengo un warnning. El codigo tiene dos warnings uno en la linea de main y el otro en la linea anterior a  
                         int isAValidDay (int nDays, int nDaysMonth){

El tema es que tengo dos warnings me podeis ayudar a solucionarlo. Gracias.

#include <stdio.h>
 
	//algorisme valiDate
 
 
 
void testNOfDaysFebruary();
void testlsACorrectDate();
int nOfdaysfebruary(int year);
 
 
 
typedef enum {FALSE, TRUE} bool;						//Estructura booleana.
 
int main(){												//La funcion main vacia?
}
 
//Variables
int nDays;
//Codigo
int nOfdaysfebruary(int year)
{
			if (year% 4 == 0 && year% 100 != 0 ){     //(year mod 4 = 0 and year mod 100 &#8800; 0) llavors
					nDays=29;
			}
				else
					nDays=28;
 
			return nDays;
}
 
void testNOfDaysFebruary()
	/*printf("Test number of days February\n");*/
{
			if (nOfdaysfebruary(2016 !=29)){
				 printf("%c ",'E'); 
				 printf("%d ",2016); 
				 printf("%c ",' ');   //Pendiente REVISAR imprimir un espacio
			} 
			if (nOfdaysfebruary(2014 !=28)){
				 printf("%c ",'E'); 
				 printf("%d ",2014); 
				 printf("%c ",' ');   //Pendiente REVISAR imprimir un espacio
			}  	
			if (nOfdaysfebruary(2000 !=29)){
				 printf("%c ",'E'); 
				 printf("%d ",2016); 
				 printf("%c ",' ');   //Pendiente REVISAR imprimir un espacio
			} 
			if (nOfdaysfebruary(1600 !=29)){
				 printf("%c ",'E');   //Imprimir caracter	
				 printf("%d ",1600); 
				 printf("%c ",' ');   //Pendiente REVISAR imprimir un espacio
			}
}
int nDaysMonth (int month, int year)
 
{
 
			if ( month ==2){
				nDays = nOfdaysfebruary(year);
			}
			else {
					if(month == 4 || month == 6 || month == 9 || month == 11 )
						nDays=30;
					else 
						nDays=31;
			}
 
 
}
int isAValidDay (int nDays, int nDaysMonth){					//FUNCION
 
	return ((nDays >= 1) & (nDays <= nDaysMonth));				//verificar  se han colocado dos parentesis
}
 
int isAvalidMonth(int month){									//FUNCION
	return (month <13);											/*retorno*/
}
 
 
int isACorrectDate(int day, int month, int year){			   // FUNCION
 
	//typedef enum {FALSE, TRUE} bool;
	bool dateOK;
	int maxDays;
 
	dateOK=FALSE;  												//Tener en cuenta en mayusculas FALSE.
 
			if (isAvalidMonth (month)){	
			maxDays=nDaysMonth(month, year);
			dateOK = isAValidDay (day, maxDays);
			}
 
	return dateOK;												/*retorno*/
}
 
/* aquesta acció us la donem ja codificada en C */
void testIsACorrectDate(){
	printf("Test isACorrectDate\n");
	if (isACorrectDate(31, 4, 2015)) printf("error 31/4/2015\n");
	if (!isACorrectDate(31, 3, 2015)) printf("error 31/3/2015\n");
	if (isACorrectDate(29, 2, 2015)) printf("error 29/2/2015\n");
	if (!isACorrectDate(29, 2, 2400)) printf("error 29/2/2400\n");
}

Mod: Tema modificado, los codigos deben ir en etiquetas GeSHi

Si no pones los warnings, quienes no lo puedan comilar no podrán verlos. Ponlos.

El main es necesario para compilar el programa. Si no está, no tiene por donde empezar el programa. En el main es donde debes testear el algoritmo.

Buenas.

En lenguaje C, deberías empezar con algo más básico, ya que ni sabes que dentro de main tiene que ir tu código. Es sí o sí. Ya que así se requiere.

Los problemas son esos, que no está dentro y no los cierras bien. Vamos es lo primero que he visto, igual tienes algo más o no, pero primera prueba con eso.

Gracias!!
He visto que era necesario poner el main.
Este ejercicio me lo han puesto como practica.
El problema de los warnings eran que faltaban return.

Otro error (uno lógico) se encuentra en todos los condicionales de la función "testNOfDaysFebruary":

void testNOfDaysFebruary(void)
{
   if (nOfdaysfebruary(2016 != 29)){
   /* ... */
   if (nOfdaysfebruary(2014 != 28)){
   /* ... */
   if (nOfdaysfebruary(2000 != 29)){
   /* ... */
   if (nOfdaysfebruary(1600 != 29)){
   /* ... */
}

Sintácticamente son correctas pero tienen el efecto de llamar a la función "nOfdaysfebruary" con el valor cero como argumento.

Hay que sacar la comparación con las literales fuera de la lista de argumentos de la función:

void testNOfDaysFebruary(void)
{
   if (nOfdaysfebruary(2016) != 29){
   /* ... */
   if (nOfdaysfebruary(2014) != 28){
   /* ... */
   if (nOfdaysfebruary(2000) != 29){
   /* ... */
   if (nOfdaysfebruary(1600) != 29){
   /* ... */
}

Un saludo