2012-10-23 3 views
0

En essayant de copier des sous-chaînes d'un caractère * vers un autre, quand j'imprime (% c) il affiche l'impression des caractères, mais à la fin de la méthode add, j'imprime printf (% s) et rien ne s'imprime. Toute aide est appréciée - peut-être aussi une meilleure méthode pour les sous-chaînes en C que char par char?C Programmation Copie Sous-chaîne

code ci-dessous:

int aliasCount; 

typedef struct Alias { 
    char* alias; 
    char* actual; 
    struct Alias* next; 
}Alias; 

Alias* aliasHead; 

void addAlias(char* new); 

AddAlias ​​méthode

void addAlias(char* new) 
{ 
    strip(new); 
    Alias* newAlias = (Alias*)malloc(sizeof(Alias)); 

    //Get start-end position for alias 
    int start=0; int end=0; int countSpace=0; 
    for(int i = 0; i < strlen(new); i++) 
    { 
    //Check for space 
    if(new[i]==' ') 
    { 
     printf("found space %d\n",i); 
     countSpace++; 
     if(countSpace==1) 
     { 
      start=i; 
     } 
     else if(countSpace==2) 
     { 
      end=i; 
      break; 
     } 
    } 
    } 

    //malloc memory 
    newAlias->next=NULL; 
    newAlias->alias=(char*)malloc(sizeof(char)*(end-start)); 
    newAlias->actual=(char*)malloc(sizeof(char)*(strlen(new)-end+1)); 

    //Get substring, 
    //Copy char by char from alias to node 
    for(int i = 0; i < strlen(new); i++) 
    { 
    if(i>start && i<end) //Alias 
    { 
     newAlias->alias[i] = new[i];printf("'%c'",newAlias->alias[i]); 
    } 
    else if(i>end) //Actual command 
    { 
     newAlias->actual[i] = new[i];printf("'%c'",new[i]); 
    } 
    } 
    printf("%s\n%s\n",newAlias->alias,newAlias->actual); 

    if(aliasCount==0) 
    { 
    aliasHead = newAlias; 
    } 
    else 
    { 
     Alias* curr = aliasHead; 
     for(int i = 0; i < aliasCount; i++) 
     { 
     curr=curr->next; 
     } 
     curr->next = newAlias; 
    } 
    aliasCount++; 
    printf("%s\n%s\n",aliasHead->alias,aliasHead->actual); 
} 

Répondre

1

Vos indices ont tort; vous copiez la plage new[start+1]-new[end-1] dans la gamme alias[start+1]-alias[end-1], lorsque vous avez réellement besoin de le copier dans la gamme alias[0]-alias[end-start-2]:

 newAlias->alias[i-start-1] = new[i];printf("'%c'",newAlias->alias[i]); 

et de même (mutatis mutandis) pour actual.

+0

Oui merci, c'est toujours quelque chose que vous avez appris dans les premiers jours de la programmation qui vous rejette n'est-ce pas. – Kairan