// (C) 2011 Underground Software
// See the README and GPLv3 files for licensing and warranty information
//
-// JLH = James L. Hammons <jlhamm@acm.org>
+// JLH = James Hammons <jlhamm@acm.org>
//
// WHO WHEN WHAT
// --- ---------- ------------------------------------------------------------
{
}
+
Line::~Line()
{
+// Taking care of connections should be done by the Container, as we don't know
+// anything about any other object connected to this one.
+#if 0
// If there are any attached Dimensions, we must set the attachment points
// to NULL since they will no longer be valid.
if (attachedDimension)
// IT WOULD BE NICE to have any object points attached to this line automagically
// connect to this dimension object at this point, instead of just becoming
// detached.
+#endif
}
+
/*virtual*/ void Line::Draw(Painter * painter)
{
painter->SetPen(QPen(Qt::red, 2.0, Qt::DotLine));
Vector current(point2 - point1);
Vector v = current.Unit() * length;
Vector v2 = point1 + v;
- painter->DrawLine((int)point1.x, (int)point1.y, (int)v2.x, (int)v2.y);
+// painter->DrawLine((int)point1.x, (int)point1.y, (int)v2.x, (int)v2.y);
+ painter->DrawLine(point1, v2);
if (current.Magnitude() > length)
{
painter->SetPen(QPen(QColor(128, 0, 0), 1.0, Qt::DashLine));
- painter->DrawLine((int)v2.x, (int)v2.y, (int)point2.x, (int)point2.y);
+// painter->DrawLine((int)v2.x, (int)v2.y, (int)point2.x, (int)point2.y);
+ painter->DrawLine(v2, point2);
}
}
+// Problem: when drawing at large zoom levels, this throws away precision thus
+// causing the line to rendered too short. !!! FIX !!! [DONE]
else
- painter->DrawLine((int)position.x, (int)position.y, (int)endpoint.x, (int)endpoint.y);
+// painter->DrawLine((int)position.x, (int)position.y, (int)endpoint.x, (int)endpoint.y);
+ painter->DrawLine(position, endpoint);
}
/*virtual*/ Vector Line::Center(void)
// (Priorities are taken care of in HitTest()...)
if (hitLine)
{
+#if 0
if (attachedDimension == NULL)
{
// How to get this object into the top level container???
But we can fix that by making this object call any attached object's (like
a dimension only) Draw() function... :-/
*/
- attachedDimension = new Dimension(&position, &endpoint, this);
+ attachedDimension = new Dimension(&position, &endpoint, DTLinear, this);
if (parent != NULL)
parent->Add(attachedDimension);
// If there's one already there, tell it to flip sides...
attachedDimension->FlipSides();
}
+#else
+ // New approach here: We look for connected objects.
+ Object * attachedDimension = FindAttachedDimension();
+
+ if (attachedDimension)
+ {
+ // If there's an attached Dimension, tell it to switch sides...
+ ((Dimension *)attachedDimension)->FlipSides();
+ }
+ else
+ {
+ // Otherwise, we make a new one and attach it here.
+ attachedDimension = new Dimension(Connection(this, 0), Connection(this, 1.0), DTLinear, this);
+ connected.push_back(Connection(attachedDimension, 0));
+ connected.push_back(Connection(attachedDimension, 1.0));
+
+ if (parent != NULL)
+ parent->Add(attachedDimension);
+ }
+#endif
return true;
}
return false;
}
+
/*virtual*/ void Line::PointerMoved(Vector point)
{
// Hit test tells us what we hit (if anything) through boolean variables. It
}
}
+
/*virtual*/ void Line::PointerReleased(void)
{
if (draggingHandle1 || draggingHandle2)
state = oldState;
}
-#if 0
-void Line::SetDimensionOnPoint1(Dimension * dimension)
+
+// Check to see if the point passed in coincides with any we have. If so, return a
+// pointer to it; otherwise, return NULL.
+/*virtual*/ Vector * Line::GetPointAt(Vector v)
{
- dimPoint1 = dimension;
+ if (v == position)
+ return &position;
+ else if (v == endpoint)
+ return &endpoint;
- if (dimension)
- dimension->SetPoint1(position);
+ return 0;
}
-void Line::SetDimensionOnPoint2(Dimension * dimension)
+
+/*virtual*/ void Line::Enumerate(FILE * file)
{
- dimPoint2 = dimension;
+ fprintf(file, "LINE (%lf,%lf) (%lf,%lf)\n", position.x, position.y, endpoint.x, endpoint.y);
+}
+
- if (dimension)
- dimension->SetPoint2(endpoint);
+/*virtual*/ Object * Line::Copy(void)
+{
+#warning "!!! This doesn't take care of attached Dimensions !!!"
+/*
+This is a real problem. While having a pointer in the Dimension to this line's points is fast & easy,
+it creates a huge problem when trying to replicate an object like this.
+
+Maybe a way to fix that then, is to have reference numbers instead of pointers. That way, if you copy
+them, ... you might still have problems. Because you can't be sure if a copy will be persistant or not,
+you then *definitely* do not want them to have the same reference number.
+*/
+ return new Line(position, endpoint, parent);
}
-#else
+
+
+/*virtual*/ Vector Line::GetPointAtParameter(double parameter)
+{
+ if (parameter <= 0)
+ return position;
+ else if (parameter >= 1.0)
+ return endpoint;
+
+ // Our parameter lies between zero and one, so calculate it!
+ Vector v(endpoint, position);
+ double length = v.Magnitude();
+ // We scale the magnitude of v so that it lies between 0 and 1...
+ // By multiplying the parameter by the magnitude, we obtain the point we
+ // want. No scaling necessary as it's inherent in the approach!
+ double spotOnLength = length * parameter;
+
+ // To get our point, we use the initial point of the line and add in our
+ // scaled point.
+ Vector result = position + (v * spotOnLength);
+ return result;
+}
+
+
void Line::SetDimensionOnLine(Dimension * dimension/*=NULL*/)
{
// If they don't pass one in, create it for the caller.
if (dimension == NULL)
{
- dimension = new Dimension(&position, &endpoint, this);
+//printf("Line::SetDimensionOnLine(): Creating new dimension...\n");
+// dimension = new Dimension(position, endpoint, DTLinear, this);
+ dimension = new Dimension(Connection(this, 0), Connection(this, 1.0), DTLinear, this);
if (parent)
+//{
+//printf("Line::SetDimensionOnLine(): Adding to parent...\n");
parent->Add(dimension);
+//}
+ }
+ else
+ {
+ dimension->Connect(this, 0);
+ dimension->Connect(this, 1.0);
}
- attachedDimension = dimension;
+ // Make sure the Dimension is connected to us...
+ Connect(dimension, 0);
+ Connect(dimension, 1.0);
+}
+
- // After we set the points here, we don't have to care about them anymore.
- if (dimension)
+Object * Line::FindAttachedDimension(void)
+{
+ // Is there anything connected to this line? If not, return NULL
+ if (connected.size() < 2)
+ return NULL;
+
+ // Otherwise, we have to search our objects to see if there's a likely
+ // candidate. In this case, we're looking for a pointer to the same object
+ // with a parameter of 0 and 1 respectively. This is O((n^2)/2).
+ for(uint i=0; i<connected.size(); i++)
{
- dimension->SetPoint1(&position);
- dimension->SetPoint2(&endpoint);
+ for(uint j=i+1; j<connected.size(); j++)
+ {
+//printf("Line: connected[i]=%X, connected[j]=%X, connected[i].t=%lf, connected[j].t=%lf\n", connected[i].object, connected[j].object, connected[i].t, connected[j].t);
+ if ((connected[i].object == connected[j].object)
+ && ((connected[i].t == 0 && connected[j].t == 1.0)
+ || (connected[i].t == 1.0 && connected[j].t == 0)))
+ return connected[i].object;
+ }
}
+
+ // Didn't find anything, so return NULL
+ return NULL;
}
-#endif
+
bool Line::HitTest(Point point)
{
// simplifies the calculation of the determinant.
//How do we determine distance here? Especially if zoomed in or out???
-#warning "!!! Distances tested for may not be valid if zoomed in or out !!!"
- if (v1.Magnitude() < 8.0)
+//#warning "!!! Distances tested for may not be valid if zoomed in or out !!!"
+// [FIXED]
+ if ((v1.Magnitude() * Painter::zoom) < 8.0)
hitPoint1 = true;
- else if (v2.Magnitude() < 8.0)
+ else if ((v2.Magnitude() * Painter::zoom) < 8.0)
hitPoint2 = true;
- else if (distance < 5.0)
+ else if ((distance * Painter::zoom) < 5.0)
hitLine = true;
return StateChanged();
t(d1x) - s(d2x) = p2x - p0x
t(d1y) - s(d2y) = p2y - p0y
-Determinant D is ad - bc where the matrix look like:
+Determinant D is ad - bc where the matrix looks like:
a b
c d