Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #1 (permalink)  
Старый 18.05.2018, 00:11
Аспирант
Отправить личное сообщение для Булат Азат улы Посмотреть профиль Найти все сообщения от Булат Азат улы
 
Регистрация: 07.09.2016
Сообщений: 75

Проверьте пожалуйста код на правильность.
Всех приветствую. Сделал аудио-плеер, и вроде всё работает. Но хотел бы попросить знающих и опытных людей посмотреть мой код - правильно ли написано. Не ресурсоемкий ли получился. Чтобы было понятно, везде написал комментарии:
// <Аудио>
var audio;
var vol = 0.5;

// Изменение звука (ползунком):
$("#vol").on("input", function(event) {
	vol = event.target.value;
	audio.volume = vol;
});

// Клик на текстовок "изображение" звука (рядом с ползунком) - выключение и включение звука:
$("#volImg").click(function() {
	if($(this).data("status") == "on") {
		audio.muted = true;
		$(this).text("????").data("status", "off");
	}else{
		audio.muted = false;
		$(this).text("????").data("status", "on");
	}
});

// Клик на знак повтора аудио:
$("#repeat").click(function() {
	if($(this).data("status") == "off") {
		audio.loop = true;
		$(this).css("color", "#FFF").data("status", "on");
	}else{
		audio.loop = false;
		$(this).css("color", "#9D9D9D").data("status", "off");
	}
});

// Клик на основную кнопку - проиграть и пауза:
$("#playPause").click(function(){
	if(audio === undefined){ // Если аудио еще не загружено
		$("#audioPlayer").slideDown(500); // По умолчанию блок с проигрывателем скрыт (кнопка "#playPause" тоже, но на него нажимается через другую кнопку)
		audio = new Audio();
		audio.autoplay = false;
		audio.controls = false;
		audio.volume = vol;
		audio.src = "001.mp3";
		$(this).text("\u23f8");
		audio.play();
	}else{ // Если аудио уже было загружено
		if($(this).text() == "\u25b6") {
			$(this).text("\u23f8");
			audio.play();
		} else {
			$(this).text("\u25b6");
			audio.pause();
		}
	}
});

// Функция дляотображения времени (то, что обновляется)
(function curTimeAud(){
	if(audio !== undefined) {
		var curT = audio.currentTime;
		var drtn = audio.duration;
		if (parseInt(curT) - parseInt(curT/60)*60 < 10) var curTSec = "0" + (parseInt(curT) - parseInt(curT/60)*60); // Если секунды меньше 10-и, ставим перед ними 0.
		else var curTSec = parseInt(curT) - parseInt(curT/60)*60;
		if(parseInt(drtn) - parseInt(drtn/60)*60 < 10) var drtnSec = "0" + (parseInt(drtn) - parseInt(drtn/60)*60); // Если секунды меньше 10-и, ставим перед ними 0.
		else var drtnSec = parseInt(drtn) - parseInt(drtn/60)*60;
		$("#curTime").text(parseInt(curT / 60) + ":" + curTSec); // Отображение текущего положения, в формате "м:сс" ("мм:сс", "ммм:сс")
		$("#maxTime").text(parseInt(drtn / 60) + ":" + drtnSec); // Отображение продолжительности аудио, в формате "м:сс" ("мм:сс", "ммм:сс")
		$("#audioTime").attr("max", drtn); // Атрибут "max" для ползунка времени аудио (type="range")
		$("#audioTime").val(curT); // Текущее положение ползунка (type="range")
		
		if(audio.ended && audio.autoplay == false){ //Что делать, когда аудио проигрывается до конца, а повтор не включен
			audio.load();
			$("#playPause").text("\u25b6");
		}
	}
	setTimeout(curTimeAud, 10); // Обновление функции каждые 10 мс
}).call(this); //Вызов функции

$("#audioTime").on("input", function(event){audio.currentTime = event.target.value}); // Перемещение ползунка времени
$("#stepFwd").click(function(){audio.currentTime = audio.currentTime + 10}); // Действие при клике кнопки "Вперед на 10 сек"
$("#stepbckw").click(function(){audio.currentTime = audio.currentTime - 10}); // Действие при клике кнопки "Назад на 10 сек"
// </Аудио>
Ответить с цитированием
  #2 (permalink)  
Старый 18.05.2018, 00:33
Аватар для j0hnik
Профессор
Отправить личное сообщение для j0hnik Посмотреть профиль Найти все сообщения от j0hnik
 
Регистрация: 01.12.2016
Сообщений: 2,967

строки 15 и 18 text("????") - зачем?
Ответить с цитированием
  #3 (permalink)  
Старый 18.05.2018, 00:58
Аспирант
Отправить личное сообщение для Булат Азат улы Посмотреть профиль Найти все сообщения от Булат Азат улы
 
Регистрация: 07.09.2016
Сообщений: 75

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



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Пожалуйста, кому не лень, посмотрите код, что он примерно делает?? arsen97 Общие вопросы Javascript 1 15.08.2014 15:51
Расшифруйте пожалуйста код! Mrs.Haneki Элементы интерфейса 2 17.04.2014 09:25
Не пойму почему не работает код? Mukhtar Events/DOM/Window 4 28.04.2013 23:01
код нe работает в фаирфохе dadli Javascript под браузер 2 28.01.2012 19:18
помогите пожалуйста улучшить код. Duda.Ml1986@gmail.com Серверные языки и технологии 4 07.01.2012 18:53