electron/docs/development/coding-style.md

56 lines
2.6 KiB
Markdown
Raw Normal View History

2015-08-31 05:31:14 +00:00
# Coding Style
These are the style guidelines for coding in Electron.
2013-09-09 07:35:57 +00:00
2016-03-07 21:58:49 +00:00
You can run `npm run lint` to show any style issues detected by `cpplint` and
`eslint`.
2016-03-07 17:46:05 +00:00
2013-08-14 22:43:35 +00:00
## C++ and Python
2015-08-31 05:31:14 +00:00
For C++ and Python, we follow Chromium's [Coding
Style](https://www.chromium.org/developers/coding-style). You can use
[clang-format](clang-format.md) to format the C++ code automatically. There is
also a script `script/cpplint.py` to check whether all files conform.
2013-08-14 22:43:35 +00:00
2015-08-31 05:31:14 +00:00
The Python version we are using now is Python 2.7.
2013-08-14 22:43:35 +00:00
2015-08-31 05:31:14 +00:00
The C++ code uses a lot of Chromium's abstractions and types, so it's
recommended to get acquainted with them. A good place to start is
Chromium's [Important Abstractions and Data Structures](https://www.chromium.org/developers/coding-style/important-abstractions-and-data-structures)
document. The document mentions some special types, scoped types (that
automatically release their memory when going out of scope), logging mechanisms
etc.
2016-03-07 17:43:04 +00:00
## JavaScript
2013-08-14 22:43:35 +00:00
* Write [standard](https://npm.im/standard) JavaScript style.
* File names should be concatenated with `-` instead of `_`, e.g.
2016-03-07 17:43:04 +00:00
`file-name.js` rather than `file_name.js`, because in
[github/atom](https://github.com/github/atom) module names are usually in
2016-03-07 17:43:04 +00:00
the `module-name` form. This rule only applies to `.js` files.
* Use newer ES6/ES2015 syntax where appropriate
* [`const`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/const)
for requires and other constants
* [`let`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let)
for defining variables
* [Arrow functions](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions)
instead of `function () { }`
* [Template literals](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Template_literals)
instead of string concatenation using `+`
2013-08-14 22:43:35 +00:00
2016-04-07 16:40:31 +00:00
## Naming Things
2013-08-14 22:43:35 +00:00
2016-04-07 16:40:31 +00:00
Electron APIs uses the same capitalization scheme as Node.js:
2016-04-07 16:49:55 +00:00
- When the module itself is a class like `BrowserWindow`, use `CamelCase`.
- When the module is a set of APIs, like `globalShortcut`, use `mixedCase`.
- When the API is a property of object, and it is complex enough to be in a
separate chapter like `win.webContents`, use `mixedCase`.
- For other non-module APIs, use natural titles, like `<webview> Tag` or
`Process Object`.
2016-04-07 16:40:31 +00:00
When creating a new API, it is preferred to use getters and setters instead of
2015-08-31 05:31:14 +00:00
jQuery's one-function style. For example, `.getText()` and `.setText(text)`
are preferred to `.text([text])`. There is a
2016-03-31 23:49:59 +00:00
[discussion](https://github.com/electron/electron/issues/46) on this.