[JS] Losowy element tablicy Opcje

[Dominator]

Czemu nie działa?

[PHP] pobierz, plaintext 
var messages = new Array();
var messages[1]="Ada";
var messages[2]="Adela";
var messages[3]="Adelajda";
var messages[4]="Adriana";
var messages[5]="Adrianna";
var messages[6]="Agata";
var messages[7]="Agnieszka";
var messages[8]="Agrypina";
 
function getMessage() {
   return messages[Math.floor(Math.random() * messages.length)];
}
[PHP] pobierz, plaintext 

[Szymciosek]

Kod

var messages = new Array();
messages[1]="Ada";
messages[2]="Adela";
messages[3]="Adelajda";
messages[4]="Adriana";
messages[5]="Adrianna";
messages[6]="Agata";
messages[7]="Agnieszka";
messages[8]="Agrypina";


console.log(messages[Math.floor(Math.random() * messages.length)]);

Sprawdzałem to sobie w konsoli, dlatego lekko przerobione

[PrinceOfPersia]

Czemu nie działa?

bo deklarujesz kilka razy tę samą zmienną (jedno var powinno być, a nie kilka). Poza tym - indeksy tablic w JS (jak i wielu innych językach) zaczynają się od zera, a nie od jedynki.

[Majcon]

Kod

var messages = new Array('Ada', 'Adela', 'Adelajda', 'Adrianna', 'Agata', 'Agnieszka', 'Agrypina');

[PrinceOfPersia]

ano. Albo jeszcze prościej, wywalmy to Array:

Kod

var messages = ['Ada', 'Adela', 'Adelajda', 'Adrianna', 'Agata', 'Agnieszka', 'Agrypina'];

[pyro]

[HTML] pobierz, plaintext 
function getRandomInt(min, max) {
  return Math.floor(Math.random() * (max - min + 1)) + min;
}
 
var randomIndex =  getRandomInt(0, (messages.length-1));
 
console.log(messages[randomIndex]);
[HTML] pobierz, plaintext