Nuevo programador Java, composición básica de Java

Question

Soy un nuevo estudiante de programación informática. Vi un video sobre Java, composición básica, y el hombre en el video hice un ejemplo sobre este tema como este:

public class PaperTray 
{ 
    int pages = 0; 
    .... 
    public boolean isEmpty() 
    { 
    return pages > 0; 
    } 
} 

---

public class Printer extends Machine 
{ 
    private PaperTray paperTray = new PaperTray(); 
    .... 
    public void print(int copies) 
    { 
    .... 
    while(copies > 0 && !paperTray.isEmpty()) 
    { 
    System.out.println("some text to print"); 
    copies--; 
    } 
    if(paperTray.isEmpty()) 
    { 
    System.out.println("load paper"); 
    } 
} 

Mi pregunta es si la bandeja de papel está vacía, entonces en clase PaperTray el método isEmpty() devolverá falso. Por lo tanto, la instrucción if en la clase Impresora no se ejecutará. Y si la bandeja de papel no está vacía, el método es Vacío() en la clase PaperTray devolverá verdadero, por lo que la instrucción while en la clase Impresora no se ejecutará. ¿Estoy equivocado o el instructor del video cometió algunos errores?

Gracias

Answer 1

si la bandeja de papel está vacía, entonces en la clase PaperTray el método isEmpty() regresará false

Debe devolver true (para cualquier aplicación sensata, es decir :-) . Para un método llamado isEmpty(), el sentido común dicta que devuelve true cuando el objeto/colección de inclusión está vacío, y false cuando no está vacío.

En otras palabras, la implementación que muestra arriba tiene un error.

Answer 2

La lógica de la isEmpty no tiene sentido: me harían lo esperaba, ya sea

public boolean isEmpty() { 
    return pages == 0; 
} 

o

public boolean isNotEmpty() { 
    return pages > 0; 
} 

Answer 3

Si lo que dice es correcto. isEmpty() debería devolver verdadero cuando está vacío. Probablemente un error en la parte de instructores.

Answer 4

Falta algo en el código: cuando se realizan copias, solo se decrementa copies. Pero el código también debería disminuir el valor de pages en PaperTray; de lo contrario, no se utiliza ningún papel para hacer las copias.

Cuando eso sucede, el código está bien:

while(copies > 0 && !paperTray.isEmpty()) 
{ 
System.out.println("some text to print"); 
copies--; 
} 
if(paperTray.isEmpty()) 
{ 
System.out.println("load paper"); 
} 

Asimismo, mantendrá haciendo que el número de copias que desee, siempre y cuando no hay papel en la bandeja. Cuando ya no quede papel, el ciclo while saldrá. De cualquier manera, se realiza una comprobación al final para indicar al usuario que rellene la bandeja de papel cuando esté vacía.

Answer 5

Parece que hay un par de problemas con este código:

public boolean isEmpty(){ 
    //return pages > 0; this doesnt make sense 
    return pages==0; 
} 

public void print(int copies){ 
    while(copies > 0 && !paperTray.isEmpty()){ 
     System.out.println("some text to print"); 
     pages--;//this is not enough. You need to decrement copies as well 
    } 

    if(paperTray.isEmpty()) 
     System.out.println("load paper"); 
} 

Answer 6

soy el instructor del vídeo. Gracias por detectar este error. Estás en lo correcto. En algún punto de la grabación del video, debo haber tenido el> 0 allí en lugar de < = 0.

Buena captura.Debe ser

public class PaperTray 
{ 
    int pages = 0; 
    .... 
    public boolean isEmpty() 
    { 
    return pages <= 0; 
    } 
} 

he comprobado el código de ejemplo para el supuesto que se puede descargar y es correcta y en todas partes esa clase aparece en el curso, por lo que debe haber sido sólo que un punto.

Gracias de nuevo por detectar ese error y mis disculpas por el error y la confusión que le costó. Es de esperar que aún pueda beneficiarse del curso.