JavaScript don't execute if statement -


i have loop , when execute code execute 'else' statement not if statement.

screenshot:

enter image description here

i recommend using dom dataset save index of picked color instead of color itself. 

var colors=["rgb(99,49,25)","rgb(115,152,170)","rgb(195,171,20)","rgb(55,49,35)","rgb(177,134,98)","rgb(247,100,160)"]    var squares = document.queryselectorall(".square");  var colordisplayed = document.queryselector("#colordisplayed");  var pickedcolor = 0;  colordisplayed.innertext = colors[pickedcolor]    var handler = function(e) {    var clickedcolor = number(e.target.dataset.colorindex); // explicitly transform dataset colorindex number    if (clickedcolor === pickedcolor) { // if don't want transforming, use `==' instead of `==='      alert("right");    } else {      alert("wrong");    }  }    (var = 0; < squares.length; i++) {    squares[i].style.background = colors[i];    squares[i].dataset.colorindex = i;    squares[i].addeventlistener("click", handler)  }
.square {width: 100px;height: 100px;border: solid 1px black;float: left;}
<div id="colordisplayed"></div>  <div><div class="square"></div><div class="square"></div><div class="square"></div><div class="square"></div><div class="square"></div>  <div class="square"></div></div>

another way using closure save it.

var colors=["rgb(99,49,25)","rgb(115,152,170)","rgb(195,171,20)","rgb(55,49,35)","rgb(177,134,98)","rgb(247,100,160)"]    var squares = document.queryselectorall(".square");  var colordisplayed = document.queryselector("#colordisplayed");  var pickedcolor = 0;  colordisplayed.innertext = colors[pickedcolor]    (let = 0; < squares.length; i++) { // notice keyword `let' here    squares[i].style.background = colors[i];    squares[i].dataset.colorindex = i;    squares[i].addeventlistener("click", function() {      if (i === pickedcolor) { // since "letted", different each in each event listener function        alert("right");      } else {        alert("wrong");      }    })  }
.square {width: 100px;height: 100px;border: solid 1px black;float: left;}
<div id="colordisplayed"></div>  <div><div class="square"></div><div class="square"></div><div class="square"></div><div class="square"></div><div class="square"></div>  <div class="square"></div></div>


Comments

Post a Comment

Popular posts from this blog

Is there a better way to structure post methods in Class Based Views -

i'm using class based views in django , never specify url in ajax calls , omit action parameter in form because know these requests go through post method of class based view. notice view - profile page make many requests depending on user updating on page - post method become convoluted , if/else mess example: def post(self, request, *args, **kwargs): # handle user changing available date if request.post.get('availabledate') != none: self.updateavailabledate(request) return jsonresponse({'result':'success'}) if request.post.get('newprojectname') != none: creative_user = creativeuserprofile.objects.get(id = request.user.id) project = project.create_project(creative_user, request.post['newprojectname']) project.save() return jsonresponse({'projectid': project.id }) if request.post.get('projectimage') != none , request.files['file']: proj...
Read more

performance - Why is XCHG reg, reg a 3 micro-op instruction on modern Intel architectures? -

i'm doing micro-optimization on performance critical part of code , came across sequence of instructions (in at&t syntax): add %rax, %rbx mov %rdx, %rax mov %rbx, %rdx i thought had use case xchg allow me shave instruction , write: add %rbx, %rax xchg %rax, %rdx however, dimay found agner fog's instruction tables , xchg 3 micro-op instruction 2 cycle latency on sandy bridge, ivy bridge, broadwell, haswell , skylake. 3 whole micro-ops , 2 cycles of latency! 3 micro-ops throws off 4-1-1-1 cadence , 2 cycle latency makes worse original in best case since last 2 instructions in original might execute in parallel. now... cpu might breaking instruction micro-ops equivalent to: mov %rax, %tmp mov %rdx, %rax mov %tmp, %rdx where tmp anonymous internal register , suppose last 2 micro-ops run in parallel latency 2 cycles. given register renaming occurs on these micro-architectures, though, doesn't make sense me done way. why wouldn't register renamer ...
Read more

c# - Asp.net web api : redirect unauthorized requst to forbidden page -

im trying redirect unauthorized request forbidden page instead i'm getting forbidden page in response body , how can fix ? here's startup class : app.createperowincontext(storecontext.create); app.createperowincontext<applicationusermanager>(applicationusermanager.create); app.createperowincontext<applicationsigninmanager>(applicationsigninmanager.create); app.usecookieauthentication(new cookieauthenticationoptions { authenticationtype = defaultauthenticationtypes.applicationcookie, expiretimespan = timespan.fromdays(30), }); app.useexternalsignincookie(defaultauthenticationtypes.externalcookie); app.usetwofactorsignincookie(defaultauthenticationtypes.twofactorcookie, timespan.fromminutes(5)); app.usetwofactorrememberbrowsercookie(defaultauthenticationtypes.twofactorrememberbrowsercookie); the method im trying reach : [httpget] [authorize(roles = "admin")] public string getcurrentusername() { return userman...
Read more