Sign in to follow this  

aa wire box neat tricks?

This topic is 1279 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

im often a little upset writing this thing (all this + - (or +0) combinations to set a 12 line segmants to draw a box)  but i from time to time do that

void DrawAAWireBox3d(float x, float y, float z, float h, unsigned colr)
{
 
 DrawLine3d(x-h,y-h,z-h,   x+h,y-h,z-h,   colr);
 DrawLine3d(x+h,y-h,z-h,   x+h,y+h,z-h,   colr);
 DrawLine3d(x+h,y+h,z-h,   x-h,y+h,z-h,   colr);
 DrawLine3d(x-h,y+h,z-h,   x-h,y-h,z-h,   colr);
 
 DrawLine3d(x-h,y-h,z+h,   x+h,y-h,z+h,   colr);
 DrawLine3d(x+h,y-h,z+h,   x+h,y+h,z+h,   colr);
 DrawLine3d(x+h,y+h,z+h,   x-h,y+h,z+h,   colr);
 DrawLine3d(x-h,y+h,z+h,   x-h,y-h,z+h,   colr);
 
 DrawLine3d(x-h,y-h,z-h,   x-h,y-h,z+h,   colr);
 DrawLine3d(x+h,y-h,z-h,   x+h,y-h,z+h,   colr);
 DrawLine3d(x+h,y+h,z-h,   x+h,y+h,z+h,   colr);
 DrawLine3d(x-h,y+h,z-h,   x-h,y+h,z+h,   colr);
 
}

are there maybe some neat tricks that would help to wrote this quickier or make shorter code here?
also could some one improve it stylistically?
Edited by fir

Share this post


Link to post
Share on other sites

To improve it stylistically, you could consider editing your post to use the forum code tags. 

i did it and it ate second part of the text - still present bug

 

Other question is that this code colorisation here is ugly, sometimes it looks worse than black font

Share this post


Link to post
Share on other sites

You can imagine a box as a min and max vector in some space, and the edges of the box are the lines between any two verts that differ in only one axis. Not sure you'll think this is much of an improvement, but it's an alternative nonetheless.

void DrawAAWireBox3d(float x, float y, float z, float h, unsigned colr)
{
    Vector3 min(x-h, y-h, z-h);
    Vector3 max(x+h, y+h, z+h);

    for (int i = 0; i < 8; i++) {
        for (int j = i + 1; j < 8; j++) {
            int diff = i ^ j; // determine number of axes these verts differ in
            if (NumberOfBitsSet(diff) == 1) { // if it's only one axis then draw a line
                Vector3 a((i & 1) ? max.x, min.x,
                          (i & 2) ? max.y, min.y,
                          (i & 4) ? max.z, min.z);
                Vector3 b((j & 1) ? max.x, min.x,
                          (j & 2) ? max.y, min.y,
                          (j & 4) ? max.z, min.z);

                DrawLine3d(a.x, a.y, a.z, b.x, b.y, b.z, colr);
            }
        }
    }
}

Share this post


Link to post
Share on other sites

 

You can imagine a box as a min and max vector in some space, and the edges of the box are the lines between any two verts that differ in only one axis. Not sure you'll think this is much of an improvement, but it's an alternative nonetheless.

void DrawAAWireBox3d(float x, float y, float z, float h, unsigned colr)
{
    Vector3 min(x-h, y-h, z-h);
    Vector3 max(x+h, y+h, z+h);

    for (int i = 0; i < 8; i++) {
        for (int j = i + 1; j < 8; j++) {
            int diff = i ^ j; // determine number of axes these verts differ in
            if (NumberOfBitsSet(diff) == 1) { // if it's only one axis then draw a line
                Vector3 a((i & 1) ? max.x, min.x,
                          (i & 2) ? max.y, min.y,
                          (i & 4) ? max.z, min.z);
                Vector3 b((j & 1) ? max.x, min.x,
                          (j & 2) ? max.y, min.y,
                          (j & 4) ? max.z, min.z);

                DrawLine3d(a.x, a.y, a.z, b.x, b.y, b.z, colr);
            }
        }
    }
}

huh, this is strange :U though at leas something (bit hard tu understand for me) - maybe something simpler? it seem that there should be something simpler

Share this post


Link to post
Share on other sites

 It's this sort of thing that makes me wish there were a GL_LINE_FAN primitive type in OpenGL

if this would be about line segments it wouldnt called a fan i think ;\

though this is good idea, define array of 8 vertices, then do some for loop that would draw a pairs of it

 

table[8] = {000,001,011,010, ...}

 

nay it wouldnt strictly work, as wirebox canot be construct this way without reapiting....

Edited by fir

Share this post


Link to post
Share on other sites

You basically need a list of the edges and then loop over them to render lines.

 

You could abstract this into a some sort of generic "geometry class" that describes a shape (box, pyramid, whatever else you have) and can give you:

-List of vertices

-List of edges (index to start and index to end vertex)

-List of surfaces

etc.

 

If using C++ I would make one class for each shape and then use templates to make a wiremesh renderer that can draw any shape (basically compile time polymorphism, you dont probably like runtime overhead...):

template T
void drawGenericShape(T shape)
{
    for each edge in shape.getEdges() do
    {
        drawLine(shape.getVertex(edge.A),shape.getVertex(edge.B))
    }
}

However, to create the specific shape class to represent a box, I would still just manually hardcode the vertices. This is because unless you want it to support 2 and 4 dimensional boxes too, its not really a useful target for a generalization. And youll have to use a different approach for every type of shape anyways. So yeah, I would just make an array of vertices and an array of edges and an array of surfaces.

Share this post


Link to post
Share on other sites

You basically need a list of the edges and then loop over them to render lines.

 

You could abstract this into a some sort of generic "geometry class" that describes a shape (box, pyramid, whatever else you have) and can give you:

-List of vertices

-List of edges (index to start and index to end vertex)

-List of surfaces

etc.

 

If using C++ I would make one class for each shape and then use templates to make a wiremesh renderer that can draw any shape (basically compile time polymorphism, you dont probably like runtime overhead...):

template T
void drawGenericShape(T shape)
{
    for each edge in shape.getEdges() do
    {
        drawLine(shape.getVertex(edge.A),shape.getVertex(edge.B))
    }
}

However, to create the specific shape class to represent a box, I would still just manually hardcode the vertices. This is because unless you want it to support 2 and 4 dimensional boxes too, its not really a useful target for a generalization. And youll have to use a different approach for every type of shape anyways. So yeah, I would just make an array of vertices and an array of edges and an array of surfaces.

 

iterating on array by for (than hardkoding 12 callls is good, but feeling it with hand returns us almost back to the first problem - There would be for example some need for formula to make up this array instead of feeling it by hand (those feeling it with hand is most troublesome,

copy and paste calls is easy)

Share this post


Link to post
Share on other sites

This topic is 1279 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

Sign in to follow this