Foro de elhacker.net

Programación => Programación C/C++ => Mensaje iniciado por: sabeeee en 28 Enero 2015, 01:47 am



Título: [Consulta] ¿Se desaprueba este código?.
Publicado por: sabeeee en 28 Enero 2015, 01:47 am
Código
  1. /*
  2. ¿Porque se ultiliza goto? Porque si, reduzco codigo.
  3. */
  4.  
  5. #include <cstdio>
  6. #include <cctype>
  7. #include <cstring>
  8. #include <cstdlib>
  9. #include <iostream>
  10. using namespace std;
  11.  
  12. FILE *archivo;
  13.   FILE *ccmd;
  14.  
  15. char *pos;
  16.  
  17. char caracteres[150];
  18.  
  19. int main( int argc, char **argv) {
  20.    char ArchivoAbrir[21];
  21. char ArchivoSaliente[21];
  22.  
  23.        if ( argc < 3 ) {
  24. cout<<"Ejemplo: programa \"Archivo a Abrir.cmd\" -c \"ArchivoSaliente.c\" "<<endl;
  25. return 1;
  26. }
  27. if ( !strcmp( argv[2], "-c" ) ) {
  28.  
  29. strncpy( ArchivoAbrir, argv[1], 20 );
  30. ArchivoAbrir[21] = '\0';
  31.  
  32. strncpy( ArchivoSaliente, argv[3], 20 );
  33. ArchivoSaliente[21] = '\0';
  34. }
  35.  
  36. archivo = fopen(argv[1],"r");
  37. ccmd = fopen(argv[3],"wt");
  38.  
  39. if (archivo == NULL)
  40. exit(1);
  41.  
  42. while (!feof(archivo))
  43. {
  44. fgets( caracteres, 150, archivo );
  45. if ((pos=strchr(caracteres, '\n')) != NULL)
  46. *pos = '\0';
  47.  
  48. while (!strcmp(caracteres,"PAUSE>NUL"))
  49. {
  50. goto PAUSANULA;
  51. }
  52.  
  53. while (!strcmp(caracteres,"PAUSE"))
  54. {
  55. fprintf(ccmd,"\tputs(\"Presione una tecla para continuar.\");\n");
  56. PAUSANULA:
  57. fprintf(ccmd,"\tgetchar();\n");
  58. caracteres[0] = '\0';
  59. }
  60.  
  61. while (!strcmp(caracteres,"EXIT"))
  62. {
  63. fprintf(ccmd,"\treturn 0 ;\n");
  64. caracteres[0] = '\0';
  65.  
  66. }
  67.  
  68. while (!strcmp(caracteres,"CLS"))
  69. {
  70. goto LIMPIADO;
  71. }
  72. while (!strcmp(caracteres,"CLEAR"))
  73. {
  74. LIMPIADO:
  75. fprintf(ccmd,"\tputs(\"\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\\n\");\n");
  76. caracteres[0] = '\0';
  77. }
  78.  
  79. while (!strcmp(caracteres,"FIN"))
  80.                {
  81. goto FIN;
  82. caracteres[0] = '\0';
  83.                }
  84.  
  85. if (!strcmp(caracteres,"\0"))
  86.                {
  87. }else{
  88. fprintf(ccmd,"\tsystem(\"%s\");\n",caracteres);
  89. caracteres[0] = '\0';
  90. }
  91.   }
  92. FIN:
  93.        fclose(archivo);
  94. fclose(ccmd);
  95. cout<<"Compilado.."<<endl;
  96. getchar();
  97. }

¿Alguna razón para que no sea así?


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: engel lex en 28 Enero 2015, 02:23 am
si se desaprueba... en lugar de usar goto puedes perfectamente usar funciones y estructurar tu código correctamente, menos codigo no hará tu programa más eficiente, ni legible, solo lo hará confuso y complicado


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: sabeeee en 28 Enero 2015, 03:15 am
Y como se podría simplificar, o mejorar. Dijkstra se debe estar levantando de la tumba para matarme, dice en wikipedia:

Código:
Respecto a su carácter árido y ácido, conocidas son su oposición a la instrucción GOTO y al lenguaje BASIC («mutila la mente más allá de toda recuperación»). Alan Kay expuso que en informática, la arrogancia se mide en nanodijkstras.


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: BlackM4ster en 28 Enero 2015, 09:06 am
El uso de goto es una práctica muy mala... el código se hace ilegible


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: exploiterstack en 28 Enero 2015, 10:17 am
Hola sabeeee,

Mira hacer uso de goto esta totalmente desaconsejado ya que crea muy mal abito y rompe por completo lo que es la programación estructurada, ahora por ejemplo no lo veras pero si analizar tu código o lo  depuras lo entenderás mucho mas fácilmente, ya que puede darse el caso que instrucciones nunca se lleguen a ejecutar cosa que va en contra de los fundamentos de la programación estructurada como bien describió Dijkstra.

Si quieres saber mas de por que no utilizar esta instrucción te aconsejo que leas sobre programación espagueti.

Espero haberte ayudado, un saludo! ;)


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: Orubatosu en 28 Enero 2015, 12:50 pm
Todo esto es muy técnico a veces, y puede parecer purismo, pero tengamos un par de cosas en cuenta.

"Goto" es una sentencia muy antigua, y proviene de lenguajes interpretados con Basic.

Incluso en lenguajes interpretados, se puede programar de forma eficiente sin algo tan contrario a la claridad como es un salto incondicional, pero en algunos interpretes muy antiguos, cabe la posibilidad de que tuviera algún sentido (poco), desde el momento en que el lenguaje interpretado va leyendo secuencialmente una lista de instrucciones.

En lenguajes compilados, usar "goto", funciones, acciones o cualquier otra estructura no va a influir en su velocidad de ejecución, porque el compilador "ya se las apaña" para desmenuzar tu código y convertirlo en "otra cosa". En lenguaje máquina si que existen obviamente los saltos incondicionales (ahí son algo diferente). Tu código en realidad solo va a ser mas eficiente y rápido mediante la elección de los tipos de variables adecuadas, la resolución mas eficiente de un algoritmo y otras, entre las cuales pueden estar (por ejemplo) que compiles específicamente para un tipo determinado de procesador y otras.

Pero, que pongas mas o menos comentarios, que definas las funciones delante o detrás del main, que uses variables globales o locales exclusivamente y otras no van a hacer que tu programa "corra mas".

Dado que tu programa en principio, puede ser igual de eficiente si usas goto que si no lo usas, ¿porque esa manía y odio furibundo hacia esa instrucción?

Por una razón muy simple. Tu código no siempre es "definitivo". Cuando se crea una aplicación grande "seria", digamos que dedicas 3 meses a completarla. ¿Ya has terminado?

No, claro que no, esa aplicación probablemente requiera retoques en el futuro, ampliaciones, corregir potenciales errores que se te hayan colado, incluir nuevas características, etc.

E incluso es posible que quien haga esas modificaciones no seas tu, sino otra persona, y que a ti te toque modificar el código de otra persona.

Y aquí es donde se justifican esas cosas. Tu trabajarás siempre sobre ese código fuente. Si esta estructurado, es claro, documentado correctamente y sigue una serie de convenciones, tu trabajo será infinitamente mas sencillo y rápido. Si el código es confuso, con saltos incondicionales a punta pala, mal documentado (o sin comentarios), aunque el programa sea rápido, eficiente y haga lo que se espera de el, modificarlo será mucho mas difícil.

Es la diferencia entre trastear con un código claro, organizado y debídamente documentado, y trabajar con un código que no lo está. En un caso es fácil, en el otro no lo es en absoluto.

Y la verdad, las personas rara vez se acuerdan perfectamente de como funciona un código que hicieron hace meses, o años incluso.

Vamos, es como si tienes que revisar un libro, y este libro está bien organizado, tiene su indice general, indice de materias, etc... y sigue un orden lógico. El otro libro tiene las páginas al azar, con señales que indican a donde debes de saltar a cada momento, no tiene índice y además está escrito de forma poco clara.

Y no hay mas misterio


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: exploiterstack en 28 Enero 2015, 13:30 pm
Por eso digo que es totalmente desaconsejado utilizar la instrucción goto, ya que puse solo uno de los motivos por los que no hay que acostumbrarse a utilizarlo.

Todos los que hemos estudiado programación sabemos que hay que evitar utilizar esa instrucción por x motivos entre ellos los que menciona Orubatosu de que raramente en una empresa seras el único en trastear el código fuente o por si en un futuro sera alguien el que se ara cargo, es aun dándoselo medio mascado con comentarios y con una buena costumbre de programación aun se le atraganta a mas de uno.

Haciendo uso de esa instrucción es una forma rápida de acceder a etiqueta que especifiques, no es cuestión de rapidez sino de como internamente trabaja. Te invito que a tu programa lo pases por un debugger. ves creando br(breakpoint) en la salida de cada instrucción goto y observa la ejecución sentencia por sentencia hasta que finalice el programa veras de lo que te hablo ;)

Te pongo por ejemplo lo que se menciona del código espageti, mira si haces un salto incondicional hacia una etiqueta estas rompiendo por completo lo que es la ejecución normal de cualquier programa estructurado ya que una vez se haya ejecutado el bloque de la etiqueta tienes que especificar de manera explicita por donde debería seguir la ejecución de la aplicación. Y para eso ya están las funciones ;)

Un saludo!


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: sabeeee en 28 Enero 2015, 18:42 pm
¿Esto desestabiliza terriblemente mi código? Porque en linux que es un código de millones de lineas y que hay varios colaboradores tengo entendido hasta hace un par de años se utilizaba el goto, en realidad mi código es otro pero lo modifique como para hacer la pregunta, porque veo que es muy cuestionado sin argumentos realmente convincentes, tampoco lo vi que este en cosas que no se pueden hacer.


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: _Enko en 28 Enero 2015, 20:20 pm
Destabilizar el codigo?

Esa afirmacion es bastante confusa.

Si con codigo estable se refiere a codigo que produce ejecutables estables... entonces goto no seria un problema alli siemper y cuando la ejecutable sea estable.

Ahora, si tienes algo asi:
Código:
if(blablabla())
   goto ALGUN_LUGAR_MUY_LEJANO_DE_AQUI;
/*
5.000 lineas de codigo mas abajo:
ALGUN_LUGAR_MUY_LEJANO_DE_AQUI:

Eso si seria molesto. Extremadamente molesto. Muy extremadamente molesto. Demasiado extremadamente molesto.

Si el goto salta 2,3 lineas... para que?  Si con sentencias de control tranquilamente se puede generar el mismo resultado.

No se me ocurre ningun caso donde un goto sea util. Es mas te digo, cuando se programa en ensamblador normalmente se utilizan saltos condicionales: jz, jnz, je, jne, etc...  Tranquilamente se puede programar evitando usar un salto incondicional (JMP) que seria el goto.

Es mas, el programa que usa goto esta desperdiciando bytes...
si pones algo como:
Código:
if(algo())
  goto ALGO;
estarias haciendo  algo asi:
Código:
   call algo
   test eax, eax
   jnz NEXT:
   JMP ALGO
NEXT:
   ;salida next

Mientras sin el goto, seria simplemente:
Código:
if(algo())
   algo2();
seria algo como
Código:
   call algo
   test eax, eax
   jnz NEXT
   call algo2
NEXT:

un JMP innecesario menos y no hay que volver a la prehistoria.
(http://imgs.xkcd.com/comics/goto.png)

Citar
ya que puede darse el caso que instrucciones nunca se lleguen a ejecutar
eso no seria culpa del goto.
Código:
if(1==2){
codigo_que_nunca_se_va_a_ejecutar();
}

Saludos.


PD: una consulta...

Que pasa con C++,  destructores y goto?
Código:
{
   MYCLASS my_class; //se llamo al constructor
   goto AVOID_DESTRUCTOR
} //<< aqui la runtime llamaria al destructor del objeto my_class
AVOID_DESTRUCTOR:
O sea, lo entiendo bien? al usar un goto y salirse del scope, el destructor nunca seria llamado?

Acabo de hacer la prueba y el destructor es llamado... raro?
Código:
#include <iostream>
using namespace std;
 


class CClass {
public:
    ~CClass(){
        cout << "destructor" << endl;;
    }
    CClass(){
        cout << "constructor" << endl;
    }
};

 
int main(int argc, char *argv[]) {
    
    {
        CClass test;
        goto AVOID;
        
    }
AVOID:
    int i;
    cin >> i;
    return 0;
}


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: Orubatosu en 28 Enero 2015, 22:30 pm
A nivel de eficacia no tiene porque dar problemas. Que demonios, es una instrucción oficial de C++, debe de funcionar como se pretende.

Pero, no te va a aportar mas velocidad ni código mas eficiente. De manera que en realidad la cuestión es la que se te a comentado varias veces.

No se usa porque el código resultante puede ser confuso, sobre todo cuando la gente usa demasiado a menudo "goto" en lugar de usar otros mecanismos.

El problema real es que simplemente, la programación estructurada es mas simple, mas legible y mas fácil de mantener. El meter montones de saltos incondicionales no hace la programación mas sencilla (es hacer las cosas de otro modo). No hace el código mas legible, y no lo hace desde luego mas fácil de mantener.

De hecho incluso en lenguajes interpretados, donde muy teóricamente "goto" podría llegar a tener algún uso (echándole imaginación) se desaconseja energicamente su uso por todos los problemas antes comentados.

Si alguna vez has tenido que depurar un programa en BASIC, con docenas y docenas de saltos condicionales, incondicionales y otras lindezas, y te has vuelto majara intentando ver por donde narices anda la ejecución entenderás porque terminas odiando esa instrucción.

En C++ simplemente no tiene sentido, es innecesario su uso


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: sabeeee en 28 Enero 2015, 23:29 pm
Bueno, creo que me aclararon la duda, incluso en C++ 14 Goto esta deprecated.


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: sabeeee en 28 Enero 2015, 23:31 pm
Compare, compilando en assembler, no se si tendrá mucho que ver y fui analizando con múltiples métodos, saludos, muchas gracias.


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: flony en 29 Enero 2015, 04:35 am
amigo ...a no ser que, como yo que iniciaste en batch en donde para hacer el for que usamos en c++ hacia falta usar goto...mucho no sirve en c++


Título: Re: [Consulta] ¿Se desaprueba este código?.
Publicado por: DarK_FirefoX en 29 Enero 2015, 05:53 am
Mi recomendación: No usarlo!
Te recomiendo la lectura de un articulo llamado "Go To Statement Considered Harmful" escrito por Edsger W. Dijkstra. El artículo original creo que es en inglés, pero estoy seguro que podrás encontrarlo en español!

Esto creo que te aclarará las dudas!
Saludos