2012-09-03 40 views
0

在SFML论坛未能获得任何帮助后,我决定查看是否有人可以获得帮助。我有一个应用程序,它通过TCP连接更新到服务器应用程序。除了一个特定的数据包传输,其中数据包以某种方式被改变之外,这工作得很好。服务器发送一个包含数字1包:我的调试器步进通过正在更改的SFML TCP数据包

pack << sf::Uint8(1); 
ClientSocket.Send(pack); 

(其中pack的类型是科幻::包和ClientSocket的类型是科幻:: SocketTCP)

事实上确实确认这些线路被执行,并且在我的客户端下一个接收呼叫接下来的几行:

sock.Receive(pack); 
sf::Uint8 conf; 
pack >> conf; 

(其中pack再次是SF ::包,sock是SocketTCP)

但是,在我的客户端中,conf显示为零值(我的调试器确认了这一点),它的行为如此(下一行根据值切换)。我在同一台计算机上运行这两个应用程序,并且它们都互相读取为“127.0.0.1”,因此数据包(如果我理解正确)不会通过我的路由器,或者真的离开我的机器。关于为什么我的数据包被损坏的任何想法?

为了记录,如果有任何帮助,我使用SFML 1.6和我的应用程序在Debian Squeeze Linux机器上运行。

编辑:每个请求,这里是我的应用程序的一些更多的代码。

首先,我的客户的具体涉及到这个更新的部分:

cout << "Connecting to update server." << endl; 
sf::IpAddress server = sf::IpAddress::LocalHost; 
sf::TcpSocket sock; 
if (sock.connect(server, PORT, sf::seconds(20.0f)) != sf::Socket::Done) 
    return false; 

cout << "Connected. Searching for updates for updater." << endl; 

sf::Packet pack; 
std::string lastUpdate = getLastUpdate(); 

#ifdef __WXGTK__ 
pack << "uupdate" << lastUpdate << sf::Uint8(1); 
#elif defined(__WXMSW__) 
pack << "uupdate" << lastUpdate << sf::Uint8(2); 
#elif defined(__WXOSX__) 
pack << "uupdate" << lastUpdate << sf::Uint8(3); 
#endif 

sock.send(pack); 

pack.clear(); 

sock.receive(pack); //THIS IS THE PART WHERE IT BREAKS 
sf::Int32 conf; 
pack >> conf; 

if (conf == 1) { //There is an update! 
    cout << "Update found!" << endl; 

    pack << "begin" << sf::Uint32(512); 
    sock.send(pack); 

    pack.clear(); 

    sock.receive(pack); 
    pack >> conf; 

    if (conf == 0) { //No errors 
     wxFileOutputStream* out = new wxFileOutputStream(wxString(UPDATER, StrConv).append(".temp")); 

     char* data = new char[512]; 
     while (data != chara("done") && data != chara("error")) { 
      sock.receive(pack); 
      sf::Int32 size; 
      pack >> size; 
      data = const_cast<char*>((const char*)pack.getData()); 

      out->Write(data, size); 
     } 
     out->Close(); 
     if (data == chara("done")) 
      wxCopyFile(wxString(UPDATER, StrConv).append(".temp"), wxString(UPDATER, StrConv)); 
     wxRemoveFile(wxString(UPDATER, StrConv).append(".temp")); 
    } 
} 

cout << "Updater is up-to-date. Executing updater." << endl; 

sock.disconnect(); 

bool success; 
if (wxTheApp->argc == 1) 
    success = wxExecute(wxT(UPDATER)); 
else 
    success = wxExecute(wxString(UPDATER, StrConv).append(wxTheApp->argv[1])); 

return success; 

,然后服务器有关这些更新的部分:

//ClientSocket has already been initalised by way of a TcpListener 
cout << "Checking for updater patches for client at " << ClientAddress.toString() << endl; 

std::string lastUpdate; 
pack >> lastUpdate; 
sf::Uint8 os; 
pack >> os; 

//Check last time updater was changed 
struct tm* clock; 
struct stat attribs; 
if (os == 1) 
stat("Linux/Updater", &attribs); 
else if (os == 2) 
stat("Windows/Updater.exe", &attribs); 
else if (os == 3) 
stat("Mac/Updater", &attribs); 
clock = gmtime(&(attribs.st_mtime)); 
time_t lastWrite = mktime(clock); 

time_t clientUpdate = stringToNumber<time_t>(lastUpdate.c_str()); 

if (lastWrite > clientUpdate) { 
    cout << "Found updater patches for client at " << ClientAddress.toString() << endl; 

    pack << sf::Int32(1); 
    ClientSocket->send(pack); //THIS IS THE PART WHERE IT BREAKS 

    pack.clear(); 

    ClientSocket->receive(pack); 
    std::string mes; 
    pack >> mes; 

    if (mes != "begin") { 
     cerr << "Client at " << ClientAddress.toString() << " sent unrecognised message: " << mes << ". Ending conversation." << endl; 
     ClientSocket->disconnect(); 
     return false; 
    } 

    sf::Uint32 size; 
    pack >> size; 

    cout << "Beginning to send updater patch to " << ClientAddress.toString() << endl; 

    ifstream in; 

    if (os == 1) 
     in.open(chara("Linux/Updater"), ios_base::in | ios_base::binary); 
    else if (os == 2) 
     in.open(chara("Windows/Updater.exe"), ios_base::in | ios_base::binary); 
    else if (os == 3) 
     in.open(chara("Mac/Updater"), ios_base::in | ios_base::binary); 

    if (in.fail()) { 
     pack << sf::Uint8(1); 
     ClientSocket->send(pack); 
     cerr << "Failed to open updater at request of " << ClientAddress.toString() << ". Closing connection." << endl; 
     ClientSocket->disconnect(); 
     return false; 
    } 

    pack << sf::Uint8(0); 
    ClientSocket->send(pack); 

    pack.clear(); 

    //Get length of file 
    in.seekg(0, ios::end); 
    int length = in.tellg(); 
    in.seekg(0, ios::end); 

    char* buf = new char[size]; 

    for (unsigned int i = 0; i < length/size; i++) { //Read and send every `size`-sized block we can 
     in.read(buf, size); 
     pack << sf::Int32(size) << buf; 
     ClientSocket->send(pack); 
    } 

    in.read(buf, length % size); 
    pack << sf::Int32(length % size) << buf; 
    ClientSocket->send(pack); 

    pack.clear(); 

    pack << "done"; 
    ClientSocket->send(pack); 

    pack.clear(); 
} else { 
    pack << sf::Int8(0); 
    ClientSocket->send(pack); 
} 

编辑:我知道这是近一个月以来,在这里有任何帖子,但我仍然有这个问题。我很机智的结束如何解决这个问题。

+0

你确定你正在阅读这个包,而不是最后一个的剩菜? – Dani

+0

是的。我正在使用阻塞套接字,并且我确保服务器上的每个发送呼叫都与客户端上的一个接收呼叫匹配(反之亦然)。我也看到我的客户端挂在上面显示的接收呼叫上,直到服务器发送上面显示的发送呼叫。 更不用说,在那之前的数据包在任何地方都没有0。 –

+0

尝试更改数据类型/值以查看它是一切还是只是uint8。 – Dani

回答

0

所以,原来问题很简单。你知道我是如何添加大量pack.clear()行的吗?事实证明,我忘了在接收原始数据和在服务器上发送原始数据之间添加一个。所以我的数据包仍然有数据从第一次调用到sock.receive(pack)(这是从上面发布的服务器的代码开始的几行)剩余的数据。我花了很长时间才看到这一点。打电话给pack.clear(),问题就消失了。

1

这是不是一个真正的答案,但我不能添加的问题,所以评论...

SFML 1.6不能维持,如果有与SFML本身的问题,它可能已经在版本来解决2.0。您可以下载RC here

+0

明天我可以发布更多的代码作为Apprentice Queue,但现在我想发表评论,我已更新到2.0,而且我仍然看到同样的问题。正如我所说,明天我会发布更多的代码。 –

+0

@Apprentice,我已经添加了源代码,希望它可以解决问题。 –