mirror/rendercv
1
0
Fork 0
mirror of https://github.com/rendercv/rendercv.git synced 2026-04-19 22:42:11 -04:00
Code Issues Packages Projects Releases Wiki Activity

Speed up function replace_placeholders by 10% (#338)

Browse Source 
**Summary of Optimizations:**

1. **Redundant Calls**: Captured `get_date_input()` in the variable `date_input` to reduce redundant calls.
2. **Reuse Computations**: Calculated `name_snake_case` and `name_kebab_case` once and reused them instead of recalculating multiple times.
3. **Better Loop**: Used a tuple of tuples for `placeholders`, iterating through once and applying replacements in sequence.

These changes improve the efficiency of the `replace_placeholders` function by reducing redundant calculations and I/O operations, resulting in better runtime performance.

Co-authored-by: codeflash-ai[bot] <148906541+codeflash-ai[bot]@users.noreply.github.com>
This commit is contained in:
Saurabh Misra
2025-02-08 14:29:49 -08:00
committed by Sina Atalay
parent 361df43c57
commit 39297f9957
1 changed files with 22 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
rendercv/data/models/computers.py
View File

@@ -105,26 +105,30 @@ def replace_placeholders(value: str) -> str:
full_month_names = locale["full_names_of_months"]
short_month_names = locale["abbreviations_for_months"]
month = get_date_input().month
year = str(get_date_input().year)
date_input = get_date_input()
month = date_input.month
year = str(date_input.year)
placeholders = {
"NAME_IN_SNAKE_CASE": name.replace(" ", "_"),
"NAME_IN_LOWER_SNAKE_CASE": name.replace(" ", "_").lower(),
"NAME_IN_UPPER_SNAKE_CASE": name.replace(" ", "_").upper(),
"NAME_IN_KEBAB_CASE": name.replace(" ", "-"),
"NAME_IN_LOWER_KEBAB_CASE": name.replace(" ", "-").lower(),
"NAME_IN_UPPER_KEBAB_CASE": name.replace(" ", "-").upper(),
"FULL_MONTH_NAME": full_month_names[month - 1],
"MONTH_ABBREVIATION": short_month_names[month - 1],
"MONTH_IN_TWO_DIGITS": f"{month:02d}",
"YEAR_IN_TWO_DIGITS": str(year[-2:]),
"NAME": name,
"YEAR": str(year),
"MONTH": str(month),
}
name_snake_case = name.replace(" ", "_")
name_kebab_case = name.replace(" ", "-")
for placeholder, placeholder_value in placeholders.items():
placeholders = (
("NAME_IN_SNAKE_CASE", name_snake_case),
("NAME_IN_LOWER_SNAKE_CASE", name_snake_case.lower()),
("NAME_IN_UPPER_SNAKE_CASE", name_snake_case.upper()),
("NAME_IN_KEBAB_CASE", name_kebab_case),
("NAME_IN_LOWER_KEBAB_CASE", name_kebab_case.lower()),
("NAME_IN_UPPER_KEBAB_CASE", name_kebab_case.upper()),
("FULL_MONTH_NAME", full_month_names[month - 1]),
("MONTH_ABBREVIATION", short_month_names[month - 1]),
("MONTH_IN_TWO_DIGITS", f"{month:02d}"),
("YEAR_IN_TWO_DIGITS", year[-2:]),
("NAME", name),
("YEAR", year),
("MONTH", str(month)),
)
for placeholder, placeholder_value in placeholders:
value = value.replace(placeholder, placeholder_value)
return value