refactor: avoid unnecessary copies, update examples, and refactor benchmarks

Author: kgryte

lib/node_modules/@stdlib/array/base/remove-at/README.md

@@ -90,11 +90,13 @@ var randi = require( '@stdlib/random/base/discrete-uniform' );
 var removeAt = require( '@stdlib/array/base/remove-at' );
 
 // Create an array of random numbers:
-var x = discreteUniform( 30, 0, 5, {
+var x = discreteUniform( 10, 0, 5, {
     'dtype': 'generic'
 });
 // returns [...]
 
+console.log( x );
+
 // Remove a random element:
 var y = removeAt( x, randi( 0, x.length-1 ) );
 // returns [...]

lib/node_modules/@stdlib/array/base/remove-at/benchmark/benchmark.js

@@ -0,0 +1,78 @@
+/**
+* @license Apache-2.0
+*
+* Copyright (c) 2024 The Stdlib Authors.
+*
+* Licensed under the Apache License, Version 2.0 (the "License");
+* you may not use this file except in compliance with the License.
+* You may obtain a copy of the License at
+*
+*    http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing, software
+* distributed under the License is distributed on an "AS IS" BASIS,
+* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+* See the License for the specific language governing permissions and
+* limitations under the License.
+*/
+
+'use strict';
+
+// MODULES //
+
+var bench = require( '@stdlib/bench' );
+var uniform = require( '@stdlib/random/array/uniform' );
+var isArray = require( '@stdlib/assert/is-array' );
+var pkg = require( './../package.json' ).name;
+var removeAt = require( './../lib' );
+
+
+// MAIN //
+
+bench( pkg+':index=-1', function benchmark( b ) {
+	var out;
+	var x;
+	var i;
+
+	x = uniform( b.iterations+1, 0.0, 10.0, {
+		'dtype': 'generic'
+	});
+
+	b.tic();
+	for ( i = 0; i < b.iterations; i++ ) {
+		out = removeAt( x, -1 );
+		if ( out.length === 0 ) {
+			b.fail( 'unexpected length' );
+		}
+	}
+	b.toc();
+	if ( !isArray( out ) ) {
+		b.fail( 'should return an array' );
+	}
+	b.pass( 'benchmark finished' );
+	b.end();
+});
+
+bench( pkg+':index=0', function benchmark( b ) {
+	var out;
+	var x;
+	var i;
+
+	x = uniform( b.iterations+1, 0.0, 10.0, {
+		'dtype': 'generic'
+	});
+
+	b.tic();
+	for ( i = 0; i < b.iterations; i++ ) {
+		out = removeAt( x, 0 );
+		if ( out.length === 0 ) {
+			b.fail( 'unexpected length' );
+		}
+	}
+	b.toc();
+	if ( !isArray( out ) ) {
+		b.fail( 'should return an array' );
+	}
+	b.pass( 'benchmark finished' );
+	b.end();
+});

lib/node_modules/@stdlib/array/base/remove-at/benchmark/benchmark.length.js

@@ -23,11 +23,13 @@ var randi = require( '@stdlib/random/base/discrete-uniform' );
 var removeAt = require( './../lib' );
 
 // Create an array of random numbers:
-var x = discreteUniform( 30, 0, 5, {
+var x = discreteUniform( 10, 0, 5, {
 	'dtype': 'generic'
 });
 // returns [...]
 
+console.log( x );
+
 // Remove a random element:
 var y = removeAt( x, randi( 0, x.length-1 ) );
 // returns [...]

lib/node_modules/@stdlib/array/base/remove-at/examples/index.js

@@ -50,11 +50,8 @@ function removeAt( x, index ) {
 	} else if ( index >= len ) {
 		return x;
 	}
-	j = 0;
-	for ( i = 0; i < len; i++ ) {
-		if ( i === index ) {
-			continue;
-		}
+	j = index;
+	for ( i = index+1; i < len; i++ ) {
 		x[ j ] = x[ i ];
 		j += 1;
 	}