You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flex.apache.org by "Justin Mclean (JIRA)" <ji...@apache.org> on 2017/06/24 02:55:00 UTC

[jira] [Commented] (FLEX-35330) [FlexJS] comparison of objects using non strict inequality is slow

    [ https://issues.apache.org/jira/browse/FLEX-35330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16061763#comment-16061763 ] 

Justin Mclean commented on FLEX-35330:
--------------------------------------

Changing the code to use equality and strict equality give similar results.

> [FlexJS] comparison of objects using non strict inequality is slow
> ------------------------------------------------------------------
>
>                 Key: FLEX-35330
>                 URL: https://issues.apache.org/jira/browse/FLEX-35330
>             Project: Apache Flex
>          Issue Type: Bug
>    Affects Versions: Apache FlexJS 0.7.0
>            Reporter: Justin Mclean
>
> Code use to test:
> {code}
> <?xml version="1.0" encoding="utf-8"?>
> <js:Application xmlns:fx="http://ns.adobe.com/mxml/2009"
>                 xmlns:js="library://ns.apache.org/flexjs/basic" applicationComplete="init()">
>     <js:valuesImpl>
>         <js:SimpleCSSValuesImpl/>
>     </js:valuesImpl>
>     <fx:Script><![CDATA[
>             protected function init():void
>             {
>                 var start:Date;
>                 var end:Date;
>                 var took:Number;
>                 var obj1:Object;
>                 var obj2:Object;
>                 var obj3:Object = null;
>                 var obj4:Object = null;
>                 var i:int;
>                 var j:int;
>                 j = 0;
>                 start = new Date();
>                 for (i = 0; i < 1e7; i++) {
>                     if (obj1 != null) {
>                         j++;
>                     }
>                 }
>                 end = new Date();
>                 took = end.getTime() - start.getTime();
>                 trace("Uninitialised != : " + took);
>                 j = 0;
>                 start = new Date();
>                 for (i = 0; i < 1e7; i++) {
>                     if (obj2 !== null) {
>                         j++;
>                     }
>                 }
>                 end = new Date();
>                 took = end.getTime() - start.getTime();
>                 trace("Uninitialised !== : " + took);
>                 j = 0;
>                 start = new Date();
>                 for (i = 0; i < 1e7; i++) {
>                     if (obj3 != null) {
>                         j++;
>                     }
>                 }
>                 end = new Date();
>                 took = end.getTime() - start.getTime();
>                 trace("Null initialised != : " + took);
>                 j = 0;
>                 start = new Date();
>                 for (i = 0; i < 1e7; i++) {
>                     if (obj4 !== null) {
>                         j++;
>                     }
>                 }
>                 end = new Date();
>                 took = end.getTime() - start.getTime();
>                 trace("Null initialised !== : " + took);
>             }
>         ]]></fx:Script>
>     <js:initialView>
>         <js:View percentWidth="100" percentHeight="100">
>             <js:Label id="result" />
>         </js:View>
>     </js:initialView>
> </js:Application>
> {code}
> Output of 5 consecutive tests:
> Uninitialised != : 18
> Uninitialised !== : 18
> Null initialised != : 16
> Null initialised !== : 13
> Uninitialised != : 20
> Uninitialised !== : 18
> Null initialised != : 17
> Null initialised !== : 14
> Uninitialised != : 18
> Uninitialised !== : 19
> Null initialised != : 16
> Null initialised !== : 14
> Uninitialised != : 18
> Uninitialised !== : 18
> Null initialised != : 16
> Null initialised !== : 14
> Uninitialised != : 19
> Uninitialised !== : 18
> Null initialised != : 17
> Null initialised !== : 14
> Smaller numbers are faster. Run on Chrome 55 on OSX.
> You can see about a 35% performance increase.
> Note there is also a small performance increase in use non strict inequality and assigning to null.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)