Как оптимизировать код?

Question

[Марк Цукенберг]

Помогите оптимизировать код!

let text = "“1orem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. ”";
let text1 = "“2orem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. ”";
let text2 = "“3orem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. ”";
let info = 'Pasha from';
let info1 = 'Linda from';
let info2 = 'Timur from';
let webSite = 'example.com'; 


$('.photo').click(function(){
    $('.photo').addClass('active-photo');
    $('.photo1').removeClass('active-photo');
    $('.photo2').removeClass('active-photo');
    $('.about-text p').text(text);
    $('.about-site p').text(info);
    $('.about-site a').text(webSite).attr("href", "#");
    $('.point li:first').addClass('active-point');
    $('.point li:nth-child(2)').removeClass('active-point');
    $('.point li:nth-child(3)').removeClass('active-point');
});

$('.photo1').click(function(){
    $('.photo1').addClass('active-photo');
    $('.photo').removeClass('active-photo');
    $('.photo2').removeClass('active-photo');
    $('.about-text p').text(text1);
    $('.about-site p').text(info1);
    $('.about-site a').text(webSite).attr("href", "#");
    $('.point li:first').removeClass('active-point');
    $('.point li:nth-child(2)').addClass('active-point');
    $('.point li:nth-child(3)').removeClass('active-point');
});

$('.photo2').click(function(){
    $('.photo2').addClass('active-photo');
    $('.photo1').removeClass('active-photo');
    $('.photo').removeClass('active-photo');
    $('.about-text p').text(text2);
    $('.about-site p').text(info2);
    $('.about-site a').text(webSite).attr("href", "#");
    $('.point li:first').removeClass('active-point');
    $('.point li:nth-child(2)').removeClass('active-point');
    $('.point li:nth-child(3)').addClass('active-point');

});

Comments

[0xD34F]

Вот это правильно, что разметку не стали показывать - так быстрее и точнее ответят.

Answer 1

[Rsa97]

$('.photo').click(function(){
    $('.photo').removeClass('active-photo');
    $(this).addClass('active-photo');
    $('.about-text p').text($(this).data('text'));
    $('.about-site p').text($(this).data('info'));
    $('.about-site a').text($(this).data('webSite')).attr("href", "#");
    $('.point li').removeClass('active-point');
    $('.point li')[$(this).data('listNumber')].addClass('active-point');
});

Comments

[Марк Цукенберг]

Не работает

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

<sarcasm>
Да, Rsa97, как не стыдно предлагать неработающий код? Может, Марк Цукенберг ещё и объяснить должен что конкретно у него не работает и есть ли какие-то ошибки в консоли? Или может, он data-атрибуты ещё должен расставлять сам в своём коде?
</sarcasm>

[Марк Цукенберг]

Алексей Уколов, извини пожалуйста сверхразум за мою тупость. А вообще я с текстом не могу разобрать. Изображения меняются, а вот текст нет.

[Rsa97]

Марк Цукенберг, https://itchief.ru/lessons/javascript/jquery-data

[Марк Цукенберг]

Rsa97, спасибо разобрался, а вот с точками не разобрался

<div class="points">
            <ul class="point" data-listnumber="">
                <li><figure></figure></li>
                <li><figure></figure></li>
                <li><figure></figure></li>
            </ul>
        </div>

Вот так вот нужно ?

[Марк Цукенберг]

Rsa97, не работает вот это

$('.point li')[$(this).data('listNumber')].addClass('active-point');