fixed the errors.

Author: jamallaqdiem

implement-shell-tools/cat/cat.js

@@ -1,21 +1,18 @@
 import process from "node:process";
 import { promises as fs } from "node:fs";
 
-// THis will give an array without the path to node and to the file.
 const argv = process.argv.slice(2);
 
-//Get line numbers.
 const showNumbers = argv.includes("-n");
 const showNonBlankNumbers = argv.includes("-b");
 
-//filter the - from the array argv as it's a flag.
 const filePaths = argv.filter((arg) => !arg.startsWith("-"));
 
 const flagsUsed = argv.filter((arg) => arg.startsWith("-"));
 const supportedFlags = ["-n", "-b"];
 for (const flag of flagsUsed) {
   if (!supportedFlags.includes(flag)) {
-    console.error(`Invalid option try 'node cat.js --help' for more info.`);
+    console.error(`Invalid option: please try "-n or "-b"`);
     process.exit(1);
   }
 }
@@ -25,9 +22,8 @@ let counterLines = 1;
 for (const path of filePaths) {
   try {
     const content = await fs.readFile(path, "utf-8");
-
-    //split the text at the new line character.
-    const splitLines = content.split("\n");
+.
+    const splitLines = content.split("\n");   
 
     splitLines.forEach((line) => {
       let prefix = "";
@@ -42,7 +38,7 @@ for (const path of filePaths) {
       console.log(`${prefix}${line}`);
     });
   } catch (error) {
-    console.error(`Could not read: ${path}`);
+    console.error(`Error ${path}: ${error.message}`);
     process.exit(1);
   }
 }

implement-shell-tools/ls/ls.js

@@ -1,32 +1,27 @@
 import fs from "node:fs";
 import process from "node:process";
 
-// This will give an array without the path to node and to the file.
 const argv = process.argv.slice(2);
 
-// filter the flag to find the target folder.
 const filePaths = argv.filter((arg) => !arg.startsWith("-"));
 const showHiddenFiles = argv.includes("-a");
 
 if (filePaths.length > 1) {
   console.error("Error: This version only supports one directory path a time.");
   process.exit(1);
 }
-// if no folder provide we use the current one
+
 const target = filePaths[0] || ".";
-// read the file.
+
 const files = fs.readdirSync(target);
 
-// save the result into the variable.
 let filteredFIles = files;
 if (!showHiddenFiles) {
   filteredFIles = files.filter((file) => !file.startsWith("."));
 } else {
-  // we use spread operator to merge the paths.
   filteredFIles = [".", "..", ...files];
 }
 
-// Print using -1 .
 filteredFIles.forEach((file) => {
   console.log(file);
 });

implement-shell-tools/wc/wc.js

@@ -8,7 +8,6 @@ let showLines = argv.includes("-l");
 let showBytes = argv.includes("-c");
 let showCharacters = argv.includes("-m");
 
-//  filter flags, and getting the string of filename
 const filePaths = argv.filter((arg) => !arg.startsWith("-"));
 // if no flags enable all.
 if (!showLines && !showCharacters && !showWords && !showBytes) {
@@ -17,30 +16,52 @@ if (!showLines && !showCharacters && !showWords && !showBytes) {
   showWords = true;
   showBytes = true;
 }
-// fix bug .length ===0 will be true if nothing provided, instead !filePath will return empty array which result true.
+
 if (filePaths.length === 0) {
   console.error("PLease provide a file path");
   process.exit(1);
 }
-// loop trough the array to get each file path.
+
+let totalLines = 0;
+let totalWords = 0;
+let totalBytes = 0;
+let totalChars = 0;
+
 filePaths.forEach((filePath) => {
-  const content = fs.readFileSync(filePath, "utf-8");
-
-  const lines = content.split("\n").length - 1;
-  const words = content
-    .trim()
-    .split(/\s+/)
-    .filter((word) => word != "").length;
-  // here I used Buffer.byteLength even if characters and bytes can be the same number .length, however sometimes an emoji or special characters can be heavier 2b or 4b
-  const bytes = Buffer.byteLength(content);
-  const characters = content.length;
-
-  let output = "";
-
-  if (showLines) output += `${lines} `;
-  if (showWords) output += `${words} `;
-  if (showBytes) output += `${bytes} `;
-  if (showCharacters) output += `${characters} `;
-
-  console.log(`${output} ${filePath}`);
-});
+  try {
+    const content = fs.readFileSync(filePath, "utf-8");
+    const lineArray = content.split("\n");
+    if (content.endsWith("\n")) lineArray.pop(); 
+    const lines = lineArray.length;
+    
+    const words = content.trim().split(/\s+/).filter((word) => word != "").length;
+    const bytes = Buffer.byteLength(content);
+    const characters = content.length;
+
+    totalLines += lines;
+    totalWords += words;
+    totalBytes += bytes;
+    totalChars += characters;
+
+    let output = "";
+    if (showLines)   output += lines.toString().padStart(8);
+    if (showWords)   output += words.toString().padStart(8);
+    if (showBytes)   output += bytes.toString().padStart(8);
+    if (showCharacters) output += characters.toString().padStart(8)
+
+    console.log(`${output} ${filePath}`); 
+  } catch(error) {
+    console.error(`Error reading ${filePath}: ${error.message}`);
+  }
+}); 
+
+if (filePaths.length > 1) {
+  let totalOutput = "";
+  if (showLines)  totalOutput += `${totalLines.toString().padStart(8)}`;
+  if (showWords)  totalOutput += `${totalWords.toString().padStart(8)}`;
+  if (showBytes)  totalOutput += `${totalBytes.toString().padStart(8)}`;
+  if (showCharacters) totalOutput += `${totalChars.toString().padStart(8)}`;
+
+  console.log(`${totalOutput} total`);
+}
+