C言語における文字列のスワップについて

Question

実現したいこと

題の通りスワップをどのように実装するのがいいのかお聞きしたいです。皆様ならばどのように実装いたしますか？
私は以下のように考えました。
MAX_LENは決められたものとします。またコンパイル方法はコードの下のコメントのようにオプションをつけます。

現在の方法

#include <stdio.h>

#define MAX_LEN 5

void copy(char *s1, char *s2, int n)
{
    while (n--) {
        if (!(*(s1++) = *(s2++)))
            break;
    }
    while (n--)
        *s1++ = '\0';
}

void swap(char *s1, char *s2)
{
    char tmp[MAX_LEN+1];
    copy(tmp, s1, MAX_LEN+1);
    copy(s1, s2, MAX_LEN+1);
    copy(s2, tmp, MAX_LEN+1);
}

int main(void) 
{
    char a[] = "Hello";
    char b[] = "World";
    printf("%s %s\n", a, b);
    swap(a, b);
    printf("%s %s\n", a, b);
    return 0;
}
/* 実行 */
/* $ gcc -O -Wall -pedantic -ansi -o t test.c */
/* $ ./t */
/* Hello World
   World Hello */

実行環境

gcc (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0

Answer 1

皆様ならばどのように実装いたしますか？

そもそも、文字列のスワップをしなくていいように、ポインタのスワップで済ませるように検討すると思います。

Answer 2

既に回答がついているように、ポインタのスワップが一番簡単ですが、

あえてするなら、 strcpy(), memcpy() とか使いますが、使ってはいけない?
また、余分なメモリ使いたくなければ、一文字ずつ、swap ですね。
(もっとも今時、この方法が効率良いかは不明)

Answer 3

こんな漢字で、

#include <stdio.h>

#define MAX_LEN 5

void strnxor(char *s1, const char *s2, size_t n) {
    while (n--) { // 終端の'\0'は無視:-)
        *(s1++) ^= *(s2++);
    }
}

void strnswap(char *s1, char *s2, size_t n) {
    strnxor(s1, s2, n);
    strnxor(s2, s1, n);
    strnxor(s1, s2, n);
}

int main(void) {
    char a[] = "Hello";
    char b[] = "World";
    printf("%s %s\n", a, b);
    strnswap(a, b, MAX_LEN + 1);
    printf("%s %s\n", a, b);
    return 0;
}

Answer 4

これが最適解かわかりませんがポインタの交換で対応いたしました。

#include <stdio.h>

#define MAX_LEN 5

void swap(char **s1, char **s2)
{
    char *tmp;
    tmp = *s1;
    *s1 = *s2;
    *s2 = tmp;
}

int main(void) 
{
    char *a = "Hello";
    char *b = "World";
    printf("%s %s\n", a, b);
    swap(&a, &b);
    printf("%s %s\n", a, b);
    return 0;
}
/* 実行 */
/* $ gcc -O -Wall -pedantic -ansi -o t test.c */
/* $ ./t */