Probleme recuperation pixel [Résolu]

cs_mano972 7 Messages postés dimanche 13 mars 2011Date d'inscription 26 avril 2011 Dernière intervention - 11 avril 2011 à 06:03 - Dernière réponse : cs_mano972 7 Messages postés dimanche 13 mars 2011Date d'inscription 26 avril 2011 Dernière intervention
- 13 avril 2011 à 17:35
Bon j'ai écrit un programme pour me permettre de récuperer et modifier des pixels d'une image mais j'aimerais savoir par quoi on peut remplacer fgetc (language c) en c+++.
Ce programme fonctionne mais pas avec le ifstream d'ou la raison du remplacement.
merci.

for (y = 0; y < LARGEUR; y++)
{
for (x = 0; x < HAUTEUR; x++)
{
r = fgetc (fichier);
g = fgetc (fichier);
b = fgetc (fichier);
niv = r * 0.299 + 0.587 * g + 0.114 * b;
fichiers<<niv
}
}
Afficher la suite 

Votre réponse

5 réponses

Meilleure réponse
cptpingu 3804 Messages postés dimanche 12 décembre 2004Date d'inscription 20 septembre 2018 Dernière intervention - 13 avril 2011 à 15:48
3
Merci
Est-ce que tu comprends ce que tu écris ?

Il y a beaucoup d'incohérences:
* image[x][y] => x et y n'ayant pas de valeur, quelle sera la taille de image ?
* std::ios::binary => C'est un fichier texte, donc pas d'ouverture binaire, non ?
* Pourquoi valmax n'est pas un entier, au lieu d'être un std::string ?
* ofstream fichiers(Fichiers.c_str(), ios::out | ios::trunc) => Deux choses bizarres: pourquoi écrire dans le fichier ou tu lis ? L'option std::ios::trunc, veut dire: "vider", donc tu vides ton fichier, alors qu'il faudrait que tu lises dedans ? Ce n'est pas gênant puisqu'à ce stade tu devrais avoir les données en mémoire, mais en général on préfère créer un nouveau fichier sans altérer l'original.
* Pourquoi ne pas découper ce code en petites fonctions, au lieu de tout avoir dans le main ?

________________________________________________________________________
Historique de mes créations, et quelques articles:
[ http://0217021.free.fr/portfolio http://0217021.free.fr/portfolio]
Merci d'utiliser Réponse acceptée si un post répond à votre question

Merci cptpingu 3

Avec quelques mots c'est encore mieux Ajouter un commentaire

Codes Sources a aidé 95 internautes ce mois-ci

Commenter la réponse de cptpingu
cs_mano972 7 Messages postés dimanche 13 mars 2011Date d'inscription 26 avril 2011 Dernière intervention - 11 avril 2011 à 06:30
0
Merci
je vous donne le programme complet
#include
#define LARGEUR 640
#define HAUTEUR 480
#include <string>
#include <fstream>
using namespace std;

int main(int argc, char* argv[])
{
// Déclaration des variables
int x,y;
unsigned char r,g,b,niv;
string Fichiers;
cin >>Fichiers;
string ligne;
string chaine1,chaine2;
int valmax,rien,riens;
// Image.ppm (la plus couramment utilisée)
ifstream fichier(Fichiers.c_str(), ios::in); // ouverture du fichier en lecture binaire

if(fichier) // si l'ouverture a réussi
{
// instructions
fichier >> chaine1 >> chaine2 >> rien >> riens>>valmax; /*on lit jusqu'à l'espace et on stocke ce qui est lu dans la variable indiquée */
for( y=0; y<LARGEUR; y++ )
{ // lecture sur l'axe Y
cout<< "." ;
for( x=0; x<HAUTEUR; x++ )
{ // lecture sur l'axe X (=lecture d'une ligne)
fichier >>r = fgetc(fichier);
fichier >>g = fgetc(fichier);
fichier >>b = fgetc(fichier);
niv = r * 0.299 + 0.587 * g + 0.114 * b;
}
}
cout <<"\n" << "Lecture de l'image terminee !\n" ;

fichier.close(); // on ferme le fichier
}
else // sinon
cerr << "le fichier est introuvable ! Le programme va s'arreter..." << endl;
return 0;

ofstream fichiers(Fichiers.c_str(), ios::out | ios::trunc); //déclaration du flux et ouverture du fichier
if (fichiers) // si l'ouverture a réussi
{
// instructions
fichiers << chaine1 << chaine2 << LARGEUR << HAUTEUR << valmax;
for (y = 0; y < LARGEUR; y++)
{
for (x = 0; x < HAUTEUR; x++)
{
fichier <<r = (fichier);
fichier <<g = (fichier);
fichier <<b = (fichier);
niv = r * 0.299 + 0.587 * g + 0.114 * b;
fichiers<<niv;
}
}

fichiers.close(); // on referme le fichier
}
else{
// sinon
cerr << "Desole, mais le fichier ne peut etre ouvert en ecriture..." << endl;
return 0;
}
Commenter la réponse de cs_mano972
cptpingu 3804 Messages postés dimanche 12 décembre 2004Date d'inscription 20 septembre 2018 Dernière intervention - 11 avril 2011 à 19:25
0
Merci
Pour du binaire, std::ios::binary => plutôt que std::ios::in, non ?

Un fichier ppm n'est pas binaire, c'est un simple fichier texte.
Au choix tu peux faire un: std::getline(file)

#include 
#include <fstream>

int main()
{
  std::fstream file("test.txt");
  if (!file)
    return 1;

  std::string line;
  while (std::getline(file, line, ' '))
    if (!line.empty()) // TODO: Dégagez aussi le \n
      std::cout << """ << line << """ << std::endl;

  return 0;
}


PS: Évite les "using namespace", voir: http://0217021.free.fr/portfolio/axel.berardino/articles/bon-usage-using-namespace
________________________________________________________________________
Historique de mes créations, et quelques articles:
[ http://0217021.free.fr/portfolio http://0217021.free.fr/portfolio]
Merci d'utiliser Réponse acceptée si un post répond à votre question
Commenter la réponse de cptpingu
cs_mano972 7 Messages postés dimanche 13 mars 2011Date d'inscription 26 avril 2011 Dernière intervention - 12 avril 2011 à 00:34
0
Merci
Je vois pas trop comment placer getline(file) pour recuperer le "r,g,b",
J'ai vu des personne utilisé dans la boucle.
fichier >> pixel;
image.pixel[x][y].r=pixel
fichier >> pixel;
image.pixel[x][y].g=pixel etc
mais le point ".r" me donne des erreurs,si quelqu'un peut m'éclairer.

je suis pas très bon en language de programmation donc j'ai essayé de le modifié, mais a la fin je recoit "erreur de segmentation" pendant la compilation sur code block? , j'ai pas encore regarder quoi remplacer pour le using namespace
#include 
#define LARGEUR 640
#define HAUTEUR 480
#include <string>
#include <fstream>
using namespace std;

int main(int argc, char* argv[])
{
// Déclaration des variables
    
    int x,y; // variables d'itération
   	string chaine1,chaine2,valmax,Fichiers;
cout << "Entrer le nom de l'image ";
cin >>Fichiers;

unsigned short int largeur,hauteur; 
    unsigned char image[x][y];//matrice de l'image
        // Image.pnm (la plus couramment utilisée)
        
        	// Lecture du ou des fichier(s)
        ifstream fichier(Fichiers.c_str(), ios::binary);		// ouverture du fichier en lecture

 	if(fichier)  // si l'ouverture a réussi
        {
            // instructions, j'ai voulu mettre getline mais j'etais obligé de mettre du string
            fichier>>chaine1;
            fichier>>chaine2;
            fichier >> largeur;
            fichier >>hauteur;
            fichier>> valmax ;
for( y=0; y<largeur; y++ )
      	{	// lecture sur l'axe Y
        cout<< "." ;
    	for( x=0; x<hauteur; x++ )
      {	// lecture sur l'axe X (=lecture d'une ligne)

        fichier >> image[x][y]; //matrice de l'image

    }
       fichier.close();  // on referme le fichier
       }
      cout <<"\n" << "Lecture de l'image terminee !\n" ;

        }
        else  // sinon
        cerr << "le fichier est introuvable ! Le programme va s'arreter..." << endl;

  // Traitement(s)
ofstream fichiers(Fichiers.c_str(), ios::out | ios::trunc);  //déclaration du flux et ouverture du fichier
if (fichiers)  // si l'ouverture a réussi
        {
            // instructions variale que je vais mettre dans le "fichiers"
        fichiers << chaine1 <<endl; //signature de l'image P3
        fichiers<< chaine2 <<endl;//commentaire de l'image
        fichiers<< largeur<<endl;//largeur de l'image
        fichiers<< hauteur <<endl;//la hauteur de l'image
        fichiers<< 255<<endl;//la profondeur de l'image, valmax ne prend pas la valeur donc je lui est mis par defaut 255
for (y = 0; y < largeur; y++)
  	{
    	for (x = 0; x < hauteur; x++)
      	{                                                                           // ici le code concernant le traitement d'images
        fichiers << (image[x*3][y] + image[x*3+1][y] + image[x*3+2][y])/3 <<endl; //mise en niveau gris de l'image grace a cette fonction

}
      	}
        fichiers.close();  // on referme le fichier

        }
        else  // sinon
        cerr << "Desole, mais le fichier ne peut etre ouvert en ecriture..." << endl;
return 0;
}
Commenter la réponse de cs_mano972
cs_mano972 7 Messages postés dimanche 13 mars 2011Date d'inscription 26 avril 2011 Dernière intervention - 13 avril 2011 à 17:35
0
Merci
Ok, c'est bon sa marche maintenant.
Commenter la réponse de cs_mano972

Vous n'êtes pas encore membre ?

inscrivez-vous, c'est gratuit et ça prend moins d'une minute !

Les membres obtiennent plus de réponses que les utilisateurs anonymes.

Le fait d'être membre vous permet d'avoir un suivi détaillé de vos demandes et codes sources.

Le fait d'être membre vous permet d'avoir des options supplémentaires.