-
Notifications
You must be signed in to change notification settings - Fork 18
Dagim H. #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Dagim H. #16
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| node_modules/ |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| { | ||
| "singleQuote": true, | ||
| "semi": true | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,13 +1,25 @@ | ||
| // This is the entrypoint for your application. | ||
| // node app.js | ||
| import { | ||
| loadBooks, | ||
| printAllBooks, | ||
| printSummary, | ||
| getBooksByGenre, | ||
| getUnreadBooks, | ||
| markAsRead, | ||
| } from './readingList.js'; | ||
|
|
||
| // TODO: Implement the main application logic here | ||
| // 1. Load books on startup | ||
| // 2. Display all books | ||
| // 3. Show summary statistics | ||
| // 4. Add example of filtering by genre or read/unread status | ||
| // 5. Add example of marking a book as read | ||
|
|
||
| console.log('📚 MY READING LIST 📚\n'); | ||
|
|
||
| // Your implementation here | ||
| const books = loadBooks(); | ||
| printAllBooks(); | ||
| printSummary(); | ||
| console.log('\nFiction Books:', getBooksByGenre('Fiction')); | ||
| console.log('\nUnread Books:', getUnreadBooks()); | ||
| markAsRead(1); |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1,37 @@ | ||
| [] | ||
| [ | ||
| { | ||
| "id": 1, | ||
| "title": "1984", | ||
| "author": "George Orwell", | ||
| "genre": "Fiction", | ||
| "read": true | ||
| }, | ||
| { | ||
| "id": 2, | ||
| "title": "Dune", | ||
| "author": "Frank Herbert", | ||
| "genre": "Sci-Fi", | ||
| "read": false | ||
| }, | ||
| { | ||
| "id": 3, | ||
| "title": "The Hobbit", | ||
| "author": "J.R.R. Tolkien", | ||
| "genre": "Fantasy", | ||
| "read": true | ||
| }, | ||
| { | ||
| "id": 4, | ||
| "title": "Sapiens", | ||
| "author": "Yuval Noah Harari", | ||
| "genre": "Non-fiction", | ||
| "read": false | ||
| }, | ||
| { | ||
| "id": 5, | ||
| "title": "Clean Code", | ||
| "author": "Robert C. Martin", | ||
| "genre": "Programming", | ||
| "read": true | ||
| } | ||
| ] |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| { | ||
| "name": "reading-list-manager", | ||
| "version": "1.0.0", | ||
| "description": "", | ||
| "main": "app.js", | ||
| "scripts": { | ||
| "test": "echo \"Error: no test specified\" && exit 1" | ||
| }, | ||
| "keywords": [], | ||
| "author": "", | ||
| "license": "ISC", | ||
| "type": "module", | ||
| "dependencies": { | ||
| "chalk": "^4.1.2" | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,53 +1,123 @@ | ||
| // Place here the file operation functions for loading and saving books | ||
| import fs from 'node:fs'; | ||
| import chalk from 'chalk'; | ||
|
|
||
| function loadBooks() { | ||
| // TODO: Implement this function | ||
| // Read from books.json | ||
| // Handle missing file (create empty array) | ||
| // Handle invalid JSON (notify user, use empty array) | ||
| // Use try-catch for error handling | ||
| } | ||
|
|
||
| try { | ||
| if (!fs.existsSync('books.json')) { | ||
| return []; | ||
| } | ||
| const data = fs.readFileSync('books.json', 'utf8'); | ||
| if (!data.trim()) { | ||
| return []; | ||
| } | ||
| return JSON.parse(data); | ||
| } catch (error) { | ||
| console.log(chalk.red('Error loading books.json - using empty list')); | ||
| return []; | ||
| } | ||
| } | ||
| function saveBooks(books) { | ||
| // TODO: Implement this function | ||
| // Write books array to books.json | ||
| // Use try-catch for error handling | ||
| try { | ||
| fs.writeFileSync('books.json', JSON.stringify(books, null, 2), 'utf8'); | ||
| } catch (error) { | ||
| console.log(chalk.red('Error saving books.json')); | ||
| } | ||
| } | ||
|
|
||
| function addBook(book) { | ||
| // TODO: Implement this function | ||
| const books = loadBooks(); | ||
| const nextId = books.length ? Math.max(...books.map((b) => b.id)) + 1 : 1; | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nice... we can' t assume an array of [1,2,3,4,5]. Maybe is is not sequentially ordered like [4,2,3,5,1]. |
||
| const newBook = { id: nextId, read: false, ...book }; | ||
| books.push(newBook); | ||
| saveBooks(books); | ||
| console.log(chalk.green('Added book:'), chalk.cyan(newBook.title)); | ||
| } | ||
|
|
||
| function getUnreadBooks() { | ||
| // TODO: Implement this function using filter() | ||
| return loadBooks().filter((book) => !book.read); | ||
| } | ||
|
|
||
| function getBooksByGenre(genre) { | ||
| // TODO: Implement this function using filter() | ||
| return loadBooks().filter( | ||
| (book) => book.genre.toLowerCase() === genre.toLowerCase(), | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. good catch to convert to lowercase first but genre can be converted outside the loop because it doesn't change (as opposed to book.genre). Devs like to see this attention to detail |
||
| ); | ||
| } | ||
|
|
||
| function markAsRead(id) { | ||
| // TODO: Implement this function using map() | ||
| const books = loadBooks(); | ||
| const updated = books.map((book) => | ||
| book.id === id ? { ...book, read: true } : book, | ||
| ); | ||
| saveBooks(updated); | ||
| console.log(chalk.green(`Book ${id} marked as read`)); | ||
| } | ||
|
|
||
| function getTotalBooks() { | ||
| // TODO: Implement this function using length | ||
| return loadBooks().length; | ||
| } | ||
|
|
||
| function hasUnreadBooks() { | ||
| // TODO: Implement this function using some() | ||
| return loadBooks().some((book) => !book.read); | ||
| } | ||
|
|
||
| function printAllBooks() { | ||
| // TODO: Implement this function | ||
| // Loop through and display with chalk | ||
| // Use green for read books, yellow for unread | ||
| // Use cyan for titles | ||
| const books = loadBooks(); | ||
| console.log(chalk.bold('\n📚 MY READING LIST 📚\n')); | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. you have this console.log('📚 MY READING LIST 📚\n'); already on line 18 of app.js :) |
||
| console.log('All Books:'); | ||
|
|
||
| books.forEach((book, index) => { | ||
| const status = book.read ? chalk.green('✓ Read') : chalk.yellow('⚠ Unread'); | ||
|
|
||
| console.log( | ||
| `${index + 1}. ${chalk.cyan(book.title)} by ${book.author} (${book.genre}) ${status}`, | ||
| ); | ||
| }); | ||
| } | ||
|
|
||
| function printSummary() { | ||
| // TODO: Implement this function | ||
| // Show statistics with chalk | ||
| // Display total books, read count, unread count | ||
| // Use bold for stats | ||
| } | ||
| const books = loadBooks(); | ||
| const total = books.length; | ||
| const readCount = books.filter((b) => b.read).length; | ||
| const unreadCount = total - readCount; | ||
|
|
||
| console.log(chalk.bold('\n📊 SUMMARY 📊')); | ||
| console.log(chalk.bold('Total Books:'), total); | ||
| console.log(chalk.bold('Read:'), readCount); | ||
| console.log(chalk.bold('Unread:'), unreadCount); | ||
| } | ||
| export { | ||
| loadBooks, | ||
| saveBooks, | ||
| addBook, | ||
| getUnreadBooks, | ||
| getBooksByGenre, | ||
| markAsRead, | ||
| getTotalBooks, | ||
| hasUnreadBooks, | ||
| printAllBooks, | ||
| printSummary, | ||
| }; | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
always nice to display the error message or just part of it like this: error. message or error.detail