Передача строки через указатель - несовместимый аргумент

Question

Я пытаюсь передать две строки функции, которая копирует вторую строку в первую строку. Получить сообщения об ошибках

"passing arguement 1/2 of 'CopyStrings'from incompatible pointer type - line 25. 
note: expected 'char*' but argument is of type 'char**' - line 8 

Вот мой код:

#include <stdio.h> 
#include <string.h> 
#include <stdlib.h> 

// a function that gets 2 strings and copies the second one into the first one 


char CopyStrings(char* string1, char* string2) 
{ 
    strcpy(string1, string2); 

    return *string1; 

} 

int main() 
{ 
    char* string1 = (char*)malloc(sizeof(char)*10); 
    char* string2 = (char*)malloc(sizeof(char)*10); 

    strcpy(string1, "ugabuga"); 
    strcpy(string2, "mukaluka"); 


    printf("%s", CopyStrings(string1, string2)); 


    free(string1); 
    free(string2); 
    return 0; 

} 

Любые предложения, что я испортил, и как я пишу указатели правильно?

EDIT - Теперь, когда я изменил

(&string1, &string2) 

в

(string1, string2) 

программа chrashes.

Answer 1

Были некоторые проблемы с кодом Ops:

В следующем коде, большинство проблем комментируются (и фиксированные) Одна вещь, чтобы знать, в C, имя массива деградирует на адрес из массив пытается всегда передавать указатель на массив, а не полный массив. Оператор преобразования %s ожидает, что связанный параметр будет адресом массива символов.

#include <stdio.h> 
#include <string.h> 
#include <stdlib.h> 

// a function that gets 2 strings and copies the second one into the first one 


char* CopyStrings(char* string1, char* string2) 
{ 
    strcpy(string1, string2); 
    return string1; // return pointer to string1 
} 

int main() 
{ 
    char* string1 = malloc(10); // char is always 1 so no need for sizeof(char) 
    char* string2 = malloc(10); 

    strcpy(string1, "ugabuga"); 
    strcpy(string2, "mukaluka"); 

    printf("%s\n", CopyStrings(string1, string2)); 

    free(string1); 
    free(string2); 
    return 0; 
} // end function: main 

Answer 2

Вы действительно хотите просто

CopyStrings(string1, string2) 

Это char* уже, и вы делаете char** от него с амперсанд.

Другая проблема заключается в функции - она ​​должна быть:

char* CopyStrings(char* string1, char* string2) 
{ 
    strcpy(string1, string2); 
    return string1; 
} 

(если вы хотите, чтобы вернуть первую строку).

Answer 3

Изменить

CopyStrings(&string1, &string2)

в

CopyStrings(string1, string2)

char* string1, здесь string1 является указателем и когда вы передаете адрес, если указатель, который является указателем на указатель, его не то, что вы имеете в виду и формальные аргументы должны быть двойными указателями. Достаточно всего лишь указателя.

Функция CopyStrings() должна вернуть строку char * после выполнения функции strcpy() для вызывающего. Вот модифицированный code.