etwas, was für mich unerklärlich ist und das Programm zum Absturz bringt:
Ich habe 4 Klassen:
` - Point

• Style

• ElementB( erbt von Style )

• ElementA( erbt von Style )`

Ungefährer Aufbau der Klassen:

struct Point
{
   double x;
   double y;

   Point( double xy = 0 ) : x( xy ), y( xy ) {};
   Point( double nx, double ny ) : x( nx ), y( ny ) {};
   Point( const Point &xy ) : x( xy.x ), y( xy.y ) {};
   ~Point( ) {}
   Point& operator=( const Point &xy )
   {
      this->x = xy.x;
      this->y = xy.y;
   }
   Point& operator=( double xy )
   {
      this->x = xy;
      this->y = xy;
   }

   //...
};

class Style
{
   private:
      Point size;

   public:
      Style( )
      {
         this->setDefault();
      }
      void setDefault()
      {
         this->size = 0;
      }
      bool setSize( const Point &newSize )
      {
         if( newSize.x == 0  ||  newSize.y == 0 ) return false;
         this->size = newSize;
         return true;
      }
      bool setSize( double newX, double newY )
      {
         return this->setSize( Point( newX, newY ) );
      }

      //...
};

class ElementB : public Style { /*...*/ };

class ElementA : public Style
{
   private:
      ElementB *elem;

   public:
      void draw()
      {
         Point s( 9.0, 11.0 );
         this->elem->setSize( 15.0, 23.0 ); //Funktioniert!
         this->elem->setSize( s ); //Programm stürtzt ab!
      }
};

An der Stelle, an der ich setSize ein Point-Objekt übergebe, anstatt zwei double-Werte, gibts nen Programmabsturz. Aber wieso?
Was muss ich machen, damit beides( setSize( double,double), setSize( Point ) ) funktioniert?

MfG. Ich

Grüssli

Aber wenn die erste Zeile funktioniert, kann das ja eigentlich nicht sein (außer der Code stimmt so nicht, wie du ihn gepostet hast).

Bei undefined behavior ist ALLES möglich, daher kann man davon nicht ausgehen

Grüssli

Und: Im Konstruktor eines jeden Elementes werden benutzte Elemente auch erzeugt. Im Beispiel würde elem also auf ein ElementB-Objekt zeigen.
Dass elem auf nichts zeigt ist sowieso ausgeschlossen, da

this->elem->setSize( 6, 7 );

ja prima funktioniert.

Benutzt ein Element ein anderes ist der Konstrukor in etwa so:

Element::Element( )
   : elem( NULL )
{
   this->elem = new ElementB( ... );
}

Ich such mal was raus und lad's hoch...

Die "Problem-Dateien" lade ich gleich mal hoch.

Hochladen?
Was verstehst du nicht an MINIMAL? Mach ein kurzes einfaches Beispielprogramm, welches dein Problem repräsentiert und den Fehler reproduziert. Es wird hier niemand deine Quellcodedateien durchschauen und sich dann noch selber ein Beispielprogramm schreiben.

ChristophLu schrieb:

Dass elem auf nichts zeigt ist sowieso ausgeschlossen, da

this->elem->setSize( 6, 7 );

ja prima funktioniert.

Nein, es ist nicht ausgeschlossen. Es wäre undefined behavior, da ist alles möglich. Es darf funktionieren, es darf nicht funktionieren, es darf die Welt untergehen, es kann dein Computer explodieren, es ist halt einfach nicht definiert, was passieren wird.

Grüssli

Die betreffenden Zeilen sind: Counter.cpp.txt-> Zeile 107 und 111

http://home.arcor.de/minicrispie/Problem/

Ich hoffe, das hilft weiter oO

So ...

Die betreffenden Zeilen sind: Counter.cpp.txt-> Zeile 107 und 111

http://home.arcor.de/minicrispie/Problem/

Ich hoffe, das hilft weiter oO

MINIMAL-Beispiel!!!!111einself
glaub nicht, dass sich hier wer den gesamten code angucken möchte (btw finde ich kommentare wie END OF FILE nicht gerade zweckmäßig)
außerdem hat man dort auch kein syntax-highliting -> post es halt hier - du kannst es danach noch immer rauseditieren, wenn dir so ist.
btw. was haben 2 buttons mit nem counter zu tun? komplett fail sind auch die default-werte für die positionen. ne gleitkommazahl zu nehmen ist auch nicht so extrem klug, weil sie eben nicht sonderlich genau ist - können also durchaus mal nen paar mehr oder weniger steps sein(und wenns ganz dumm läuft, dann auch unendlich viele)... viel mehr hab ich mir aber nicht angeguckt...

bb

Aber wenn du ein funkionierendes Beispiel willst, bitte:

#include <iostream>
using namespace std;

class Point
{
	public:
		double x;
		double y;

		Point( double xy = 0 ) : x( xy ), y( xy ) {}
		Point( double nx, double ny ) : x( nx ), y( ny ) {}
		Point( const Point &p ) : x( p.x ), y( p.y ) {}
		~Point( ) {}

		Point& operator=( double xy )
		{
			this->x = xy;
			this->y = xy;
			return *this;
		}
};

class Style
{
	protected:
		Point size;

	public:
		Style( )
		{
			this->reset();
		}
		~Style( ) {}
		void reset()
		{
			this->size = 10;
		}
		bool setSize( const Point &newSize )
		{
			if( newSize.x == 0  ||  newSize.y == 0 ) return false;
			this->size = newSize;
			return true;
		}
		bool setSize( double x, double y )
		{
			return this->setSize( Point( x, y ) );
		}
};

class ElementA : public Style
{
	public:
		ElementA( )
			: Style()
		{};
		~ElementA( ) {}
};

class ElementB : public Style
{
	private:
		ElementA *elem;

	public:
		ElementB( )
			: Style(), elem( NULL )
		{
			this->elem = new ElementA();
		}
		~ElementB( ) {}

		void draw( )
		{
			Point s( 2, 3 );
			this->elem->setSize( s ); //Funktioniert
			this->elem->setSize( 15, 23 ); //Funktioniert ebenfalls
		}
};

int main()
{
	ElementB elem;
	elem.draw();
	cout << "Hallo..." << endl;
	return 0;
}

Aber im Originalcode funkioniert es eben nicht mit setSize( s ) .. bei diesem Funktionsaufruf stürtzt es ab.

bb

edit: wie wärs eigtl mal mit der main.cpp am anfang?

Wahrsager:
Ich habe eine gute und eine schlechte Nachricht für Sie.
Die Gute: sie gewinnen nächste Woche viel Geld!

Die Schlechte: sie sterben vorher bei einem Unfall.

btw. was haben 2 buttons mit nem counter zu tun? komplett fail sind auch die default-werte für die positionen. ne gleitkommazahl zu nehmen ist auch nicht so extrem klug, weil sie eben nicht sonderlich genau ist - können also durchaus mal nen paar mehr oder weniger steps sein(und wenns ganz dumm läuft, dann auch unendlich viele)

Es ist ein Counter-Element. Also sowas:
http://www.mikrocontroller.net/attachment/8990/terminal.png ( oben in der Mitte, nach "Count" )

Was meinst du wo mit den Steps?

ich meine so was in etwa(gerad keinen compiler zur hand - kann also sein, dass es dummerweise gerade trotzdem geht):
startwert: 1e15
endwert: 1e16
schrittweite: 1e-16

dem user (der klasse) so was zu erlauben, macht nicht viel sinn - soll er sich doch selbst drum kümmern, das umzurechnen...
ein counter sollte einfach nur zählen können - der rest hat in nem counter nix zu suchen...
ich bau doch auch keine klasse number und füg nen label mit dazu oO

bb

Da ist nichts mit unendlich viele steps oder so...

[versucht schlechtes design wegzuerklären, liest nicht richtig und zeigt auch sonst 0 eigeninitiative]

bye

Es ist kein schlechtes Design sondern einfach nur ein Steuerelement wie ein Button, ein Textfeld oder sonst irgendwas.

Und deine Klassen solltest du nochmal genauer anschauen.
Warum erben alle Klassen von Style samt virtuellen Methoden, Style hat aber keinen virtuellen Destructor?
Eine weitere Basis (die scheinbar dann auch genutzt werden soll für Elemente) basic_element wird eingeführt. redraw() gibts in Style, ein draw(Buffer, ..) erst mit basic_element.
Die Bezeichnung "Counter" ist irreführend, da nicht irgend welche Events oder sonstwas mitgezählt wird sondern eine doublebasierte Spinbox samt User-Input gemodelt wird.
Außerdem ist das mit den doubles echt ein Problem, da bedingt durch Ungenauigkeiten in der Darstellung zwei eigentlich gleiche Werte beim op== ein false liefern können, umgekehrt zwei verschiedene ein true. Damit ist nicht garantiert, dass dein Counter bei einem +2.0 auch in den Bounds bleibt.

Und Rechtschreibung wäre auch nicht schlecht (das heißt "receive" und nicht "recive")

Wegen dem Vergleich meinst du diese Zeile?

if( this->current == newCurrent ) return true;

Da bei einer Änderung des Wertes neu gezeichnet werden muss, wollte ich unnötiges Neuzeichnen ersparen, wenn der Wert gleich bleibt. Steht im Textfeld also 2.0 und der User will auf 2.0 setzen( setCurrent( 2.0 ) ), sollte nicht neu gezeichnet werden.
Das mit den Ungenauigkeiten ist mir bekannt. Wie vergleicht man sowas trotz der Ungenauigkeiten?

Meiner Meinung nach kann current nicht aus den Bounds hinausgehen, da ich die Grenzen mit <= und >= prüfe. Oder spielen die Ungenauigkeiten hier auch eine Rolle?
Problem ist: Der User soll mit dieser Spinbox auch Kommazahlen verwenden können. Also zum Beispiel:
`Step: 0.05

Min: 2.0

Max: 8.45

Current: 3.75`
Sowas sollte auch Möglich sein. Hier sollen Mögliche Werte auch 2.0 und 8.45 sein.

Ich dachte, es hieße Counter, aber Spinbox heißt es eigentlich. Werde ich umbenennen.( genauso wie receive )