Added the optimized solution for promblem 44#1324
Added the optimized solution for promblem 44#1324utkarsh-shrivastav77 wants to merge 1 commit intoTheAlgorithms:masterfrom
Conversation
|
Sir please check the code let me know if there is any error |
There was a problem hiding this comment.
CI is failing.
Please adapt the tests; they can't possibly work for this, since you renamed the function.
Second, the comments are very redundant with the code (and also read as if they had been written by a certain large language model). pentagonalIndices isn't a set of indices, it's a set of numbers.
I also don't see how this is supposed to be "O(n)" considering it has two nested for loops which I would assume to be O(n²) given that both for loops look like they are roughly O(k). What even is n?
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Optimised the solution for problem 44 in o(n) time complexity.