Dessiner sur une image : assez complexe

Citation:

---

Envoyé par yan

Quelle est le problème exactement? Qu'es ce qui ne marche pas?

---

j'ai joins le code source ça me semble juste, je vois pas de probléme mais ça marche pas quand je clique sur la souris ça bogue et ça marche plus et mon pc se plante!!

c'est une méthode entre autre mais le but etant toujours de mettre en rouge le pixels cliqué sur l'image!!De ce fais j'ai deux buffers disponible l'un contenant les pixels de l'image et l'autre contenant que des zeros et pour afficher je compare les deux fichiers les pixels de l'image affichée corresponde à zeros sur l'autre buffer et les pixels en rouge de l'image correspondde a 1 sur l'autre buffer et a chaque clique je modifie la valeur 0 de ce buffer à 1 et je refais une comparaison pour affcihé en rouge le pixel cliqué!!mais ça marche pas!!

Merci d'avance j'ai besoin de votre help please!!:roll:

Citation:

---

Envoyé par lisco

j'ai joins le code source ça me semble juste, je vois pas de probléme mais ça marche pas quand je clique sur la souris ça bogue et ça marche plus

---

Peut tu être précis?

Citation:

---

Envoyé par lisco

et mon pc se plante!!

---

8O tu plante ton pc avec un simple programme!!!!

Première piste, vérifie la valeur de

Code:

---

1 2	int PosX = event->x() ; int PosY = event->y() ;

---

pour être sure qu'ils sont dans ton images

pourquoi
ui->ZoneImage->setWidget(m_label) ;

Citation:

---

Envoyé par yan

Peut tu être précis?

---

enfin je veux dire que je trouve juste ce que j'ai fait8O

Citation:

---

Envoyé par yan

8O tu plante ton pc avec un simple programme!!!!

---

je suis obligé de fermer l'application de force sinon aucune touche ne marche meme pas la souris.

Citation:

---

Envoyé par yan

Première piste, vérifie la valeur de

Code:

---

1 2	int PosX = event->x() ; int PosY = event->y() ;

---

pour être sure qu'ils sont dans ton images !!!!

---

tu veux dire dans le repere de l'image?Non je pense pas mais sont dans le repere de l'ihm mais vu que je clique sur l'image et que je modifie le buffer à la position [PosX + PosY * NbColonnes] je n'avais pas pris compte de cela.
je pense ajouté ça :

Code:

---

if (m_label && event->button() == Qt::LeftButton)

---

??

Citation:

---

Envoyé par yan

pourquoi
ui->ZoneImage->setWidget(m_label) ;

---

parce que vu mon algo je suis obligé de réafficher mon image à chaque pour faire apparaitre le pixel mis à 1 en rouge et en ecrasant l'ancienne image.Quelque precision ui-> : parce que ce sont des widget de mon ihm fait avec qt designer et le m_label est contenu dans un QScrollArea d'ou le setWidget!

Citation:

---

Envoyé par lisco

parce que vu mon algo je suis obligé de réafficher mon image à chaque pour faire apparaitre le pixel mis à 1 en rouge et en ecrasant l'ancienne image.Quelque precision ui-> : parce que ce sont des widget de mon ihm fait avec qt designer et le m_label est contenu dans un QScrollArea d'ou le setWidget!

---

C'est pas la bonne solution. La bonne solution c'est de mettre une image dans ton label. Après tu récupères cette image de la modifier et de remettre l'image après modification dans ton label.
Je pense que le principe du code que je t'ai donné fonctionne après le seul problème je pense c'est les coordonnée qui sont pas dans le repère image. Mais çà c'est a toi de voir. Ou autre solution tu crée un classe fille de QLabel et tu redéfinis l'événement mousePressEvent(QMouseEvent *event) et là normalement tu seras dans le repère du QLabel.

Bonjour,

J'ai regardé rapidement les solutions proposées, mais il manque le changement de repère.
Le plus simple est d'hériter une classe à partir d'un QLabel dans laquelle on surcharge la méthode de clic (pour faciliter le repérage)
Ensuite, ll faut réadapter les coordonnées de la souris en rapport avec la taille de l'image.

Donc dans la méthode de clique sur l'image :

Code:

---

1 2	posXImage = posXSouris * largeurImage / largeurLabel; posYImage = posYSouris * hauteurImage / hauteurLabel;

---

Code:

---

1 2	posXImage = posXSouris * largeurImage / largeurLabel; posYImage = posYSouris * hauteurImage / hauteurLabel;

---

Le problème c'est que ceci va dépendre de la méthode utiliser pour afficher l'image.
Si scaledContents est à true, tu as raison
Je lui avez montrer un code pour afficher une images et connaitre la position de la souris dans l'image
http://www.developpez.net/forums/d72...r/#post4228537

En gros :

CImgLabel.h

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

#include <QLabel> #include <QImage> #include <QPixmap> #include <QMouseEvent>   class CImgLabel : public QLabel { Q_OBJECT   public: CImgLabel(QWidget *parent = 0);   protected: void mousePressEvent(QMouseEvent *event);   private: QImage m_img; };

---

CImgLabel.cpp

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

#include "CImgLabel.h"   CImgLabel::CImgLabel(QWidget *parent) : QLabel(parent) { m_img.load("c:/tests/image.jpg"); this->setPixmap(QPixmap::fromImage(m_img)); }   void CImgLabel::mousePressEvent(QMouseEvent *event) { int posXImage = event->pos().x() * m_img.width() / width(); int posYImage = event->pos().y() * m_img.height() / height();     m_img.setPixel(posXImage, posYImage, qRgb(255,0,0)); this->setPixmap(QPixmap::fromImage(m_img)); }

---

Citation:

---

Envoyé par yan

Si scaledContents est à true, tu as raison

---

Et si c'est à faux, le rapport "largeurImage / largeurLabel" vaut 1. Donc ça marche aussi non ?

HS : ça a l'air pratique les eventFilter...

Citation:

---

Envoyé par LeGars

Et si c'est à faux, le rapport "largeurImage / largeurLabel" vaut 1. Donc ça marche aussi non ?

---

non car le label n'est pas forcement à la taille de l'image.

Citation:

---

Envoyé par LeGars

HS : ça a l'air pratique les eventFilter...

---

:D, carrément!!!
C'est une autre façon d'utiliser les event. Je trouve cela beaucoup plus puissante. Bien sure si tu dérive d'un widget autant réimplémenter les fonctions liées. Ca dépend ce que tu veut faire. J'ai vue plusieurs personnes voulant emettre un signal lors d'un click dans un QLabel. L'utilisation de eventfilter est beaucoup plus simple et adapter, et sans surcharger le code.

j'en connait un ou deux qui vont encore se moquer de moi...

Citation:

---

Envoyé par yan

j'en connait un ou deux qui vont encore se moquer de moi...

---

Je ne sais vraiment pas de qui tu parles.:koi:

Citation:

---

Envoyé par yan

non car le label n'est pas forcement à la taille de l'image.

---

Effectivement...

Dans ce cas, l'utilisation d'un QGraphicsView permet de faire tout le boulot :

CImgLabel.h

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

#include <QtGui>   class CImgLabel : public QGraphicsView { Q_OBJECT   public: CImgLabel(QWidget *parent = 0);   protected: void mousePressEvent(QMouseEvent *event);   private: QImage m_img; QGraphicsScene m_scene; QGraphicsPixmapItem *pixItm; };

---

CImgLabel.h

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

#include "CImgLabel.h"   CImgLabel::CImgLabel(QWidget *parent) : QGraphicsView(parent) { m_img.load("c:/tests/image.jpg"); setScene(&m_scene); pixItm = m_scene.addPixmap(QPixmap::fromImage(m_img));   }   void CImgLabel::mousePressEvent(QMouseEvent *event) { QPointF ptScene = this->mapToScene(event->pos());   m_img.setPixel(ptScene.x(), ptScene.y(), qRgb(255,0,0)); pixItm->setPixmap(QPixmap::fromImage(m_img)); }

---

De cette manière peut importe la taille du QGraphicsView, l'image est contenu dans la scene créant son propre repère.

Citation:

---

Envoyé par yan

J'ai vue plusieurs personnes voulant emettre un signal lors d'un click dans un QLabel. L'utilisation de eventfilter est beaucoup plus simple et adapter, et sans surcharger le code.

---

Je vais pas me moquer, mais juste dire qu'un QPushButton frameless semble plus adapté :aie:

Je ne connaissais pas je me disais bien qu'il fallait que je me mette aux classes QGraphics. Merci pour l'info LeGars. :ccool:

Citation:

---

Envoyé par IrmatDen

Je vais pas me moquer, mais juste dire qu'un QPushButton frameless semble plus adapté

---

Tu veux énerver Yan ? :aie:

Citation:

---

Envoyé par IrmatDen

Je vais pas me moquer, mais juste dire qu'un QPushButton frameless semble plus adapté :aie:

---

le but était de connaitre la position du click et non de faire un boutton :langue:

merci pour toutes les reponses apportées!J'ai hate de devenir Modérateur:ccool: car dur d'etre débutant:aie:

Citation:

---

Envoyé par lisco

merci pour toutes les reponses apportées!J'ai hate de devenir Modérateur:ccool: car dur d'etre débutant:aie:

---

tu as réussi?

Citation:

---

Envoyé par yan

tu as réussi?

---

Oui et thanks you for your help :ccool:

Citation:

---

Envoyé par lisco

Oui et thanks you for your help :ccool:

---

Qu'elle etait la solution alors?

cela peut servir à d'autre personne.

Citation:

---

Envoyé par yan

Qu'elle etait la solution alors?

cela peut servir à d'autre personne.

---

Code:

---

1 2 3 4 5 6 7 8 9

mousePressEvent( QMouseEvent *event) { img->pixmap()->toImage() ; img.setPixel(PositionX, PositionY, qRgb(255, 0, 0)) ; m_label->setPixmap(QPixmap::fromImage(img)) ; ui->ZoneImage->setWidget(m_label) ;   }

---

sachant que PositionX et PositionY sont les coordonnées de caque pixel pointé qui est specifié dans le repere du QLabel!