ChromaGaming/AliPresents
1
0
Fork 0
Code Issues 13 Pull requests Projects Releases Packages Wiki Activity

Make variable names more meaningful #50

New issue
Open
opened 2017-01-27 21:42:15 +00:00 by alisolarflare · 2 comments
alisolarflare commented 2017-01-27 21:42:15 +00:00 (Migrated from github.com)
Copy link
  • AliLink -> RedstoneLink

Find a new name for:

  • Component -> SubPlugin? Module? Registration? Package? Main? Base? Root? Builder? Activator? Factory? Nothing at all? Package?

  • Architecture -> Abstract?

  • Graveyard -> Deleted? Unused?

  • Creative Boundaries -> Actually pretty descriptive

  • Flaircolour contains username-letter-dyes and flair-changing-portals

  • Magic (My personal favourite)

  • API

  • PlotChangeListener

  • SetForceBoundaries

  • GetLetterDye

  • PlayerProximityLoop

  • SetProximityLocation

  • PortalListener

  • FlairMe

  • FlairColour

  • GPower

  • EnchantingLoop

  • Insurance (in general)

  • Ali Arrow -> ArrowTrail

All things considered, my variable names are unmeaningful as FUCK
Time to refactor

- [x] AliLink -> RedstoneLink Find a new name for: - [x] Component -> SubPlugin? Module? Registration? Package? Main? Base? Root? Builder? Activator? Factory? Nothing at all? Package? - [ ] Architecture -> Abstract? - [x] Graveyard -> Deleted? Unused? - [x] Creative Boundaries -> Actually pretty descriptive - [ ] Flaircolour contains username-letter-dyes and flair-changing-portals - [ ] Magic (My personal favourite) - [ ] API - [ ] PlotChangeListener - [ ] SetForceBoundaries - [ ] GetLetterDye - [ ] PlayerProximityLoop - [ ] SetProximityLocation - [ ] PortalListener - [ ] FlairMe - [x] FlairColour - [x] GPower - [ ] EnchantingLoop - [ ] Insurance (in general) - [ ] Ali Arrow -> ArrowTrail All things considered, my variable names are unmeaningful as *FUCK* Time to refactor
👍 1
alisolarflare commented 2017-01-27 21:43:02 +00:00 (Migrated from github.com)
Copy link

Inspired by:
https://www.youtube.com/watch?v=brfqm9k6qzc

Inspired by: https://www.youtube.com/watch?v=brfqm9k6qzc
alisolarflare commented 2017-01-27 21:58:55 +00:00 (Migrated from github.com)
Copy link
  • ListLinks -> List[verb] and List[noun] confusion
  • UnpressTask
- [ ] ListLinks -> List[verb] and List[noun] confusion - [ ] UnpressTask
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
No results found.
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
help wanted

   
invalid

   
question

   
wontfix
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: ChromaGaming/AliPresents#50
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?