How to include .txt on Qt

Payx

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

What you want to do is loop on the map and if any of the colour ÌsCloseColour` then use that one.

Sorry if i don't understand well but """IsCloseColor(BaseColor, colour) == true""" dont mean that if any of the color IsCloreColour ?

SGaist

No, that function checks the whether the colours passed as parameters are close.

What you want to do is check which one of the map entries is close to the reference.

Take a look at your code and the order in which you wrote it. You are not testing the content of the Costs map, you are just checking whether colour is close to BaseColor which currently you can't create since your key variable is inside your loop and not outside where you declared BaseColor.

Payx

Ok,

					foreach( QRgb key, Costs.keys() ) {
					QColor BaseColor( key );
					if (IsCloseColor(BaseColor, colour) == true){

so first i loop on my map, i declare my BaseColor who is the color in my map, and i check if it's close to my color's reference, right ?

SGaist

Looks better, yes.

Don't forget to break out of the loop once you found a match.

Payx

so it compile but i got the 3 same errors :

QPixmap::scaled: Pixmap is a null pixmap

setGeometry: Unable to set geometry 5x13+640+280 on QWidgetWindow/'QLabelClassWindow'. Resulting geometry:  120x13+640+280 (frame: 8, 31, 8, 8, custom margin: 0, 0, 0, 0, minimum size: 0x0, maximum size: 16777215x16777215).

yep its no good

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);




				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 {
					foreach( QRgb key, Costs.keys() ) {
					QColor BaseColor( key );
					if (IsCloseColor(BaseColor, colour) == true){

						CostInfo& ci = Costs[colour.rgb()];
						QLabel abc;
						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 put my code here

SGaist

Because you are still trying to get the wrong entry from your map. If you enter your foreach loop, it means that colour can't be found in your map thus you can't use it to get the value from the map.

IsCloseColor tells you that BaseColor and colour are close, then you must use key to get the CostInfo you want.

Payx

Ok i understand, so it doesnt work because i still search my color in my map.

So i have to replace color by basecolor (when IsCloseColor(BaseColor, colour) == true) right ? and search for basecolor in my map ?

Flotisable

@Payx
yes

Payx

am i right if i write

CostInfo& ci = Costs[BaseColor.rgb()];

?

Flotisable

yes

by the way, you don't need to write if( IsCloseColor( BaseColor , colour ) == true ) explicitly. Yout can write if( IsColseColor( BaseColor , colour ) ) since the return value of IsCloseColor is already a bool and if can test it

Payx

Ok so it compile,

but i have the same error again :

yep its no good
yep its no good
yep its no good
setGeometry: Unable to set geometry 8x7+640+280 on QWidgetWindow/'QLabelClassWindow'. Resulting geometry:  120x7+640+280 (frame: 8, 31, 8, 8, custom margin: 0, 0, 0, 0, minimum size: 0x0, maximum size: 16777215x16777215).

*1000

SGaist

Again checkout what you get, whether what it contains is valid.

And again 2, you are creating QLabels that will be destroyed before they can be shown.

Payx

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

And again 2, you are creating QLabels that will be destroyed before they can be shown.

So maybe write "QLabel abc;" in my .h ?

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

Again checkout what you get, whether what it contains is valid.

i have

						CostInfo& ci = Costs[BaseColor.rgb()];
						QPixmap pix( ci.ImageName );

Maybe i don't understand well this 2 lines, i explain what i understand and tell me if i'm wrong.

The first line put in my variable ci the "line" referent to my BaseColor ( it stock the Costs (who have a QString and a Int))

the second line create a pixmap from the ImageName referent to my BaseColor. So something like (QPixmap pix(:/img/ballepeche.png))

mrjj

Hi
Make sure that
BaseColor.rgb() actually give a 100% MATCH else u insert empty one with no file name and ImageName will be empty!

I think u are using it wrongly.

try
if ( Costs.contains( BaseColor.rgb()))
qDebug () << "its OK with base color";
else
qDebug () << " base color had no match!!"

before
CostInfo& ci = Costs[BaseColor.rgb()];
QPixmap pix( ci.ImageName );

As if base color is NOT 1 of the colors in qmpa, it dont work and image is empty.

Flotisable

@mrjj
@Payx's BaseColor is constructed from the key of Costs, so I think there is no problem

Payx

its OK with base color
setGeometry: Unable to set geometry 8x7+640+280 on QWidgetWindow/'QLabelClassWindow'. Resulting geometry:  120x7+640+280 (frame: 8, 31, 8, 8, custom margin: 0, 0, 0, 0, minimum size: 0x0, maximum size: 16777215x16777215).
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
its OK with base color

So no problem about BaseColor !

Flotisable

@Payx
I think you can output the imageName in the foreach to check if there is a file name.

and you can try to use debug mode and execute step by step to find out which line cause the problem.

maybe it will be more clear what's wrong with it.

mrjj

Hi
Yes as @Flotisable says, you should check what is going on with the file name

so

xxxx
QPixmap pix( ci.ImageName );
qDebug() << "image i get from ci.ImageName : " << ci.ImageName;

and see what it really is.

Payx

with qDebug() << "image i get from ci.ImageName : " << ci.ImageName;

image i get from ci.ImageName :  ":/img/noir.png"
setGeometry: Unable to set geometry 8x7+640+280 on QWidgetWindow/'QLabelClassWindow'. Resulting geometry:  120x7+640+280 (frame: 8, 31, 8, 8, custom margin: 0, 0, 0, 0, minimum size: 0x0, maximum size: 16777215x16777215).
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
its OK with base color
image i get from ci.ImageName :  ":/img/noir.png"
setGeometry: Unable to set geometry 8x7+640+280 on QWidgetWindow/'QLabelClassWindow'. Resulting geometry:  120x7+640+280 (frame: 8, 31, 8, 8, custom margin: 0, 0, 0, 0, minimum size: 0x0, maximum size: 16777215x16777215).
yep its no good

mrjj

Ok so path is good ? seems ok.
So I guess it does load it there. ( in pix)
But why does

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

still trigger then ?

or is that from
if (IsCloseColor(BaseColor, colour) == true){
xxx
}
else {
qDebug() << "yep its no good";
}

?