文字列を入れ替える関数

twitterで見かけたので真面目に考えてみる。

引数で渡された２つの文字列（s1,s2）の中身を入れ替える関数を作りなさい。
http://d.hatena.ne.jp/tek_koc/20080518/1211095764

意図はなんだろう。
単純に数値を入れ替える関数の文字列版でいいのかな。

#define MAX 15
void strchange(char s1[], char s2[])
{
    char tmp[MAX];

    strncpy(tmp, s1,  MAX);
    strncpy(s1,  s2,  MAX);
    strncpy(s2,  tmp, MAX);
}

tmpを使って処理しなさいということならこれで良さそう。分解すると

#define MAX 15
void strchange(char s1[], char s2[])
{
    char tmp[MAX];
    int i;

    for (i = 0; i<MAX; i++){
        tmp[i] = s1[i];
    }
    for (i = 0; i<MAX; i++){
        s1[i] = s2[i];
    }
    for (i = 0; i<MAX; i++){
        s2[i] = tmp[i];
    }

}

こんな感じなのかな。
\0 があるかないかに関わらず動くので安全。
でもMAXが微妙だ。それを直すついでにfor文もまとめると

void strchange(char s1[], char s2[], int size)
{
    char tmp;
    int i;

    for (i = 0; i<size; i++){
        tmp = s1[i];
        s1[i] = s2[i];
        s2[i] = tmp;
    }
}

こうなる。
サイズを受け取る引数は無しなんだろうか。それは困る。
意外にむずい。

というかこれはポインタを使うという問題？コメントでもポインタ使ってるしなぁ。
ポインタを使えという意図なら機械的に置換で。

void strchange(char *s1, char *s2)
{
    char tmp;
    int count = 0;

    while(count != 3){
        tmp = *s1;
        *s1 = *s2;
        *s2 = tmp;

        if (*s1 == '\0')
            count |= 1;
        if (*s2 == '\0')
            count |= 2;
        s1++;
        s2++;
    }
}