Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > comp.lang.javascript > #27776

Re: Merge indexes..

Show key headers only | View raw

John Harris wrote:

> […] Ben Bacarisse […] wrote:
>> jonas.thornvall@gmail.com writes:
>>> <script>
>>> function merge(){
>>>   for (i=0;i<myArr.length;i++){
>>>       myInt=(myArr[i]*x)+myInt;
>>>       x=x*10;
>>>   }
>>> }
>>> myArr =[7,9,8]
>>> myInt=0;
>>> x=1;
>>> merge();
>>> document.write(myInt);
>>> </script>
>>
>>Don't make you code worse by adding yet more function that communicate
>>via globals.  This operation is a self-contained function of the array.
>>Pass the array as a parameter and return the result from the function.
> 
> Even better, pass the base, x, as a parameter as well.

And for bases from 2 to 36, just return parseInt(a.join(""), base), with 
both “a” and “base” being formal parameters of merge().  No need to reinvent 
Wheel 0.1:

<http://ecma-international.org/ecma-262/5.1/#sec-15.1.2.2>

Finally, make merge() the valueOf() method of your own object type, so that 
“a” and maybe even “base” can be replaced by references to properties of the 
instance.

-- 
PointedEars
FAQ: <http://PointedEars.de/faq> | SVN: <http://PointedEars.de/wsvn/>
Twitter: @PointedEars2 | ES Matrix: <http://PointedEars.de/es-matrix>
Please do not cc me. / Bitte keine Kopien per E-Mail.

Back to comp.lang.javascript | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Merge indexes.. jonas.thornvall@gmail.com - 2015-05-19 23:59 -0700
  Re: Merge indexes.. jonas.thornvall@gmail.com - 2015-05-20 01:20 -0700
    Re: Merge indexes.. Ben Bacarisse <ben.usenet@bsb.me.uk> - 2015-05-20 11:42 +0100
      Re: Merge indexes.. jonas.thornvall@gmail.com - 2015-05-20 06:16 -0700
      Re: Merge indexes.. John Harris <niam@jghnorth.org.uk.invalid> - 2015-05-20 14:21 +0100
        Re: Merge indexes.. Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2015-05-20 15:32 +0200
          Re: Merge indexes.. Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2015-05-20 15:52 +0200
            Re: Merge indexes.. jonas.thornvall@gmail.com - 2015-05-20 08:23 -0700
            Re: Merge indexes.. jonas.thornvall@gmail.com - 2015-05-21 05:51 -0700
              Re: Merge indexes.. "Evertjan." <exxjxw.hannivoort@inter.nl.net> - 2015-05-21 15:44 +0200
                Re: Merge indexes.. jonas.thornvall@gmail.com - 2015-05-21 07:24 -0700
                Re: Merge indexes.. Ben Bacarisse <ben.usenet@bsb.me.uk> - 2015-05-21 15:52 +0100
                Re: Merge indexes.. Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2015-05-23 09:40 +0200
                Re: Merge indexes.. "Evertjan." <exxjxw.hannivoort@inter.nl.net> - 2015-05-23 11:54 +0200
                Re: Merge indexes.. "Evertjan." <exxjxw.hannivoort@inter.nl.net> - 2015-05-21 22:49 +0200
  Re: Merge indexes.. Andrew Poulos <ap_prog@hotmail.com> - 2015-05-20 20:55 +1000
    Re: Merge indexes.. "Michael Haufe (TNO)" <tno@thenewobjective.com> - 2015-05-20 04:50 -0700
      Re: Merge indexes.. Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2015-05-20 14:32 +0200
        Re: Merge indexes.. "Christoph M. Becker" <cmbecker69@arcor.de> - 2015-05-20 15:12 +0200

csiph-web