Skip to content

Simplify Capitalize Function #12879

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.

Sign up for GitHub

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

Merged
merged 5 commits into from
Aug 24, 2025
Merged

Simplify Capitalize Function #12879

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3e43345
Simplify the capitalize function using ASCII arithmetic to make the a…
Miladkhoshdel Aug 4, 2025
743f95a
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Aug 4, 2025
d219a3c
Update capitalize.py
MaximSmolskiy Aug 24, 2025
c22d862
Update capitalize.py
MaximSmolskiy Aug 24, 2025
50d99d1
Merge branch 'master' into perf/capitalize-ascii-optimization
MaximSmolskiy Aug 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 1 addition & 6 deletions strings/capitalize.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
from string import ascii_lowercase, ascii_uppercase


def capitalize(sentence: str) -> str:
"""
Capitalizes the first letter of a sentence or word.
Expand All @@ -19,11 +16,9 @@ def capitalize(sentence: str) -> str:
if not sentence:
return ""

# Create a dictionary that maps lowercase letters to uppercase letters
# Capitalize the first character if it's a lowercase letter
# Concatenate the capitalized character with the rest of the string
lower_to_upper = dict(zip(ascii_lowercase, ascii_uppercase))
return lower_to_upper.get(sentence[0], sentence[0]) + sentence[1:]
return sentence[0].upper() + sentence[1:]


if __name__ == "__main__":
Expand Down