✅ Deploy Preview for livecodes ready!

Name	Link
🔨 Latest commit	4b2bf6f
🔍 Latest deploy log	https://app.netlify.com/sites/livecodes/deploys/67fa879eb8c6dd0008e43248
😎 Deploy Preview	https://deploy-preview-788--livecodes.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Thank you @Seth0x41
That is truly great work.

I have added some edits, mostly organizational for consistency.
The most notable changes:

• added Java prettier plugin for code formatting
• added codemirror language support for java (monaco & codejar already support java out-of-the-box)
• fixed handling translation
• changed compiler option liveReload to true. This avoids full page reload with every code change, leading to the need to re-download and initialize the environment. Instead, the code is sent to the page to run it in the same environment.
• removed extra console logs to keep the output cleaner.
• refactored initializing FS. Previously it checked if initialization was complete while it was still initializing, so files were downloaded twice.

Do we need this file? public/assets/lang-java-worker.js

Please review the changes and let me know what you think.
Thank you very much. I really appreciate your effort.

Hi @hatemhosny,
Thanks a lot for these changes! Actually, I had noticed an issue like the FS initialization and already fixed it on my side, but I was digging into the reloading issue and was planning to ask you about it today, looks like you beat me to it 😄
No, we don’t need public/assets/lang-java-worker.js], sorry about that, I forgot to delete it while working.
I think everything is good now.
Thanks so much for this experience, and if I optimize anything later, I’ll let you know.
All the best!

Thank you @Seth0x41

I have added docs for Java: https://deploy-preview-788--livecodes.netlify.app/docs/languages/java

It covers the live reload vs full reload. It also covers helper methods & properties that allow communication between Java & JavaScript in the page.

Please review and let me know if you have any comments or additions.

Excellent work, @hatemhosny
I just have one comment, I think we need to change the demo in the Usage section, It's currently printing:

WARN: Warn message  
ERROR: Error message

and this might be confusing, especially since it's printed right after the initialization, so someone might mistake it for a compiler error (I was one of them 😄).
If you don’t have any alternatives, we could use this snippet from The Algorithms instead:

// from https://github.com/TheAlgorithms/Java/blob/master/src/main/java/com/thealgorithms/searches/BinarySearch.java

public class BinarySearchSnippet {

  /**
   * Search an item with binarySearch algorithm.
   *
   * @param arr sorted array to search
   * @param item an item to search
   * @return if item is found, return the index position of the array item otherwise return -1
   */

  public static int binarySearch(int[] arr, int left, int right, int item) {
    if (right >= left) {
      int mid = left + (right - left) / 2;
      if (arr[mid] == item) {
        return mid;
      }

      if (arr[mid] > item) {
        return binarySearch(arr, left, mid - 1, item);
      }

      return binarySearch(arr, mid + 1, right, item);
    }
    return -1;
  }

  public static void main(String[] args) {
    int[] sortedArray = {1, 3, 5, 7, 9, 11, 13, 15};
    int itemToSearch = 7;

    int result = binarySearch(sortedArray, 0, sortedArray.length - 1, itemToSearch);

    if (result == -1) {
      System.out.println("Result: Item not found in the array.");
    } else {
      System.out.println("Result: Item found at index -> " + result);
    }
  }
}

Good suggestion 😄
I will do that 👍

Thank you.

Quality Gate passed

Issues
7 New issues
3 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

i18n Actions

Source PR has been merged into the default branch.

Maintainers can comment .i18n-update-push to trigger the i18n update workflow and push the changes to Lokalise.

Thank you @Seth0x41
This is merged now. It should be available in the next release isA.
Meanwhile you can use it on the development build: https://dev.livecodes.io/

It would be great if we can later on upgrade the Java version.

Very grateful ❤️
Looking forward for your next contribution.

.i18n-update-push

i18n Actions: .i18n-update-push

Localization updated and pushed to Lokalise.

Maintainers can comment .i18n-update-pull after translation is done to trigger the i18n pull workflow and pull the changes back to Github.

.i18n-update-pull

i18n Actions: .i18n-update-pull

Localization pulled from Lokalise.