Add beautiful website interface and Z-Algorithm implementation by Adithyakp86 · Pull Request #356 · shivamm-verma/Daily-DSA

Adithyakp86 · 2025-09-08T02:11:19Z

Add beautiful website interface and Z-Algorithm implementation

Created modern website with search, filtering, and modal system
Added complete Z-Algorithm problem with Python, Java, C++ solutions
Fixed navigation issues and 404 errors
Added comprehensive documentation and test cases
Enhanced user experience with responsive design
Fixed import error in Best Time to Buy and Sell Stock problem"
===>
fixes
[New Problem] Sort a Linked List #354
[New Problem] Z-Algorithm for Pattern Matching #355

Daily-DSA/
├── index.html (NEW)
├── style.css (NEW)
├── script.js (NEW)
└── Strings/
└── Z-Algorithm/ (NEW FOLDER)
├── Python_code.py (NEW)
├── Java_code.java (NEW)
├── C++_code.cpp (NEW)
└── Markdown.md (NEW)

Issue number if any:-

Fixes #<issue_number>

✅ Contribution Checklist (make sure all are checked)

📝 Type of Addition (check any one)

DSA problem addition
Fix mistakes in existing codebase
Add better solutions
Make docs better

📊 Last check (make sure all are checked

Have you checked that this problem isn't already in the codebase?🔍
Did you cross-check that your code/explanation is right?

🏢 Your Organization?

GsSOC'25

github-actions · 2025-09-08T02:11:27Z

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

shivamm-verma · 2025-09-08T19:54:37Z

Sorry @Adithyakp86
But you can't raise a PR, until you create an issue, and it get assigned to you!

Copilot

Pull Request Overview

This PR adds a modern website interface for the Daily-DSA repository and implements a complete Z-Algorithm problem solution. The changes create a beautiful, responsive web interface with search and filtering capabilities while adding comprehensive Z-Algorithm implementations in Python, Java, and C++.

Added complete website interface with modern UI/UX design
Implemented Z-Algorithm pattern matching with full documentation
Fixed import error in Best Time to Buy and Sell Stock problem

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
index.html	Main website interface with search, filtering, and modal system
style.css	Comprehensive CSS with responsive design and modern styling
script.js	JavaScript functionality for search, filtering, and navigation
Strings/Z-Algorithm/Python_code.py	Complete Python implementation with test cases
Strings/Z-Algorithm/Java_code.java	Java implementation with multiple variants
Strings/Z-Algorithm/C++_code.cpp	C++ implementation with performance optimizations
Strings/Z-Algorithm/Markdown.md	Comprehensive documentation and algorithm explanation
Arrays/Best_Time_to_Buy_and_Sell_Stock/Python_code.py	Fixed missing import statement

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-09-08T19:54:59Z

+    // Map display names to actual folder names
+    const folderMap = {
+        'Arrays': 'Arrays',
+        'Binary Search': 'Binary Search',
+        'Binary Trees': 'Binary Trees',
+        'Dynamic Programming': 'Dynamic Programming',
+        'Graphs': 'Graphs',
+        'Strings': 'Strings',
+        'Linked Lists': 'Linked Lists',
+        'Stack and Queue': 'Stack and Queue',
+        'Sorting': 'Sorting',
+        'Hashing': 'Hashing',
+        'Two Pointers': 'Two Pointers',
+        'Sliding Window': 'Sliding Window',
+        'Greedy': 'Greedy',
+        'Recursion and Backtracking': 'Recursion and Backtracking',
+        'Matrix': 'Matrix',
+        'Math': 'Math',
+        'Bit Manipulation': 'Bit Manipulation',
+        'Heap': 'Heap',
+        'Tries': 'Tries'
+    };
+
+    const folderName = folderMap[topicName] || topicName.replace(/\s+/g, '_');


[nitpick] The folderMap object contains redundant mappings where the key and value are identical. This adds unnecessary code complexity. Consider removing entries where the display name matches the folder name exactly, or use a Set for topics that don't need mapping and only map the exceptions.

Suggested change

// Map display names to actual folder names

const folderMap = {

'Arrays': 'Arrays',

'Binary Search': 'Binary Search',

'Binary Trees': 'Binary Trees',

'Dynamic Programming': 'Dynamic Programming',

'Graphs': 'Graphs',

'Strings': 'Strings',

'Linked Lists': 'Linked Lists',

'Stack and Queue': 'Stack and Queue',

'Sorting': 'Sorting',

'Hashing': 'Hashing',

'Two Pointers': 'Two Pointers',

'Sliding Window': 'Sliding Window',

'Greedy': 'Greedy',

'Recursion and Backtracking': 'Recursion and Backtracking',

'Matrix': 'Matrix',

'Math': 'Math',

'Bit Manipulation': 'Bit Manipulation',

'Heap': 'Heap',

'Tries': 'Tries'

};

const folderName = folderMap[topicName] || topicName.replace(/\s+/g, '_');

// Use topicName directly as folder name, replacing spaces with underscores

const folderName = topicName.replace(/\s+/g, '_');

Copilot · 2025-09-08T19:55:00Z

+        n = len(s)
+        z = [0] * n
+
+        # Z[0] is always 0 as it's the entire string


The comment on line 19 is incorrect. Z[0] is set to n (the length of the entire string), not 0. The comment should read '# Z[0] is always n as it represents the entire string length' or similar to accurately reflect the implementation.

Suggested change

# Z[0] is always 0 as it's the entire string

# Z[0] is always n as it represents the entire string length

Copilot · 2025-09-08T19:55:00Z

+        int n = s.length();
+        int[] z = new int[n];
+
+        // Z[0] is always 0 as it's the entire string


The comment on line 18 is incorrect. Z[0] is set to n (the length of the entire string), not 0. The comment should read '// Z[0] is always n as it represents the entire string length' to match the actual implementation.

Suggested change

// Z[0] is always 0 as it's the entire string

// Z[0] is always n as it represents the entire string length

Copilot · 2025-09-08T19:55:01Z

+        int n = s.length();
+        vector<int> z(n, 0);
+
+        // Z[0] is always 0 as it's the entire string


The comment on line 23 is incorrect. Z[0] is set to n (the length of the entire string), not 0. The comment should read '// Z[0] is always n as it represents the entire string length' to accurately describe the implementation.

Suggested change

// Z[0] is always 0 as it's the entire string

// Z[0] is always n as it represents the entire string length

shivamm-verma · 2025-09-08T19:55:09Z

Please create an issue first!

shivamm-verma · 2025-09-08T19:56:22Z

Or just tell me about your intuition for this!

Adithyakp86 · 2025-09-09T12:29:46Z

i changed some files add z algorithm and simple website for this

shivamm-verma · 2025-09-09T18:30:22Z

You can't add everything in just a one PR @Adithyakp86
For this you would have to create 3 multiple PRs(and so 3 Issues)

shivamm-verma · 2025-09-09T18:34:56Z

As from this, I would suggest you to just raise an Issue for "Z Algo" in strings as

shivamm-verma · 2025-09-09T18:40:41Z

plus your website creation with script.js also does make sense here, You can create an issue for that too @Adithyakp86 !

Question: script.js for this will be automatically updated with the new/latest codebase, right? or is it pre fixed–no updates/sync.

Add beautiful website interface and Z-Algorithm implementation

1b6f2a2

shivamm-verma requested a review from Copilot September 8, 2025 19:54

Copilot AI reviewed Sep 8, 2025

View reviewed changes

shivamm-verma closed this Sep 8, 2025

shivamm-verma reopened this Sep 8, 2025

shivamm-verma added the invalid This doesn't seem right label Sep 8, 2025

shivamm-verma added enhancement New feature or request questioned Further information is requested labels Sep 9, 2025

Adithyakp86 added 2 commits September 11, 2025 21:29

Reverse string and extra some feauture

2b04584

Reverse string extra feauture

5a28b4e

shivamm-verma added Pause⌛ Paused working for this Issue. and removed questioned Further information is requested labels Oct 4, 2025

Navigation Menu

Add beautiful website interface and Z-Algorithm implementation - #356

Add beautiful website interface and Z-Algorithm implementation#356
Adithyakp86 wants to merge 3 commits into
shivamm-verma:mainfrom
Adithyakp86:main

Adithyakp86 commented Sep 8, 2025 •

edited

Loading

Uh oh!

github-actions Bot commented Sep 8, 2025

Uh oh!

shivamm-verma commented Sep 8, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Sep 8, 2025

Uh oh!

Copilot AI Sep 8, 2025

Uh oh!

Copilot AI Sep 8, 2025

Uh oh!

Copilot AI Sep 8, 2025

Uh oh!

shivamm-verma commented Sep 8, 2025

Uh oh!

shivamm-verma commented Sep 8, 2025

Uh oh!

Adithyakp86 commented Sep 9, 2025

Uh oh!

shivamm-verma commented Sep 9, 2025

Uh oh!

shivamm-verma commented Sep 9, 2025

Uh oh!

shivamm-verma commented Sep 9, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	# Z[0] is always 0 as it's the entire string
	# Z[0] is always n as it represents the entire string length

	// Z[0] is always 0 as it's the entire string
	// Z[0] is always n as it represents the entire string length

Conversation

Adithyakp86 commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue number if any:-

✅ Contribution Checklist (make sure all are checked)

📝 Type of Addition (check any one)

📊 Last check (make sure all are checked

🏢 Your Organization?

Uh oh!

github-actions Bot commented Sep 8, 2025

Thanks for creating a PR for your Issue! ☺️

Uh oh!

shivamm-verma commented Sep 8, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Sep 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 8, 2025

Choose a reason for hiding this comment

Uh oh!

shivamm-verma commented Sep 8, 2025

Uh oh!

shivamm-verma commented Sep 8, 2025

Uh oh!

Adithyakp86 commented Sep 9, 2025

Uh oh!

shivamm-verma commented Sep 9, 2025

Uh oh!

shivamm-verma commented Sep 9, 2025

Uh oh!

shivamm-verma commented Sep 9, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Adithyakp86 commented Sep 8, 2025 •

edited

Loading