Empezamos a crear la ats, por fin...

Y bueno, estoy con la 1º función, y por mensajería, discutía amistosamente con nicolas_cof, sobre como implementar la siguiente función:

 
// STDIO.H
 
/*
 * By: Sergio López & elhacker.net- 5/6/2010
 * GNU GPLv3
 * 
 * Function returns an integer
 *
 * Syntax example: var =atsInNum(1, 6);
 * Returns a value between 1 and 6
 */
 
int atsInNum(int ATSMIN, int ATSMAX)
{
	int atsint =0;
 
	do
	{
		fscanf(stdin, "%d", &atsint);
	} while (atsint<=ATSMIN || atsint>=ATSMAX);
 
	return atsint;
}
 
 

Varias cosas:

* nicolas_cof opina que el switch con las opciones de texto sobra, pues se sale del objetivo de la función. Yo opino que es un extra a la hora de usarla, ya que te ahorras un par de lineas gracias a ATSOP.

* No me gusta demasiado usar fscanf, así que pensé en usar fgetc y atoi. Espero opiniones al respecto.

Espero que comentéis sobre estos 2 asuntos, y aportéis nuevas ideas, reportéis errores, etc. ^^

http://foro.elhacker.net/programacion_cc/libreria-t294373.0.html

La verdad es que el propósito de la función esta como a medias. Tengo que preguntar cual es el propósito que intentaste darle a la función, ya que no me queda claro todavía para que la queres utilizar específicamente.

No entiendo la utilidad del tercer parámetro. Primero, estoy de acuerdo con Nicolas_cof, la función debe tener un objetivo primario. Si lo analizas, el código de los mensajes ocupa mas que la función en si, lo cual no tiene mucho sentido.
En cuanto al tercer parámetro, supongo que es la opción "idioma" lo cual me parece bastante innecesario y ademas esta mal codificado debido a estos 2 detalles:

Si el switch no tiene un break, si ingresas un 1 en el tercer parámetro se van a mostrar los dos mensajes.

El segundo detalle esta relacionado con el anterior, el texto carece de formato, por lo tanto se veria algo como:


Lo cual es incorrecto.

Ahora bien, en cuanto al propósito general de la función, he de decirte que no tiene mucho sentido. Vamos a analizarla por partes:

	do
	{
		fscanf(stdin, "%d", &atsint);//Lees un entero
	} while (atsint<=ATSMAX && atsint>=ATSMIN);//Mientras sea menor o igual que MAX Y mayor o igual que MIN.
 
	return atsint; //retorno el primer numero que ingrese que NO sea menor o igual que MAX Y mayor o igual que min

Y ahora veamos los comentarios:

 * Function returns an integer if it works
 *
 * Syntax example: var =atsInNum(1, 6, 1);
 * Returns a value between 1 and 6, displaying a message

Entenderás que no corresponde una cosa con la otra por lo tanto se torna bastante confuso.

Despacio y con análisis que van a salir cosas buenas, no apurarse.

Saludos

Claro pero vos retornas precisamente lo contrario, porque solo salís del do while cuando ingresas un entero que no esta comprendido entre esos valores.

Y ahora te quedas en un bucle infinito. Podes ingresar números toda la vida que siempre se va a cumplir la condición.


func(0,5)

ingreso 5;

es mayor o igual que 0? si
o
es menor o igual que 5? si

ingreso 20;

es mayor o igual que 0? si
o
es menor o igual que 5? no


De cualquiera de las dos formas el ciclo no termina.

Corregido el maldito error de máximos y mínimos. Gracias Littlehorse por hacerme pensar XD