SoloCodigo

Hola
el siguiente programa es para asignar una de cinco PCs a un usuario q ingresa su matricula (EJ.:  20920)

he tratado de corregir un error q creo q esta el parte de LOGIN
si alguien puede ayudarme corrigiendo el error si es posible q me lo haga saber . se lo agradeceria bastante

#include<stdio.h>
#include<conio.h>
void main (void)
{
int mat,p[4],cont;
char q;

ini:
printf("\n\nQué desea hacer? log(i)n, log(o)ff, (e)stado: ");
scanf("%s",&q);
if(q=='i')goto login;
if(q=='o')goto logoff;
if(q=='e')goto estado;

login:                      //esta parte asigna una pc a la matricula
printf("Matricula: ");
scanf("%d",&mat);
for(cont=0;cont<=5;cont++)
if(p[0]==0)p[cont]=mat;
goto ini;

                             //esta parte saca una matricula del sistema
logoff:
printf("Matricula: ");
scanf("%d",&mat);
for(cont=0;cont<=5;cont++)
{
if(mat==p[cont])p[cont]=0;
}
goto ini;

                              //esta parte muestra las PCs vacias y las ocupadas
estado:
for(cont=0;cont<=5;cont++)
{if(p[cont]==0)printf("\nPc %d desocupada",cont);else printf("\nPc %d ocupada por %d",cont,p[cont]);}
goto ini;


}

me uno.

el goto es una instruccion muy utilizada en los tiempos de la programacion no estructurada ('spagetti'), esta instruccion se caracteriza por producir un codigo desordenado y dificil de entender, su existencia actualmente es mas por compatibilkidad con las viejas aplicaciones , ya que su uso hoy en dia esta abolido.
todo lo que se puede hacer con un goto se puede hacer con un while o con un for.

revisa este link de una discucion en el foro el año pasado.
sobre goto y una ayudita

no es una buena practica,
un desarrollador serio no la usa, no la uso ni la he usado y la verdad lo hago porque es el conocimiento que me han impartido durante toda mi carrera en la universidad.
cite el mensaje de gmantil porque él es un profesor de mi universidad, expretoen sistemas operativos y programacion.

En todo caso cada cual es libre de usar lo que necesita, pero el goto es una instruccion que degrada la calidad del codigo, no hay nada que se haga con un goto que no se pueda hacer de una manera mas clara con un while o un for los cuales conservan las practicas de codigo estructurado.


  El solo hecho de utilizar la instruccion dificulta el seguimiento, la depuracion y el mantenimiento del codigo fuente.


Pero si les funciona y creen que les sirve usenlo.  

Si a los desarrolladores del kernel de linux no se les puede considerar serios...

grep -R  goto /usr/src/linux/

Más de 19000 goto -kernel 2.4.25-

Aunque te tengo que dar la razón en que el seguimiento del código en este caso es todo un reto

En ese caso tenes razon, pero tambien deberias tener en cuenta que es el kernel. Y generalmente se mezcla C y ASM o se usa mucho codigo que trabaja a muy bajo nivel.

Segun mucho programadores goto solo se debe usar en casos donde no tenes otra opcion sino se debe omitir.

Un Saludo

Buenas. Apoyo el no uso del GOTO, la verdad que los tiempos de usar esta sentencia han desaparecido. Ya han dado multitud de razones, asi que lo dejo como esta  

Respecto a como salir de dos o mas bucles a la vez.....en mi opinion si esto sucede es una mala planificación de los mismos, y hay que revisar el código, pero me ha ocurrido mas de una vez y para poder continuar y más tarde revisarlo utilizo esta forma:

bool pasalir=true;
for(x=0;x<10 && pasalir;x++)
{
for(y=0;y<10 && pasalir;y++)
{
...
if ("quieressalir") pasalir=false
}
}

pero ya te digo, son métodos que uso hasta q tengo tiempo de depurar el código

Deberías eliminar esos goto's como todos dicen y mejor utilizar funciones que te devuelvan un valor boolean, así puedes saber si el código se ejecutó bien o no.....