docs: add advanced/loader.md #198
Merged
Conversation
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
Current coverage is 97.49% (diff: 100%)@@ master #198 diff @@
==========================================
Files 34 35 +1
Lines 863 959 +96
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
+ Hits 843 935 +92
- Misses 20 24 +4
Partials 0 0
|
popomore
force-pushed
the
doc-loader
branch
3 times, most recently
from
9c4bcfc
to
e574250
Compare
popomore
requested review from
fengmk2,
shaoshuai0102,
leoner,
dead-horse and
gxcsoccer
|
可以 review 了 |
fengmk2
reviewed
Jan 9, 2017
|
|
||
| ## 应用、框架和插件 | ||
|
|
||
| egg 是一个底层框架，应用可以直接使用，但 egg 本身的插件比较少，应用需要自己配置插件增加各种特性，比如 mysql。 |
fengmk2
reviewed
Jan 9, 2017
| } | ||
| } | ||
|
|
||
| // plugin.js |
fengmk2
reviewed
Jan 9, 2017
| +-----------------------------------+--------+ | ||
| ``` | ||
|
|
||
| ## loadUnit |
fengmk2
reviewed
Jan 9, 2017
|
|
||
| ``` | ||
| app | ||
| | ├─- plugin2 (依赖 plugin3) |
fengmk2
reviewed
Jan 9, 2017
|
|
||
| plugin1 为 framework1 依赖的插件，配置合并后 object key 的顺序会优先级于 plugin2/plugin3。因为 plugin2 和 plugin3 的依赖关系，所以交换了位置。framework1 继承了 egg，顺序会晚于 egg。应用最后加载。 | ||
|
|
||
| 请查看 [Loader.getLoadUnits 方法](https://github.com/eggjs/egg-core/blob/65ea778a4f2156a9cebd3951dac12c4f9455e636/lib/loader/egg_loader.js#L233) |
There was a problem hiding this comment.
请查看 Loader.getLoadUnits 方法
|
加上 init.js 说明就好了，没看出什么问题。 |
dead-horse
reviewed
Jan 9, 2017
| }); | ||
| ``` | ||
|
|
||
| 加载后 `app.serviceClasses.user` 就是 UserService，当调用 `ctx.service.user` 时会实例化 UserService。所以这个方法加载的文件是会在每次请求实例化的。 |
There was a problem hiding this comment.
所以这个类只有每次请求中首次访问到时才会实例化，实例化后会被缓存，同一个请求多次调用也只会实例化一次。
dead-horse
reviewed
Jan 9, 2017
|
|
||
| #### override [Boolean] | ||
|
|
||
| 是否覆盖已经存在的文件，如果出现同名文件会报错，默认为 false。 |
There was a problem hiding this comment.
遇到已经存在的文件时是直接覆盖还是抛出异常，默认为 false
这样好点？
dead-horse
approved these changes
Jan 9, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist
npm testpassesAffected core subsystem(s)
doc
Description of change
loader