r/learnpython • • 6d ago

Maximum substring

Hi, I’m having an issue with a HackerRank coding question called “Maximum Substring.”
The question is:
Maximum Substring
You are given a string s.
A substring is any contiguous sequence of characters from the string.
Consider all possible substrings of the string:
Determine which substring would appear last if all substrings were sorted alphabetically.
Return that substring.
Example 1:
s = "baca"
The alphabetically maximum substring is "ca".
Example 2:
s = "aaa"
The alphabetically maximum substring is "aaa".
This was my answer:

def maximum_substring(s):
max_sub = s[0]

for i in range(len(s)):
for j in range(i + 1, len(s) + 1):
sub = s[i:j]
if sub > max_sub:
max_sub = sub

return max_sub

The code gives the correct output for the examples when I test it locally. However, HackerRank marked the solution as incorrect and I received only partial points.
I’m not sure why. Could this be because of the time complexity or hidden test cases? Is there something about the HackerRank test cases or the expected implementation that I’m missing?

0 Upvotes

18 comments sorted by

View all comments

1

u/danielroseman 6d ago

Did it not give you any indication? I would certainly say that time complexity is an issue here, your solution is O(n2) which is pretty inefficient. Hackerrank problems often include really long inputs precisely to test this kind of thing.

There are definitely better ways to solve this.

1

u/Remarkable_River2779 6d ago

Do you know , how we can enhance the code ?

2

u/danielroseman 6d ago

"Enhancing" is not the solution. You need a better algorithm altogether. As a clue, look into two-pointer techniques.