Last active
December 1, 2017 01:16
-
-
Save 3ZsForInsomnia/16a4cbb11ad6a75f70b6ae4786796079 to your computer and use it in GitHub Desktop.
Example of method wrapped usage of constants
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
// Import the constants file | |
const VideoPlayerComponent = React.createClass({ | |
getInitialState: function() { | |
return { | |
// Default view - it does not matter how or when it is updated, so that is left out | |
view = views.viewing | |
}; | |
}, | |
render: function() { | |
return // some HTML | |
// We do not use a "switch" here since it is easier to work with expressions without, I find | |
// Also, these checks can be done anywhere in the rendered HTML | |
if (isViewStateViewing(this.view)) { | |
// Render some HTML | |
} | |
// Maybe some more HTML? | |
if (isViewStateRecording(this.view)) { | |
// Render some HTML | |
} else if (isViewStateEditing(this.view)) { | |
// Render different HTML! Exciting. | |
} | |
// Yet more HTML | |
} | |
)}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
// Imagine a video player with a playlist of video clips, where you can record new clips and then add metadata to them | |
// In this example we imagine it has a playlist viewing state, a state for creating a new clip, and a state for editing a clip | |
// Enum of possible viewing states | |
export const views = { | |
viewing: 'viewing', | |
recording: 'recording', | |
editing: 'editing' | |
} | |
// Checks against a given view state | |
// These methods don't care what else is going on, just that the thing to be checked is provided as an argument | |
export function isViewStateViewing(view) { | |
return view === views.viewing; | |
} | |
export function isViewStateRecording(view) { | |
return view === views.recording; | |
} | |
export function isViewStateEditing(view) { | |
return view === views.editing; | |
} | |
// There is no reason that these checks can't be more complicated, e.g. that multiple properties on state combine to give us the view | |
// However, passing objects does introduce a level of fragility as the object must have the necessary properties to check |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment