Hacer dos comprobaciones en un WHILE con funciones

[aguml]

Te doy un consejo que me dieron a mi hace mucho. "Un return por funcion". Nada de 20 return repartidos por toda la función. Pon un return al final y que retorne el valor de una variable y lo que tienes que modificar es el valor de esa variable en los sitios deseados. Muchos return hace que el código cueste más interpretarlo y que tengas fallos como lo que por ejemplo te pasa en el tuyo, que sales con un return y no cierras el fichero por ejemplo.
Si miras en los códigos de ejemplo de la msdn veras que usan muchos return pero que ibas a esperar de Microsoft jajaja.