Генерировать случайные объекты в Java

Question

im делает обзор производительности графических баз данных в контексте моделирования процессов. Поэтому я установил простой тестовый файл, который отслеживает производительность простых вставок во встроенную базу данных N4J. Это отрывок из моего теста.

@Test 
    public void insertQuants(){ 
     int size =750; 
     /** 
     * create 10 different languages and store them in the ArrayList 
     */ 
     ArrayList<Language> lngs = new ArrayList<Language>(); 
     for(int i=0; i<10; i++){ 
      String title = "Testsprache " + String.valueOf(i); 
      String description = "Beschreibung " +String.valueOf(i); 
      lngs.add(modelservice.createLanguage(title,description)); 
     }   
     /** 
     * Create different models and assign them to different 
     * language objects 
     */ 
     for(int i=0; i<size; i++){ 
      String title = "Testmodel " + String.valueOf(i); 
      //get random between 0 and 9 
      int fin = (int) Math.round(Math.random()*9); 
      //fetch random Language object out of ArrayList 
      Language l = lngs.get(fin); 
      Model m = modelservice.createModel(l, title); 
      l.addModel(m); 
      template.save(l); 
     } 

методы createModel в modelservice возвращают определенный созданный объект (модель или язык). Проблема, с которой я сейчас сталкиваюсь, заключается в том, что случайное присвоение языков моделям не работает. Скрипт присваивает каждой модели одному языковому объекту. На мой взгляд, это скорее должно случайным образом привязывать языки к модели.

Answer 1

Проблема не с рандомизации:

int fin = (int) Math.round(Math.random()*9); 

Math.random() возвращается в диапазоне double: [0,1), умножая на 9 дает: [0,9), и округление дает [0,9]. Как CPerkins указал, используя Random может выглядеть проще и, вероятно, лучше (поскольку вероятность генерации чисел с 0 по 9 неравномерна вашему методу), но ваш диапазон генерации случайных чисел правильный. Проблема не в том коде, который вы нам показали. Возможно, он находится в addModel() или save().

Answer 2

Это не ваша проблема, но это a проблема: Math.random возвращает double между 0 и 1. Умножение на 9 и округление не дает вам справедливого распределения (ноль и девять будут отображаться только вдвое меньше, чем вам хотелось бы). Изменение:

int fin = (int) Math.round(Math.random()*9); 

к чему-то больше похоже на

Random randomGenerator = new Random(); 
... 
int fin = randomGenerator.nextInt(10);