Как правильно использовать входящий параметр в функцию?

Question

[Андрей]

Помогите разобраться начинающему

<div class="panel panel-default" id="VIP" onclick="o('VIP');">

function o(type){
type = "'" + type + "'";
$(type).removeClass('panel-default');
$(type).addClass('panel-success');
};

В чем моя ошибка

Answer 1

[Владислав Савельев]

Зачем использовать входящий параметр? там же просто "VIP"

function o(type){
$(this).removeClass('panel-default');
$(this).addClass('panel-success');
};

Ну если уж вообще прижмет то

function o(type){
$('#' + type).removeClass('panel-default');
$('#' + type).addClass('panel-success');
};

А вообще использовать onclick и прочие - плохая практика.
Правильней всего будет это делать через jQuery

$('#VIP').click(function() {
$(this).removeClass('panel-default');
$(this).addClass('panel-success');
})

Comments

[Андрей]

Пробовал и так не работает(( alert делал VIP приходит.. но не хочет класс менять почему-то по нажатию(

[Андрей]

хмм.. вот так не пробовал щас проверю

[Андрей]

тоже не меняет.. бред какой-то чет у меня(

[Андрей]

я делал через onclick потому-что у меня 4 панели и разные id

[Владислав Савельев]

Андрей Григораш: https://jsfiddle.net/4wx2qn32/
посмотрите, всё работает.

можете прописать всем элементам один класс и так же написать, только в селекторе точка вместо решетки.
$('.classname').click(...)

[Андрей]

Илья Шамов: пасиб попробую :)

[Владислав Савельев]

Андрей Григораш: кстати, что в консоли то пишет?