Como indentar/tabular un switch?

[stramin]

Hola estimados, siempre he tenido esta pregunta y me gustaría saber como lo hacen ustedes, esto me ayudará a elegir un estandar para mis códigos.

Algunos ejemplos que he usado:

dentador (php.net): indenta cada instruccion como si fuesen llaves

Código PHP:

switch($b){
    case "cont":
        $x=1;
        $y=1;
        break;
    case "meas":
        $x=2;
        $y=2;
        break;
    case "pass":
        $x=3;
        $y=3;
        break;
}
 

canónico (dreamweaver): solo las llaves indentan

Código PHP:

switch($b){
    case "cont": $x=1;
    $y=1;
    break;
    case "meas": $x=2;
    $y=2;
    break;
    case "pass": $x=3;
    $x=3;
    break;
}
 

económico (php.net, flash 4 y w3cJS): solo indenta el código

Código PHP:

switch($b){
case "cont":
    $x=1;
    $y=1;
    break;
case "meas":
    $x=2;
    $y=2;
    break;
case "pass":
    $x=3;
    $x=3;
    break;
}
 

cortador (aporte en php.net): usa el break como un cierre de llave

Código PHP:

switch($b){
case "cont":
    $x=1;
    $y=1;
break;
case "meas":
    $x=2;
    $y=2;
break;
case "pass":
    $x=3;
    $y=3;
break;
}
 

cerrajero (php.net): si no hay llaves, las crea

Código PHP:

switch($b){
    case "cont":{
        $x=1;
        $y=1;
        break;
    }
    case "meas":{
        $x=2;
        $y=2;
        break;
    }
    case "pass":{
        $x=3;
        $y=3;
        break;
    }
}
 

Ustedes como hacen el switch?

[Patriarka]

ninguna de ellas me parecen identadas!

mira como lo hago yo:

Código PHP:

<?php
    switch($b){
        case "cont": 
            $x=1;
            $y=1;
        break;
        case "meas": 
            $x=2;
            $y=2;
        break;
        case "pass": 
            $x=3;
            $x=3;
        break;
    }
?>
 

poner llaves dentro de un switch es irrespetuoso, pner varias instrucciones en la misma linea definitivamente n es tabular, etc etc

tenes que mirar todo como si fuera una tabla html

Código HTML:

<table>
    <tr>
        <td></td>
        <td></td>
    <tr>
    <tr>
        <td></td>
        <td></td>
    <tr>
    <tr>
        <td></td>
        <td></td>
    <tr>
<table>
 

ves? todo en su lugar, no hay tr y td en la misma linea ni mada que rompa la estructura
de todos los ejemplos que pusiste yo no tomaria ni uno por que estan todos mal y me produce desprecio por esos developer´s! jajaja es un chite malo

[jc_moj]

Hola

Yo uso el primero que pusiste. @Patriarka me odia

Saludos

[pateketrueke]

Ustedes como hacen el switch?

Siguiendo la guía de estilo definida aquí: http://phptherightway.com/

[stramin]

El estilo de Patriarka es parecido al "Cortador" pero indentado, supongo que llamaré a este estilo "Patriarka".

Entonces jc_moj prefiere la indentación de php.net

No me quedo claro como lo hace pateketrueke :(

He estado buscando más información por ahí y he encontrado que originalmente los case: eran etiquetas y por lo tanto nunca deberían tabularse, o sea como el "económico".

When a label in a switch follows a "break" statement, the two are separated by a blank line. If the preceding statement is not a break, there is no blank line.

Blank lines separate each of the main sections in a file, including prologue comment (with copyright or version information, etc.), package declaration, imports and interface or class. Within a class, at least all constructors, methods and inner classes should be separated from each other and from any other fields by blank lines (always before the prologue comment, of course).

A blank line may optionally be inserted anywhere to improve readability but the use of two or more contiguous blank lines is unnecessary and should be avoided.

Sin embargo otras fuentes dicen que todo lo que está entre {llaves} debería indentarse.

Sin embargo en otro foro un usuario publicó este código:

Código PHP:

switch (1) {
    case 1:
        break;
    case 2:
        break;
    case 3:
        break;
    default:
        break;
}
 

y un corrector de scripts lanzó esta salida:

line 3, col 5, Expected 'case' to have an indentation at 1 instead at 5.
line 4, col 9, Expected 'break' to have an indentation at 5 instead at 9.
line 5, col 5, Expected 'case' to have an indentation at 1 instead at 5.
line 6, col 9, Expected 'break' to have an indentation at 5 instead at 9.
line 7, col 5, Expected 'case' to have an indentation at 1 instead at 5.
line 8, col 9, Expected 'break' to have an indentation at 5 instead at 9.
line 9, col 5, Expected 'default' to have an indentation at 1 instead at 5.
line 10, col 9, Expected 'break' to have an indentation at 5 instead at 9.

Por lo tanto también apoya a estilo "económico".

Al parecer, no hay una forma "correcta" o "incorrecta", pero esta encuesta me sirve para elegir un estandar :)

Y ustedes como hacen el switch?

[ocp001a]

Yo lo escribo igual que Patriarka, con el Break a la misma altura que el Case.

[Triby]

Yo lo hago como el primer ejemplo que pusiste (php.net)

Zend sugiere esto: http://framework.zend.com/manual/1.1...tements.switch

CodeIgniter no lo menciona: http://ellislab.com/codeigniter/user...tyleguide.html

Laravel solo dice que la llave de apertura no debe estar en la misma línea: https://github.com/laravel/framework...ONTRIBUTING.md

Sugiero que no te compliques tanto la vida, elige el "formato" que mejor te acomode y aplícalo siempre que requieras esa estructura. No podrás satisfacer a todos y, además, eres tú quien debe sentirse cómodo.

[Patriarka]

coincido con TRiby,
busca tu estilo

[stramin]

Gracias por la opinión ocp001a, la verdad creo que en un comienzo (hace unos 10 años) yo también usaba el break como cierre, no recuerdo por que dejé de hacerlo, quizás fue por que el break muchas veces no tuve que ponerlo y quedaba sin ese "cierre".

Muchas gracias por esos aportes triby, a mi también me gusta el primer estilo, pero elegir cual es el que más me acomode no ha sido tan fácil, esto es por que ninguno me convence al 100%, hay muchas reglas que me parecen rotas en la estructura de un switch, patriarka mencionó algunas y siempre me da la sensación de que hay algo mal en mis switches XD... y bueno, por eso quise hacer esta encuesta y conocer las opiniones del foro.

Gracias por su ayuda!