La construcción de lazo más elegante?

Question

Lo siento por la nueva pregunta. Todavía estoy aprendiendo programación. Así que estoy usando C++, y tengo que hacer algo como esto:

int n; 
do { 
    n = get_data(); 
    if(n != -1) 
     send(n); 
} while(n != -1); 

Esto es solo un boceto. De todos modos, no se siente realmente elegante. Tengo que tener mi prueba dos veces. Podría simplemente probar una vez y establecer una bandera, pero eso tampoco parece realmente elegante ya que tengo que revisar la bandera dos veces. Simplemente parece que debería haber una manera de hacer esto más fácilmente, ya que es muy fácil lo que quiero hacer.

Answer 1

Howabout usando break:

int n; 
while(1) { 
    n = get_data(); 
    if(n == -1) 
     break; 
    send(n); 
} 

De esta manera sólo prueba una vez, y dejar de fumar inmediatamente si get_data no devuelve lo que quiere.

Answer 2

int n; 
while (-1 != (n = get_data())) { 
    send(n); 
} // while 

Answer 3

Su versión original está bien.

En mi último trabajo de programación C tuvimos que seguir los estándares de codificación MISRA.

Bajo las reglas Misra, esto:

int n; 
while(1) { 
    n = get_data(); 
    if(n == -1) 
     break; 
    send(n); 
} 

está prohibido debido a la break, y esto:

while((n = get_data()) != -1) { send(n); } 

está prohibido debido a una asignación aparece en un contexto booleano, así que estoy en el hábito de escribir bucles similares a su versión original.

Usted podría una variable booleana si usted piensa que hará que su intención más clara o si la prueba es una expresión compleja:

int n; 
bool valid; 
do { 
    n = get_data(); 
    valid = n != -1; 
    if(valid) 
     send(n); 
} while(valid); 

Pero para una prueba simple como "n != -1" puede que no sea la pena hacer el programa más largo.

Answer 4

similares a eed3si9n de, pero sin duda más fácil de leer:

int n; 
while (n = get_data(), n != -1) 
{ 
    send(n); 
} 

Answer 5

for (int n = get_data(); n != -1; n = get_data()) { 
    send(n); 
} 

Answer 6

construcción con un goto :)

int n; 
goto inside; do { 
    send(n); 
inside: 
    n=get_data(); 
} while(n!=-1); 

Answer 7

int get_data() 
{ 
... 
} 

void _send(int) 
{ 
... 
} 

int send(int (*a)()) 
{ 
    int n = a(); 

    if (n == -1) 
     return n; 

    _send(n); 
    return 1; 
} 

int main() 
{ 
    int (*fp)(); 
    fp = get_data; 
    while (send(fp)!= -1); 

    return 0; 
} 

HTH

Answer 8

int n; 
n = get_data(); 
while (n != -1) { 
    send(n); 
    n = get_data(); 
} 

Answer 9

En Python que podría hacer esto:

while True: 
    n = get_data() 
    if n == -1: 
     break 
    send(n) 

Como se puede ver, esto no es mucho más corta que la versión C. Normalmente, un código Python es de 5 a 10 veces más pequeño que su equivalente en C. Este no es el caso aquí, por lo que puede disfrutar de su fragmento de texto legible, suficientemente corto y rápido, y centrarse en otra cosa.

Si quiere acortarlo, consulte la respuesta de Jonathan, pero realmente esto no importa.

Answer 10

/* This is cleaner */ 
AGAIN:; 
    int n = get_data(); 
    if (n != -1) 
    { 
     send(n); 
     goto AGAIN; 
    } 


/* This has some charm as well */ 
int n; 
while ((n = get_data()) != -1) 
    send(n); 
/* and now i see that this is the top answer. Oh well */