Este problema en el fondo es la composición de los siguientes problemas:
Problema 1. Hacer el manejador genérico:
Para ello hay muchas alternativas, ninguna de ellas extremadamente sencilla. Queremos simular el paso de un parámetro al manejador y los parámetros son pasados por el emisor del evento, así que no hay una forma limpia de aportarlos mediante los eventArgs.
Sin embargo,
1.a. puedes hacer una clase que contenga como atributo un flag que sea la operación mostrar u ocultar e implementar algo parecido a lo que propone Dariel Ramos Díaz de Villegas
1.b. Puedes crear una expresión lambda que genere una expresión lambda para mostrar u ocultar en función del parámetro de la primera.
Problema 2. Reutilizar el timer
Para reutilizar el timer, recuerda que el evento Tick del timer es eso, un evento. Admite múltiples manejadores. Tan solo asígnale el manejador de abrir, el de cerrar, ambos o ninguno para abrir, cerrar, abrir y cerrar (dos formularios distintos) o no hacer nada en absoluto (aunque para no hacer nada en absoluto puedes deshabilitar el timer).
Si deseas una implementación de lo que propongo, comenta mi respuesta pidiéndolo y así lo haré.
Con esto considero que he respondido a la pregunta, pero me parece muy importante mencionar lo siguiente:
La eficiencia no consiste en esto. Tu código no va a ser más eficiente o menos eficiente por usar un timer en vez de dos. Como tampoco lo va a ser por el número de líneas de código de tu código fuente (de hecho hay técnicas que incrementan la longitud del código pero lo hacen más eficiente). El código es reutilizado por motivos de mantenibilidad, no por eficiencia.
Considera que estás escribiendo código en un lenguaje de alto nivel. Dado que estás usando .Net, tú código es transformado a un código intermedio en un lenguaje conocido como CIL. A su vez, este código es transformado de nuevo por el CLR en código nativo.
Con esto en mente, observa que un timer es en el fondo una representación abstracta para poder desencadenar un evento periódicamente. Tener dos timers significa ocupar algunos bytes más en memoria y tener los dos timers activos implica realizar desencadenar dos eventos en vez de uno, pero la diferencia entre tener dos timers con un manejador en cada uno y tener un timer con dos manejadores es despreciable (tan despreciable que te animo a que hagas la media de mil ejecuciones de mil ciclos por cada alternativa, verás que, a lo sumo, hay unos pocos milisegundos de diferencia).
No voy a tratar en detalle el tema de las optimizaciones de código porque es muy (pero que muy) amplio, pero si tu idea es optimizar el código, no van por ahí los tiros. Es mucho más importante tener en cuenta los mecanismos de caché y el acceso a bases de datos (por ejemplo) que la reutilización de una variable.
Un saludo y siéntete libre de preguntar lo que quieras.