Skip to content

comment and modification from Jerry #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
fafrifa wants to merge 1 commit into
base: master
Choose a base branch
from
Open

comment and modification from Jerry #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions nodejs/站点建设/imooc/app.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,10 @@ app.set('views', './views/pages');
//使用的模板引擎
app.set('view engine', 'jade');


app.use(bodyParser.urlencoded({extended: false}));
// extended A new body object containing the parsed data is populated on the request object after the middleware (i.e. req.body). This object will contain key-value pairs, where the value can be a string or array (when extended is false), or any type (when extended is true).
// 设置为false时只能解析 value 为 string 和 数组，为true的时候可以解析value 是任意的。
// 我测试了下，用true 才能得到结果^^
app.use(bodyParser.urlencoded({extended: true}));
app.use(bodyParser.json());

//静态资源的路径
Expand Down Expand Up @@ -102,6 +104,9 @@ app.post('/admin/movie/new', function (res, req) {
var id = res.body.movie._id;
var movieObj = req.body.movie;
var _movie;
// 我卡在这好久，原来视频写的也是 !=='underfined'但是这表明id被建立但是没有赋值的情况。
// 最后改成 id!=="" 才顺利post成功。
// 其实if(id)就可以了，只要值为trusy就好，""本身就是falsy所以会建立 Movie
if (id !== 'undefined') {
Movie.findById(id, function (err, movie) {
if (err) {
Expand Down
3 changes: 2 additions & 1 deletion nodejs/站点建设/imooc/schemas/movie.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,8 @@ MovieSchema.pre('save', function () {
} else {
this.meta.updateAt = Date.now()
}

// 经常容易忘了next() 然后就悲剧了
// 这种middle ware 一定要加next ^_^
next();
});

Expand Down
5 changes: 4 additions & 1 deletion nodejs/站点建设/imooc/views/pages/admin.jade
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@ block content
.container
.row
form.form-horizontal(method="post",action="/admin/movie/new")
input(type="hidden",name="movie[_id]",value="#{movie{_id}")
//- typo -> #{movie{_id}}
//- input#title.form-control(type="text", name="movie[title]", value=movie.title)
//- 不太确定 name="movie{...}" 能不能把post的数据装到movie对象中~ 请帮我解答下 ，多谢
input(type="hidden",name="movie[_id]",value="#{movie_id}")
.form-group
label.col-sm-2.control-label(for="title") 电影名字
.col-sm-10
Expand Down