Почему gulp-uglify выдаёт такой результат?

Question

[Артур Карапетян]

Таск

gulp.task('js', function() {
	return gulp.src([
		'assets/app/libs/jquery/dist/jquery.min.js',
		// Всякие библиотеки
		'assets/app/js/common.js', // Всегда в конце
		])
	.pipe(concat('scripts.min.js'))
	.pipe(uglify())
	.pipe(gulp.dest('assets/app/js'))
	.pipe(browserSync.reload({stream: true}));
});

Я пишу в common.js

$(".form").submit(function(e) {
		e.preventDefault();

		var th = $(this),
				reqInputs = th.find(".required")

		for (var i = 0; i < reqInputs.length; i++) {
			reqInputs[i].addClass("error")
		}

	});

И что я получаю в scripts.min.js

$(".form").submit(function(e){e.preventDefault();for(var t=$(this),n=t.find(".required"),i=0;i<n.length;i++)n[i].addClass("error")})

Что случилось с циклом и названиями переменных?

Comments

[zorca]

Вы бы хоть почитали про библиотеку, которую используете.
lisperator.net/uglifyjs

Answer 1

[Александр Таратин]

А что с ними не так? Все сжалось очень хорошо.

Comments

[Артур Карапетян]

Эм var t=$(this),n=t.find(".required") оказалось в цикле. Это норма?

[Артур Карапетян]

переменная reqInputs заменилась на n

[Александр Таратин]

Да. Экономия на лишних var

[Анатолий Медведев]

Артур, ну да, имена переменных сократил и всего то, вас это пугает?

[Артур Карапетян]

Александр Таратин, написал в common.js
$(".form").submit(function(e) {
e.preventDefault();

var th = $(this);
var reqInputs = th.find(".required");

for (var i = 0; i < reqInputs.length; i++) {
reqInputs[i].addClass("error")
}

});
получил

$(".form").submit(function(e)e.preventDefault();for(vart=$(this).find(".required"),n=0;n<t.length;n++)t[n].addClass("error")})

[Артур Карапетян]

Здесь явно что-то не так. Но что?

[Алексей Уколов]

Эм var t=$(this),n=t.find(".required") оказалось в цикле. Это норма?

Оно не "оказалось в цикле", а точно так же как и раньше инициалицируется перед стартом цикла, только с экономией пары байт при передаче по сети.

Записи

var th = $(this);
var reqInputs = th.find(".required");

for (var i = 0; i < reqInputs.length; i++) {
reqInputs[i].addClass("error")
}

и

for (var n = $(this).find(".required"), i = 0; i < n.length; i++) {
n[i].addClass("error")
}

абсолютно идентичны по сути и делают одно и то же.

Вы зачем uglify запускали, если не знаете, что он делает и зачем нужен?

[Артур Карапетян]

Анатолий Медведев, он засунул объявление переменных в цикл и цикл не работает

[Артур Карапетян]

Хорошо, тогда почему в консоли вылетает ошибка "t[n].addClass is not a function"

[zorca]

Артур, все супер, отлично сжато, устранен лишний код.

[Артур Карапетян]

zorca,

Хорошо, тогда почему в консоли вылетает ошибка "t[n].addClass is not a function"

[Алексей Уколов]

Вообще, весь цикл можно заменить на $(this).find(".required").addClass("error");

[Артур Карапетян]

Алексей Уколов, там будет проверка на заполненность поля, это просто для проверки написал