sprintf()

Shade Of Mine

sorry - hab den falschen code erwischt.

deiner ist besser, gibt eigentlich nix zu beanstanden nur n paar kosmetik sachen:

#include <stdio.h>
#include <netdb.h>
#include <string.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>
#define POP_PORT 110
//POP_PORT sollte vielleicht besser ein const int sein
#define BUFFSIZE 1500

struct pop_user
{
    char username[30];
    char passwd[30];
    char dns_addr[20];
    int sockfd;
    struct sockaddr_in servaddr;
    char buffer[BUFFSIZE];
};

int connect_pop_server(struct pop_user *pop_data);
int check_messages(struct pop_user *pop_data);
int registrate(struct pop_user *pop_data);
int get_answere(struct pop_user *pop_data);
//schoen, das gefaellt mir!

int main(void) 
{
    struct pop_user pop_data;

    strcpy(pop_data.dns_addr,"pop.gmx.de");
    strcpy(pop_data.username,"test@gmx.de");
    strcpy(pop_data.passwd,"test");
    //uU besser eine Funktion die pop_data fuellt
    //damit kannst du besser kapseln

    connect_pop_server(&pop_data);
    printf("Ok, you are connected :)\n");
    registrate(&pop_data);
    printf("Messages: %i\n",check_messages(&pop_data));
    return 0;
}
int registrate(struct pop_user *pop_data) 
{
    /* send the username */
    snprintf(pop_data->buffer,BUFFSIZE,"USER %s\n",pop_data->username);
    //vielleicht solltest du checken ob username abgeschnitten wurde
    //(snprintf returned dann -1)
    //vielleicht brauchst du das aber nicht, da Buffer sowieso immer
    //gross genug ist! (sizeof(username) + sizeof("USER \n") < BUFFERSIZE)
    if((send(pop_data->sockfd, pop_data->buffer, strlen(pop_data->buffer), 0))== -1) {
        perror("send() failed");
        return 1;
        //fehlermeldung die in main nicht abgefangen wird - also entwder in main
        //abfangen oder abort() aufrufen
    }
    if((get_answere(pop_data)) == -1)
        return  1; //siehe oben
    /* send the passwd */
        snprintf(pop_data->buffer,BUFFSIZE,"PASS %s\n",pop_data->passwd);
        //siehe oben
        if((send(pop_data->sockfd, pop_data->buffer, strlen(pop_data->buffer), 0))== -1) {
                perror("send() failed");
                return 1; //siehe oben
        }
        if((get_answere(pop_data))== -1)
        return  1; //siehe oben
    return 0;
}
int connect_pop_server(struct pop_user *pop_data)
{
    struct hostent *host;
    pop_data->servaddr.sin_port = htons(POP_PORT);
    pop_data->servaddr.sin_family = AF_INET;
    /* get the IP adress */
    host = gethostbyname(pop_data->dns_addr);
    pop_data->servaddr.sin_addr = *(struct in_addr*) host->h_addr;

    /* get a socket file deskriptor */
    if((pop_data->sockfd = socket(AF_INET, SOCK_STREAM, 0))== -1) {
        perror("socket() failed");
        return -1; //fehler abfangen oder aborten
    }
    /* connect to the server */
    if((connect(pop_data->sockfd, &pop_data->servaddr, sizeof(pop_data->servaddr)))== -1) {
        perror("connect() failed");
        return -1; //siehe oben
    }
    /* get the answer from the server */
    if((get_answere(pop_data))== -1)
        return -1; //siehe oben
    return 0;
}
int check_messages(struct pop_user *pop_data)
{
    int msg = 1; 
    while(1) {
        snprintf(pop_data->buffer,BUFFSIZE,"LIST %i\n",msg);
        //sprintf tut es auch
        if((send(pop_data->sockfd, pop_data->buffer, strlen(pop_data->buffer), 0))== -1) {
            perror("send() failed");
            return -1; //siehe oben
        }

        if((get_answere(pop_data))== -1)
            break;
        msg++;
    }
    return msg-1;
}
int get_answere(struct pop_user *pop_data)
{
    char buffer[BUFFSIZE];
    int bytes;
    bytes = recv(pop_data->sockfd, buffer, sizeof(buffer), 0);
    buffer[bytes] = '\0';
    printf("--> %s",buffer);
    if(buffer[0] == '-')
        return -1;
    return 0;
}

die Kapselung ist schoen!!

Ok, vielen Dank für deine Rezension.
Du musst nämlich wissen, dass ich in diesem Gebiet manchmal ein wenig perfektionistisch bin....

mfg grottenolm

hallo,
hab mir mal diesen interessanten beitrag durchgelsen und hab dazu mal eine kleine frage:

int VerbindungAufbauen( char *szUsername , char *szPasswort , int nPort , char *szPOP , int nSocket )
{//szUsername und szPassowort und szPOP sollten const char* sein

warum das const noch zusätzlich dazu ?
mfg ronny

Shade Of Mine

stichwort const-correctness

das ist in C nicht ganz so wichtig, aber dennoch schadet es nicht (und kann aber helfen)

wenn eine funktion den Usernamen nicht verändert, dann gibt es 2 möglichkeiten dies zu dokumentieren:

void nochange(char* username);
/*username wird nicht verändert*/

oder
void nochange(const char* username);

letzteres verhindert ein unwissentliches ändern von username - gegen ein mutwilliges kann man sich nicht schützen.

In C ist soetwas leider kein muss in C++ allerdings schon.

Bashar

in C gibt es keine const-correctness, das Überschreiben auf ein als const deklariertes Objekt führt aber zu undefiniertem Verhalten. const wird lediglich vom Compiler zu Optimierungszwecken verwendet (genau wie restrict und volatile, wobei man bei letzterem vielleicht Pessimierungszwecke sagen sollte ;))

ahso, danke euch beiden