Mensajes

No pases variable alguna, solo devuelve el valor que te lo interesa.

Tipicamente será

condicion = ...  // Chequear dato en nodo actual

return condicion + f(nodo izquierdo) + f(nodo derecho)

Posiblente el modo null será especial.

En el caso del ABB todas las veces (o la mayoría) que ingresas a la función haces 2 comparaciones, contra null primero y luego si el de la derecha es null. Dada esta última, no es posible que las llamadas recursivas se encuentren con punteros null.

Puedes ahorrarte la primera comparación. Un truco es tener 2 funciones, una que hace la comparación contra null que no es recursiva y que invoca a otra que si es recursiva, pero que no compara contra null.

Sugeriría escribir los métodos de conversión fuera de las clases, para no crear una dependencia entre ellas.

El tipo de dato que leerIibros dice devolver  no está bien. Corrijelo.

Arroja una excepcion.

Aprende cómo se hace que una clase derive de otra y haz que Piano derive de Instrumento.

El Piano es un tipo de Instrumento, no es un tipo de tecla y no es un tipo de pedal. No tiene sentido que haya derivación de teclas o.pedales.

En vez de definir el método cargar define y usa el constructor.

Si le pides al usuario el índice de las teclas que van de 1 a 7, por qué le sumas 1?

El método apretar del piano no ejecuta el método apretar de la tecla.

Te falta la clase Instrumento.
La clase piano deriva de instrumento

El problema está en que todo el switch es un gran bloque y cada case es un label y por tanto existen labels que al saltar a ellos, hay variables que se saltan sus inicializaciones, lo que no está permitido en C++.

Define la(s) variable(s) fuera del switch o.bien
Define un bloque usando {} dentro del case donde vas a usar la(s) variable(s).