Why is this laggin/crashing my server.

Is it because when i have 1 location i spawn 2 stacks of bottles at once? Or because of a leak or something.

Code:

            List<Player> onlinePlayers = new ArrayList<Player>();
            for(Player p : Bukkit.getOnlinePlayers()){
                onlinePlayers.add(p);
            }
            int amountXpOrbs = (128 * onlinePlayers.size()) / main.locations.size();
            String world = player.getWorld().getName();
            for(Location l : main.locations){
                for(int x = 0; x < amountXpOrbs; x =+1){
                    Bukkit.getWorld(world).spawn(l, ExperienceOrb.class);
                }
            }

 

Maybe because you're spawning 128 experience orbs per location, per player. Not to mention adding all the online players to a new collection for no reason. As well as retrieving the world by name for every single orb when you already have a reference available via player.getWorld()