Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add some changes #42

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add some changes #42

wants to merge 3 commits into from

Conversation

yagatastudio
Copy link
Contributor

Ajout du zoom/dézoom sur la voiture.
Ajout centrage caméra sur la voiture.
Ajout changement de voiture aléatoire et centrage sur la nouvelle.
Ajout sauvegarde mais pas restauration.

PS: Pour la sauvegarde j'ai utilisé newtonjson avec une class savePrefab qui stock toutes les informations.
Le seul problème c'est lorsque je dois recréer l'objet savePrefab depuis le fichier json, pour directement passer les informations au panel, j'ai une erreur de constructeur voiture.

Ajout du zoom sur la voiture courante avec les touches F2 et F3.
Ajout du changement de voiture aléatoire avec la touche F1.
Création de l'objet savePrefab pour l'enregistrement fonctionnelle mais recréation de celui-ci depuis le fichier save.json impossible.
@@ -51,7 +51,7 @@ public void EditionRoute(Orientation orientation)
default:
break;
}
Route.DictionaireObstacles = dico;
Route.DictionnaireObstacles = dico;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 !

case Orientation.OUEST:
return Orientation.EST;
}
return Orientation.NORD;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

si Orientation.SUD est (aussi) le default, vous gagnerez en clareté.

:base(rectangle, v, vMax)
{

}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

est-ce vraiment requis de redéfinir chaque constructeur?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants