splice() funzt in Array.prototype nicht

[tomez]

Hi folks,

wollte ein Prototype erstellen, welcher mir ein Array mit Zufallszahlen füllt.

x0 = Min Value z.B. 1
x1 = Max Value z.B. 49
num = Zahlenmenge welche ünernommen werden soll z.B. 6

Array.prototype.fillRandomNum = function(x0, x1, num) {
	for (var i = x0; i <= x1; i++) {
		this.push(i);
	}
	this.sort(function () {
		return (random(3) - 1);
	});
	this = this.splice(0, (num ? num : this.length));
};

theArray = [];
theArray.fillRandomNum(1, 49, 6);

Leider gibt er mir ein Array zurück, das alle Zahlen zwischen 1 und 49 enthält. Die Schleife und die Sotiertmethode funzt also, nur die splice-Methode nich...?

Please help!

[Janoscharlipp]

Willkommen im Forum!

Du verwendest Array.splice falsch.
Array.slice wäre die Funktion die du so einsetzen könntest, wie du es gerade vorhast, allerdings halte ich es für eine schlechte Idee, this zu überschreiben.

Da Array.splice ein bestehendes Array verändert könntest du die hinteren (nicht benötigten) Elemente mit this.splice(num, this.length) abschneiden.

[hgseib]

x0 und x1 sind recht anonyme bezeichnungen. eine sinnvolle namensgebung verringert die fehlermöglichkeiten und erspahrt unnötige beschreibungen ;-)
wobei eine beschreibungen kein fehler ist.

// gibt anzahl zufallszahlen zurück, im zahlenwert von..bis
Array.prototype.fillRandomNum = function(von, bis, anzahl) {
	while (von<=bis) {
		this.push(von++);
	}
	this.sort(function () {
		return (random(3)-1);
	});
	if (anzahl && anzahl <this.length) {
		this.length = anzahl;
	}
};
//
theArray = [];
theArray.fillRandomNum(5, 49, 6);
trace(theArray.join(','));

[tomez]

Danke Jungs!

@Janoscharlipp: okay, da hätte ich eigentlich drauf kommen können Danke für den Hinweis!
@hgseib: die namensgebung hab ich auch geändert - muss aber leider immer alles in englisch halten

Array.prototype.fillRandomNum = function(from, to, num) {
	for (var i = from; i <= to; i++) {
		this.push(i);
	}
	this.sort(function () {
		return (random(3) - 1);
	});
	this.splice((num ? num : this.length), this.length);
};

...so funzt das nun wunderbar!