Нужна функция в JavaScript, чтобы поменять 2 элемента в массиве - PullRequest
0 голосов
/ 14 апреля 2010

Программа для обмена двумя числами

        /*
        /*
        Function to swap two numbers.
        Function takes an argument which is an array of two elements.
        Function returns a new array containing the same two elements
                       as the argument array but in reverse order.
        */

        function swap(anArray)           
        {
        // declare and initialise a variable to hold the length of the 
                          argument array
            var length = anArray.length;

            //declare an array to be returned by the function
            var returnArray = new Array(length);

            //copy each element of the argument array to the  
                                  return array
            for (var i = 0; i < length; i = i + 1)
            {
                returnArray[i] = anArray[i];
            }


        var anArray [0] = 250;
        var anArray [1] = 100;

        var tempArray [0] = 0;
        var tempArray [1] = 0;

        tempArray [0] = anArray [1];
        tempArray [1] = anArray [0];


        }

        document.write('A program to swap two numbers.');

        //  PLACE YOUR CODE FOR THE MAIN PROGRAM HERE

        var anArray = [250,100];

        // Display the original array values,
        document.write('The original array was ' + anArray[i] + '<BR>');

        // invoke swap() with two element array argument
        function swap(anArray);

        // display final array values
        document.write('This array now becomes ' + returnArray[i] + '<BR>');

    </SCRIPT>
</HEAD>
<BODY>

</BODY>

Ответы [ 3 ]

10 голосов
/ 14 апреля 2010

Ваш код смехотворно длинный. Если массив всегда содержит два элемента, почему бы не сделать это?

function swap(arr) {
    return [arr[1], arr[0]];
}

Кроме того, правильный способ вызова функции:

arr = swap(arr);

Если вы хотите, чтобы функция изменила свой аргумент, вместо этого сделайте следующее:

function swap(arr) {
    var tmp = arr[1];
    arr[1] = arr[0];
    arr[0] = tmp;
}

... также есть встроенный метод reverse для массивов:

arr.reverse();
0 голосов
/ 14 апреля 2010
    function swap(anArray) { 
        var length = anArray.length; 
        var returnArray = new Array(length); 
        for (var i = 0, j = length - 1; i < length; i++, j--) { 
            returnArray[j] = anArray[i]; 
        } 
        return returnArray;
    }
0 голосов
/ 14 апреля 2010

Немного не по теме, но этот метод Array меняет местами два элемента своего массива (общий случай) самым быстрым из возможных способов:

// swaps elements at index i and j in array this
// feature detection for destructuring assignment
Array.prototype.swap = (function () {
    var i=0, j=1;
    try { [i,j]=[j,i]; }
    catch (e) {}
    if(i) {
        return function(i,j) {
            [this[i],this[j]] = [this[j],this[i]];
            return this;
        }
    } else {
        return function(i,j) {
            var temp = this[i];
            this[i] = this[j];
            this[j] = temp;
            return this;
        }
    }
})();
...