Ваш код в порядке.
Однако я хотел бы рассмотреть одну вещь: хотите ли вы продолжать вызывать document.getElementById()
(внутри функции), если возникает ситуация, когда вам нужно выполнить несколько изменений одного и того же элемента. То, что я собираюсь предложить, это излишнее, чтобы показать вам больше возможностей, но учтите, что вы можете передать Id своей функции, или передать ссылку на элемент напрямую, или иметь функцию, которая принимает строку или элемент справка и вычисляется по типу параметра:
function objStyleById(oId,p,v){
document.getElementById(oId).style[p]=v;
}
function objStyle(o,p,v) {
o.style[p] = v;
}
function objStyleAuto(o,p,v) {
if (typeof o === "string")
o = document.getElementById("o");
// else not a string so assume o is element reference
o.style[p] = v;
}
objStyleById('object','property','value');
var myEl = document.getElementById("someElement");
objStyle(myEl,"prop","val");
objStyle(myEl,"prop2","val");
// some other non-style operation on myEl, e.g.,
myEl.className = "something";
myEl.innerHTML = "something";
objStyle(myEl.parentNode,"prop","value");
objStyleAuto('object','property','value');
objStyleAuto(myEl,'property','value');