Datei einlesen und uebertragen(Binär)

Reply to Datei einlesen und uebertragen(Binär) on Sun, 07 Oct 2007 13:16:50 GMT

HilfeNoetig — Sun, 07 Oct 2007 13:16:50 GMT

Hallo,
ich möchte gerne eine Datei versenden und habe dies auch schon mit folgendem Code geschafft:

std::ifstream thefile("Bla.dat", std::ios::in | std::ios::binary);
//...
std::string buffer;
while (std::getline(thefile, buffer)) {
	sendStr(connectedSocket, buffer + "\n");
}
thefile.close();

Das Problem ist, dass dieser Code zu langsam ist, da die Datei viele Zeilen enthält die nicht sehr lang sind! Ich habe also folgendes ausprobiert:

std::ifstream thefile("Bla.dat", std::ios::in | std::ios::binary);
//...
char buffer[512];
while (true) {
	thefile.read(buffer, sizeof(buffer));
	sendStr(connectedSocket, buffer);
	if (screenshot.eof()) { break; }
}
thefile.close();

Die Datei wird leider nicht komplett übertragen... Wo liegt der Fehler?

Vielen Dank im Voraus!!!

HilfeNoetig

Reply to Datei einlesen und uebertragen(Binär) on Sun, 07 Oct 2007 14:43:01 GMT

Fake oder Echt — Sun, 07 Oct 2007 14:43:01 GMT

#include 
#include 
#include 
#include 
#include 

#pragma comment( lib, "ws2_32.lib" )

typedef struct _NETFILEHEADER
{
    unsigned long FileSize;
    char pFileName[256];
} NETFILEHEADER;

long SendFileToNet(const char *pFileName,
                   const char *pIP,
                   unsigned short Port)
{
    FILE *pFile;
    unsigned long FileSize;
    char buffer[4096];
    NETFILEHEADER FileHeader;
    SOCKET sender;
    WSADATA wsa;
    SOCKADDR_IN addr;
    int addrlen = sizeof(SOCKADDR_IN);

    pFile = fopen(pFileName, "rb");

    if(!pFile) return 0;

    fseek(pFile, 0, SEEK_END);
    FileSize = ftell(pFile);
    fseek(pFile, 0, SEEK_SET);

    FileHeader.FileSize = FileSize;
    memset(FileHeader.pFileName, 0, 256);
    memcpy(FileHeader.pFileName, pFileName, strlen(pFileName));

    WSAStartup(MAKEWORD(2,0),&wsa);

    sender = socket(AF_INET, SOCK_STREAM, 0);

    memset(&addr, 0, sizeof(SOCKADDR_IN));
    addr.sin_family = AF_INET;
    addr.sin_port = htons(Port);
    addr.sin_addr.s_addr = inet_addr(pIP);

    connect(sender, (SOCKADDR*)&addr, addrlen);

    send(sender, (char*)&FileHeader, sizeof(NETFILEHEADER), 0);

    while(FileSize >= 4096)
    {
        fread(buffer, 4096, 1, pFile);
        send(sender, buffer, 4096, 0);

        FileSize -= 4096;
    }

    if(FileSize)
    {
        fread(buffer, FileSize, 0, pFile);
        send(sender, buffer, FileSize, 0);
    }

    closesocket(sender);

    fclose(pFile);

    WSACleanup();

    return FileSize;
}

long GetFileFromNet(unsigned short Port)
{
    SOCKET server;
    SOCKET client;
    WSADATA wsa;
    SOCKADDR_IN addr;
    char buffer[4096];
    long rval;
    FILE *pFile;
    NETFILEHEADER FileHeader;
    long FileSize;

    int addrlen = sizeof(SOCKADDR_IN);

    WSAStartup(MAKEWORD(2,0),&wsa);

    server = socket(AF_INET, SOCK_STREAM, 0);

    memset(&addr, 0, sizeof(SOCKADDR_IN));
    addr.sin_family = AF_INET;
    addr.sin_port = htons(Port);
    addr.sin_addr.s_addr = INADDR_ANY;
    bind(server, (SOCKADDR*)&addr, sizeof(SOCKADDR_IN));

    listen(server, 10);

    client = accept(server, (SOCKADDR*)&addr, &addrlen);

    recv(client, (char*)&FileHeader, sizeof(NETFILEHEADER), 0);

    FileSize = FileHeader.FileSize;

    pFile = fopen(FileHeader.pFileName, "wb");

    while(FileSize > 0)
    {
        rval = recv(client, buffer, 4096, 0);
        fwrite(buffer, rval, 1, pFile);

        FileSize -= rval;
    }

    fclose(pFile);

    closesocket(client);
    closesocket(server);

    WSACleanup();

    return FileHeader.FileSize;
}

void main(void)
{
    //SendFileToNet("net.cpp", "127.0.0.1", 2007);
    GetFileFromNet(2007);
}

Das sind mal 2 Funktionen zum versenden und empfangen von Dateien. Um ehrlich zu sein war ich zu faul bei deinem Code auf Fehlersuche zu gehen.

Reply to Datei einlesen und uebertragen(Binär) on Sun, 07 Oct 2007 14:46:11 GMT

HilfeNoetig — Sun, 07 Oct 2007 14:46:11 GMT

Und dein Code hilft mir rein gar nicht weiter...

Wir sind hier übrigens im C++ Forum...

Trotzdem Danke!

Reply to Datei einlesen und uebertragen(Binär) on Sun, 07 Oct 2007 14:52:17 GMT

Fake oder Echt — Sun, 07 Oct 2007 14:52:17 GMT

Oh oh, jetzt wirste patzig. Einen geschenkten Gaul guckt man nicht ins Maul.
Du überprüfst in deinem Mist nichteinmal wie groß die Datei ist, so dass auch zu viel bzw. zu wenig gesendet werden könnte. Aber das kann man aus meinem Beispiel ja nicht sehen, besonders nicht wenn man es sich nichteinmal anguckt.

Reply to Datei einlesen und uebertragen(Binär) on Sun, 07 Oct 2007 15:04:01 GMT

camper — Sun, 07 Oct 2007 15:04:01 GMT

HilfeNoetig schrieb:

Wo liegt der Fehler?

Ich kenne deine sendStr-Funktion nicht, würde aber darauf tippen, dass deinem String der Terminator fehlt bzw. du keine Länge angibst. Da eine Binärdatei auch eingebettete Nullen enthalten kann, ist es sowieso schlecht, mit terminierten Strings zu arbeiten.

char buffer[512];
while (true) {
    if (int x = thefile.readsome(buffer, sizeof buffer))
        sendStr( connectedSocket, std::string(buffer,x));
    else
        break;
}
thefile.close();

Reply to Datei einlesen und uebertragen(Binär) on Sun, 07 Oct 2007 15:29:09 GMT

HilfeNoetig — Sun, 07 Oct 2007 15:29:09 GMT

@camper:

void sendStr(SOCKET socket, const std::string& str) {
	send(socket, str.c_str(), (int)str.length(), 0);
}

Dein Code erzeugt bei mir leider eine leere Datei...

Reply to Datei einlesen und uebertragen(Binär) on Sun, 07 Oct 2007 15:34:24 GMT

camper — Sun, 07 Oct 2007 15:34:24 GMT

char buffer[512];
while (thefile.good()) {
    thefile.read(buffer, sizeof buffer))
    if ( thefile.gcount() )
        sendStr( connectedSocket, std::string(buffer,thefile.gcount()));
}

oder so ähnlich.

Reply to Datei einlesen und uebertragen(Binär) on Sun, 07 Oct 2007 15:57:18 GMT

HilfeNoetig — Sun, 07 Oct 2007 15:57:18 GMT

Danke! Klappt perfekt

for (;;) {
  std::cout << "Vielen Dank!\n";
}

Viele Gruesse,
HilfeNoetig