Return a string from an input written the other way around

Question

This code is supposed to return a string from an input, written the other way around but it doesn't work... Can someone help me?

function Change1() {
    document.getElementById("top").innerHTML="Inroduceti cuvantul";
    document.getElemenyById("button1").setAttribute('onclick','StringBackFlip(document.getElementById("insert").value)');
}
function StringBackFlip(str) {
    var res = "";
    var i = str.lenght;
    while (i>0) {
        res += str[i-1];
        i--;
    }
    document.getElementById("result").value=res;
    document.getElementById("top").innerHTML="Introduceti alt cuvant";
}

This is the code. I need help so tell me what I've done wrong.

Warning: http://stackoverflow.com/a/16776380/1636522. – Dec 28 '15 at 16:12 — , Dec 28 '15 at 16:12

epascarello · Answer 1 · 2015-12-28T15:54:25.200

0

That is not how you use setAttribute. There should be no equal sign.

setAttribute=('onclick'
           ^^^

Add events with addEventListener.

And also

var i = str.lenght; <-- spelled wrong

edited Dec 28 '15 at 15:54

answered Dec 28 '15 at 15:39

epascarello

204,599
20
195
236

I actually noticed that after posting the question. In my code it is without the equal sign. However the code doesn't work... – Whitewolf3131 Dec 28 '15 at 15:52

score 0 · Answer 2 · answered Dec 28 '15 at 15:54

You have several syntax errors in your code, but this code does what you asked. I suggest the following really great articles, to give you a better understanding of js and it's beauty and it's quirks

 document.getElementById("button1").addEventListener('click',function(ev){
   var forward = document.getElementById('insert').value;
   var backward = forward.split('').reverse().join('');
   document.getElementById('result').innerHTML = backward;
 });

<p id="top">Introduceti alt cuvant</p>

<input type="text" name="insert" id="insert" value="hello" />  
<button id="button1" type="button">reverse it</button>

<p id="result"></p>

The code you gave me is good. The problem I have more than one function which is supposed to run on the same button but if I start the first one and after the second, the first function will also start on the second's one turn. What should I do to fix that? — Whitewolf3131, Dec 28 '15 at 20:10

Return a string from an input written the other way around

2 Answers2