Según lo que he leído, para algunos usar break
no tiene nada de malo.
Usar break
es tan malo como usar cualquier otra utilidad de C++: nada malo. La maldad no está en la utilidad usada si no en el uso que se le da. Tal y como has señalado suele usarse para alterar el flujo natural de un bucle, por ejemplo iterando una colección hasta dar con un elemento concreto:
Ejemplo1
int indice = 0;
for (const int final = principes.size(); indice != final; ++indice)
{
// Eres tu mi principe azul que yo sonye?
if (es_azul(principes[indice])
break;
}
El código anterior no tiene nada de malo: recorre una colección de objetos (principes
) a los que les realiza un test (es_azul
) y al finalizar el bucle la variable indice
contiene el índice del primer objeto que cumpla dicha condición o principes.size()
en caso de no haber ninguno; el código anterior es casi equivalente a este:
Ejemplo2
int indice = 0;
bool encontrado = false;
for (const int final = principes.size(); !encontrado && (indice != final); ++indice)
{
// Eres tu mi principe azul que yo sonye?
encontrado = es_azul(principes[indice]);
}
Pueden haber acalorados debates sobre cuál de las dos versiones es mejor pero en general, estas discusiones no se centrarán en el uso de break
si no en cantidad de instrucciones (el bucle de Ejemplo2
da una vuelta más) o claridad (Ejemplo1
puede ser más difícil de entender).
Sea cual sea el caso, dificilmente verás Ejemplo1
o Ejemplo2
en C++ moderno pues lo habitual será delegar este tipo de operaciones a la cabecera <algorithm>
:
auto azul = std::find_if(principes.begin(), principes.end(),
[](const auto &principe){ return es_azul(principe); });
bool encontrado = (azul != principes.end());
Cada compilador implementará std::find_if
de <algorithm>
a su manera pero en general la implementación podría parecerse a:
template<class InputIt, class UnaryPredicate>
InputIt find_if(InputIt first, InputIt last, UnaryPredicate p)
{
for (; first != last; ++first) {
if (p(*first)) {
return first;
}
}
return last;
}
La instrucción return first;
del interior del bucle funciona de manera parecida a los break
que pones en tu ejemplo: alteran el flujo natural del bucle y como puedes ver se usa en código de librerías estándar, nada que objetar ¿verdad?
auto_ptr
o la especificación de excepciones en funciones). Si hubiera algo de dañino o infame en goto, ya no estaría ahí, aunque lo sigan repitiendo los más listillos.