Opened 3 years ago
Closed 3 years ago
#33063 closed Cleanup/optimization (wontfix)
Use string module instead of writing all of characters
| Reported by: | sammiee5311 | Owned by: | nobody |
|---|---|---|---|
| Component: | Utilities | Version: | 3.2 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
From
Code highlighting:
RANDOM_STRING_CHARS = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789'
To
Code highlighting:
RANDOM_STRING_CHARS = string.ascii_letters + string.digits
It looks more clean with string module than writing all characters.
Note:
See TracTickets
for help on using tickets.
The length of alphabet is important in few places, that's why I would prefer the current form. I don't think it's worth changing.