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

Remplir struct en fonction du fichier #1

Closed
Hagbuck opened this issue Oct 13, 2017 · 8 comments
Closed

Remplir struct en fonction du fichier #1

Hagbuck opened this issue Oct 13, 2017 · 8 comments
Assignees
Labels
Milestone

Comments

@Hagbuck
Copy link
Owner

Hagbuck commented Oct 13, 2017

En lecture d'un fichier.

  • Vérifier si il s'agit bien d'un P2
  • Récupérer les informations et les mettre dans la structure
@Hagbuck Hagbuck added the To Do label Oct 13, 2017
@Hagbuck Hagbuck added this to the PGM milestone Oct 13, 2017
@Hagbuck Hagbuck self-assigned this Oct 13, 2017
@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 13, 2017

La fonctionPGM_image* PGM_get_image_from_file(FILE * file) remplit la structure PGM_image. Elle utilise plusieurs fonctions intermédiaires permettant d'éviter la réécriture du code.

Il faut penser à commenter le fonctionnement des fonctions

@Hagbuck Hagbuck added Done and removed To Do labels Oct 13, 2017
@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 13, 2017

Commentaires ajoutés

@Hagbuck Hagbuck added Doing and removed Done labels Oct 17, 2017
@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 17, 2017

Notre méthode fonctionne bien pour un petit fichier, mais un fichier de 512*512 ne fonctionne pas (http://people.sc.fsu.edu/~jburkardt/data/pgma/lena.png)

Il faut considérer que les les lignes ne doivent pas dépasser 70 caractères ( et cela dans TOUT le programme).

Il faut également comprendre où notre méthode échoue et comment corriger le tir.

@Hagbuck Hagbuck added the PRIO label Oct 17, 2017
@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 17, 2017

On lit actuellement ligne par ligne, sauf que une ligne du fichier ne correspond pas forcement a une ligne de pixel !

@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 17, 2017

Une solution potentiel est la suivante :

Lire le fichier caractère par caractère
On compte les retour à la ligne pour savoir notre position
    Si ligne == 1 Alors Magic word
    Si ligne == 2 Alors width, height
    Si ligne == 3 Alors v_max
Si caractère = # alors on lit jusqu'à retour à la ligne

Ensuite pour le tableau de pixel, après avoir allouer dynamiquement la mémoire
On lit chaque caractère jusqu'à former un entier, une fois l'entier lut et confirmé, on l'injecte dans le tableau de pixels.

Avec cette méthode au évite de ce prendre la tête avec les retour à la ligne pour le tableau de pixel

@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 19, 2017

Créer une nouvelle branch pour coder la modification

@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 19, 2017

La nouvelle fonction lit correctement les fichiers (branch fill_struct_from_big_picture)

@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 19, 2017

Par contre le code est trop répétitif sur la lecture des nombres, il faut coder une fonction par fraction la fonction qui est illisible --> #16

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

No branches or pull requests

1 participant