2009-07-25 12 views
1

기본적으로 실행 루프는 내 응용 프로그램에서 매 2 ~ 2 초가 걸리는 반면, 동시에 다른 스레드는 listenForPackets 메서드를 통해 반복됩니다. broadcastMessage은 다른 작업 방법이 발생할 때만 시작됩니다. 이 질문의 중요한 부분은 리스너 스레드가 주 스레드에서 별도로 실행될 때 인쇄 명령을 전혀 인쇄하지 않으며 외부에있는 recvMessage이라는 정의 된 전역 변수에 대한 액세스를 허용하지 않는 것 같습니다. 인터페이스 및 구현 섹션.objective-C NSThread는 전역 변수에 액세스 할 수 있습니까?

내 코드에서는 메인 루프를 돌 때마다 내 GUI에서 UILabel을 업데이트하도록 설정했습니다. 앱이 실행 중일 때 내 레이블은 항상 비어 있으며 변경되지 않습니다. 나는 GUI를 두 번 체크했는데 모든 것이 올바로 연결되어 있고 내 레이블도 올바르게 인스턴스화되어 있습니다 (아래 코드에서 UILabel의 인스턴스로 "label"이라는 이름을 사용합니다). 누구든지 내 레이블이 업데이트되는 이유를 알고 있습니까? 사물의 네트워킹 측면은 훌륭합니다. 왜냐하면 방금 그 일을 끝내고 모든 것이 "말하고 있습니다". 아마도 내가 알지 못하는 가변 범위 문제 일 수도 있고, 아래에 사용 된 (rcvMessage)와 같이 전역 변수에 액세스 할 수있는 별도의 스레드가있을 수도 있습니다. 나는 멀티 스레드 응용 프로그램에 상당히 익숙하지만 NSThread (단 한 줄의 코드)를 사용하여 구현하기가 정말 어렵다고 생각하지 않습니다.

글로벌 변수

NSString *recvMessage; 

홈페이지 Runloop - 라벨을가 runloop

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"); 
} 
+0

이 질문은 모든 관련없는 소켓 코드를 제거하고 질문과 관련된 부분을 게시 한 경우 사람들이 대답하기가 훨씬 쉬울 것입니다. – smorgan

답변

4

예, 모든 스레드는 전역 변수에 액세스 할 수 있습니다. 확실히 전역 변수를 사용하는 방법에는 문제가 있습니다. 변수가 업데이트 될 때마다 생성하는 NSString을 누출시키고 액세스 제어없이 두 스레드에서 동일한 메모리에 액세스하는 것입니다. 그러나 변수를 방지 할 수있는 방법은 없습니다 업데이트되지 않습니다.

로그 메시지가 인쇄되지 않는 경우 코드가 실행되지 않으므로 변수가 변경되지 않는 것이 문제입니다. 이 새로운 스레드를 시작하기로되어있는 코드를 살펴 봐야합니다.

+0

감사합니다. 많은 도움을 받았습니다. 변수 액세스 제어 및 메모리 누수 방지 방법에 대한 기사 나 문서가 있습니까? 내가 찾을 수있는 NSThread 문서에는 그다지 많은 것이 없습니다. –

+0

누수와 관련하여 스레드와 관련된 것은 없습니다. 이를 위해서는 http://developer.apple.com/documentation/Cocoa/Conceptual/MemoryMgmt/MemoryMgmt.html과 같이 Cocoa의 메모리 관리에 대한 기본 안내서 만 있으면됩니다. – smorgan

0

UI 구성 요소를 업데이트 할 때 "performSelectorOnMainThread"메서드를 사용하여 레이블 텍스트 또는 다른 GUI 요소의 값 설정을 수행해야합니다. 이 값은 백그라운드 스레드에서 업데이트되지 않습니다.

관련 문제