How to include .txt on Qt

[KzR69100]

i got :

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< z){	// i block for the z how can i say "the smallest" ?
		return c1;


	}
}

i got a lot of errors, i tried to google it but no answer :

like : request for meme "key in 'i'", which is of non-class type 'int'

template argument 2 is invalid (QMap<QRgb, CostInfo ">" Costs =)

scalar object 'Costs' requires one element in initializer

[mrjj]

@KzR69100

z you must decide for.
Best is to test with different colors and see how close its still ok.

With the exact code and the exact line it's impossible to say what error comes from.

You can try my test project. It compiles and you can test the color matching.
https://www.dropbox.com/s/6mc3e4qe375qzih/mylookup.zip?dl=0

[KzR69100]

My code is : (cpp)

#include "mainwindow.h"
#include "ui_mainwindow.h"
#include <QPixmap>
#include <QImage>
#include <QFileDialog>
#include <QColor>
#include <QPoint>
#include <QSize>
#include <iostream>
using namespace std;


QColor Couleurdominante(const QImage& image, const QPoint& topLeft, const QSize& rectSize);
void Remplissage(QImage& image, const QPoint& topLeft, const QSize& rectSize, const QColor& colour);


MainWindow::MainWindow(QWidget *parent) :
	QMainWindow(parent),
	ui(new Ui::MainWindow)
{
		ui->setupUi(this);

}


MainWindow::~MainWindow()
{
	delete ui;
}




void MainWindow::on_push_clicked()
{
	QString fileName = QFileDialog::getOpenFileName(this,
	tr("Open Image"), "/", tr("Image Files (*.png *.jpg *.bmp)"));
	QPixmap pix(fileName);
	ui->label->setPixmap(pix);
	const QSize s = pix.size();
	pixi = QImage(pix.toImage());

	ui->label_2->setText( "Size: " + QString::number(s.width()) +" "+ QString::number(s.height()) );

}

void MainWindow::on_push2_clicked()
{


	for (int i=0;i<pixi.width();i+=a){
						 for (int j=0;j<pixi.height();j+=b){
	Remplissage(pixi,QPoint(i,j),QSize(a,b),Couleurdominante(pixi,QPoint(i,j),QSize(a,b)));
	}}

	pixa=QPixmap::fromImage(pixi);
	ui->label_3->setPixmap(pixa);

}




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);
			QMapIterator<QRgb, CostInfo >i;
			while (i.hasNext()) {
				i.next();
				QColor BaseColor( i.key() );
				if (Costs.contains( colour.rgb() ) || IsCloseColor(BaseColor, colour) )  {
					CostInfo& ci = Costs[colour.rgb()];
					int Cost = ci.Cost;
					QPixmap pix( ci.ImageName );
				}
			}
		}
	}
}


QColor Couleurdominante(const QImage & image,const QPoint & topLeft, const QSize & rectangle)
{
		int rouge = 0, vert = 0, bleue = 0;

		int  X = topLeft.x() + rectangle.width();
		int  Y = topLeft.y() + rectangle.height();

		for (int y = topLeft.y(); y < Y; y++)  {
				for (int x = topLeft.x(); x < X; x++)   {
						QRgb pixel = image.pixel(x, y);

						rouge += qRed(pixel);
						vert += qGreen(pixel);
						bleue += qBlue(pixel);
				}
		}

		int n = rectangle.width() * rectangle.height();

		Q_ASSERT(n);
		if (n <= 0)
				return Qt::black;

		return QColor(rouge / n, vert / n, bleue / n);
}



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


	}
}

i took z = 100. For the moment

.h :

#ifndef MAINWINDOW_H
#define MAINWINDOW_H

#include <QMainWindow>
#include <QPixmap>
#include <QPoint>
#include <QSize>
#include <iostream>

namespace Ui {
	class MainWindow;
}

class MainWindow : public QMainWindow
{
	Q_OBJECT

public:
	explicit MainWindow(QWidget *parent = 0);


	~MainWindow();





private slots:
	void on_push_clicked();
	struct CostInfo {
		QString ImageName;
		int Cost;
	};

	void on_push2_clicked();

	void on_verticalSlider_sliderMoved(int position);

private:
	Ui::MainWindow *ui;
	QImage pixi;
	QPixmap pixa;
	int a;
	int b;
};

#endif // MAINWINDOW_H

[mrjj]

Hi
In your
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 c1; <<<<<< you should return true
    else //////////////////you need this part also
       return false;  /////////

[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
    }