Partage
  • Partager sur Facebook
  • Partager sur Twitter

fonction generation aleatoire labyrinthe

plutot urgent

4 janvier 2024 à 1:25:20

bonjour,ceci est une fonction que j'utilise pour generer un labyrinthe aleatoire 

laby* creation(int p, int q) {
    srand(time(0));


    laby* L = new laby;

    init(*L, p, q);

    pile P;

    int taillePile = L->p * L->q * 2;

    init(P, taillePile);

    couple c = {0, 0};

    empiler(P, c);

    cout << "Entering creation loop" << endl;

    while (!vide(P)) {

        c = depiler(P);

        L->tab[c.i][c.j].etat = true;

        couple T[4];

        int nbe = 0;

        if (c.i > 0 && !L->tab[c.i][c.j].N) {

            T[nbe++] = {c.i - 1, c.j};

        }

        if (c.j > 0 && !L->tab[c.i][c.j].W) {

            T[nbe++] = {c.i, c.j - 1};

        }

        if (c.i < L->p - 1 && !L->tab[c.i][c.j].S) {

            T[nbe++] = {c.i + 1, c.j};

        }

        if (c.j < L->q - 1 && !L->tab[c.i][c.j].E) {

            T[nbe++] = {c.i, c.j + 1};

        }

        if (nbe > 0) {

            int d = rand() % nbe;

            couple cprime = T[d];

            char dep = deterDirec(c, cprime);

            open(*L, c, cprime, dep);

            L->tab[cprime.i][cprime.j].etat = true;

            empiler(P, cprime);

            empiler(P, c);

            cout << "Random value: " << d << endl;

        }

    }

    return L;

}



cependant l'execusion du code ne mene pas a la generation aleatoire
pour L1 = creation ( L1->p ,L1->q) tout les portes sont ouverte et pour creation ( L1->p ,L1->q) le meme labyrinte ce genere
  je voudrais donc avoir de l'aide sur cela.merci.(je tien a precicer que normalement la pile,la fonction deplacement,direction,init,couple ect fonctionnent normalement)
voici le main de la fonction
int main() {
    laby* L1 = lire_Laby("laby.txt");
    creation(L1->p, L1->q);
    ecrire_laby("laby_copy.txt", *L1);
    InitCurses();
    Show_Lab(*L1);
    deplacement(*L1);
    int ch = wgetch(stdscr);
    for (int i = 0; i < L1->p; ++i) {
    delete[] L1->tab[i];}
    delete[] L1->tab;
    EndCurses();
    return 0;
}

-
Edité par FeMo1 4 janvier 2024 à 10:38:03

  • Partager sur Facebook
  • Partager sur Twitter
4 janvier 2024 à 1:44:48

Salut,

Je passe outre la non utilisation des balises code pour poster les sources (ou pas).

Ton code est difficile à comprendre, faute de noms de variables parlantes (en d'autre termes: Grande paresse dans le choix des noms).
Si "laby" est une classe, c'est à son constructeur de génrer le labyrinthe et personne d'autre !!
On fait du C++ moderne, tu n'as aucune raison d'utiliser des pointeurs nu.

Bref, reprend ton code, sort toi les doigts du ... et fournit un code moderne, correcte, lisible et compréhensible.

  • Partager sur Facebook
  • Partager sur Twitter
4 janvier 2024 à 4:09:26

Bonjour, Merci d'indiquer un titre de sujet en rapport avec votre problématique et d'insérer votre code avec le bouton code </> de la barre d'outil du forum.

Le message qui suit est une réponse automatique activée par un membre de l'équipe de modération. Les réponses automatiques leur permettent d'éviter d'avoir à répéter de nombreuses fois la même chose, ce qui leur fait gagner du temps et leur permet de s'occuper des sujets qui méritent plus d'attention.
Nous sommes néanmoins ouverts et si vous avez une question ou une remarque, n'hésitez pas à contacter la personne en question par Message Privé.

Pour plus d'informations, nous vous invitons à lire les règles générales du forum

Merci de colorer votre code à l'aide du bouton Code </>

Les forums d'Openclassrooms disposent d'une fonctionnalité permettant de colorer et mettre en forme les codes source afin de les rendre plus lisibles et faciles à manipuler par les intervenants. Pour cela, il faut utiliser le bouton  </> de l'éditeur, choisir un des langages proposés et coller votre code dans la zone prévue. Si vous utilisez l'éditeur de messages en mode Markdown, il faut utiliser les balises <pre class="brush: php;">Votre code ici</pre>.

Merci de modifier votre message d'origine en fonction.


Mauvais titre

Le titre est un élément important qui ne doit pas être négligé. N'oubliez pas cette règle simple : le titre idéal résume la question que vous allez poser en une petite phrase. Il doit permettre aux visiteurs de se repérer facilement dans le forum visité et d'identifier le sujet à sa seule lecture.

Vous pouvez utiliser divers préfixes comme [Erreur], [MySQL], [Compatibilité], etc... Aussi, pensez à consulter les règles propres à chaque forum (visibles dans les topics épinglés en haut des sections).

De plus, choisir un bon titre permet de rendre plus faciles les recherches des autres membres.

Les titres de type "besoin d'aide" ou "problème" ne sont pas tolérés.

Merci de modifier votre titre. Pour cela, éditez le premier message de votre sujet.

(titre originel : probleme labyrinthe,besoin d'aide)

Liens conseillés

  • Partager sur Facebook
  • Partager sur Twitter
4 janvier 2024 à 4:39:20

On va éviter l'erreur fondamentale d'attribution, @Deedolith, Ok ? ;-)

@FeMo1, ce genre de code pouvait être "tolérable" il y a 30 ans, mais clairement, si c'est votre source d'apprentissage qui vous montre cette manière de coder, changez s'en très rapidement. Vous perdez votre temps sinon.

-
Edité par bacelar 4 janvier 2024 à 4:39:39

  • Partager sur Facebook
  • Partager sur Twitter
Je recherche un CDI/CDD/mission freelance comme Architecte Logiciel/ Expert Technique sur technologies Microsoft.
4 janvier 2024 à 9:27:49

Deedolith a écrit:

Salut,

Je passe outre la non utilisation des balises code pour poster les sources (ou pas).

Ton code est difficile à comprendre, faute de noms de variables parlantes (en d'autre termes: Grande paresse dans le choix des noms).
Si "laby" est une classe, c'est à son constructeur de génrer le labyrinthe et personne d'autre !!
On fait du C++ moderne, tu n'as aucune raison d'utiliser des pointeurs nu.

Bref, reprend ton code, sort toi les doigts du ... et fournit un code moderne, correcte, lisible et compréhensible.


bonjour.ceci n'est pas le code complet evidement car plusieur autre fichier et fonction sont  inclus.de deux je ne comprend pas de quoi vous parler avec "variable parlante" et de quoi vous parler quand vous dite je ne fait pas du C++ moderne,je ne suis qu'en premiere anné de license informatique donc soyer comprehensive si mon code n'a pas l'air """"""moderne"""""".je fait juste ce que on 'm'a appris.comprener que je suis en premiere année,pas tres forte en c++,utilise chat gpt pour comprehndre,et je base ce que fait sur ce que l'on ma appris

-
Edité par FeMo1 4 janvier 2024 à 9:35:53

  • Partager sur Facebook
  • Partager sur Twitter
4 janvier 2024 à 9:36:05

Alors donnez un coup de pied au cul de votre prof., de notre part.

(ou vous avez complètement compris de travers son enseignement ?)

Il a quel âge votre prof. ? 90 ans, sans se remettre à jour depuis 30 ans (et il y a 30 ans, c'était déjà très très mauvais) ?

Si votre prof. est une buse, n'en devenez pas une, vous avez Internet pour apprendre, vous.

  • Partager sur Facebook
  • Partager sur Twitter
Je recherche un CDI/CDD/mission freelance comme Architecte Logiciel/ Expert Technique sur technologies Microsoft.
4 janvier 2024 à 10:14:08

bacelar a écrit:

Alors donnez un coup de pied au cul de votre prof., de notre part.

(ou vous avez complètement compris de travers son enseignement ?)

Il a quel âge votre prof. ? 90 ans, sans se remettre à jour depuis 30 ans (et il y a 30 ans, c'était déjà très très mauvais) ?

Si votre prof. est une buse, n'en devenez pas une, vous avez Internet pour apprendre, vous.


bon esceque je peut quand meme avoir de l'aide sur cela,sans vous offencer c'est bien beau d'insulter mon code ou mon prof mais c'est pas cela qui reparera le code.je suis justement sur ce forum pour comprhendre ou est le probleme.
  • Partager sur Facebook
  • Partager sur Twitter
4 janvier 2024 à 10:14:28

@FeMo1 Bonjour, mon message plus haut n'est pas la pour faire joli, merci de modifier votre titre et d'en profiter pour rendre votre code plus lisible sur le forum.

La modération

  • Partager sur Facebook
  • Partager sur Twitter
4 janvier 2024 à 10:16:28

AbcAbc6 a écrit:

@FeMo1 Bonjour, mon message plus haut n'est pas la pour faire joli, merci de modifier votre titre et d'en profiter pour rendre votre code plus lisible sur le forum.

La modération


oh mes excuse,comprehensible je vais le changer dés maintenant.desoler pour toute inconveniance
  • Partager sur Facebook
  • Partager sur Twitter
4 janvier 2024 à 12:39:29

Bonjour, je retire des spams ce sujet, le forum est un poil chatouilleux quand on édite plusieurs fois son message surtout pour les nouveaux inscrit.

Si cela arrive encore vous pouvez poster dans ce sujet Si votre message est considéré comme spam

Comme vous avez créé un nouveau sujet, on continue la discussion sur l'autre et je ferme celui-ci. 

Doublon

Les doublons nuisent au bon fonctionnement du forum et sont donc interdits. Si vous vous êtes trompé de section, il suffit de signaler votre sujet au staff pour qu'il le déplace au bon endroit.

Je vous invite à continuer la discussion sur l'autre sujet : https://openclassrooms.com/forum/sujet/fonction-generation-aleatoire-labyrinthe-39375

-
Edité par AbcAbc6 4 janvier 2024 à 12:40:48

  • Partager sur Facebook
  • Partager sur Twitter