Search code examples
c++fltk

FLTK fl_line and fl_point don't work


I want to place a pixel on the window in accordance to the result of a calculation. The calculation works, but no part of the graph I want to generate ever shows up. Compiler and linker both find no error and I have no idea why no pixels are put on the window. Since I can't figure out what the problem is and not narrow it down either I am posting the entire code:

#include <FL/Fl.H>
#include <FL/Fl_Window.H>
#include <FL/fl_draw.H>
#include <FL/Fl_Button.H>
#include <Fl/fl_Input.H>
#include <Fl/fl_Output.H>
#include <Physics.h>
#include <string.h>
#include <stdlib.h>
#include <Windows.h>

using namespace std;

#define WINSIZE 700
#define TIMESTEP 0.02f

class Graph : public Fl_Widget
{
public:
    Graph(int X, int Y, int W, int H) : Fl_Widget(X, Y, W, H) {}

    void draw(void) 
    {
        fl_color(FL_RED);
        fl_point(pos_x, (500 - pos_y));
    }

    int pos_x, pos_y;
};
//------------------------------------------------------------------------    

class WindowReq : public Fl_Window
{
public:
    WindowReq(int W, int H, const char* Title);

    Fl_Input*   InVel;
    Fl_Input*   InDeg;
    Fl_Output*  OutT;
    Fl_Output*  OutX;
    Fl_Output*  OutY;
    Fl_Button*  Calc;
    Graph*      graph;

private:
    static void calc_cb(Fl_Widget* o, void* v);
    inline void copyValueToX(char* c);
    inline void copyValueToY(char* c);
    inline void copyValueToT(char* c);
};

WindowReq::WindowReq(int W, int H, const char* Title) : Fl_Window(W, H, Title)
{
    begin();
        Calc = new Fl_Button((WINSIZE - 150), 50, 100, 30, "Calculate path");
        Calc->callback(&WindowReq::calc_cb, this);

        InVel = new Fl_Input(70, 50, 100, 30, "Velocity:");

        InDeg = new Fl_Input(((WINSIZE / 3) + 50), 50, 100, 30, "Angle:");

        OutX = new Fl_Output(70, 100, 100, 30, "Distance:");

        OutY = new Fl_Output(((WINSIZE / 3) + 50), 100, 100, 30, "Height:");

        OutT = new Fl_Output((WINSIZE - 150), 100, 100, 30, "Time:");

        graph = new Graph(0, 150, 700, 500);

        redraw();
    end();

    show();
}

void WindowReq::calc_cb(Fl_Widget* o, void* v)
{
    Projectile Stone;
    WindowReq* win = (WindowReq*)v;
    Coordinates2D Position = { 0,500 }, Buffer = Position;
    double velocity;
    double Degree;
    double Time;
    char Stuff[1000];

    velocity = atof(win->InVel->value());
    Degree = atof(win->InDeg->value());

    Stone.initialize(velocity, Degree, TIMESTEP);

    for (int i = 0; i < 700; i++)
    {
        fl_point(i, 400);
    }

    do
    {
        Position = Stone.getpos();
        Time = Stone.gettime();
        sprintf(Stuff, "%.2f", Time);
        win->copyValueToT(Stuff);
        sprintf(Stuff, "%d", Position.x);
        win->copyValueToX(Stuff);
        sprintf(Stuff, "%d", Position.y);
        win->copyValueToY(Stuff);
        Stone.simulatestep();
        win->graph->pos_x = Position.x;
        win->graph->pos_y = Position.y;
        win->graph->draw();
        Fl::check();
        Sleep(100);
    }while(Position.y >= 0);
}

void WindowReq::copyValueToX(char c[1000])
{
    OutX->value(c);
}

void WindowReq::copyValueToY(char c[1000])
{
    OutY->value(c);
}

void WindowReq::copyValueToT(char c[1000])
{
    OutT->value(c);
}

//============================================================
int main()
{
    WindowReq win(WINSIZE, WINSIZE, "Balistics Simulator");
    Fl::run();
}

Solution

  • The problem is most likely related either to your callback function called in the constructor body of WindowReq:

    Calc->callback(&WindowReq::calc_cb, this);
    

    try changing it to:

    Calc->callback(calc_cb, this);
    

    or in the action function calc_cb(), in the line:

    win->graph->draw();
    

    after the above line try adding:

    redraw();
    

    and replace the explicit type casting:

     WindowReq* win = (WindowReq*)v;
    

    with:

     WindowReq* win = reinterpret_cast<WindowReq*>(v);
    

    or in the definition of the function draw(), within class Graph. Those are the places related with not displaying the fl_point()s, representing pos_x and pos_y.

    Side note:

    there are some C-style lines in your code that should be replaced with their C++ alternatives, for example:

     copyValueToX(char c[1000]) 
    

    could be replace with std::string c and become:

     copyValueToX(string c) 
    

    similarly:

     sprintf(Stuff, "%d", Position.x);
    

    could become:

    std::cout << Position.x << std::endl;
    

    Finally, add few comments in your member functions to clarify statements and expressions in them.