Update sync and async file open

Author: keybraker

electron/guiExample.html

@@ -20,7 +20,9 @@
       />
       <button id="send_code">Interact with program</button>
       <button id="stop_code">Terminate program</button>
-      <button id="open_file">How to open a file</button>
+      <hr>
+      <button id="open_file_sync">Open a file sync</button>
+      <button id="open_file_async">Open a file async</button>
       <p><a href="https://github.com/keybraker">Made by Keybraker</a></p>
     </div>
   </body>

electron/guiExample.js

@@ -1,7 +1,10 @@
-const exec = require("child_process").exec;
+const { exec } = require("child_process");
 const nodeConsole = require("console");
 const { ipcRenderer } = require("electron");
 
+const terminalConsole = new nodeConsole.Console(process.stdout, process.stderr);
+let child;
+
 ipcRenderer.send("run-command", "ls");
 ipcRenderer.on("run-command-result", (event, result) => {
   if (result.error) {
@@ -11,77 +14,76 @@ ipcRenderer.on("run-command-result", (event, result) => {
   }
 });
 
-const terminalConsole = new nodeConsole.Console(process.stdout, process.stderr);
-let child;
+const printBoth = (str) => {
+  console.log(`Javascript: ${str}`);
+  terminalConsole.log(`Javascript: ${str}`);
+};
 
-function print_both(str) {
-  console.log("Javascript: " + str);
-  terminalConsole.log("Javascript: " + str);
-}
-
-function send_to_program(str) {
+const sendToProgram = (str) => {
   child.stdin.write(str);
-  child.stdout.on("data", function (data) {
-    print_both(
-      "Following data has been piped from python program: " +
-        data.toString("utf8")
+  child.stdout.on("data", (data) => {
+    printBoth(
+      `Following data has been piped from python program: ${data.toString(
+        "utf8"
+      )}`
     );
   });
-}
+};
 
-// starts program execution from within javascript and
-function start_code_function(evt) {
-  print_both("Initiating program");
+const startCodeFunction = () => {
+  printBoth("Initiating program");
 
-  child = exec(
-    "python -i ./python/pythonExample.py ",
-    function (error, stdout, stderr) {
-      if (error !== null) {
-        print_both("exec error: " + error);
-      }
+  child = exec("python -i ./python/pythonExample.py", (error) => {
+    if (error) {
+      printBoth(`exec error: ${error}`);
     }
-  );
+  });
 
-  child.stdout.on("data", function (data) {
-    print_both(
-      "Following data has been piped from python program: " +
-        data.toString("utf8")
+  child.stdout.on("data", (data) => {
+    printBoth(
+      `Following data has been piped from python program: ${data.toString(
+        "utf8"
+      )}`
     );
   });
-}
+};
 
-// sends data to program
-function send_code_function(evt) {
-  let string_to_send = document.getElementById("string_to_send").value;
-  print_both('Sending "' + string_to_send + '" to program');
-  send_to_program(string_to_send);
-}
+const sendCodeFunction = () => {
+  const stringToSend = document.getElementById("string_to_send").value;
+  printBoth(`Sending "${stringToSend}" to program`);
+  sendToProgram(stringToSend);
+};
 
-// sends termination message to python program and closed stream
-// that receives information from it
-function stop_code_function(evt) {
-  print_both("Terminated program");
-  send_to_program("terminate");
+const stopCodeFunction = () => {
+  printBoth("Terminated program");
+  sendToProgram("terminate");
   child.stdin.end();
-}
+};
 
-// requests main.js to open a file from the filesystem
-function open_file_function(evt) {
-  print_both("From guiExample.js sending a request to main.js via ipc");
-  ipcRenderer.send("open_json_file");
-}
+const openFileFunctionSync = () => {
+  printBoth("From guiExample.js sending a request to main.js via ipc");
+  ipcRenderer.send("open_json_file_sync");
+};
 
-document.addEventListener("DOMContentLoaded", function () {
+const openFileFunctionAsync = () => {
+  printBoth("From guiExample.js sending a request to main.js via ipc");
+  ipcRenderer.send("open_json_file_async");
+};
+
+document.addEventListener("DOMContentLoaded", () => {
   document
     .getElementById("start_code")
-    .addEventListener("click", start_code_function);
+    .addEventListener("click", startCodeFunction);
   document
     .getElementById("send_code")
-    .addEventListener("click", send_code_function);
+    .addEventListener("click", sendCodeFunction);
   document
     .getElementById("stop_code")
-    .addEventListener("click", stop_code_function);
+    .addEventListener("click", stopCodeFunction);
+  document
+    .getElementById("open_file_sync")
+    .addEventListener("click", openFileFunctionSync);
   document
-    .getElementById("open_file")
-    .addEventListener("click", open_file_function);
+    .getElementById("open_file_async")
+    .addEventListener("click", openFileFunctionAsync);
 });

electron/main.js

@@ -3,30 +3,29 @@ const { app, BrowserWindow, ipcMain, Notification } = require("electron");
 const exec = require("child_process").exec;
 const path = require("path");
 
-var nodeConsole = require("console");
-var my_console = new nodeConsole.Console(process.stdout, process.stderr);
-var child;
+const nodeConsole = require("console");
+const myConsole = new nodeConsole.Console(process.stdout, process.stderr);
+let child;
 
-function print_both(str) {
+function printBoth(str) {
   console.log("main.js:    " + str);
-  my_console.log("main.js:    " + str);
+  myConsole.log("main.js:    " + str);
 }
 
+// Create the browser window.
 function createWindow() {
-  // Create the browser window.
   const mainWindow = new BrowserWindow({
     width: 990,
     height: 600,
     resizable: true,
     webPreferences: {
       preload: path.join(__dirname, "guiExample.js"),
       contextIsolation: true,
-      nodeIntegration: true
+      nodeIntegration: true,
     },
   });
 
-  // and load the index.html of the app.
-  // mainWindow.loadFile("./electron/guiExample.html");
+  // Load the index.html of the app.
   mainWindow.loadFile(path.join(__dirname, "guiExample.html"));
 
   // Open the DevTools.
@@ -64,32 +63,27 @@ ipcMain.on("execute", (command) => {
   });
 });
 
-ipcMain.on("open_json_file", () => {
-  var fs = require("fs");
-  var fileName = "./config.json";
-  var file = require(fileName);
+ipcMain.on("open_json_file_sync", () => {
+  const fs = require("fs");
 
-  // Asynchronous read
-  // fs.readFile('config.json', function (err, data) {
-  //   if (err) {
-  //     return console.error(err);
-  //   }
-  //   console.log("Asynchronous read: " + data.toString());
-  // });
+  fs.readFile("config.json", function (err, data) {
+    if (err) {
+      return console.error(err);
+    }
+    printBoth("Called through ipc.send from guiExample.js");
+    printBoth("Asynchronous read: " + data.toString());
+  });
+});
+
+ipcMain.on("open_json_file_async", () => {
+  const fs = require("fs");
 
-  // Synchronous read
-  var data = fs.readFileSync(fileName);
-  var json = JSON.parse(data);
+  const fileName = "./config.json";
+  const data = fs.readFileSync(fileName);
+  const json = JSON.parse(data);
 
-  print_both("Called through ipc.send from guiExample.js");
-  print_both(
-    "Data from config.json:\nA_MODE = " +
-      json.A_MODE +
-      "\nB_MODE = " +
-      json.B_MODE +
-      "\nC_MODE = " +
-      json.C_MODE +
-      "\nD_MODE = " +
-      json.D_MODE
+  printBoth("Called through ipc.send from guiExample.js");
+  printBoth(
+    `Data from config.json:\nA_MODE = ${json.A_MODE}\nB_MODE = ${json.B_MODE}\nC_MODE = ${json.C_MODE}\nD_MODE = ${json.D_MODE}`
   );
 });