Gulpfile.js смотреть лучшие практики

Question

вот мой текущий список наблюдения для моих gulpfile.js

// Gulp watchlist 
gulp.task('watch', ['browserSync', 'sass'], function(){ 
    gulp.watch('app/scss/**/*.scss', ['sass']); 
    gulp.watch('app/*.html').on('change', browserSync.reload); 
    gulp.watch('app/js/**/*.js').on('change', browserSync.reload); 
    //add more watchers here 
}); 

и это работает.

но учебник я слежу есть что-то немного другое:

gulp.task('watch', ['browserSync', 'sass'], function(){ 
    gulp.watch('app/scss/**/*.scss', ['sass']); 
    // Reloads the browser whenever HTML or JS files change 
    gulp.watch('app/*.html', browserSync.reload); 
    gulp.watch('app/js/**/*.js', browserSync.reload); 
}); 

мне нужен .По ('изменения', browserSync.reload)? оно работает; Мне просто интересно, что то, что я делаю, - это не хорошая практика. Благодаря!

Answer 1

Нет, вам не нужно добавлять пользовательские on('change'). Вы должны передать обработчик события при инициализации наблюдателя, как учебник. В противном случае, это похоже на то, что я говорю gulp, чтобы ничего не делать при изменении по умолчанию, а затем добавить своего слушателя.

Другим преимуществом является то, вы можете передать несколько обработчиков событий на одном дыхании, как массив, если ваши обработчики глотком задачи сами gulp.watch(..., ['task1', 'task2']) вместо цепочки on('change', task1Function).on('change', task2Function)

Answer 2

Глоток watch всегда возвращает EventEmitter, испускающее событие изменения, поэтому вызов gulp.watch('app/*.html').on('change', browserSync.reload) можно записать в виде:

var watcher = gulp.watch('app/*.html'); 
watcher.on('change', browserSync.reload); 

watch функция может быть вызвана следующим Params: gulp.watch(glob [, opts], tasks) или gulp.watch(glob [, opts, cb]).

Таким образом, вы можете использовать оба варианта, но предпочтительнее использовать более короткий метод.