How to include .txt on Qt

KzR69100

Thank you i corrected.

And for my other error ?

in : if (Costs.contains( colour.rgb() ) || IsCloseColor(BaseColor, colour) )

for the function IsCloseColor, its not better to return a color ? like QColor IsCloseColor();, i don't understand what we can do if we return "false" or "true"

Payx

i up ! need help ^^

Payx

for the function IsCloseColor, its not better to return a color ? like QColor IsCloseColor();, i don't understand what we can do if we return "false" or "true"

mrjj

@Payx

It just checks if the color is "close " to base color
No reason to return color ?
we say
if (Costs.contains( colour.rgb() ) || IsCloseColor(BaseColor, colour) )

So if we have direct match OR
the colour is close to BaseColor then we think its ok.

Payx

Okay thank you,

i have

diffBlue+diffRed+diffGreen< 100

but if many colors correspond ?
it can return many colors no ?

mrjj

@Payx
No it cant..
IsCloseColor tells if 2 colors are close. It return true/false.
it dont return any color. it tells you if they color u have "in the hand" matches the
base color given.
You already have the color (colour var) so no reason to return it.

Payx

So it cant be possible to have 2 colors who is <100 ?

mrjj

@Payx
ofc, just compare 2 times or more times. Once pr color.
IsCloseColor(BaseColor, col1)
IsCloseColor(BaseColor, col2)
IsCloseColor(BaseColor, col3)
....

Payx

So if i have like 27 colors, i have to do 27 function ?

mrjj

@Payx
No. use a list (of colors) and loop over it, checking each.

Payx

Okay, so it's useless to write :

bool IsCloseColor( QColor c1, QColor c2 ) {
	int diffRed   = Math.abs(c1.red() - c2.red());
	int diffGreen = Math.abs(c1.green() - c2.green());
	int diffBlue  = Math.abs(c1.blue()  - c2.blue());

	if (diffBlue+diffRed+diffGreen< 100){	/
		return true;.
		else
				return false;


	}
}

So if i understood well the "<100" is now useless. Can't we just return the color who have the minimum diffBlue+diffRed+diffGreen ?

mrjj

@Payx
The < 100 is the factor on How close the color was to the Base Color.
So i do not understand u think its useless.

How else would u compare them?

Payx

u said "do a list", instead of write ...<100, we can do a "for" and "return" the minimal "diffBlue+diffRed+diffGreen" or the all color no ?

jsulm

@Payx If you want to return the closest color from a list then you need to write a function similar to IsCloseColor. You iterate over the list, calculate the diff and store the color with the smallest diff and its diff in local variables, then return the color from that local variable.

Payx

I learn how to browse number (from 0 to 50) but how to browse a list ?

mrjj

@Payx
No sure what "browse a list is"

    QList<QColor> ColorList;
    ColorList << QColor(100,100,100) << QColor(200,200,200) << QColor(0,0,100) ;
    for( int c=0; c < ColorList.size();c++ ) {
    QColor CurCol=ColorList[c]; // take from list
    // ... do what u want
    }

Payx

like compare all the color with a for

you said it to me ^^

@Payx
No. use a list (of colors) and loop over it, checking each.

i already have this (you give it to me)

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

bool IsCloseColor( QColor c1, QColor c2 ) {
	int diffRed   = Math.abs(c1.red() - c2.red());
	int diffGreen = Math.abs(c1.green() - c2.green());
	int diffBlue  = Math.abs(c1.blue()  - c2.blue());

	if (diffBlue+diffRed+diffGreen< 100){	/
		return true;.
		else
				return false;


	}
}

the function i want to create is to browse the Qmap and pick the color who got the minimum ""diffBlue+diffRed+diffGreen< 100""

mrjj

Hi
ok. i see. we had the code higher up.

Qcolor colour; // the one you compare too. set to something...
QMapIterator<QRgb, CostInfo >i(Costs);
while (i.hasNext()) {
i.next();
QColor BaseColor( i.key() ); // the color in table
if (Costs.contains( colour.rgb() ) || IsCloseColor(BaseColor, colour) ) { // directly have colour or it is < 100
//// we got a match
}
}// while

Payx

Okay,

but we keep the code ?

because the <100 isn't good, you said we have to compare all the colors and see what colors match the best

mrjj

Yes ?
Then change the 100.
Its just a value. You should examine your color and see
the value for color that you consider "The same"
Maybe its need to be 80 or 110. Depends on how close
the color must be for you to think its the same.