Placement de bloc aléatoire

[blueblasterz]

Bonjour/Bonsoir à tous.

Aujourd'hui je m'en remet à la communauté skript pour m'aider sur l'un de mes projet, le skript Random Zone.

Voici mon problème : Je cherche à ajouter un moyen simple de modifier le nombre de blocs de chaque type devant apparaître, sous forme de pourcentage. Par exemple, la commande /rz apply 70%stone,20%coal ore, 10%iron ore doit signifier "on rempli 70% de la zone avec de la stone, 20% avec du minerai de charbon, et 10% avec du minerai de fer". Et en fait mon problème ne se situe pas là. La commande est déjà prête, les pourcentages sont pris en compte, mais voici 4 résultats sur un carré de 10*10 avec cette commande:

[ATTACH=full]3192[/ATTACH] [ATTACH=full]3193[/ATTACH] [ATTACH=full]3194[/ATTACH] [ATTACH=full]3195[/ATTACH]

Vous voyez bien le soucis, les bloc ayant un petit pourcentage apparraîssent tout en bas, ratatinés.

Voici le code concerné (si vous voulez le reste allez voir sur le post de la ressource):

loop {_randfloor.blocks::*}:
          if "%loop-value%" does not contain "%%": # si on a pas de pourcentage dans cet élément
            message "{@logo} &4Erreur : &cVeuillez spécifier un pourcentage pour tout les blocs, ou bien ne spécifiez &naucun&c pourcentage"
            message "{@logo} &cproblème : ""%loop-value%"" "
            stop trigger     
          add "%loop-value%" to {_blocks::*} # on add la valeur à une liste
        set {_verifPourc} to 0 # déclaration en dehors de la loop (vaut pour toutes les instructions de ce type
        loop {_blocks::*}:
          set {_temp::*} to loop-value split by "%%" # on casse les info en 2, pour avoir d'un coté le pourcentage, de l'autre le bloc
          if {_temp::1} parsed as number is smaller than 1: # autrement dit 0 ou moins
            message "{@logo}&4 Erreur&c : Vous ne pouvez pas mettre de pourcentage négatif ou nul:"
            message "{@logo}&c %{_temp::1}%%%%{_temp::2}%"
            stop trigger
          add "%{_temp::1}%|%{_temp::2}%" to {_blocks1::*} # Cette instruction n'est pas obligatoire, mais elle permet de me rappeler qu'on a finis une étape
          add {_temp::1} parsed as number to {_verifPourc} # au passage on ajoute tout les pourcentages

        if {_verifPourc} is smaller than 100: # pour vérifier que le total est bien de 100
          message "{@logo}&4 Erreur&c : Vous n'avez indiqué que %{_verifPourc}%%% des blocs devant être posés !"
          message "{@logo}&c Ce n'est pas assez -> revoyez vos pourcentages"
          stop trigger
        if {_verifPourc} is greater than 100:
          message "{@logo}&4 Erreur&c : Vous avez indiqué %{_verifPourc}%%% de blocs deant être posés !"
          message "{@logo}&c C'est trop -> revoyez vos pourcentages"
          stop trigger


        set {_placeP} to 0
        set {_blocks2::*} to "" and ""
        loop {_blocks1::*}:
          add 1 to {_placeP}
          # message "%loop-value%"
          set {_temp::*} to loop-value split by "|"
          set {_nbBlPos} to {_tailleZone} / 100 * {_temp::1} parsed as number # on transforme le pourcentage en nombre de blocs avec ce calcul
          # message "%{_nbBlPos}% blocs de %{_temp::2}%"
          set {_blocks2::%{_placeP}%} to "%{_nbBlPos}%|%{_temp::2}%"# et on re compresse les valeurs

        loop blocks within {_loc1} to {_loc2}: # début de la partie qui ne marche pas bien
          set {_blockNum} to random integer between {_nbBlock}-({_nbBlock}-1) and {_nbBlock} # random number between 1 and {_nbBlock} # C'est ce choix de nombre qui fait que  les blocs sont "écrasés" en bas du carré (j'explique plus bas)
          set {_temp::*} to {_blocks2::%{_blockNum}%} split by "|" # on sépare -> en ::1 le nb de blocs, en ::2 le bloc

          while {_temp::1} parsed as number is 0: #si le nb de bloc est tombé à zéro (on a "utilisé" tout es bloc dispo
            set {_blockNum} to random integer between {_nbBlock}-({_nbBlock}-1) and {_nbBlock} # random number between 1 and {_nbBlock} # alors on en choisi un autre
            set {_temp::*} to {_blocks2::%{_blockNum}%} split by "|"

          set {_temp::1} to {_temp::1} parsed as number - 1 # pour indiquer qu'on en a bien posé 1
          set {_blocks2::%{_blockNum}%} to "%{_temp::1}%|%{_temp::2}%" # on update la variable
          set block at location of loop-block to "%{_temp::2}%" parsed as item # et on pose le bloc

Voici les explication (de là ou j'ai mis que j'expliquai plus bas): en gros quelque soit le nombre de bloc de chaque type, le code va :

- compter combien il y a de type de bloc,

- attribuer à chacun de ces blocs un nombre, qui correspond à leur pourcentage d'apparition

- un fois ceci fait, lance une boucle qui fait défiler tous les blocs de la zone,

- pour chaque loop-block, il choisis un type de bloc aléatoire, et le pose si son nombre d'apparition n'est pas nul.

 

Le soucis c'est que le choix de ce bloc aléatoire n'est pas pondéré selon le nombre d'apparition : c'est donc logique que les blocs ayant une faible apparition apparraîssent en bas du carré.

Ma question est donc, avez vous une idée de comment régler ce soucis, et faire apparaître régulièrement tout les blocs ??

Et merci d'avance à ceux qui ont eu le courage de lire mon pavé, et encore plus merci à ceux qui m'aideront :)

[EdwinnSs]

Bonjour,

 

Avant de te répondre, sache que tu pourrais utiliser la commande WorldEdit qui existe déjà pour cela :)

 

Mais sinon, c'est un sujet assez intéressant.

 

Déjà tu as un gros problème : tous les block pris en compte dans le pourcentage d'apparition on la même chance d'apparaître au début de ton skript, ce qui est donc totalement faux d'un point de vue probabilités.

 

Je m'explique : au tout début tu as 4 types de blocks possible. Tu vas faire un random entre 1 et 4 ... chacun des block à donc 25℅ de chance d'apparaître ! Et ce n'est pas ce que tu veux.

Si les blocks ont respectivement 70%, 20%, 5% et 5% de chance d'apparaître il faut que ce soit le cas dès le tirage au début.

Tu dois donc modifier ton algorithme. Pour que ce soit déjà plus juste, il faut qu'a chaque tirage de block tu y appliques la probabilité de chaque block d'être tiré.

 

Bonne chance :)

[blueblasterz]

Le but est de ne pas utiliser World Edit ;)

Je suis bien conscient du problème de ce pourcentage d'apparition, j'ai même déjà fais le skript que tu me dis avec un "chance of %le pourcentage%:"

Le problème est visible en regardant les résultat avec, disons ne serai-ce que 2 blocs, l'un avec 70% et l'autre avec 30%, en inversant le 30% et le 70%:

 

 

[ATTACH=full]3198[/ATTACH]

 

[ATTACH=full]3199[/ATTACH]

Avec ce code:

 

loop {_randfloor.blocks::*}:
          add loop-value split by "%%" to {_blocksFinal::*}

        loop blocks within {_loc1} to {_loc2}:
          # message "---"
          set {_verifSet} to 0

          while {_verifSet} is not equal to 1:
            set {_place} to 0
            loop {_nbBlock} times:
              add 1 to {_place}
              set {_placeB} to {_place} + 1
              set {_chance} to "%{_blocksFinal::%{_place}%}%"
              chance of {_chance} parsed as number%:
                set {_finalBlock} to "%{_blocksFinal::%{_placeB}%}%"
                set block at location of loop-block-1 to {_finalBlock} parsed as item
                # message "%{_finalBlock}%"
                set {_verifSet} to 1
              add 1 to {_place}

On voit bien que le résultat est différent, même si je ne saisi pas tout à fait pourquoi.

Merci de ton aide, tout de même :)

[EdwinnSs]

En effet, je vais te détailler l'algo que je pensais .

Le but et de transformer le carré en matrice.

 

Je ferais dans cet ordre :

- enregistrer les coordonnées de chaque blocks de la zone dans une liste

- en même temps, compter le nombre de block total

- tu enregistres le pourcentages de blocks correspondants au total des block (ex: 70% blue = 140 blocks, 30% red = 60 blocks)

- tu prends un nombre aléatoire dans ta liste de tes coordonnées entre 1 et la taille de ta liste (200)

- tu le met dans la catégorie d'un des totaux (ici le choix importe peu), par exemple au 140 blocks donc ce sera un blue

- tu enlèves donc un block du total (tu auras 139 blocks blue à poser)

- tu enlèves la coordonnée de la liste, et sa taille passe à 199

 

Tu auras donc exactement le nombre de block que tu veux à un emplacement aléatoire.

J'espère que c'est plus clair (ou pas) xD

 

PS : bonne chance pour gérer la liste ;)

[minouis55]

J'ai peut-être une idée pour t'aider:

 

- Pour chaque pourcentage tu fais ceci:

loop (ton pourcentage) times:

add (item du pourcentage correspondant) to {_liste::*}

 

- Puis lorsque tu places tes blocs, pour le choix de l'item, tu fais ça:

set {_item} to random string from {_liste::*}

 

Je ne sais pas si tu comprend ce que je dis ;) Mais ducoup meme si le totale des pourcentage n'est pas égal à 100, ca marchera :)

[blueblasterz]

@EdwinnSs et @minouis55, merci de votre aide à tout les 2, je vais tester vos solutions ^^

 

EDIT : la solution de @minouis55 semble marcher, son seul inconvénient est qu'elle n'est pas très précise ^^ Par exemple si je met 1%de bloc de diamant et 99% de pierre sur une zone de 100 blocs, je peut me retrouver avec 2 ou 3 bloc de diamant ou même aucun ^^ néanmoins elle nécessite beaucoup moins de travail que tout ce que j'avais écrit jusqu'alors. Je teste quand même la solution de EdwinnSs et je reviens ^^

 

EDIT 2 : Je testerai cette 2eme solution demain ^^

[EdwinnSs]

- Puis lorsque tu places tes blocs, pour le choix de l'item, tu fais ça:

set {_item} to random string from {_liste::*}

Dans ce cas il faut faire en sorte de ne pas prendre deux fois le même block, sinon ça fausse ^^

[blueblasterz]

Pour ta solution @EdwinnSs, je me confronte à un problème au niveau de cette instruction :

- tu enlèves la coordonnée de la liste, et sa taille passe à 199

Mais après quelques tests ratés, j'ai testé ce petit code :

on right click:
  set {_temp::*} to "1" and "2" and "3" and "4" and "5"
  message "%{_temp::*}%"
  remove {_temp::2} from {_temp::*}
  message "%{_temp::*}%"
  message "%{_temp::1}%,%{_temp::2}%,%{_temp::3}%,%{_temp::4}%,%{_temp::5}%"

Et e, voici le résultat :

 

[ATTACH=full]3213[/ATTACH]

Skript considère apparemment que la variable garde la même taille. Donc évidemment ton code ne marche pas (du moins de la manière dont je l'ai appliqué). Le seul moyen que je verrai pour résoudre ce problème serai de loop la variable et de l'ajouter toutes ces valeurs à une nouvelle variable, sauf celles qui sont "<none>". Mais dans ce cas bonjour les lags...

Je te passe le code que j'ai fais selon tes instructions, dis moi si tu vois quelque chose de mal fait :

set {_placeP} to 0
        set {_blocks2::*} to "" and ""
        loop {_blocks1::*}:
          add 1 to {_placeP}
          # message "%loop-value%"
          set {_temp::*} to loop-value split by "|"
          set {_nbBlPos} to {_tailleZone} / 100 * {_temp::1} parsed as number # on transforme le pourcentage en nombre de blocs avec ce calcul
          # message "%{_nbBlPos}% blocs de %{_temp::2}%"
          set {_blocks2::%{_placeP}%} to "%{_nbBlPos}%|%{_temp::2}%"# et on re compresse les valeurs
          # message "loop-value: %loop-value%"
          # message "bloc %{_temp::2}% -> %{_nbBlPos}% apparitions"


        set {_loop} to {_tailleZone}
        set {_pos2} to 0
        set {_tailleReste} to {_tailleZone}
        loop {_loop} times:
          # message ""
          set {_pos2} to 0
          set {_pos} to a random integer between 1 and {_tailleReste}
          # message "pos : %{_pos}%"
          add 1 to {_pos2}
          # message "pos2 : %{_pos2}%"
          set {_temp::*} to {_blocks2::%{_pos2}%} split by "|"
          # message "blocks2::pos2 -> %{_blocks2::%{_pos2}%}%"

          while {_temp::1} is "0":
            # message "temp::1 dans while: %{_temp::1}%"
            set {_pos2} to {_pos2} + 1
            set {_temp::*} to {_blocks2::%{_pos2}%} split by "|"
            # message "block2 dans while : %{_blocks2::%{_pos2}%}%"

          set block at {_posZone::%{_pos}%} to {_temp::2} parsed as item
          # message "set block at %{_posZone::%{_pos}%}% to %{_temp::2}%"

          set {_blocks2::%{_pos2}%} to "%{_temp::1} parsed as number - 1%|%{_temp::2}%"
          # message "blocks 2 après : %{_blocks2::%{_pos2}%}%"
          remove {_posZone::%{_pos}%} from {_posZone::*}
          set {_tailleReste} to {_tailleReste} - 1
          # message "taille restante : %{_tailleReste}%"

Le code n'est pas complet, la variable {_blocks1::*} est de la forme "<pourcentage>%|<bloc>".

Voilà, merci de ton aide, et puis si on arrive pas à corriger ça je prendrai la solution de minouis55, qui marche quand même très bien ^^