Vanilla Javascript: counter variable of for loop undefined when used in function -


this sudoko generator i'm programming in vanilla javascript:

jsfiddle

nicer looking full screen jsfiddle

if click on 1 of fields, popup shown 3x3 fields 1 9. if click on 1 of fields of popup f. ex. 5, should change parent field 5. problem changes parent field undefined.

here relevant code generate popup:

    // create popup     function createpopup(position) {         var popup = document.createelement('dialog');         popup.classname = 'popup';         popup.id = 'window_' + position;          var dialogblock = popup.appendchild(document.createelement('div'));         dialogblock.classname = 'popupblock';          (var z = 1; z <= 9; z++) {             var div = dialogblock.appendchild(document.createelement('div'));              div.classname = 'popupcell';             div.id = position + 'z' + z;             div.appendchild(document.createtextnode(z));              div.onclick = function (e, z) {                 document.getelementbyid(position).innerhtml = z;                 e.stoppropagation();                 popup.close();             };         }          return popup;     }

the problem seems z in scope when passing div.onclick. how can pass z function?

problem #1

the event handler function assigned globaleventhandlers.onclick passed 1 argument when called.

div.onclick = function (e, z)

there no second argument, rid of , z here.

problem #2

because onclick function not executed until element receives click event, need wrap body of for-loop in invoked function expression1,2 in order create new lexical scope each iteration of loop.

this makes z in context of event handler reference value of z in current iteration of loop instead value of z @ at last iteration of loop.

jsfiddle

for (var z = 1; z <= 9; z++) (function(z){      var div = dialogblock.appendchild(document.createelement('div'));      div.classname = 'popupcell';     div.id = position + 'z' + z;     div.appendchild(document.createtextnode(z));      div.onclick = function (e) {         console.log(e, z);         document.getelementbyid(position).innerhtml = z;         e.stoppropagation();         popup.close();     }; })(z);

Comments

Post a Comment

Popular posts from this blog

java - WARN : org.springframework.web.servlet.PageNotFound - No mapping found for HTTP request with URI [/board/] in DispatcherServlet with name 'appServlet' -

this question has answer here: why spring mvc respond 404 , report “no mapping found http request uri […] in dispatcherservlet”? 3 answers i trying making simple bbs i'm encountering following error, warn : org.springframework.web.servlet.pagenotfound - no mapping found http request uri [/board/] in dispatcherservlet name 'appservlet' i have tried solve error many times, have failed. how can solve error? here web.xml, controllerand , servlet-context. my web.xml <?xml version="1.0" encoding="utf-8"?> <web-app version="2.5" xmlns="http://java.sun.com/xml/ns/javaee" xmlns:xsi="http://www.w3.org/2001/xmlschema-instance" xsi:schemalocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/web-app_2_5.xsd"> <filter> <filter-nam...
Read more

1111. appearing after print sequence - php -

morning, i have code takes string of shoe sizes, explodes , puts in table. code works , info displayed correctly on page after table there few 1's after table. i'm confused this. so table looks (in columns of 3 on webpage) available in following sizes: 3 4 5 6 9 10 111 here's codes, purpose of example $x = 3|4|6|10|9 function get_sizes_pipe($x) { $counter = 0; $splits = explode("|",$x); asort($splits); $x = print "<table class='greentable'>"; $x .= print "<thead><tr><th scope='col' colspan='3' abbr='starter'>available in following sizes:</th></tr></thead><tbody><tr>"; foreach ($splits $split) { $entry = print "<td>".$split."</td>"; if($counter == 2){ $entry .= print "</tr><tr>"; $counter =0; } else { $counter++; ...
Read more

android - How to create dynamically Fragment pager adapter -

i unable create dynamic layout each tabs, able create tabs dynamically constructor(public mypageradapter(fragmentmanager fm , int nooftabs)), coudn't inflate view each tabs creating each fragment should dyanmic. public class mypageradapter extends fragmentpageradapter { private final string[] titles = { "categories", "home", "top paid", "top free", "top grossing", "top new paid", "top new free", "trending" }; int nooftabs; **public mypageradapter(fragmentmanager fm , int nooftabs) { super(fm); this.nooftabs = nooftabs; }** /*public mypageradapter(fragmentmanager fm) { super(fm); }*/ @override public charsequence getpagetitle(int position) { return titles[position]; } @override public int getcount() { ...
Read more