Pick a random string from an array

Question

So I'm having a brain fart at the moment and can't figure out how to finish this part of the code:

String[] diceNumbers = {"1", "2", "3", "4", "5", "6"};
String pickedNumber = (diceNumbers[new Random()]);

I saw this on another StackedOverflow past but I can't find it again.

I want it to be stored as a String variable so I can add it into another line of code because I'm using it in a Minecraft plugin

Bukkit.broadcastMessage(p.getName() + "has rolled a" + pickedNumber);

is what I am trying to do

The random number must be in the range of 0 to array length - 1; Now you can search for that. — Scary Wombat, Jul 03 '19 at 05:54
@ScaryWombat yes. That is what I have been doing so it get a random number within the length of the array Edit: Zheka just answered it :) Thanks for the help — secon19665, Jul 03 '19 at 05:55
Why don't you do this: make random number like this `r.nextInt(5) + 1` to make a random number between 1 and 6. and put it directly in your string — Pouria Moosavi, Jul 03 '19 at 05:56
"_That is what I have been doing so it get a random number within the length of the array_", so you had a solution from the start ? — AxelH, Jul 03 '19 at 06:00
@AxelH Techinally yes but I completely forgot how to do it and couldn't find the answer anywhere — secon19665, Jul 03 '19 at 06:06
@ScaryWombat if you are going to mark a post as a duplicate, you should seriously provide the link to the original post — secon19665, Jul 03 '19 at 06:08
@secon19665 If you are going to criticize me, you should seriously look at the link at the top of this page which links to the original post. Strange eh? — Scary Wombat, Jul 03 '19 at 06:11

score 0 · Accepted Answer · answered Jul 03 '19 at 05:55

0

String pickedNumber = diceNumbers[ThreadLocalRandom.current().nextInt(diceNumbers.length)];

answered Jul 03 '19 at 05:55

ZhekaKozlov

It would be nice to explain why this would be a solution. – AxelH Jul 03 '19 at 05:57
This just answered my question. I have been using ```String pickedNumber = (diceNumbers[new Random().nextInt(diceNumbers.length)]);``` – secon19665 Jul 03 '19 at 05:57
@secon19665 This is semantically the same but `ThreadLocalRandom` is generally faster than creating a `Random` instance every time – ZhekaKozlov Jul 03 '19 at 06:04
@ZhekaKozlov I'm only using it once in the plugin that I am making so it doesn't fully matter to me. I don't see why people are downvoting your answer because it answers my question haha. I will be accepting your answer! – secon19665 Jul 03 '19 at 06:05

1 Answers1