Como llamar a una funcion

[fedefrankk]

Hola a todos, tengo un boton que hace lo siguiente.

Código Javascript:

function quienesom(){
        var S1=document.getElementById('img005');
        var S2=document.getElementById('img006');
        var S3=document.getElementById('mov700');
        var S4=document.getElementById('ser1');
        S1.style.display='none';
        S2.style.display='none';
        S3.style.display='none';
       var q1=document.getElementById('qui1');
       q1.style.display='block';
     
}
 

En otro boton tengo que pegar lo que es S1. S2. S3., como puedo llamar solamente ese fragmento en otra funcion, ya que sino tengo que copiar y pegar en cada boton que tengo lo mismo.
Espero que se me entienda,
Gracias por adelantado
Saludos a todos
Fede

[AlvaroG]

Hazte un objeto que cargue los valores en algún otro momento (quizás en el onload) y al que puedas acceder desde tu función:

Código Javascript:

elementos = {
    S1 = null,
    S2 = null,
    S3 = null,
    S4 = null,
 
    inicializar : function() {
        elementos.S1=document.getElementById('img005');
        elementos.S2=document.getElementById('img006');
        elementos.S3=document.getElementById('mov700');
        elementos.S4=document.getElementById('ser1');        
    }
}
 
function quienesom(){
        elementos.S1.style.display='none';
        elementos.S2.style.display='none';
        elementos.S3.style.display='none';
       var q1=document.getElementById('qui1');
       q1.style.display='block'; 
}
 
 
window.onload = function() {
    elementos.inicializar();
}
 

Ya que estás en eso, quizás quieras poner también la función en un objeto en vez de dejarla como global, así sea nada más por prolijidad.


saludos

[fedefrankk]

Hola, muchas gracias por tu tiempo, esta muy bueno el codigo, me dices que hay otra manera de hacerla mas prolija puede decirme como?, si no es molestia, con lo que me as esplicado esta re bien.

Saludos y gracias.!
Fede
PD: quisas vuelva hacerte unas preguntas....

[AlvaroG]

Simplemente evitando las funciones globales (que resultan de escribirlas sin estar dentro de un objeto). No te hará ninguna diferencia si lo que necesitás es solamente un par de funciones, pero puede evitarte colisiones de nombres en proyectos grandes. Además contribuye a la modularidad del código.
En este caso podrías, por ejemplo, hacer algo como:

Código Javascript:

var MiObjeto = {
    S1 = null,
    ....
 
    inicializar : function() {
        ....
    },
 
    quienessom : function() {
        ....
    }
};
 

Siempre recordá llamar a todas las funciones y atributos por su nombre completo (ej. MiObjeto.quienessom() )


saludos.

[fedefrankk]

Hola, Buenisimo, y es mas simplificado.!!
Muchas gracias.!!!
Saludos
Fede