Estabas cerca de la respuesta Alvaro lo unico que hice fue ponerle el `display: block;` por default a la etiqueta de tu parrafo y luego cree una nueva funcion que controla el estado del display con un ternario.. en caso que sea block llama tu funcion `ocultar()` en caso que sea none la funcion `mostrar()`.. El condicional no lo puedes llamar asi.. o puedes pero lo que hace es ejecutarse una vez y luego no funciona nuevamente.. por eso haces una funcion metes la condicion y cuando la necesites 
 llamas la fncion.. este es el codigo saludos..

<!-- begin snippet: js hide: false console: true babel: false -->

<!-- language: lang-html -->

    <p id="parrafo" style='display:block'>Lorem ipsum, dolor sit amet consectetur adipisicing elit.
            Voluptatem ut non sint excepturi molestias perspiciatis
            doloremque, provident ducimus est quo, dolorum at rerum!
            Hic accusamus mollitia unde, quisquam ratione repellendus!</p>
        <button type="button" id="boton" onclick='mostrarOcultar()'>boton</button>
        <script>
            var parr = document.getElementById("parrafo")

            function ocultar() {
                parr.style.display = 'none';
            }
            function mostrar() {
                parr.style.display = "block";
            }

            function mostrarOcultar(params) {
                parr.style.display == 'block' ? ocultar() : mostrar()
            }

        </script>

<!-- end snippet -->