Прозрачной замены указателей на символьные массивы в C++

Question

у меня есть 2D массив символов:
char nm[MAX1][MAX2] = { "john", "bob", "david" };
Я хочу поменять два из этих элементов (без std::swap) просто писать
swapPointers(nm[0], nm[1]);
где swapPointers выглядит следующим образом

void swapPointers(char *&a, char *&b) 
{ 
    char *temp = a; 
    a = b; 
    b = a; 
} 

Однако это не скомпилировано (и при добавлении отбрасываний компиляция указателей в конечном итоге указывает на неправильные/странные местоположения).

Может ли кто-нибудь помочь?
Спасибо!

Answer 1

Zan близок, но его проблема в том, что его функция «swap» может принимать любые указатели на символы. Это может вызвать проблемы при неправильном использовании. Вот более безопасная версия:

void swap(char (&x)[MAX2], char (&y)[MAX2]) 
{ 
    char temp[MAX2]; 

    memcpy(temp, x, MAX2); 
    memcpy(x, y, MAX2); 
    memcpy(y, temp, MAX2); 
} 

Существует также непонимание со стороны плаката: «нм» является 2-мерный массив символов. Нет указателей. nm [0], nm [2] и т. д. также не являются указателями - они все еще (одномерные) массивы. Тот факт, что одномерные массивы неявно конвертируемы в указатели, вызывает такой путаницу среди многих программистов на C и C++.

Чтобы поменять местами данные в двумерном массиве, вы должны поменять местаки памяти размером MAX2 - как указано обеим «swap» функциями Zan и я написал.

Answer 2

void swapPointers(char** ppa, char** ppb) 
{ 
    char* ptemp = *ppa; 
    *ppb = *ppa; 
    *ppa = ptemp; 
} 

swapPointers(&nm[0], &nm[1]); 

Answer 3

Вы не можете поменять эти указатели, переназначив указатели, потому что эти указатели указывают на двухмерный массив символов.

нм [a] и nm [b] очень сильно const, потому что nm - это действительно const объект. Если это не так, вы можете переместить переменные С в ОЗУ, переназначив их имена.

Просто подумайте о хаосе! Поэтому вы не можете этого сделать. :-)

Чтобы поменять местами, на которые указывают эти указатели, вам необходимо поменять значения в этих местах массива.

swap(char *a, char *b) 
{ 
    char temp[MAX1]; 
    memcpy(temp, a, MAX1); 
    memcpy(b, a, MAX1); 
    memcpy(a, temp, MAX1); 
} 

Answer 4

Ваши swapPointers() свопы указатели, в то время как вы пытаетесь передать его массивы.

Если вы изменяете

char nm[MAX1][MAX2] 

в

char *nm[MAX1] 

и исправить небольшую ошибку в swapPointers() (последней строке должно быть b = temp;), она работает.

Answer 5

Реальная точка, если вы используете C++, то вы должны использовать зЬй :: вектор станд :: строку вместо:

std::vector<std::string> nm; 
nm.push_back("john"); 
nm.push_back("bob"); 
nm.push_back("david"); 
std::swap(nm[0], nm[1]); 

Примечание: не тестировалось.