这个函数有什么不对的地方吗？

function a(){

    var z= document.getElementById("search")

    if (z.style.display="none")

      {

        z.style.display= "block";

      }

    else

      {

        z.style.display= "none";

      }

}