[RPG-MAKER.FR] Oniromancie: tout l'univers de RPG Maker en français - Les forums

Bien ouej

C'est plus user-friendly que ce que j'aurais fait (j'aurais juste eu un .txt ou tu configures ce que tu veux puis tu lances un .exe sans interface graphique). C'est quand même chelou le temps d'execution, je me serais attendu à quelque chose de l'ordre de quelques secondes. T'as essayer de detecter quelle partie cause tant de soucis?

Je connais pas le JS et je trouve ça illisible mais en survolant j'ai l'impression qu'il y a des endroits de ton code qui peuvent être optimisés très facilement:

Portion de code : Tout sélectionner

 
for (let j=0; j<nbreVerticChunks; j++) {
        console.log('Preparing file. Reading line '+(j+1)+'/'+(nbreVerticChunks)+'...')
        for (let i=0; i<nbreHorizChunks; i++) {
          const chunk = await Jimp.read(map)

Là on dirait que tu recharges l'image à chaque passage de la boucle au lieu de le faire une seule fois en dehors de la boucle.

Portion de code : Tout sélectionner

let repeatingPixels;
 
          if (chunksImgs.length > 0) {
            for (let k=0; k<chunksImgs.length; k++) {
              repeatingPixels = 0;
              if (chunk[1] === chunksHash[k]) {
                for (let m=0; m<chunckWidth; m++) {
                  for (let n=0; n<chunkHeight; n++) {
                    if (chunk[0].getPixelColor(m, n) === chunksImgs[k].getPixelColor(m, n)) {
                      repeatingPixels++;
                    }
                  }
                }
              }
              if (repeatingPixels === chunckWidth*chunkHeight) {
                break;
              }
            }
          }

Là tu comptes le nombre de pixels identiques alors que tu peux t'arrêter dès que tu en trouves deux différents. Et puis tu compares le hash de l'image avec tous les hash precedements créés. Faut utiliser un conteneur plus approprié (une table de hashage justement) ! Il doit bien y avoir une implémentation d'une table de hashage (ou d'un autre type de conteneur qui permet de tester si un objet est déjà dedans) en JS ? Là tu perds la moitié des avantages du hashage.

Et tu fais ces mêmes trucs plus loin dans ton code.

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71

package main import ( "fmt" "image" "image/draw" "image/png" "os" ) func readImage(filePath string) (*image.RGBA, error) { f, err := os.Open(filePath) if err != nil { return nil, err } defer f.Close() img, _, err := image.Decode(f) b := img.Bounds() rgba_img := image.NewRGBA(image.Rect(0, 0, b.Dx(), b.Dy())) draw.Draw(rgba_img, rgba_img.Bounds(), img, b.Min, draw.Src) return rgba_img, err } func cropImage(img *image.RGBA, crop image.Rectangle) *image.RGBA { simg := img.SubImage(crop) b := simg.Bounds() rgba_img := image.NewRGBA(image.Rect(0, 0, b.Dx(), b.Dy())) draw.Draw(rgba_img, rgba_img.Bounds(), simg, b.Min, draw.Src) return rgba_img } func writeImage(img image.Image, name string) error { fd, err := os.Create(name) if err != nil { return err } defer fd.Close() return png.Encode(fd, img) } func main() { complete_img, err := readImage("azdu.png") if err != nil { os.Exit(1) } bounds := complete_img.Bounds() pixel_width := bounds.Dx() pixel_height := bounds.Dy() tiles := make(map[[4 * 16 * 16]uint8]struct{}) count := 0 for x := 0; x < pixel_width; x += 16 { for y := 0; y < pixel_height; y += 16 { simg := cropImage(complete_img, image.Rect(x, y, x+16, y+16)) var pix_array [4 * 16 * 16]uint8 copy(pix_array[:], simg.Pix) if _, ok := tiles[pix_array]; !ok { tiles[pix_array] = struct{}{} count += 1 } } } fmt.Printf("count %d\n", count) }