Há com disparar eventos a partir do onclick?

[Micilini Roll 49]

pessoal seguinte eu tenho este link que da um alert na tela:

 

<a href="#" onclick="alerta()" value="de casa" id="chave">Dispare o alerta</a>

<script>
function alerta(){
var valor = document.getelementbyid("chave").value;
alert("Alerta disparado!! "+ valor );
}
</script>

 

 

so que eu nao quero fazer isso eu gostaria de colocar este comando de pegar o value e o alert dentro do onclick ficando +- assim:

 

<a href="#" onclick="alerta(){var valor = document.getelementbyid("chave").value; alert("Alerta disparado!! "+ valor );
}" value="de casa" id="chave">Dispare o alerta</a>

 

 

so que nao pareçe funcionar!!! tem como fazer isso?

[William Bruno 1501]

não faça isso.. é gambiarra.

 

o correto em js é não usar o atributo onclick, e muito menos ficar escrevendo código inline no html.

[Bergs 53]

Chamando a função dentro do onclick você já está disparando o evento.

Você precisa abstrair o javascript do html, o atributo onclick assim como outros onsubmit, onblur...

[cristianoolv 93]

<a não recebe atributo "value", logo qualquer coisa parecida com quer voce não dará certo.

[Micilini Roll 49]

entendi pessoal,mas o william bruno disse uma coisa que nao entendi:

 

 

o correto em js é não usar o atributo onclick,

 

 

como assim? espere em todos os meus codigos de meu site 60% de links que estao lá! ultilizam onclick para chamar varias funçoes diferentes?! isso é errado? entao como serio o certo?

perante ao código inline no html ja esta safo! nao ultilizo mais isso!

[William Bruno 1501]

o correto é não colocar onclick no html, deixando tudo no js. Assim:

window.onload = function(){
   document.getElementById('t').onclick = function(){
       alert('Aeeeee');
   }
}

e ai o html fica limpo:

<p id="t">Clique aqui</p>

[Micilini Roll 49]

aaata entendi! vlw pelas dicas william bruno, e todos que me ajudaram!!! abraços!!!

[#eclipse 1]

outra maneira :

 

window.onload = function(){
document.getElementsByName("p")[0].onclick = function(){
alert('Aeeeee');
}
}

 

<p>Clique aqui</p>

[Micilini Roll 49]

é isso ai #eclipse ! agradeço a sua contribuição para o nosso forum!