javascript - Why does this && not short-circuit? -

-

this code node module templatizer.

if (i === 3 && node.type === "expressionstatement" && node.expression.callee.object.name === "buf" &&        node.expression.arguments.length === 1 && node.expression.arguments[0].type === "literal") {         // save simple string        simplestring = node.expression.arguments[0].value;        cnt++;     }

the value of node.type 'variabledeclaration' logical expression false , node.expression shouldn't evaluated appears so...

typeerror: cannot read property 'object' of undefined @ /node_modules/templatizer/lib/simplifytemplate.js:34:89 @ array.foreach (native)

it means 'object' in 'callee'. expression , callee both undefined. step on conditional node crashes.

edit

i think javascript works fine , perhaps async code resulting in strange results form debugger. if put console.log @ top of loop gives output makes sense...

/usr/local/bin/node bin/www count: 1 i:0 node.type: variabledeclaration expression: undefined node.expression.callee: undefined  count: 2 i:1 node.type: variabledeclaration expression: undefined node.expression.callee: undefined  count: 3 i:2 node.type: variabledeclaration expression: undefined node.expression.callee: undefined  count: 4 i:3 node.type: expressionstatement expression: [object object] node.expression.callee: [object object]  count: 5 i:4 node.type: returnstatement expression: undefined node.expression.callee: undefined  count: 6 i:0 node.type: variabledeclaration expression: undefined node.expression.callee: undefined  count: 7 i:1 node.type: variabledeclaration expression: undefined node.expression.callee: undefined  count: 8 i:2 node.type: variabledeclaration expression: undefined node.expression.callee: undefined  count: 9 i:3 node.type: expressionstatement expression: [object object] node.expression.callee: undefined   /users/me/webstormprojects/mysite/node_modules/templatizer/lib/simplifytemplate.js:41 = 3 && node.type === "expressionstatement" && node.expression.callee.object.na                                                                     ^ typeerror: cannot read property 'object' of undefined     @ /users/me/webstormprojects/mysite/node_modules/templatizer/lib/simplifytemplate.js:41:89     @ array.foreach (native)     @ module.exports (/users/me/webstormprojects/mysite/node_modules/templatizer/lib/simplifytemplate.js:15:18)     @ /users/me/webstormprojects/mysite/node_modules/templatizer/templatizer.js:111:20     @ array.foreach (native)     @ module.exports (/users/me/webstormprojects/mysite/node_modules/templatizer/templatizer.js:95:15)     @ object.<anonymous> (/users/me/webstormprojects/mysite/app.js:23:1)     @ module._compile (module.js:456:26)     @ object.module._extensions..js (module.js:474:10)     @ module.load (module.js:356:32)  process finished exit code 8

so looks bug in module.

seems debugger showing values different part of execution 1 caused crash. surely must running things asynchronously somewhere. bug in module. temporary fix add check conditional.

if (i === 3 && node.type === "expressionstatement" && node.expression.callee && node.expression.callee.object.name === "buf" &&                 node.expression.arguments.length === 1 && node.expression.arguments[0].type === "literal")

i opened issue.


Comments

Post a Comment

Popular posts from this blog

c++ - QTextObjectInterface with Qml TextEdit (QQuickTextEdit) -

-
i registred handler simple qtextobjectinterface , draws 10x10 red rectangle. when used qtextedit in normal qwidget app, worked. when used qquicktextedit (textedit qml component) in qt quick app, doesn't worked (nothing drawed, rectangle in textedit reserved, because when change cursor position, notice there something, empty space, nothing drawed. qtextobjectinterface intrinsicsize method called (that explains why see there empty space 10x10), drawobject method isn't. i did research , found problem here: qquicktextedit.cpp qt 5.3.0 sources (line 1821) qsgnode *qquicktextedit::updatepaintnode(qsgnode *oldnode, updatepaintnodedata *updatepaintnodedata) { . . . if (textframe->firstposition() > textframe->lastposition() && textframe->frameformat().position() != qtextframeformat::inflow) { updatenodetransform(node, d->document->documentlayout()->frameboundingrect(textframe).topleft()); c...
Read more

javascript - angular ng-required radio button not toggling required off in firefox 33, OK in chrome -

-
i trying toggle required attribute on radio button using angular show html5 popup. i cannot following code function correctly in firefox 33. function correctly in chrome. required property being toggled via checkbox value, in firefox, when set false radio button still acts required , shows html5 required popup. see plnkr below example doesn't work me in ff 33. <!doctype html> <html> <head> <script data-require="angular.js@*" data-semver="1.3.1" src="//code.angularjs.org/1.3.1/angular.js"></script> <link href="style.css" rel="stylesheet" /> <script src="script.js"></script> </head> <body ng-app="myapp"> <script> angular.module('myapp',[]) .controller('mycontroller',['$scope',function($scope) { $scope.foorequired=true; $scope.submitform = function() { $scope.submitmessage = ...
Read more

xcode - Swift Playground - Files are not readable -

-
Image
files not readable in swift playground . how make files readable? same code runs on xcode terminal app, fails on swift playground. demo code below. import foundation println("hello, world!") var fname:string = "/users/holyfield/desktop/com.apple.iconcomposer.plist" var fm:nsfilemanager = nsfilemanager.defaultmanager() if(fm.fileexistsatpath(fname)){ println("file exists") if(fm.isreadablefileatpath(fname)){ println("file readable") var fd:nsdata? = nsdata(contentsoffile: fname) println(fd?.length) let pl = nsdictionary(contentsoffile: fname) println(pl?.count) println(pl?.allkeys) }else{ println("file not readable") } } else{ println("file not exists") } sample images: i have thank nate cook first quick response , solid answer. just case share answer post, title misleading. see nate's answer here: https://stackoverf...
Read more