How to include .txt on Qt

mrjj

@Payx said in How to include .txt on Qt:

if (pix.isNull()){
QPixmap pix(":/img/ballebleue.png");
};

Well that would not really change anything as you create a new local pix and its not related to the "pix" outside
You should find out if its invalid and then why it is

if (pix.isNull()){
qDebug() << "yep its no good";
return;
};

Payx

okay so i have that.

yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
QImage::pixel: coordinate (144,540) out of range
QImage::pixel: coordinate (145,540) out of range
QImage::pixel: coordinate (146,540) out of range
QImage::pixel: coordinate (147,540) out of range
QImage::pixel: coordinate (148,540) out of range
QImage::pixel: coordinate (149,540) out of range
QImage::pixel: coordinate (150,540) out of range
QImage::pixel: coordinate (151,540) out of range
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
yep its no good
QImage::pixel: coordinate (168,540) out of range
QImage::pixel: coordinate (169,540) out of range
QImage::pixel: coordinate (170,540) out of range
QImage::pixel: coordinate (171,540) out of range
QImage::pixel: coordinate (172,540) out of range
QImage::pixel: coordinate (173,540) out of range
QImage::pixel: coordinate (174,540) out of range
QImage::pixel: coordinate (175,540) out of range
QImage::pixel: coordinate (168,541) out of range
QImage::pixel: coordinate (169,541) out of range
QImage::pixel: coordinate (170,541) out of range
QImage::pixel: coordinate (171,541) out of range
QImage::pixel: coordinate (172,541) out of range
QImage::pixel: coordinate (173,541) out of range
QImage::pixel: coordinate (174,541) out of range
QImage::pixel: coordinate (175,541) out of range
QImage::pixel: coordinate (168,542) out of range
QImage::pixel: coordinate (169,542) out of range
QImage::pixel: coordinate (170,542) out of range
QImage::pixel: coordinate (171,542) out of range
QImage::pixel: coordinate (172,542) out of range
QImage::pixel: coordinate (173,542) out of range
QImage::pixel: coordinate (174,542) out of range

that's mean that my pix is null right?

mrjj

@Payx
Yes this means that the pictures in the table cannot be loaded.
Are they in a resource ?

Payx

Yes i have that exactly :http://hpics.li/cc11b5c

mrjj

@Payx
Oh, they are under IMG

Right click one image file.
Find Get Path
should be something like

:/IMG/filename.xxx

Fix the names in the table.
As far as i recall they start like ":/filename.xxx" and that dont match.

alt text

Note: Here i do NOT have them in a sub folder like you have (img) so path is :/p1.png

Payx

QMap<QRgb, CostInfo > Costs = {
	{ QColor(255 , 0 , 0 ).rgb(), { ":/img/fraise.png", 10 }},
	{ QColor(0 , 255 , 0 ).rgb(), { ":/img/balleverte.png", 20 }},	
	{ QColor(0 , 0 , 255 ).rgb(), { ":/img/balebleue.png", 20 }},
	{ QColor(255 , 255 , 255 ).rgb(), { ":/img/balleblanche.png", 20 }},
	{ QColor(255 , 128 , 0 ).rgb(), { ":/img/ballepeche.png", 20 }},
	{ QColor(0 , 0 , 0 ).rgb(), { ":/img/noir.png", 20 }},
	{ QColor(102 , 51 , 0 ).rgb(), { ":/img/marron.png", 20 }},
	{ QColor(255 , 102 , 78 ).rgb(), { ":/img/rose.png", 20 }},
	{ QColor(0 , 204 , 204 ).rgb(), { ":/img/turquoise.png", 20 }},
	{ QColor(255 , 178 , 102 ).rgb(), { ":/img/beige.png", 20 }},
	{ QColor(76 , 0 , 153 ).rgb(), { ":/img/violet.png", 20 }},
	{ QColor(100 , 100 , 100 ).rgb(), { ":/img/gris.png", 20 }},
};

i didnt mention that but i had already changed the path

mrjj

@Payx
oh, it seems correct. You checked they are truly correct?

Please qDebug() << ci.ImageName to verify the variable contains the path from
the table.

just above where you try to make pixmap

qDebug() << "check me--->" << ci.ImageName;
QPixmap pix( ci.ImageName );

Payx

I have

check me---> ""

so there is nothing

i declared as QString ImageName
so maybe it is that ?

mrjj

@Payx
Not sure. check the code that look it up.
If from table it should not be empty :)

SGaist

Are you sure you are retrieving a valid value from your map ?

Payx

No it's sure that it's not a valid value !

QPixmap pix( ci.ImageName );

so it create a pix from a QString no ? that why it didnt work ?

SGaist

From what you wrote before, ci.ImageName is empty, correct ?

If so, check the key that you use to retrieve the data from the map. If not part of the map, depending on the technique used, you will get a default constructed object.

Payx

Yes it's empty.

			foreach( QRgb key, Costs.keys() ) {
				QColor BaseColor( key );
				if (Costs.contains( colour.rgb() ) || IsCloseColor(BaseColor, colour) )  {
					CostInfo& ci = Costs[colour.rgb()];
					QLabel abc;
//          int Cost = ci.Cost;
					qDebug() << "check me--->" << ci.ImageName;
					QPixmap pix( ci.ImageName );
					if (pix.isNull()){
					qDebug() << "yep its no good";
					return;
					};
					QPixmap scaledPix = pix.scaled(   8, 8,
																				 Qt::KeepAspectRatio,
																				 Qt::SmoothTransformation
																				 );
					abc.setPixmap(scaledPix);
					abc.show();

it's what i have.

SGaist

Your logic is flawed. If Costs contains colour.rgb() then you can go on with the rest of your logic else you should go through your map and use IsCloseColor to determine which key is good. Then you can use that key to go further.

Payx

So i have to separate the

colour.rgb() ) || IsCloseColor(BaseColor, colour)

?

can you tell me how can i got through my map i've already search and asked but no one answer

SGaist

You're already doing it with your foreach loop and the documentation of QMap shows several ways of looping through its content.

Payx

something like that ?

			foreach( QRgb key, Costs.keys() ) {
				QColor BaseColor( key );
				if (Costs.contains( colour.rgb() ))  {
					CostInfo& ci = Costs[colour.rgb()];
					QLabel abc;
//          int Cost = ci.Cost;
					qDebug() << "check me--->" << ci.ImageName;
					QPixmap pix( ci.ImageName );
					if (pix.isNull()){
					qDebug() << "yep its no good";
					return;
					};
					QPixmap scaledPix = pix.scaled(   8, 8,
																				 Qt::KeepAspectRatio,
																				 Qt::SmoothTransformation
																				 );
					abc.setPixmap(scaledPix);
					abc.show();

				}
				else {
					if (IsCloseColor(BaseColor, colour) == true){
						CostInfo& ci = Costs[colour.rgb()];
						QColor BaseColor( key );
						QPixmap pix( ci.ImageName );
						QPixmap scaledPix = pix.scaled(   8, 8,
																					 Qt::KeepAspectRatio,
																					 Qt::SmoothTransformation
																					 );
						abc.setPixmap(scaledPix);
						abc.show();
					}
					else {
						qDebug() << "yep its no good";
					}
				}

SGaist

Move the foreach loop in the else part. There's no sense in looping through the whole map if it contains colour.rgb(). Also note that you won't see any QLabel since it's going out of scope.

Payx

void Remplissage(QImage& image, const QPoint& topLeft, const QSize& rectangle, const QColor& colour) {

	int maxX = topLeft.x() + rectangle.width();
	int maxY = topLeft.y() + rectangle.height();

	for(int x = topLeft.x(); x < maxX; ++x) {
		for(int y = topLeft.y(); y < maxY; ++y) {
			image.setPixelColor(x, y, colour);


				QColor BaseColor( key );
				if (Costs.contains( colour.rgb() ))  {
					CostInfo& ci = Costs[colour.rgb()];
					QLabel abc;
//          int Cost = ci.Cost;
					qDebug() << "check me--->" << ci.ImageName;
					QPixmap pix( ci.ImageName );
					if (pix.isNull()){
					qDebug() << "yep its no good";
					return;
					};
					QPixmap scaledPix = pix.scaled(   8, 8,
																				 Qt::KeepAspectRatio,
																				 Qt::SmoothTransformation
																				 );
					abc.setPixmap(scaledPix);
					abc.show();

				}
				else {
					if (IsCloseColor(BaseColor, colour) == true){
						foreach( QRgb key, Costs.keys() ) {
						CostInfo& ci = Costs[colour.rgb()];
						QLabel abc;
						QColor BaseColor( key );
						QPixmap pix( ci.ImageName );
						QPixmap scaledPix = pix.scaled(   8, 8,
																					 Qt::KeepAspectRatio,
																					 Qt::SmoothTransformation
																					 );
						abc.setPixmap(scaledPix);
						abc.show();
					}
					else {
						qDebug() << "yep its no good";
					}
				}

i changed what u said

SGaist

I'd guess it doesn't compile.

Your BaseColor variable doesn't make sense where it's declared. The if (IsCloseColor... also doesn't make sense where you put it. You are searching the map to find which of its key is close to the colour you received. But currently if BaseColor fits you are just trying to show each and every element of Costs.

Again: your QLable objects will be destroyed before they are even shown.