Цикл для создания переменных, цикл функции / массива

Question

Я запрограммировал программу для выделения выбранной кнопки при нажатии и отмены выделения при нажатии другой кнопки. Это работает, но мне нужно сделать его менее повторяющимся (возможно, используя циклы).

var txt = $(".txt");
var xls = $(".xls");
var jpg = $(".jpg");
var png = $(".png");
var pdf = $(".pdf");
var mp3 = $(".mp3");
var mp4 = $(".mp4");
var gif = $(".gif");
var flac = $(".flac");
var html = $(".html");
var js = $(".js");
var aac = $(".aac");
var ai = $(".ai");
var avi = $(".avi");
var psd = $(".psd");
var tiff = $(".tiff");

var fileTypes = [txt, xls, jpg, png, pdf, mp3, mp4, gif, flac, html, js, aac, ai, avi, psd, tiff];

$(fileTypes[0]).click( function() {
    for(i=0; i<fileTypes.length; i++) 
    {
        fileTypes[i].removeClass("selected");
        fileTypes[i].addClass("not-selected");
    }

    fileTypes[0].removeClass("not-selected");
    fileTypes[0].addClass("selected");
});

$(fileTypes[1]).click( function() {
    for(i=0; i<fileTypes.length; i++) 
    {
        fileTypes[i].removeClass("selected");
        fileTypes[i].addClass("not-selected");
    }

    fileTypes[1].removeClass("not-selected");
    fileTypes[1].addClass("selected");

});

$(fileTypes[2]).click( function() {
    for(i=0; i<fileTypes.length; i++) 
    {
        fileTypes[i].removeClass("selected");
        fileTypes[i].addClass("not-selected");
    }

    fileTypes[2].removeClass("not-selected");
    fileTypes[2].addClass("selected");

});  // It does this 16 times, the length of the fileTypes array

Я пытался использовать цикл for для решения этой проблемы, но мне это не удалось. Пожалуйста помоги. Спасибо.

Answer 1

Как то так?

var toggelf = function(event) {
    for(i=0; i<fileTypes.length; i++) 
    {
        fileTypes[i].switchClass("selected","not-selected");
    }
    $(event.currentTarget).switchClass("not-selected","selected");
}

for(i=0; i<fileTypes.length; i++) 
{
    $(fileTypes[i]).click(toggelf);
}

Я не проверял, но это должно работать. По сути, я создаю анонимную функцию, присваиваю ее переменной и использую в качестве обработчика для кликов по всем вашим объектам.

Answer 2

Я бы предпочел не определять каждый тип в javascript, а использовать сам dom для определения типов (как вам угодно). Это может быть достигнуто путем упаковки ваших объектов в контейнер. Например:

const buttons = '.buttons button'

$(buttons).click(function() {
  $(buttons).removeClass('selected')
  $(this).addClass('selected')
})

.selected {
  background:red;
}

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.0.0/jquery.min.js"></script>
<div class="buttons">
  <button>txt</button>
  <button>xls</button>
  <button>jpg</button>
  <button>png</button>
  <button>pdf</button>
  <button>mp3</button>
  <button>mp4</button>
  <button>gif</button>
  <button>flac</button>
  <button>html</button>
  <button>js</button>
  <button>aac</button>
  <button>ai</button>
  <button>avi</button>
  <button>psd</button>
  <button>tiff</button>
</div>

Я также не выбрал бы и выбранные, и не выбранные классы, так как один является избыточным.

---

Просто обратите внимание, что я считаю, что вам не следует дублировать набор типов файлов в dom и javascript; просто выберите одно место. В конечном счете, я бы предпочел сделать это полностью на основе данных из javascript, используя что-то вроде реакции, но с jquery имеет смысл описать его в dom.

Answer 3

Возможно, вы могли бы упростить ваш скрипт, выполнив что-то вроде следующего:

var extensions = ['.txt',
'.xls',
'.jpg',
'.png',
'.pdf',
'.mp3',
'.mp4',
'.gif',
'.flac',
'.html',
'.js',
'.aac',
'.ai',
'.avi',
'.psd',
'.tiff']

// Create a selector that selects all classes in
// the extensions array
var selector = extensions.join(',');

// Assign a click handler to each element in the
// selected string
$(selector).click(function() {

  // When clicked, remove the selected 
  // class, and add not-selected to all 
  // elements in the selector
  $(selector)
  .removeClass('selected')
  .addClass('not-selected')

  // For this item, remove not-selected
  // class and add selected class
  $(this)
  .removeClass('not-selected')
  .addClass('selected')

})

.selected {
background:red;
}

.not-selected {
background:white;
}

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.0.0/jquery.min.js"></script>
<button class="txt">txt</button>
<button class="xls">xls</button>
<button class="jpg">jpg</button>
<button class="png">png</button>
<button class="pdf">pdf</button>
<button class="mp3">mp3</button>
<button class="mp4">mp4</button>
<button class="gif">gif</button>
<button class="flac">flac</button>
<button class="html">html</button>
<button class="js">js</button>
<button class="aac">aac</button>
<button class="ai">ai</button>
<button class="avi">avi</button>
<button class="psd">psd</button>
<button class="tiff">tiff</button>