Question

我不想每次将新图像添加到HTML时都向JS代码添加新行。但是我不知道该怎么办。

$(document).ready(function() {
$('.btn').click(function() {
var bid = $(this).attr('id');
if(bid=="img1" || bid == "img2" || bid == "img3"){
    document.getElementById("img1").style.display = "none";
    document.getElementById("img2").style.display = "none";
    document.getElementById("img3").style.display = "none";
        } else {
            document.getElementById("img4").style.display = "none";
            document.getElementById("img5").style.display = "none";
        }
        document.getElementById(bid).style.display = "block";
    });
});

CSS

.textures {
    background: url("images/room.png") center / cover no-repeat;
    position: static;
    height: 600px;
    width: 800px;
}
div>img {
    display: none;
    position: absolute;
}
[type=radio] {
    position: absolute;
    opacity: 0;
    width: 0;
    height: 0;
}
[type=radio] + img {
    cursor: pointer;
    height: 60px;
    width: 80px
}
[type=radio]:checked + img {
    outline: 2px solid #000;
}

HTML。在这里，您可以查看是否单击了类别中的半径输入，它将显示来自材质的图像之一。

<!--Room materials-->
<div class="textures">
    <div class="wall">
        <img id="img1" src="images/wall/blue.png" /> 
        <img id="img2" src="images/wall/brick.png" /> 
    </div>

    <div class="floor">
        <img id="img3" src="images/floor/cedro.png" /> 
        <img id="img4" src="images/floor/cipres.png" />
    </div>
</div>

<!--Categories-->
<h5>Walls</h5>
<label>
    <input type="radio" class="btn" id="img1" name="test" value="small" />
    <img src="images/thumbnail/wall/blue.png"/>
</label>
<label>
    <input type="radio" class="btn" id="img2" name="test" value="small" />
    <img src="images/thumbnail/wall/brick.png"/> 
</label>

<h5>Floors</h5>
<label>
    <input type="radio" class="btn" id="img3" name="test" value="small" />
    <img src="images/thumbnail/floor/cedro.png"/> 
</label>
<label>
    <input type="radio" class="btn" id="img4" name="test" value="small" />
    <img src="images/thumbnail/floor/cipres.png"/> 
</label>

示例here。

this确实是我想要实现的。

Answer 1

首先，在一个文档中具有重复的ID是无效的HTML 。要将数据表示与另一个元素的连接的HTML数据，可以改用data属性。提取数据属性，隐藏所有图像，然后在选择器中使用数据属性：

<input type="radio" class="btn" data-image="img4" name="test" value="small" />
                                ^^^^^^^^^^^^^^^^^

$('.btn').on('change', function() {
  const id = $(this).data('image');
  $('.textures img').hide();
  $('#' + id).show();
});

如何使此功能更清洁？

1 个答案: