holm/naming-cheatsheet
1
0
Fork 0
forked from extern/naming-cheatsheet
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #17 from keis/patch-1

Browse Source 
Drop plural from count variable
This commit is contained in:
Artem Zakharchenko 2021-01-17 17:34:24 +01:00 committed by GitHub
commit f55da865cc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
README.md
View File

@ -51,11 +51,11 @@ const pages_count = 5
const shouldUpdate = true
/* Good */
const pagesCount = 5
const pageCount = 5
const shouldUpdate = true
/* Good as well */
const pages_count = 5
const page_count = 5
const should_update = true
```
@ -70,13 +70,13 @@ A name must be _short_, _intuitive_ and _descriptive_:
```js
/* Bad */
const a = 5 // "a" could mean anything
const isPaginatable = postsCount > 10 // "Paginatable" sounds extremely unnatural
const shouldPaginatize = postsCount > 10 // Made up verbs are so much fun!
const isPaginatable = a > 10 // "Paginatable" sounds extremely unnatural
const shouldPaginatize = a > 10 // Made up verbs are so much fun!
/* Good */
const postsCount = 5
const hasPagination = postsCount > 10
const shouldDisplayPagination = postsCount > 10 // alternatively
const postCount = 5
const hasPagination = postCount > 10
const shouldDisplayPagination = postCount > 10 // alternatively
```
## Avoid contractions
@ -111,11 +111,11 @@ A name should reflect the expected result.
```jsx
/* Bad */
const isEnabled = itemsCount > 3
const isEnabled = itemCount > 3
return <Button disabled={!isEnabled} />
/* Good */
const isDisabled = itemsCount <= 3
const isDisabled = itemCount <= 3
return <Button disabled={isDisabled} />
```
@ -154,7 +154,7 @@ The verb part of your function name. The most important part responsible for des
Accesses data immediately (i.e. shorthand getter of internal data).
```js
function getFruitsCount() {
function getFruitCount() {
return this.fruits.length
}
```