update

Author: ZabihollahNamazi

implement-shell-tools/cat/cat.js

@@ -10,6 +10,9 @@ const showNonEmptyNumbers = args.includes("-b");
 // getting the paths of files
 const paths = args.filter(arg => arg !== "-n" && arg !== "-b");
 
+// helper: mimic real cat line number formatting
+const padLineNumber = (num) => String(num).padStart(6, " ");
+
 // loop over each file
 for (const path of paths) {
   try {
@@ -21,24 +24,24 @@ for (const path of paths) {
 
     let lineNumber = 1; // tracks line numbers for -b and -n
 
-    lines.forEach((line) => {
+    for (const line of lines) {
       if (showNonEmptyNumbers) {
         // -b: number only non-empty lines
-        if (line.trim() !== "") {
-          console.log(`${lineNumber} ${line}`);
+        if (line !== "") {
+          console.log(`${padLineNumber(lineNumber)}  ${line}`);
           lineNumber++;
         } else {
           console.log(line); // shows empty line with no number
         }
       } else if (showAllNumbers) {
         // -n: number all lines
-        console.log(`${lineNumber} ${line}`);
+        console.log(`${padLineNumber(lineNumber)}  ${line}`);
         lineNumber++;
       } else {
         // no flags: just print line
         console.log(line);
       }
-    });
+    }
   } catch (err) {
     console.error(`Error reading file "${path}": ${err.message}`);
   }

implement-shell-tools/ls/ls.js

@@ -10,19 +10,32 @@ const showAllFilesWithHidden = args.includes("-a");
 const path = args.find(arg => !arg.startsWith("-")) || ".";
 // if we do console.log("path=> ",path," args=> " ,args); it will give us this =>: path=>  sample-files  args=>  [ '-1', '-a', 'sample-files' ]
 
-const direc = await fs.readdir(path)
-// if the path is <sample-files> console.log(direc) gives us =>: [ '.hidden.txt', '1.txt', '2.txt', '3.txt', 'dir' ]
+try {
+  const direc = await fs.readdir(path);
 
-if(showOnePerLine && showAllFilesWithHidden){ 
-    direc.forEach(element => {
-        console.log(element)
-    })
-}
-else if(showOnePerLine){
-    const visibleFiles = direc.filter(element => !element.startsWith(".")); // filtering hidden files which starts with "."
-    visibleFiles.forEach(element => {
-        console.log(element)
+  // handle hidden files (-a flag controls this)
+  let files = direc;
+
+  if (!showAllFilesWithHidden) {
+    files = files.filter(file => !file.startsWith("."));
+  }
+
+  // one file per line
+  if (showOnePerLine) {
+    files.forEach(file => {
+      console.log(file);
     });
+  }
+
+  // default behavior: also one per line (simple version of ls)
+  else {
+    files.forEach(file => {
+      console.log(file);
+    });
+  }
+
+} catch (err) {
+  console.error(`Error reading directory "${path}": ${err.message}`);
 }
 
 

implement-shell-tools/wc/wc.js

@@ -1,60 +1,55 @@
 import { promises as fs } from "node:fs";
 
 const args = process.argv.slice(2);
-console.log(args)
 
 const showLines = args.includes("-l");
 const showWords = args.includes("-w");
 const showChars = args.includes("-c");
-const paths = args.filter(arg => !arg.startsWith("-")) || ".";
-console.log(paths)
+let paths = args.filter(arg => !arg.startsWith("-"));
+if (paths.length === 0) paths = ["."];
 
-// const direct = await fs.readdir(path);
+// helper for formatting like real wc
+const pad = (n) => String(n).padStart(8, " ");
 
-// console.log(direct);
 let totalLines = 0;
 let totalWords = 0;
-let totalchars = 0;
+let totalChars = 0;
 
-if(showLines){
-    for (const path of paths){
-    const content = await fs.readFile(path, "utf-8"); 
-    const lines = content.split("\n").length;
+for (const path of paths) {
+    try {
+        const content = await fs.readFile(path, "utf-8");
 
-    totalLines += lines;
+        const lines = content.split("\n").length;
+        const words = content.split(/\s+/).filter(Boolean).length;
+        const chars = content.length;
+
+        totalLines += lines;
+        totalWords += words;
+        totalChars += chars;
+
+    } catch (err) {
+        console.error(`Error reading file "${path}": ${err.message}`);
     }
-    console.log("line: ", totalLines);
+
+}
+if (showLines) {
+    console.log("lines:", totalLines);
 }
-else if(showWords){
-    for (const path of paths){
-    const content = await fs.readFile(path, "utf-8");
-    const words = content.split(/\s+/).filter(Boolean).length;
 
-    totalWords += words;
-    }
-    console.log("words: ", totalWords);
+if (showWords) {
+    console.log("words:", totalWords);
 }
-else if(showChars){
-    for (const path of paths){
-    const content = await fs.readFile(path, "utf-8");
-    const char = content.length; 
 
-    totalchars += char;
-    }
-    console.log("chars", totalchars)
+if (showChars) {
+    console.log("chars:", totalChars);
 }
-else{
-    for (const path of paths){
-    const content = await fs.readFile(path, "utf-8");
-    const lines = content.split("\n").length;
-    const words = content.split(/\s+/).filter(Boolean).length;
-    const char = content.length; 
-
-    totalLines += lines;
-    totalWords += words;
-    totalchars += char;
-    }
-console.log("lines: ", totalLines, " words", totalWords, " char:", totalchars)
+
+
+// default output when no flags
+if (!showLines && !showWords && !showChars) {
+    console.log(
+    `${pad(totalLines)}\t${pad(totalWords)}\t${pad(totalChars)}\ttotal`
+  );
 }