А надо, чтобы выполнялось сразу же с первого раза.
function f1(){
//var pcolor = document.getElementsByTagName('p');
var one = document.getElementById('one');
if(one.style.background == 'black') {
one.style.background = 'yellow';
one.style.transition = 'all ease 1s';
}
else {
one.style.background = 'black';
one.style.transition = 'all ease 1s';
}
/*for(i=0; i < pcolor.length; i++) {
pcolor.style.backgroundColor = 'blue';
}*/
}
p {
width: 50px;
height: 10px;
background-color: black;
display: inline-block;
}
<p id="one"></p>
<p></p>
<p></p>
<p></p>
<p></p>
<p></p>
<button onclick='f1()'>Press</button>