France-ioi / AlgoreaBackend

Backend for the new Algorea platform
MIT License
1 stars 2 forks source link

1. Rename 'ok' meaning 'watched_group_id is set' into 'watchedGroupIDSet', then 2. Rename 'watchedGroupIDSet' into 'watchedGroupIDIsSet' #1097

Closed zenovich closed 4 weeks ago

zenovich commented 1 month ago

Since constructs like 'if ok {', where 'ok' actually means 'watched_group_id is set' especially when they are located far from the definition of 'ok' variable are not very meaningful, it's been decided to get rid of 'ok' meaning 'watched_group_id is set'.

In this change, 'ok' meaning 'watched_group_id is set' gets renamed into 'watchedGroupIDSet' first. Then, 'watchedGroupIDSet' gets renamed into 'watchedGroupIDIsSet' to improve readability even beyond.