Ich habe hier zwei Funktionen die an sich gleich sind nur ein unterschiedliches Interface haben

Variante 1:

void Ploter::plot_line(std::shared_ptr<QwtPlotCurve>&myLine, std::vector<punkt<double>> &points, ploting_range &std, bool show, QPen pen)
{
	QwtSplineCurveFitter *fitter = new QwtSplineCurveFitter();
	fitter->setFitMode(QwtSplineCurveFitter::Spline);
	fitter->setSplineSize(points.size());
	QVector<double> x, y;
	for (size_t i = 0; i < points.size(); i++) {
		x.push_back(points.at(i).get_x());
		y.push_back(points.at(i).get_y());
	}

	myLine->setSamples(x, y);
	myLine->setCurveFitter(fitter);

	draw_spline(myLine, pen, show);
}

Variante 2:

void Ploter::plot_line(std::shared_ptr<QwtPlotCurve>&myLine, funktion &q ,ploting_range &std, bool show, QPen pen )
{
std::vector<punkt<double>>fkt_werte = q.werte(std.von, std.bis, std.schrittweite);
	QwtSplineCurveFitter *fitter = new QwtSplineCurveFitter();
	fitter->setFitMode(QwtSplineCurveFitter::Spline);
	fitter->setSplineSize(fkt_werte.size());
	QVector<double> x, y;
	for (size_t i = 0; i < fkt_werte.size(); i++) {
		x.push_back(fkt_werte.at(i).get_x());
		y.push_back(fkt_werte.at(i).get_y());
	}

	myLine->setSamples(x, y);
	myLine->setCurveFitter(fitter);

	draw_spline(myLine, pen, show);
}

Beide Funktionen machen das gleiche. Der Unterschied besteht nur darin das Variante 1 ein vector mit Punkten übergeben wird und Variante 2 eine Funtkion um sich die Punkte selbst zu errechnen.
Es wird dabei eine QwtPlotCurve mit Werten gefüttert. Die QwtPlotCurve wird durch den Aufrufer erstellt und als shared_ptr übergeben.
Die draw_spline - Methode ist nur dazu da die QwtPlotCurve in einem QwtPlot zu zeichen.

void Ploter::draw_spline(std::shared_ptr<QwtPlotCurve>&myLine,QPen pen, bool show) {
	myLine->setPen(pen);
	myLine->attach(plot);
	myLine->setVisible(show);
	plot->replot();
}

Die Klasse "Plotter" enthält hierbei den QwtPlot als pointer.

Das eigentliche Problem ist das Variante 1 von "plot_line" nichts im Plot darstellt, obwohl sämtliche Wertepaare in der QwtPlotCurve enthalten sind (zumindest laut dem Debugger).

Was mich zur Verzweiflung bringt ist das ich einfach nicht verstehe warum, denn die Funktionen sind vom Prinzip her gleich.

Wo liegt denn der Fehler? Was übersehe ich seit zwei Tagen?

denn die Funktionen sind vom Prinzip her gleich.

Wo liegt denn der Fehler? Was übersehe ich seit zwei Tagen?

Das ist schon schlecht. Doppelter Code bedeutet doppelter Aufwand und doppelte Fehlermöglichkeiten => DRY

Hast du mal versucht, das Ergebnis der Funktion q dem Aufruf der ersten Funktion plot_line zu übergeben?

Wenn du sowieso beide Varianten hast, dann verwende doch:

void Ploter::plot_line(std::shared_ptr<QwtPlotCurve>&myLine, funktion &q ,ploting_range &std, bool show, QPen pen)
{
  std::vector<punkt<double>> fkt_werte = q.werte(std.von, std.bis, std.schrittweite);

  plot_line(myLine, fkt_werte, std, show, pen);
}

(wobei 'std' ja in der ersten Variante gar nicht als Parameter benutzt wird - und außerdem noch namensgleich mit dem Namespace "std" ist).

PS: Es heißt korrekt "Plotter".

Mal davon abgesehen, dass eine Referenz auf einen shared_ptr ziemlicher Mist ist

Ja das war nur ein verzweifelter Versuch...

Th69 schrieb:

Hast du denn fkt_werte bzw. fkt_werte.size() mal überprüft?

Ja habe ich, enthält exakt die übergebenen Werte. Auch die QwtPlotCurve enhält intern die übergebenen Punkte.

Habe das ganze jetzt so umgebaut wie TH69 es vorgeschlagen hat, das ich nur noch eine Variante von plot_line habe. Das macht das ganze jetzt aber noch konfuser, denn es hat sich dadurch nichts geändert. Jetzt ist der aufruf von "plot_line" in allen "create_spline" - Methoden gleich. Die create_spline Methode dient als public Funktion dazu den Plotter aufzurufen.

create_spline Varainte 1:

std::shared_ptr<QwtPlotCurve> Plotter::create_spline(std::vector<punkt<double>> points, bool show, QPen pen)
{
	std::shared_ptr<QwtPlotCurve>myLine(new QwtPlotCurve(QString("")));
	std::thread t(static_cast<void(Plotter::*)(std::shared_ptr<QwtPlotCurve>, std::vector<punkt<double>>&, bool, QPen)>(&Plotter::plot_line), this, std::ref(myLine), std::ref(points), show, pen);
	t.join();
	return myLine;
}

create_spline Variante 2:

std::shared_ptr<QwtPlotCurve> Plotter::create_spline(funktion &q, ploting_range &range, bool show, QPen pen) {
	std::shared_ptr<QwtPlotCurve>myLine(new QwtPlotCurve(QString("")));
	std::vector<punkt<double>>points = q.werte(range.von, range.bis, range.schrittweite);

	std::thread t(static_cast<void(Plotter::*)(std::shared_ptr<QwtPlotCurve>, std::vector<punkt<double>>&, bool, QPen)>(&Plotter::plot_line), this, std::ref(myLine), std::ref(points), show, pen);
	t.join();
	return myLine;
}

Damit bleibt für mich nur noch der Aufruf von "create_spline" als Fehlerquelle.
Variante 1 wird so aufgerufen:

void datatable::draw() {
	QColor color = Plotter::randomColor();
	for (punkt<double>&t : m_points) {
		m_plot->create_point(t.get_x(), t.get_y(),color);
	}

	m_plot->create_spline(m_points,true,QPen(color,3));
}

Dabei ist die Klasse "datatable" eine UI für eine Wertetabelle.

Variante 2 wird aus der Klasse des MainWindows heraus aufgerufen

QVariant data;
		data.setValue(plot->create_spline(function, function.range, true, Plotter::stdPen));

Dabei wird der shared_ptr<QwtPlotCurve> in einem QVariant gespeichert.

Ich komm einfach nicht drauf wo jetzt der Fehler liegt, denn der aufruf aus der MainWindow-Klasse klappt wunderbar, nur der aus der datatable-Klasse nicht.

Wo kommen denn auf einmal die Threads her? Und welchen Sinn macht es, einen Thread zu starten und dann gleich mit join darauf zu warten?

Weil es unterm Strich ein paar ms Laufzeit ausmacht, was sich bei einem just-in-time plotting smother anfühlt.

Es ändert aber nichts. Auch ohne Thread passiert nichts anderes. Also der Fehler ist mit und ohne Thread da.

Weil es unterm Strich ein paar ms Laufzeit ausmacht, was sich bei einem just-in-time plotting smother anfühlt.

Du startest einen Thread und hältst den andern an - das fühlt sich bestimmt smoother an.

axels. schrieb:

Weil es unterm Strich ein paar ms Laufzeit ausmacht, was sich bei einem just-in-time plotting smother anfühlt.

Du startest einen Thread und hältst den andern an - das fühlt sich bestimmt smoother an.

Das ist doch jetzt nicht das Thema.

Das ist doch jetzt nicht das Thema.

Was hier letztendlich das Thema ist, ist völlig unklar. Du führst hier mit jedem Post komplett anderen Code ein.

axels. schrieb:

Das ist doch jetzt nicht das Thema.

Was hier letztendlich das Thema ist, ist völlig unklar. Du führst hier mit jedem Post komplett anderen Code ein.

Es ist doch wohl klar das man sich dem Fehler stück für Stück nähern muss, und eventuelle Fehlerquellen nach und nach eliminieren muss, deswegen immer wieder anderer Code der aber im zusammenhang mit allem anderen steht.

Also jetzt hier für den manni den aktuellen Stand zusammengefasst:

Zwei Funktionen die sich nur durch ihr Interface unterscheiden liefern verschiedene Ergebnisse.

Variante 1

std::shared_ptr<QwtPlotCurve> Plotter::create_spline(std::vector<punkt<double>> points, bool show, QPen pen)
{
	std::shared_ptr<QwtPlotCurve>myLine(new QwtPlotCurve(QString("")));
	plot_line(myLine, points, show, pen);
	return myLine;
}

Variante 2

std::shared_ptr<QwtPlotCurve> Plotter::create_spline(funktion &q, ploting_range &range, bool show, QPen pen) {
	std::shared_ptr<QwtPlotCurve>myLine(new QwtPlotCurve(QString("")));
	std::vector<punkt<double>>points = q.werte(range.von, range.bis, range.schrittweite);
	plot_line(myLine, points,show, pen);
	return myLine;
}

Beim aufruf von Varainte 2 wird die Kurve gezeichnet, bei Varainte 1 nicht.
Alles war vorher in dem Thread diskutiert wurde ist das was in der Funktion "create_spline" passiert. Da dies jetzt identisch ist, muss der Fehler nicht innerhalb der Funktion liegen. Ich verstehe aber nicht wo! Und genau das ist das aktuelle Thema!
Also nochmals die Frage, was ist falsch das eben jenes Verhalten auftritt?

Tja, wenn man jetzt wüsste was plot_line macht ...

Hier bitte. Da diese in beiden Fällen gleich ist, würde ich in meiner naivität diese als Fehlerquelle ausschließen.

void Plotter::plot_line(std::shared_ptr<QwtPlotCurve>myLine, std::vector<punkt<double>> &points, bool show, QPen pen)
{
	QwtSplineCurveFitter *fitter = new QwtSplineCurveFitter();
	fitter->setFitMode(QwtSplineCurveFitter::Spline);

	fitter->setSplineSize(points.size());
	QVector<double> x, y;
	for (size_t i = 0; i < points.size(); i++) {
		x.push_back(points.at(i).get_x());
		y.push_back(points.at(i).get_y());
	}

	myLine->setSamples(x, y);
	myLine->setRenderHint(QwtPlotCurve::RenderAntialiased);
	myLine->setCurveFitter(fitter);

	draw_spline(myLine, pen, show);
}

Für den Falls das du "draw_spline" auch noch sehen willst
Hier bitte:

void Plotter::draw_spline(std::shared_ptr<QwtPlotCurve>myLine,QPen pen, bool show) {
	myLine->setPen(pen);
	myLine->attach(plot.get());
	myLine->setVisible(show);
	plot->replot();
}

std::vector<punkt<double>> points

identisch zu denen sind, die dieser Aufruf liefert:

std::vector<punkt<double>>points = q.werte(range.von, range.bis, range.schrittweite);

Mir scheint, dass dieser Aufruf nicht das liefert, was du haben möchtest.

Mir scheint, dass dieser Aufruf nicht das liefert, was du haben möchtest.

Doch genau das tut er. Wenn ich mir die Daten im vector anschaue finde ich exakt die ich eingeben habe. Das ist es ja was mich so verrückt macht.

std::shared_ptr<QwtPlotCurve> Plotter::create_spline(funktion &q, ploting_range &range, bool show, QPen pen) {
    std::shared_ptr<QwtPlotCurve>myLine(new QwtPlotCurve(QString("")));
    std::vector<punkt<double>>points = q.werte(range.von, range.bis, range.schrittweite);
    return create_spline(points, show, pen);
}

Dann würdest du in Variante 1 garantiert den selben Vector benutzen, den du sonst in Variante 2 verwendet hast.

Edit: Gibt es einen Grund, dass du in create_spline keine Referenz auf den Vektor übergibst?

Was passiert denn, wenn du aus Variante 2, Variante 1 aufrufst?

Dann tritt ebenfalls der Fehler auf.
Dennoch weiß ich jetzt noch nicht woran es liegt.
Demnach muss es irgendwo in der Funktion

std::shared_ptr<QwtPlotCurve> Plotter::create_spline(std::vector<punkt<double>>& points, bool show, QPen pen)
{
	std::shared_ptr<QwtPlotCurve>myLine(new QwtPlotCurve(QString("")));
	plot_line(myLine, points, show, pen);
	return myLine;
}

ein Problem geben, aber welches?

Schlangenmensch schrieb:

Edit: Gibt es einen Grund, dass du in create_spline keine Referenz auf den Vektor
übergibst?

Keinen expliziten...

Gibt es drei überladene Funktionen, in meinem Fall wird die zweite Verwendet.

Wie sieht denn QwtPlotCurve::setSamples aus?

Ich bin der Meinung, es spielt überhaupt keine Rolle, was in plotLine oder noch tiefer in setSamples passiert.

axels. schrieb:

Variante 1

std::shared_ptr<QwtPlotCurve> Plotter::create_spline(std::vector<punkt<double>> points, bool show, QPen pen)
{
    std::shared_ptr<QwtPlotCurve>myLine(new QwtPlotCurve(QString("")));
    plot_line(myLine, points, show, pen);
    return myLine;
}

Variante 2

std::shared_ptr<QwtPlotCurve> Plotter::create_spline(funktion &q, ploting_range &range, bool show, QPen pen) {
    std::shared_ptr<QwtPlotCurve>myLine(new QwtPlotCurve(QString("")));
    std::vector<punkt<double>>points = q.werte(range.von, range.bis, range.schrittweite);
    plot_line(myLine, points,show, pen);
    return myLine;
}

plot_line bekommt 4 parameter. Aus deinem Post geht hervor, dass der erste ( myLine ) bei beiden identisch ist, weil innerhalb von create_spline identisch erzeugt. Wenn die beiden Varianten mit demselben show und demselben pen aufgerufen werden, kann der Fehler doch nur beim Parameter points liegen.

Ich bleibe dabei: stelle sicher, dass die 4 Parameter, die plot_line nimmt bei beiden Aufrufen identisch sind, dann muss auch dasselbe rauskommen. (Ist vielleicht beim einen Aufruf, bei dem kein Plot angezeigt wird, show == false ...?)

Keinen expliziten...

Hast du es denn mal mit der Referenz versucht? (Ich wollte nur ausschließen, dass mit dem Copy ctor von "punkt" etwas nicht stimmt.)

Ansonsten wäre evt. dann tatächlich die Implementation von punkt Interessant und wie die übergebene Funktion q, bzw. q.werte(....) aussieht.

hustbaer schrieb:

Wie sieht denn QwtPlotCurve::setSamples aus?

Ich bin der Meinung, es spielt überhaupt keine Rolle, was in plotLine oder noch tiefer in setSamples passiert.

Na, schön für dich.

Ich wollte einfach wissen ob setSamples ne "deep copy" macht. Tut es. Damit ist es OK. Täte es das nicht, hätte es der Grund für den Fehler sein können.

plot_line bekommt 4 parameter. Aus deinem Post geht hervor, dass der erste ( myLine ) bei beiden identisch ist, weil innerhalb von create_spline identisch erzeugt. Wenn die beiden Varianten mit demselben show und demselben pen aufgerufen werden, kann der Fehler doch nur beim Parameter points liegen.

Ich bleibe dabei: stelle sicher, dass die 4 Parameter, die plot_line nimmt bei beiden Aufrufen identisch sind, dann muss auch dasselbe rauskommen. (Ist vielleicht beim einen Aufruf, bei dem kein Plot angezeigt wird, show == false ...?)

Selbst wenn ich beide Funktionen direkt hintereinander aufrufe, mit den selben Parmetern tritt der Fehler auf.

Schlangenmensch schrieb:

Hast du es denn mal mit der Referenz versucht? (Ich wollte nur ausschließen, dass mit dem Copy ctor von "punkt" etwas nicht stimmt.)

Ja habe ich versucht. Hat nichts gebracht.

Schlangenmensch schrieb:

Ansonsten wäre evt. dann tatächlich die Implementation von punkt Interessant und wie die übergebene Funktion q, bzw. q.werte(....) aussieht.

punkt ist wie folgt implementiert:

template<typename T>
class punkt
{
public:
	punkt() :m_x(0), m_y(0) {};
	punkt(T x,T y){
		m_x = x;
		m_y = y;
	}
	punkt(std::string &v) {
		int o_brack = v.find("(");
		int mid = v.find("|");
		int c_brack = v.find(")");
		try
		{
			m_x = T(std::stod(v.substr(o_brack + 1, (o_brack + 1 - mid))));
			m_y = T(std::stod(v.substr(mid + 1, (mid + 1 - c_brack))));
		}
		catch (const std::invalid_argument&)
		{
			throw std::exception("Error! Invalid Expression");

		}
		catch (const std::out_of_range &) {
			throw std::exception("Error! Invalid Expression");
		}
	}
	T& get_x() {
		return m_x;
	}
	T& get_y() {
		return m_y;
	}

	T get_x() const {
		return m_x;
	}
	T get_y() const  {
		return m_y;
	} 

	T Round(T Zahl, int Stellen);

	static bool compare_x(const punkt& first, const punkt& second) {
		return first.get_x() < second.get_x();
	}

	bool operator!=(const punkt &rhs);
	bool operator==(const punkt& rhs);

	std::string to_string();
	template<typename T>
	friend std::ostream &operator<<(std::ostream &ostr, const punkt<T> &a);
	template<typename T>
	friend std::istream &operator>>(std::istream &istr, punkt<T> &a);

	virtual ~punkt() {}
protected:
	T m_x, m_y;

	void set_x(T t) {
		m_x = t;
	}
	void set_y(T t) {
		m_y = t;
	}

};

template<typename T>
inline T punkt<T>::Round(T Zahl, int Stellen)
{
	double v[] = { 1, 10, 1e2, 1e3, 1e4, 1e5, 1e6, 1e7, 1e8, 1e9, 1e10, 1e11,1e12,1e13,1e14,1e15,1e16,1e17 };  // mgl. verlängern 
	return floor(Zahl * v[Stellen] + 0.5) / v[Stellen];
}

template<typename T>
inline bool punkt<T>::operator!=(const punkt &rhs) {
	if (rhs.get_x() == get_x())
		if (rhs.get_y() == get_y())
			return false;
		else
			return true;
	else
		return true;
}

template<typename T>
inline bool punkt<T>::operator==(const punkt &rhs) {
	if (rhs.get_x() == get_x())
		if (rhs.get_y() == get_y())
			return true;
		else
			return false;
	else
		return false;
}

template<typename T>
inline std::ostream &operator<<(std::ostream &ostr, const punkt<T> &a) {
	ostr << "(" << a.get_x() << "|" << a.get_y() << ")\n";
	return ostr;
}

template<typename T>
inline std::istream &operator >> (std::istream &istr, punkt<T> &a) {
	std::string tmp; 
	std::getline(istr, tmp, '\n');
	a = punkt<T>(tmp);
	return istr;
}
template<typename T>
inline std::string punkt<T>::to_string() {
	return std::string("(" + boost::lexical_cast<std::string>(get_x()) + "|" + boost::lexical_cast<std::string>(get_y()) + ")\n");
}

Langsam wird's echt seltsam.

Hat noch jemand eine Idee?
Evtl. die Gegebenheiten des Aufrufers?

Prinzipiell ein minimales aber kompilierbares Beispiel bei dem der Fehler auftritt. Damit man alles zusammen hat, wo der Fehler liegen kann und sich das mal selbst im Compiler / Debugger anschauen kann.

Ich weiß schon, nur um ein kompilierbares Beispiel wird hier sehr schnell sehr groß, weil einfach ein rießen Ratenschwanz hinten dran hängt.