Thread: Why does this keep frezzing up when I run it?

No errors when compiling and I can comment out most of the main function and it still frezzes.
Remove the SDL parts and it won't freeze up but then again the SDL is most of the program.


CODE:

Code:

//Main.cpp
#include <SDL.h>
#include <iostream>
#include "Socket.h"
#include <fstream>
#include <string>

using namespace std;

SDL_Surface *screen;
SDL_Event event;

void apply_surface( int x, int y, SDL_Surface* source)
{
    //Make a temporary rectangle to hold the offsets
    SDL_Rect offset;
    
    //Give the offsets to the rectangle
    offset.x = x;
    offset.y = y;
    //Blit the surface
    SDL_BlitSurface( source, NULL, screen, &offset );
    
    SDL_Flip(screen);
    
    
}

void clearscreen(){
            SDL_FillRect( screen, &screen->clip_rect, SDL_MapRGB( screen->format, 0x00, 0x00, 0x00 ) );
}













class Dot
{
    private:
    //The X and Y offsets of the dot
    int x, y;



    public:
    //Initializes the variables
    Dot();

    //Takes key presses and adjusts the dot's velocity
    void handle_input(int xz, int yz);

    //Moves the dot
    void move();

    //Shows the dot on the screen
    void show();
    
    int getx();
    int gety();
};






Dot::Dot()
{
    //Initialize the offsets
    x = 0;
    y = 0;

}



void Dot::handle_input(int xz,int yz)
{
x = xz;
y = yz;
}

void Dot::show()
{
SDL_Rect basd;
basd.w = 10;
basd.h = 10;
basd.y = y;
basd.x = x;
SDL_FillRect( screen, &basd, SDL_MapRGB( screen->format, 0x00, 0x99, 0x00 ) );
}


int Dot::getx(){
    return x;
}
int Dot::gety(){
    return y;
}


int main(int argc, char* args[])
{
              screen = SDL_SetVideoMode(500,500,32,0);
      //Dot ase;
    int choice;
    int port = 4521;
    //char *ipAddress = "127.0.0.1";
    string ipAddress;
    bool done = false;
    char recMessage[STRLEN];
    char sendMessage[STRLEN];


        //Client

        
        ClientSocket sockClient;

        sockClient.ConnectToServer("127.0.0.1", 4753 );


        //Connected

//74.76.149.190

          string jppa;
          int pos1,pos2;




        while ( true )
        {     

sockClient.RecvData(recMessage,STRLEN);
string sad = recMessage;

if(strlen(sad.c_str()) > 6){

int pos = sad.find("BA"); 
string xc,yc;

xc = sad.substr(0,3); 
yc = sad.substr(pos+3);

int ll,kk;

ll = atoi(xc.c_str());
kk = atoi(yc.c_str());


}


SDL_Delay(150); 
}



        }

I don't see, in your main program, where it does anything that you can see. You're constantly getting messages, and searching through messages, but.

There is a delay of 150 milliseconds on the loop.
That should slow it down decently.

It sends nothing.
Recives some.

Why should it send anything? I don't see any send functions in the code either.

(And just for reference -- why use strlen on a std::string? It has a size() built in.)

So, I'd use:

string bob = "HI";
int saize = bob.size();

For strlen from now on?


Also there are NO send functions in it, that is correct.

Originally Posted by azjherben

So, I'd use:

string bob = "HI";
int saize = bob.size();

For strlen from now on?

That is correct.

Originally Posted by azjherben

Also there are NO send functions in it, that is correct.

Then I guess that's why it wasn't sending anything.

It's not supposted to.
It crashes, like it frezzes up and I gotta get task manager to close it.

I'm not sure what exactly you want to happen here. You have written a program that doesn't do anything (including stop) and therefore that's what it does. What do you want it to do?

Here is my UPDATED PROGRAM:
(way better; actually runs; but still frezzes after 5-10 seconds)

Code:

#include <Socket.h>
#include <SDL.h>
#include <iostream>
#include <sstream>
SDL_Event event;
SDL_Surface *screen;
ClientSocket client;
void apply_surface( int x, int y, SDL_Surface* source)
{
    //Make a temporary rectangle to hold the offsets
    SDL_Rect offset;
    
    //Give the offsets to the rectangle
    offset.x = x;
    offset.y = y;
    //Blit the surface  
    SDL_BlitSurface( source, NULL, screen, &offset );
    
    SDL_Flip(screen);
    
    
}

void clearscreen(){
            SDL_FillRect( screen, &screen->clip_rect, SDL_MapRGB( screen->format, 0x00, 0x00, 0x00 ) );
}













class Dot
{
    private:
    //The X and Y offsets of the dot
    int x, y;



    public:
    //Initializes the variables
    Dot();

    //Takes key presses and adjusts the dot's velocity
    void handle_input(int xx,int yy);

    //Moves the dot
    void move();

    //Shows the dot on the screen
    void show();
    
    int getx();
    int gety();
};






Dot::Dot()
{
    //Initialize the offsets
    x = 0;
    y = 0;

}



void Dot::handle_input(int xx,int yy)
{
y = yy;
x = xx;
}

void Dot::show()
{
SDL_Rect basd;
basd.w = 10;
basd.h = 10;
basd.y = y;
basd.x = x;
SDL_FillRect( screen, &basd, SDL_MapRGB( screen->format, 0x00, 0x99, 0x00 ) );
}


int Dot::getx(){
    return x;
}
int Dot::gety(){
    return y;
}

int main(int argc, char* args[])
{
string aaw;
    char coords[11];
   client.ConnectToServer("74.76.147.199",4753);
    
    screen = SDL_SetVideoMode(500,500,32,0);
    clearscreen();
int zx,zy;
Dot mydot;
Dot hisdot;
bool quit = false;





while(quit == false){
zx = 50;
zy = 50;
for(int p = 1; p<500; p++){


aaw = coords;

if (aaw.size() > 4){

//                100 BA 100

int pos = aaw.find("BA");
string px1 = aaw.substr(0,3);
string px2 = aaw.substr(pos+3,3);              

zx = atoi(px1.c_str());
zy = atoi(px2.c_str());
}

if(zx < 3){
      zx = 150;
      }

if(zy < 3){
      zy = 150;
      }
}
mydot.handle_input(zx,zy);
clearscreen();
mydot.show();
SDL_Flip(screen);



SDL_Delay(100);


client.RecvData(coords,11);

SDL_Delay(100);
}

}

Maybe put an SDL_Quit at the end? It also kills the window.

I tried SDL_Quit()...
I'll add SDL_Init() ...

Think that will help at all?

We're all on pins and needles awaiting your test :-)

Yes, I think that should help - the docs say it is required, so it probably is for good reason.

I'm sorry to say that just adding the function SDL_Init(blah) right before I set up the screen (screen = SDL_SETVIDEOMODE(blah,blah,blah,blah)) did little to help.