diff options
author | Marc Delisle <marc@infomarc.info> | 2013-03-30 14:10:59 +0400 |
---|---|---|
committer | Marc Delisle <marc@infomarc.info> | 2013-03-30 14:10:59 +0400 |
commit | a746469e9785b12c0bd49b4ad89df3ccca202807 (patch) | |
tree | c5c487291d04fbb89b95c08806787ab0162a7103 /js/keyhandler.js | |
parent | 1e5d225bbabc8a36695447a32a837bc707b09303 (diff) |
Unnecessary loop
Diffstat (limited to 'js/keyhandler.js')
-rw-r--r-- | js/keyhandler.js | 60 |
1 files changed, 27 insertions, 33 deletions
diff --git a/js/keyhandler.js b/js/keyhandler.js index c5f9e9e860..8f9ccec527 100644 --- a/js/keyhandler.js +++ b/js/keyhandler.js @@ -47,46 +47,40 @@ function onKeyDownArrowsHandler(e) return; } - var x = pos[2], y=pos[1]; + var x = pos[2], y = pos[1]; var nO = null; - // skip non existent fields - for (var i=0; i < 10; i++) - { - switch(e.keyCode) { - case 38: - // up - y--; - break; - case 40: - // down - y++; - break; - case 37: - // left - x--; - break; - case 39: - // right - x++; - break; - default: - return; - } + switch(e.keyCode) { + case 38: + // up + y--; + break; + case 40: + // down + y++; + break; + case 37: + // left + x--; + break; + case 39: + // right + x++; + break; + default: + return; + } - var id = "field_" + y + "_" + x; + var id = "field_" + y + "_" + x; + nO = document.getElementById(id); + if (! nO) { + id = "field_" + y + "_" + x + "_0"; nO = document.getElementById(id); - if (!nO) { - id = "field_" + y + "_" + x + "_0"; - nO = document.getElementById(id); - } - if (nO) { - break; - } } - if (!nO) { + // skip non existent fields + if (! nO) { return; } nO.focus(); |