Нахождение запятые внутри массива символов

Question

У меня есть 2 проблемы с этой функцией:

char * getHeader(char localString[], int seperatorNum) 
{ 
    int maxL=50; 
    const char seperator=':'; 
    char newdata[maxL]; 
    int counter=0; 
    for(int k=0;k<maxL;k++) 
    { 
     if (localString[k]== seperator ) 
     { 
      counter++; 
      if(counter==seperatorNum) 
      { Serial.println(newdata); return newdata;  } 
     } 
     newdata[k]=localString[k]; 
    } 
} 

1. Когда я напечатаю newdata я получаю мусор, но localString[k] показывает хорошие результаты.
2. Как я могу вернуть массив - мой путь хорош?

Редактировать: я получаю bb после каждого результата. так:

aaa:sss (send) 
aaabb (get) 
kkk:ooo (send) 
kkkbb (get) 
ppp:ttt 
pppbb 

Answer 1

очистка массива, который будет иметь некоторые значения в нем, будет решить эту проблему, и это происходит потому, что при печати, и есть некоторые другие ценности в ней, вы получите их напечатанные слишком

for(int i = 0; i < maxL; ++i) 
    newdata[i] = (char)0; 

решить полностью.

EDIT: А для тех, кто нуждается, это полная функция, которая работает:

char * getHeader(char localString[], int seperatorNum) 
{ 


    int maxL=50; 
    const char seperator=':'; 
    char newdata[maxL]; 
    int counter=0; 
    int divider=0; 
    //clear array when it has garbage it added 
    for(int i = 0; i < maxL; ++i) 
     newdata[i] = (char)0; 

    for(int k=0;k<maxL;k++) 
    { 


     if (localString[k]== seperator ) 
     { 

      counter++; 
      divider=k+1; 
      if(counter==seperatorNum) 
      { return newdata;  } 
     } 
     if( (seperatorNum-1) ==counter) 
      newdata[k-divider]=localString[k]; 

    } 


} 

Answer 2

newdata[k]=localstring[k]; 

Если должно прийти только после того, как

for(int k=0;k<maxL;k++) 
{ 
//here 

И если это не решает тогда просьба представить связанную полный код.

Answer 3

char * getHeader(char localString[], int seperatorNum) 
{ 
    int maxL=50; 
    const char seperator=':'; 
    char newdata[maxL]; 
    memset(newdata, '\0', maxL);/////// 
    int counter=0; 
    int i = 0; /////// 
    for(int k=0;k<maxL;k++) 
    { 
     if (localString[k] == seperator ) 
     { 
      counter++; 
      if(counter == seperatorNum) 
      { 
       Serial.println(newdata);  
       return newdata;  
      } 
     }else{ 
      newdata[i] = localString[k];//outside the else this was alway executed. I assume you didn't want the seperator in this 
      i++; 
     } 
    } 
}