From 4ac689ec15123b7c8ccab3993a8e63502db4147c Mon Sep 17 00:00:00 2001 From: Chris Cromer Date: Sat, 2 Jul 2016 21:10:00 -0400 Subject: [PATCH] Cleanup code. --- src/cl/cromer/estructuras/Grafico.java | 94 +++++++++++++----------- src/cl/cromer/estructuras/HashItem.java | 6 +- src/cl/cromer/estructuras/HashTable.java | 5 +- src/cl/cromer/estructuras/Palabras.java | 2 +- 4 files changed, 58 insertions(+), 49 deletions(-) diff --git a/src/cl/cromer/estructuras/Grafico.java b/src/cl/cromer/estructuras/Grafico.java index 858f31d..9506926 100644 --- a/src/cl/cromer/estructuras/Grafico.java +++ b/src/cl/cromer/estructuras/Grafico.java @@ -48,7 +48,7 @@ public class Grafico { /** * Los elementos destacados. */ - private List destacados; + final private List destacados; /** * Graficar una escena. @@ -257,45 +257,6 @@ public class Grafico { return stackPane; } - /** - * Crear un animacion de transicion usando colores que cambian. - * - * @param rectangle Rectangle: El objeto que desea animar. - * @param colorBackground Color: Color del fondo de destacar. - * @return PauseTransition: La transition creado con los elementos y colores. - */ - private static PauseTransition createPauseTransition(Rectangle rectangle, Color colorBackground) { - PauseTransition changeColor = new PauseTransition(new Duration(DURACION)); - changeColor.setOnFinished(actionEvent -> rectangle.setFill(colorBackground)); - return changeColor; - } - - /** - * Crear un animacion de transicion usando colores que cambian. - * - * @param circle Circle: El objeto que desea animar. - * @param colorBackground Color: Color del fondo de destacar. - * @return PauseTransition: La transition creado con los elementos y colores. - */ - private static PauseTransition createPauseTransition(Circle circle, Color colorBackground) { - PauseTransition changeColor = new PauseTransition(new Duration(DURACION)); - changeColor.setOnFinished(actionEvent -> circle.setFill(colorBackground)); - return changeColor; - } - - /** - * Crear un animacion de transicion usando colores que cambian. - * - * @param text Text: El texto que desea animar. - * @param colorText Color: Color del texto. - * @return PauseTransition: La transition creado con los elementos y colores. - */ - private static PauseTransition createPauseTransition(Text text, Color colorText) { - PauseTransition changeColor = new PauseTransition(new Duration(DURACION)); - changeColor.setOnFinished(actionEvent -> text.setStroke(colorText)); - return changeColor; - } - /** * Destacar un elemento * @@ -348,6 +309,48 @@ public class Grafico { destacados.get(destacados.size() - 1).getSequentialTransition().play(); } + /** + * Crear un animacion de transicion usando colores que cambian. + * + * @param rectangle Rectangle: El objeto que desea animar. + * @param colorBackground Color: Color del fondo de destacar. + * + * @return PauseTransition: La transition creado con los elementos y colores. + */ + private static PauseTransition createPauseTransition(Rectangle rectangle, Color colorBackground) { + PauseTransition changeColor = new PauseTransition(new Duration(DURACION)); + changeColor.setOnFinished(actionEvent -> rectangle.setFill(colorBackground)); + return changeColor; + } + + /** + * Crear un animacion de transicion usando colores que cambian. + * + * @param circle Circle: El objeto que desea animar. + * @param colorBackground Color: Color del fondo de destacar. + * + * @return PauseTransition: La transition creado con los elementos y colores. + */ + private static PauseTransition createPauseTransition(Circle circle, Color colorBackground) { + PauseTransition changeColor = new PauseTransition(new Duration(DURACION)); + changeColor.setOnFinished(actionEvent -> circle.setFill(colorBackground)); + return changeColor; + } + + /** + * Crear un animacion de transicion usando colores que cambian. + * + * @param text Text: El texto que desea animar. + * @param colorText Color: Color del texto. + * + * @return PauseTransition: La transition creado con los elementos y colores. + */ + private static PauseTransition createPauseTransition(Text text, Color colorText) { + PauseTransition changeColor = new PauseTransition(new Duration(DURACION)); + changeColor.setOnFinished(actionEvent -> text.setStroke(colorText)); + return changeColor; + } + /** * Remover todos los elementos destacados. */ @@ -383,19 +386,22 @@ public class Grafico { /** * El tipo de objeto que está destacado. */ - private int tipo; + final private int tipo; + /** * El id del elemento destacado. */ - private String id; + final private String id; + /** * El color anterior del elemento destacado. */ - private Color color; + final private Color color; + /** * La animación del elemento destacado. */ - private SequentialTransition sequentialTransition; + final private SequentialTransition sequentialTransition; /** * Inicilizar. diff --git a/src/cl/cromer/estructuras/HashItem.java b/src/cl/cromer/estructuras/HashItem.java index 91065f8..a7f4ed9 100644 --- a/src/cl/cromer/estructuras/HashItem.java +++ b/src/cl/cromer/estructuras/HashItem.java @@ -1,9 +1,11 @@ package cl.cromer.estructuras; public class HashItem { + final private String llave; + + final private int valor; + private int indice; - private String llave; - private int valor; public HashItem(String llave, int valor) { this.llave = llave; diff --git a/src/cl/cromer/estructuras/HashTable.java b/src/cl/cromer/estructuras/HashTable.java index 5c271ed..37a72e9 100644 --- a/src/cl/cromer/estructuras/HashTable.java +++ b/src/cl/cromer/estructuras/HashTable.java @@ -1,8 +1,9 @@ package cl.cromer.estructuras; public class HashTable { - private HashItem hashArray[]; - private int tamano; + final private HashItem hashArray[]; + + final private int tamano; private int size; public HashTable(int tamano) { diff --git a/src/cl/cromer/estructuras/Palabras.java b/src/cl/cromer/estructuras/Palabras.java index 5091138..8e8ed80 100644 --- a/src/cl/cromer/estructuras/Palabras.java +++ b/src/cl/cromer/estructuras/Palabras.java @@ -5,7 +5,7 @@ import java.util.List; import java.util.Random; public class Palabras { - List palabras; + final private List palabras; public Palabras() { palabras = new ArrayList<>();