1

わかりました。基本的に、アプリケーションで1〜2秒ごとに実行ループが実行されますが、同時に、listenForPacketsメソッドをループする別のスレッドが実行されます。broadcastMessage別のアクションメソッドが実行されたときにのみ開始されます。この質問の重要な部分は、リスナースレッドがメインスレッドとは別に実行されている場合、印刷コマンドを出力せず、定義したグローバル変数へのアクセスを許可しないように見えることですrecvMessage。インターフェイスと実装のセクション。

私のコードでは、メインの実行ループを実行するたびにGUIのUILabelを更新するように設定しています。アプリの実行中、ラベルは常に空白のままで、変更されることはありません。GUIを再確認したところ、すべてが正しくリンクされており、ラベルも正しくインスタンス化されています(以下のコードでは、UILabelのインスタンスとして「label」という名前を使用しています)。私のレーベルが更新されている理由を誰かが知っていますか?私はそれをやったばかりで、すべてが「話している」ので、物事のネットワーキングの側面は素晴らしいと私は信じています。多分それは私が知らない変数スコープの問題であるか、または私が以下で使用したもの(rcvMessage)のようなグローバル変数にアクセスすることを許可された別々のスレッドですか?私はマルチスレッドアプリケーションにかなり慣れていませんが、実装するのがそれほど難しいとは思いません。

グローバル変数

NSString *recvMessage;

メインランループ-ランループを通過するたびにラベルを更新するセクション

if (label.text != recvMessage)
    label.text = recvMessage

トーカー方式

-(void)broadcastMessage { // (NSString*)msg {
    msg = @"From_Master";

    NSLog(@"broadcastMessage - Stage 1");
    mc_ttl = 15; // number of node hops the message is allowed to travel across the network
    // define the port we will be using
    mc_port = MYPORT;

    // create a socket for sending to the multicast address
    if ((sock = socket(AF_INET, SOCK_DGRAM, IPPROTO_UDP)) < 0) {
        NSLog(@"ERROR: broadcastMessage - socket() failed");
        return;
    }

    memset(&mc_addr, 0, sizeof(mc_addr));
    mc_addr.sin_family      = AF_INET;
    mc_addr.sin_addr.s_addr = inet_addr(GROUP_ADDRESS);
    mc_addr.sin_port        = htons(MYPORT);

    NSLog(@"broadcastMessage - Stage 2");

// set the TTL (time to live/hop count) for the send
    if ((setsockopt(sock, IPPROTO_IP, IP_MULTICAST_TTL, &mc_ttl, sizeof(mc_ttl))) < 0) {
        NSLog(@"ERROR: broadcastMessage - setsockopt() failed");
        return;
    }   

    NSLog(@"broadcastMessage - Stage 3");

    // clear send buffer
    memset(send_str, 0, sizeof(send_str));

    // convert the message to a C string to send
    [msg getCString:send_str maxLength:MAX_LEN encoding:NSASCIIStringEncoding];

    //while (fgets(send_str, MAX_LEN, stdin)) {
    NSLog(@"broadcastMessage - Stage 4");
    NSLog(@"Message =");
    printf(send_str);

    // send string to multicast address
    if ((sendto(sock, send_str, sizeof(send_str), 0, (struct sockaddr *)&mc_addr, sizeof(mc_addr))) < 0) {
        NSLog(@"ERROR: broadcastMessage - sendto() sent incorrect number of bytes");
        //return;
    }
    NSLog(@"Sent Message -");
    printf(send_str);
    NSLog(@"broadcastMessage - Stage 5");

    // clear send buffer
    memset(send_str, 0, sizeof(send_str));
    NSLog(@"broadcastMessage - Stage 6 - Complete");
    close(sock);
}

リスナーメソッド

-(void)listenForPackets {
    listeningFlag_on = 1; // allows reuse of the same socket

    NSLog(@"listenForPackets - Stage 1");
    if ((listeningSock = socket(AF_INET, SOCK_DGRAM,IPPROTO_UDP)) < 0) {
        NSLog(@"ERROR: listenForPackets - socket() failed");
        return;
            // make the method return an int instead of void and use this statement to check for errors
    }

    NSLog(@"listenForPackets - Stage 2");

    // set reuse port to on to allow multiple binds per host
    if ((setsockopt(listeningSock, SOL_SOCKET, SO_REUSEADDR, &listeningFlag_on, sizeof(listeningFlag_on))) < 0) {
        NSLog(@"ERROR: listenForPackets - setsockopt() Reuse failed");
        return;
            // make the method return an int instead of void and use this statement to check for errors
    }

    // construct a multicast address structure after erasing anything in the listeningmc_addr structure
    memset(&listeningmc_addr, 0, sizeof(listeningmc_addr));
    listeningmc_addr.sin_family      = AF_INET;
    listeningmc_addr.sin_addr.s_addr = htonl(INADDR_ANY); // different from sender
    listeningmc_addr.sin_port        = htons(MYPORT);

    // bind multicast address to socket
    if ((bind(listeningSock, (struct sockaddr *)&listeningmc_addr, sizeof(listeningmc_addr))) < 0) {
        NSLog(@"ERROR: listenForPackets - bind() failed");
        perror("Bind() -");
        return;                         // make the method return an int instead of void and use this statement to check for errors
    }

    //*********************************************************************************

    NSString *ipAddress = [[NSString alloc] initWithString:self.getIPAddress];
    const char *tmp = [ipAddress UTF8String];
    listeningMc_addr_str = tmp;

    printf("%s\n", listeningMc_addr_str);

    listeningMc_req.imr_multiaddr.s_addr = inet_addr(GROUP_ADDRESS);
    listeningMc_req.imr_interface.s_addr = htonl(INADDR_ANY);
    // send an ADD MEMBERSHIP message via setsockopt
    if ((setsockopt(listeningSock, IPPROTO_IP, IP_ADD_MEMBERSHIP, &listeningMc_req, sizeof(listeningMc_req))) < 0) {
        NSLog(@"ERROR: listenForPackets - setsockopt() failed");
        int err = errno;
        NSLog(@"errno - %i", err);
        NSLog(@"Error = %s", strerror(err));
        perror("ERROR");
        return;                         // make the method return an int instead of void and use this statement to check for errors
    }

    NSLog(@"listenForPackets - Stage 3");
    for (;;) {          // loop forever

        // clear the receive buffers & structs
        memset(listeningRecv_str, 0, sizeof(listeningRecv_str));
        listeningFrom_len = sizeof(listeningFrom_addr);
        memset(&listeningFrom_addr, 0, listeningFrom_len);

        NSLog(@"Test #1 Complete");
        //msgStatus.text = @"Waiting...";

        // block waiting to receive a packet
        listeningFrom_len = sizeof(listeningmc_addr);
        if ((listeningRecv_len = recvfrom(listeningSock, listeningRecv_str, MAX_LEN, 0, (struct sockaddr*)&listeningmc_addr, &listeningFrom_len)) < 0) {
            NSLog(@"ERROR: listenForPackets - recvfrom() failed");
            return;                     // make the method return an int instead of void and use this statement to check for errors
        }
        NSLog(@"Test #2 Complete - Received a Message =");
        NSLog(@"listenForPackets - Stage 4");

        // listeningRecv_str
    **tmpString = [[NSString alloc] initWithCString:listeningRecv_str encoding:NSASCIIStringEncoding];
            NSLog(@"Message Received =");
            NSLog(tmpString);
            recvMessage = tmpString;**
        //}
        // received string
        printf("Received %d bytes from %s: ", listeningRecv_len, inet_ntoa(listeningFrom_addr.sin_addr));
        printf("%s", listeningRecv_str);
        //}
    }
    // send a DROP MEMBERSHIP message via setsockopt
    if ((setsockopt(listeningSock, IPPROTO_IP, IP_DROP_MEMBERSHIP, (void*) &listeningMc_req, sizeof(listeningMc_req))) < 0) {
        NSLog(@"ERROR: listenForPackets - setsockopt() drop membership failed");
        //return 1;                         // make the method return an int instead of void and use this statement to check for errors
    }
    close(listeningSock);
    NSLog(@"listenForPackets - Stage 5 - Complete");
}
4

2 に答える 2

4

はい、すべてのスレッドがグローバル変数にアクセスできます。グローバルの使用方法には確かに問題があります-変​​数が更新されるたびに作成するNSStringをリークし、アクセス制御なしで2つのスレッドから同じメモリにアクセスしています-しかし、変数を妨げるものは何もありません更新されないようにします。

ログメッセージが出力されない場合、問題はコードが実行されないことです。これが変数が変更されない理由です。この新しいスレッドを開始することになっているコードを確認する必要があります。

于 2009-07-25T23:30:44.170 に答える
0

また、UIコンポーネントを更新する場合は、メソッド「performSelectorOnMainThread」を使用して、ラベルテキストまたはその他のGUI要素の値を設定する必要があることに注意してください。値はバックグラウンドスレッドから更新されません。

于 2009-07-27T01:15:29.947 に答える