[Optimize]

Optimize the code for improved readability.

Dockerfile

@@ -1,5 +1,5 @@
 FROM eclipse-temurin:17-jre
-MAINTAINER admin@keke125.com
+LABEL maintainer="admin@keke125.com"
 
 COPY target/*.jar /app/pafw.jar
 EXPOSE 8080

src/main/java/com/keke125/pixel/Application.java

@@ -3,6 +3,7 @@
 import com.keke125.pixel.data.service.UserRepository;
 import com.vaadin.flow.component.page.AppShellConfigurator;
 import com.vaadin.flow.theme.Theme;
+
 import javax.sql.DataSource;
 
 import org.springframework.boot.SpringApplication;
@@ -13,10 +14,9 @@
 
 /**
  * The entry point of the Spring Boot application.
- *
+ * <p>
  * Use the @PWA annotation make the application installable on phones, tablets
  * and some desktop browsers.
- *
  */
 @SpringBootApplication
 @Theme(value = "pixel-art-filter-web")
@@ -28,10 +28,12 @@ public static void main(String[] args) {
     }
 
     @Bean
-    SqlDataSourceScriptDatabaseInitializer dataSourceScriptDatabaseInitializer(DataSource dataSource,
-                                                                               SqlInitializationProperties properties, UserRepository repository) {
+    SqlDataSourceScriptDatabaseInitializer dataSourceScriptDatabaseInitializer
+            (DataSource dataSource, SqlInitializationProperties properties,
+             UserRepository repository) {
         // This bean ensures the database is only initialized when empty
-        return new SqlDataSourceScriptDatabaseInitializer(dataSource, properties) {
+        return new SqlDataSourceScriptDatabaseInitializer(dataSource,
+                properties) {
             @Override
             public boolean initializeDatabase() {
                 if (repository.count() == 0L) {

src/main/java/com/keke125/pixel/components/appnav/AppNav.java

@@ -7,6 +7,7 @@
 import com.vaadin.flow.component.dependency.JsModule;
 import com.vaadin.flow.component.dependency.NpmPackage;
 import com.vaadin.flow.dom.Element;
+
 import java.util.Optional;
 
 @JsModule("@vaadin-component-factory/vcf-nav")
@@ -23,8 +24,7 @@ public AppNav() {
     /**
      * Creates a new menu with the given label.
      *
-     * @param label
-     *            the label to use
+     * @param label the label to use
      */
     public AppNav(String label) {
         setLabel(label);
@@ -33,8 +33,7 @@ public AppNav(String label) {
     /**
      * Adds menu item(s) to the menu.
      *
-     * @param appNavItems
-     *            the menu item(s) to add
+     * @param appNavItems the menu item(s) to add
      * @return the menu for chaining
      */
     public AppNav addItem(AppNavItem... appNavItems) {
@@ -50,8 +49,7 @@ public AppNav addItem(AppNavItem... appNavItems) {
      * <p>
      * If the given menu item is not a child of this menu, does nothing.
      *
-     * @param appNavItem
-     *            the menu item to remove
+     * @param appNavItem the menu item to remove
      * @return the menu for chaining
      */
     public AppNav removeItem(AppNavItem appNavItem) {
@@ -88,8 +86,7 @@ public String getLabel() {
      * This can help the end user to distinguish groups of navigation items. The
      * label is also available for screen reader users.
      *
-     * @param label
-     *            the label to set
+     * @param label the label to set
      * @return this instance for chaining
      */
     public AppNav setLabel(String label) {
@@ -127,9 +124,9 @@ public boolean isCollapsible() {
      * <p>
      * NOTE: The navigation has to have a label for it to be collapsible.
      *
-     * @param collapsible
-     *            true to make the whole navigation component collapsible, false
-     *            otherwise
+     * @param collapsible true to make the whole navigation component
+     *                    collapsible, false
+     *                    otherwise
      * @return this instance for chaining
      */
     public AppNav setCollapsible(boolean collapsible) {

src/main/java/com/keke125/pixel/components/appnav/AppNavItem.java

@@ -8,6 +8,7 @@
 import com.vaadin.flow.router.RouteConfiguration;
 import com.vaadin.flow.router.Router;
 import com.vaadin.flow.server.VaadinService;
+
 import java.util.Optional;
 
 /**
@@ -20,36 +21,34 @@
 public class AppNavItem extends Component {
 
     /**
-     * Creates a menu item which does not link to any view but only shows the given
+     * Creates a menu item which does not link to any view but only shows the
+     * given
      * label.
-     * 
-     * @param label
-     *            the label for the item
+     *
+     * @param label the label for the item
      */
     public AppNavItem(String label) {
         setLabel(label);
     }
 
     /**
-     * Creates a new menu item using the given label that links to the given path.
-     * 
-     * @param label
-     *            the label for the item
-     * @param path
-     *            the path to link to
+     * Creates a new menu item using the given label that links to the given
+     * path.
+     *
+     * @param label the label for the item
+     * @param path  the path to link to
      */
     public AppNavItem(String label, String path) {
         setPath(path);
         setLabel(label);
     }
 
     /**
-     * Creates a new menu item using the given label that links to the given view.
-     * 
-     * @param label
-     *            the label for the item
-     * @param view
-     *            the view to link to
+     * Creates a new menu item using the given label that links to the given
+     * view.
+     *
+     * @param label the label for the item
+     * @param view  the view to link to
      */
     public AppNavItem(String label, Class<? extends Component> view) {
         setPath(view);
@@ -59,13 +58,10 @@ public AppNavItem(String label, Class<? extends Component> view) {
     /**
      * Creates a new menu item using the given label and icon that links to the
      * given path.
-     * 
-     * @param label
-     *            the label for the item
-     * @param path
-     *            the path to link to
-     * @param icon
-     *            the icon for the item
+     *
+     * @param label the label for the item
+     * @param path  the path to link to
+     * @param icon  the icon for the item
      */
     public AppNavItem(String label, String path, Component icon) {
         setPath(path);
@@ -74,26 +70,24 @@ public AppNavItem(String label, String path, Component icon) {
     }
 
     /**
-     * Creates a new menu item using the given label that links to the given view.
-     * 
-     * @param label
-     *            the label for the item
-     * @param view
-     *            the view to link to
-     * @param icon
-     *            the icon for the item
+     * Creates a new menu item using the given label that links to the given
+     * view.
+     *
+     * @param label the label for the item
+     * @param view  the view to link to
+     * @param icon  the icon for the item
      */
-    public AppNavItem(String label, Class<? extends Component> view, Component icon) {
+    public AppNavItem(String label, Class<? extends Component> view,
+                      Component icon) {
         setPath(view);
         setLabel(label);
         setIcon(icon);
     }
 
     /**
      * Adds menu item(s) inside this item, creating a hierarchy.
-     * 
-     * @param appNavItems
-     *            the menu item(s) to add
+     *
+     * @param appNavItems the menu item(s) to add
      * @return this item for chaining
      */
     public AppNavItem addItem(AppNavItem... appNavItems) {
@@ -109,9 +103,8 @@ public AppNavItem addItem(AppNavItem... appNavItems) {
      * Removes the given menu item from this item.
      * <p>
      * If the given menu item is not a child of this menu item, does nothing.
-     * 
-     * @param appNavItem
-     *            the menu item to remove
+     *
+     * @param appNavItem the menu item to remove
      * @return this item for chaining
      */
     public AppNavItem removeItem(AppNavItem appNavItem) {
@@ -125,7 +118,7 @@ public AppNavItem removeItem(AppNavItem appNavItem) {
 
     /**
      * Removes all menu items from this item.
-     * 
+     *
      * @return this item for chaining
      */
     public AppNavItem removeAllItems() {
@@ -135,7 +128,7 @@ public AppNavItem removeAllItems() {
 
     /**
      * Gets the label for the item.
-     * 
+     *
      * @return the label or null if no label has been set
      */
     public String getLabel() {
@@ -146,9 +139,8 @@ public String getLabel() {
      * Set a textual label for the item.
      * <p>
      * The label is also available for screen rader users.
-     * 
-     * @param label
-     *            the label to set
+     *
+     * @param label the label to set
      * @return this instance for chaining
      */
     public AppNavItem setLabel(String label) {
@@ -170,9 +162,8 @@ private Element getLabelElement() {
 
     /**
      * Sets the path this item links to.
-     * 
-     * @param path
-     *            the path to link to
+     *
+     * @param path the path to link to
      * @return this instance for chaining
      */
     public AppNavItem setPath(String path) {
@@ -182,13 +173,13 @@ public AppNavItem setPath(String path) {
 
     /**
      * Sets the view this item links to.
-     * 
-     * @param view
-     *            the view to link to
+     *
+     * @param view the view to link to
      * @return this instance for chaining
      */
     public AppNavItem setPath(Class<? extends Component> view) {
-        String url = RouteConfiguration.forRegistry(getRouter().getRegistry()).getUrl(view);
+        String url =
+                RouteConfiguration.forRegistry(getRouter().getRegistry()).getUrl(view);
         setPath(url);
         return this;
     }
@@ -203,7 +194,8 @@ private Router getRouter() {
             router = VaadinService.getCurrent().getRouter();
         }
         if (router == null) {
-            throw new IllegalStateException("Implicit router instance is not available. "
+            throw new IllegalStateException("Implicit router instance is not " +
+                    "available. "
                     + "Use overloaded method with explicit router parameter.");
         }
         return router;
@@ -225,10 +217,10 @@ private int getIconElementIndex() {
     /**
      * Sets the icon for the item.
      * <p>
-     * Can also be used to set a custom component to be shown in front of the label.
-     * 
-     * @param icon
-     *            the icon to show
+     * Can also be used to set a custom component to be shown in front of the
+     * label.
+     *
+     * @param icon the icon to show
      * @return this instance for chaining
      */
     public AppNavItem setIcon(Component icon) {
@@ -245,8 +237,7 @@ public AppNavItem setIcon(Component icon) {
     /**
      * Sets the expanded status of the item.
      *
-     * @param value
-     *            true to expand the item, false to collapse it
+     * @param value true to expand the item, false to collapse it
      */
     public AppNavItem setExpanded(boolean value) {
         if (value) {

src/main/java/com/keke125/pixel/core/AppConfig.java

@@ -9,39 +9,58 @@
 @Component
 @Data
 public class AppConfig {
+
     // default password encoder can be set by idForEncode
     // BCrypt pbkdf2 argon2
     // @Value("{security.password.encoder:argon2}")
     private String idForEncode;
+
     // only file size below 10MB can be uploaded
     // you can modify this value, but the limit is 2047MB
     // @Value("{upload.image.size:10}")
     private int maxImageSizeInMegaBytes;
+
     // max 3 files can be uploaded
     // you can modify this value
     // @Value("{upload.image.number:3}")
     private int maxImageFiles;
+
     // only file size below 3MB can be uploaded
     // you can modify this value, but the limit is 2047MB
     // @Value("{upload.avatar.size:3}")
     private int maxAvatarSizeInMegaBytes;
+
     // The default image size limit for new sign-up users
     private int newSignupImageSizeLimit;
+
     // website location country
     private String webCountry;
+
     // website name
     private String webName;
+
     // website link
     private String webLink;
+
     // admin contact email
     private String adminContactEmail;
-    // web name
+
+    // web name TC
     private String webNameTC;
+
+    // web name EN
     private String webNameEN;
-    // web description
+
+    // web description TC
     private String webDescriptionTC;
+
+    // web description EN
     private String webDescriptionEN;
-    // login form additional information
+
+    // login form additional information TC
     private String loginInfoTC;
+
+    // login form additional information EN
     private String loginInfoEN;
+
 }