v0.4.4

Działający proces własnej aktualizacji

Author: koder95

pom.xml

@@ -9,7 +9,7 @@
     <url>https://github.com/koder95/eMetrykant</url>
     <groupId>pl.koder95</groupId>
     <artifactId>eMetrykant</artifactId>
-    <version>0.4.3</version>
+    <version>0.4.4</version>
     <scm>
         <url>https://github.com/koder95/eMetrykant</url>
         <tag>search, metric, church</tag>

src/main/java/pl/koder95/eme/Main.java

@@ -21,6 +21,7 @@
 import javafx.geometry.Pos;
 import javafx.scene.Parent;
 import javafx.scene.Scene;
+import javafx.scene.control.Alert;
 import javafx.scene.control.Label;
 import javafx.scene.control.ProgressBar;
 import javafx.scene.image.Image;
@@ -35,6 +36,7 @@
 import pl.koder95.eme.dfs.IndexList;
 import pl.koder95.eme.git.RepositoryInfo;
 
+import java.io.IOException;
 import java.text.Collator;
 import java.util.Arrays;
 import java.util.Locale;
@@ -45,7 +47,7 @@
  * Klasa uruchamiająca i inicjująca podstawowe elementy aplikacji.
  *
  * @author Kamil Jan Mularski [@koder95]
- * @version 0.4.3, 2024-12-01
+ * @version 0.4.4, 2024-12-02
  * @since 0.0.201
  */
 public class Main extends Application {
@@ -82,10 +84,12 @@ public static void main(String[] args) {
             if (args[0].equals("-v")) {
                 System.out.println(Version.get());
                 System.exit(0);
+                return;
             }
             else if (args[0].equals("-u")) {
                 SelfUpdate su = new SelfUpdate();
                 new Thread(su).start();
+                return;
             }
         }
         Main.launch(args);
@@ -105,7 +109,7 @@ public static void releaseMemory() {
     @Override
     public void init() throws Exception {
         super.init();
-        RepositoryInfo.get().reload();
+        task = new SelfUpdateTask();
         if (getParameters().getUnnamed().isEmpty()) {
             Arrays.stream(IndexList.values()).forEach(IndexList::load);
         }
@@ -119,7 +123,6 @@ public void init() throws Exception {
         worker.load();
 
         root = FXMLLoader.load(ClassLoader.getSystemResource("pl/koder95/eme/fx/PersonalDataView.fxml"), BUNDLE);
-        task = new SelfUpdateTask();
     }
 
     @Override

src/main/java/pl/koder95/eme/au/SelfUpdate.java

@@ -26,9 +26,19 @@
 
 /**
  * Klasa odpowiedzialna za aktualizowanie własnej wersji do najnowszej.
+ * Działa w następujący sposób:
+ * <ol>
+ *     <li>Usuwanie zawartości folderu tymczasowego, gdzie pobrane zostaną pliki ostatniego wydania.</li>
+ *     <li>Pobieranie najnowszego wydania programu w postaci archiwum ZIP.</li>
+ *     <li>Wypakowanie zawartości archiwum do folderu tymczasowego.</li>
+ *     <li>Usunięcie pobranego archiwum ZIP.</li>
+ *     <li>Wygenerowanie skryptu samoaktualizacji.</li>
+ *     <li>Uruchomienie skryptu i zamknięcie programu.</li>
+ * </ol>
+ * Generator skryptu aktualizującego powinien zadbać o uruchomienie programu po skończeniu aktualizacji.
  *
  * @author Kamil Jan Mularski [@koder95]
- * @version 0.4.3, 2024-12-01
+ * @version 0.4.4, 2024-12-02
  * @since 0.4.3
  */
 public class SelfUpdate implements Runnable {
@@ -56,26 +66,33 @@ public class SelfUpdate implements Runnable {
      * Tworzy domyślną instancję klasy.
      */
     public SelfUpdate() {
-        this((workDone, max) -> {
-            double percent = 100 * workDone.doubleValue()/max.doubleValue();
-            System.out.println("Processing... " + (int) percent);
-        }, msg -> System.out.println("INFO: " + msg), title -> System.out.println('\n' + title + '\n'));
+        this((workDone, max) -> {}, msg -> {}, title -> System.out.println('\n' + title + '\n'));
     }
 
     @Override
     public void run() {
+        updateTitle.accept("Rozpoczęcie aktualizacji");
         try {
-            // usuwanie plików z katalogu tymczasowego:
+            updateMessage.accept("Usuwanie plików z katalogu tymczasowego...");
             clear();
-            // pobieranie zip'a i rozpakowywanie do folderu tymczasowego:
+            updateMessage.accept("Przygotowywanie do pobierania...");
             prepare();
         } catch (IOException e) {
             throw new RuntimeException(e);
         }
+        updateTitle.accept("Ponowne uruchamianie w celu zapisania zmian...");
+        updateProgress.accept(0, 0);
+        updateMessage.accept("");
+        try {
+            Thread.sleep(200);
+        } catch (InterruptedException e) {
+            // ignoruj
+        }
         restart(); // restartowanie z wywołaniem skryptu
     }
 
     private void clear() throws IOException {
+        if (Files.notExists(TEMP_DIR)) return;
         List<Exception> exceptions = new LinkedList<>();
         try (Stream<Path> paths = Files.walk(TEMP_DIR).sorted(Comparator.reverseOrder())) {
             paths.forEach(path -> {
@@ -105,16 +122,25 @@ private void restart() {
     }
 
     private void prepare() throws IOException {
+        updateMessage.accept("Tworzenie katalogu tymczasowego...");
+        Files.createDirectories(TEMP_DIR);
+        updateMessage.accept(Files.exists(TEMP_DIR)? "Stworzono katalog tymczasowy." : "Błąd tworzenia katalogu tymczasowego!");
+        updateMessage.accept("Sprawdzanie ostatniej wersji...");
+        RepositoryInfo.get().reload();
         if (extractZip(downloadZip())) {
             Map<Path, Path> updateMap = new HashMap<>();
             try (Stream<Path> paths = Files.list(TEMP_DIR)) {
-                paths.forEach(path -> updateMap.put(path, WORKDIR.resolve(path.getFileName().toString())));
+                paths.forEach(path -> {
+                    String fileName = path.getFileName().toString();
+                    if (fileName.startsWith("eMetrykant") && fileName.endsWith(".jar"))
+                        fileName = SELF.getFileName().toString();
+                    updateMap.put(path, WORKDIR.resolve(fileName));
+                });
             }
             updateScriptGenerator.generateUpdateScript(updateMap);
         }
     }
 
-
     private Path downloadZip() throws IOException {
         updateTitle.accept("Pobieranie " + RepositoryInfo.get().getLatestReleaseName());
         return tryTransfer(RepositoryInfo.get().getLatestReleaseBrowserURL());
@@ -172,7 +198,7 @@ private boolean extractZip(Path forExtract) throws IOException {
                     Files.createFile(outFile);
 
                     if (Files.isRegularFile(outFile)) {
-                        tryTransfer(zip, entry, outFile, workDone, total);
+                        workDone = tryTransfer(zip, entry, outFile, workDone, total);
                     }
                 } else if (Files.notExists(outFile)) {
                     Files.createDirectories(outFile);
@@ -199,20 +225,21 @@ private static long getTotal(ZipFile zip) throws IOException {
         return total;
     }
 
-    private void tryTransfer(ZipFile zip, ZipEntry entry, Path outFile, long workDone, long total) throws IOException {
+    private long tryTransfer(ZipFile zip, ZipEntry entry, Path outFile, long workDone, long total) throws IOException {
         try (InputStream input = zip.getInputStream(entry);
              OutputStream output = Files.newOutputStream(outFile)) {
-            transfer(input, output, workDone, total);
+            return transfer(input, output, workDone, total);
         }
     }
 
-    private void transfer(InputStream input, OutputStream output, long workDone, long total) throws IOException {
+    private long transfer(InputStream input, OutputStream output, long workDone, long total) throws IOException {
         while (input.available() > 0) {
             int b = input.read();
             output.write(b);
             output.flush();
             updateProgress.accept(++workDone, total);
             updateMessage.accept(NumberFormat.getPercentInstance().format((double) workDone / total));
         }
+        return workDone;
     }
 }

src/main/java/pl/koder95/eme/au/SelfUpdateTask.java

@@ -20,26 +20,24 @@
 
 /**
  * Klasa definiuje metody samoaktualizacji, które współpracują z JavaFX. Jest to szczególna implementacja
- * klasy abstrakcyjnej {@link Task}, której metoda {@link #call()} zwraca zawsze {@code null} i działa
- * w następujący sposób:
- * <ol>
- *     <li>Usuwanie zawartości folderu tymczasowego, gdzie pobrane zostaną pliki ostatniego wydania.</li>
- *     <li>Pobieranie najnowszego wydania programu w postaci archiwum ZIP.</li>
- *     <li>Wypakowanie zawartości archiwum do folderu tymczasowego.</li>
- *     <li>Usunięcie pobranego archiwum ZIP.</li>
- *     <li>Wygenerowanie skryptu samoaktualizacji.</li>
- *     <li>Uruchomienie skryptu i zamknięcie programu.</li>
- * </ol>
+ * klasy abstrakcyjnej {@link Task}, której metoda {@link #call()} zwraca zawsze {@code null}.
  * Generator skryptu aktualizującego powinien zadbać o uruchomienie programu po skończeniu aktualizacji.
+ *
+ * @author Kamil Jan Mularski [@koder95]
+ * @version 0.4.4, 2024-12-02
+ * @since 0.4.1
+ * @see SelfUpdate
  */
 public class SelfUpdateTask extends Task<Void> {
 
+    private final SelfUpdate su = new SelfUpdate(this::updateProgress, this::updateMessage, this::updateTitle);
+
     /**
      * @return zawsze {@code null}
      */
     @Override
     protected Void call() throws Exception {
-        new SelfUpdate(this::updateProgress, this::updateMessage, this::updateTitle).run();
+        su.run();
         return null;
     }
 

src/main/java/pl/koder95/eme/au/UpdateScriptGenerator.java

@@ -33,7 +33,7 @@
  * Klasa dostarcza metody do generowania skryptów aktualizacyjnych.
  *
  * @author Kamil Jan Mularski [@koder95]
- * @version 0.4.3, 2024-12-01
+ * @version 0.4.4, 2024-12-02
  * @since 0.4.1
  */
 public class UpdateScriptGenerator {
@@ -58,9 +58,7 @@ public Path getPath() {
      * @return nowa instancja generatora skryptu
      */
     public static UpdateScriptGenerator create(Path out) {
-        return new UpdateScriptGenerator(IS_WINDOWS_OS?
-                out.endsWith(WIN_EXT)? out : Paths.get(out + WIN_EXT)
-                : out);
+        return new UpdateScriptGenerator(IS_WINDOWS_OS && !out.endsWith(WIN_EXT)? Paths.get(out + WIN_EXT) : out);
     }
 
     /**
@@ -71,7 +69,7 @@ public static UpdateScriptGenerator create(Path out) {
     public void generateUpdateScript(Map<Path, Path> updateMap) throws IOException {
         if (Files.notExists(path)) Files.createFile(path);
         try (BufferedWriter writer = Files.newBufferedWriter(path, StandardOpenOption.WRITE)) {
-            if (path.endsWith(WIN_EXT)) {
+            if (IS_WINDOWS_OS) {
                 generateWinUpdateScript(writer, updateMap);
             } else {
                 generateUnixScript(writer, updateMap);
@@ -92,15 +90,21 @@ private void generateUnixScript(BufferedWriter writer, Map<Path, Path> updateMap
                 writer.write(newFile.toString());
                 writer.write('"' + " " + '"');
                 writer.write(oldFile.toString());
-                writer.write('"' + " /y");
+                writer.write('"' + " 2>/dev/null");
                 writer.newLine();
+            } else throw new FileNotFoundException(BUNDLE.getString("THR_UPDATE_FILES_NOT_FOUND"));
+        }
+        for (Path newFile : updateMap.keySet()) {
+            if (Files.exists(newFile)) {
                 writer.write("rmdir -r " + '"');
-                writer.write(oldFile.getParent().toString());
-                writer.write('"' + " /y");
+                writer.write(newFile.getParent().toString());
+                writer.write('"' + " 2>/dev/null");
                 writer.newLine();
             } else throw new FileNotFoundException(BUNDLE.getString("THR_UPDATE_FILES_NOT_FOUND"));
         }
-        writer.write("java -jar \"" + SELF + '"');
+        writer.write("rm -rf \"" + this.path + "\" 2>/dev/null");
+        writer.newLine();
+        writer.write('"' + System.getProperty("java.home") + "\\bin\\java\" -jar \"" + SELF + '"');
         writer.newLine();
     }
 
@@ -118,15 +122,21 @@ private void generateWinUpdateScript(BufferedWriter writer, Map<Path, Path> upda
                 writer.write(newFile.toString());
                 writer.write('"' + " " + '"');
                 writer.write(oldFile.toString());
-                writer.write('"' + " /y");
+                writer.write('"' + " > nul");
                 writer.newLine();
-                writer.write("rmdir -r " + '"');
-                writer.write(oldFile.getParent().toString());
-                writer.write('"' + " /y");
+            } else throw new FileNotFoundException(BUNDLE.getString("THR_UPDATE_FILES_NOT_FOUND"));
+        }
+        for (Path newFile : updateMap.keySet()) {
+            if (Files.exists(newFile)) {
+                writer.write("rmdir /s /q " + '"');
+                writer.write(newFile.getParent().toString());
+                writer.write('"' + " > nul");
                 writer.newLine();
             } else throw new FileNotFoundException(BUNDLE.getString("THR_UPDATE_FILES_NOT_FOUND"));
         }
-        writer.write("java -jar \"" + SELF + '"');
+        writer.write("del \"" + this.path + "\" /q");
+        writer.newLine();
+        writer.write('"' + System.getProperty("java.home") + "\\bin\\java.exe\" -jar \"" + SELF + '"');
         writer.newLine();
     }
 }

src/main/java/pl/koder95/eme/git/GitHubRepositoryController.java

@@ -27,7 +27,7 @@
  * Klasa dostarcza metod kontrolujących połączenie z repozytorium GitHub.
  *
  * @author Kamil Jan Mularski [@koder95]
- * @version 0.4.1, 2021-11-07
+ * @version 0.4.4, 2024-12-02
  * @since 0.4.1
  */
 public class GitHubRepositoryController {
@@ -92,13 +92,9 @@ private void selectRepository() throws IOException {
     /**
      * Wprowadza podstawowe ustawienia do połączenia z repozytorium: wybiera użytkownika i repozytorium.
      */
-    public void init() {
-        try {
-            selectUser();
-            selectRepository();
-        } catch (IOException e) {
-            throw new ExceptionInInitializerError(e);
-        }
+    public void init() throws IOException {
+        selectUser();
+        selectRepository();
     }
 
     /**