1

ポインターと関数がどのように機能するかについて少し質問があります。関数がどのように見えるかを確認したいのですがqsort()、独自の関数を使用して要素を交換し、要素を比較する必要があります。私の関数がデータを交換しないことを知って非常に驚いています...

私のコード:

//prototypes file: other.h

void Sort(char* pcFirst, int nNumber, int size, void (*Swap)(void*, void*), int (*Compare)(void*, void*) ); //sorts any arrays
void SwapInt(void* p1, void* p2); // swap pointers
int CmpInt(void* p1, void* p2); // compare poineters 

//realisation file: other.cpp

#include "other.h"
void Sort(char* pcFirst, int nNumber, int size,
     void (*Swap)(void*, void*), int (*Compare)(void*, void*) )
{
    int i;
    for( i = 1; i < nNumber; i++)
        for(int j = nNumber - 1; j >= i; j--)
        {
            char* pCurrent = pcFirst + j * size;
            char* pPrevious = pcFirst + (j - 1) * size;
            if( (*Compare)( pPrevious, pCurrent ) > 0 )// if > 0 then Swap
            {
                (*Swap)( pPrevious, pCurrent );
            }
        }
}

void SwapInt(void* p1, void* p2) 
{
    int * ptmp1 = static_cast<int*>(p1);
    int * ptmp2 = static_cast<int*>(p2);
    int * ptmp = ptmp1;
    ptmp1 = ptmp2;
    ptmp2 = ptmp;
}

int CmpInt(void* p1, void* p2)
{
    int nResult;
    int * ptmp1 = static_cast<int*>(p1);
    int * ptmp2 = static_cast<int*>(p2);
    nResult = (*ptmp1 - *ptmp2);
    return nResult;
}

//main file: lab.cpp
#include <tchar.h>
#include <iostream>
#include <cstdio>
#include <cmath>
#include "other.h"

int _tmain()
{
int nAr[] = {33,44,55,22,11};   //array for sort
    int nTotal = sizeof(nAr) / sizeof(int); //number of elements
for ( int i = 0; i < nTotal; i++)
    {
        printf("%d ",nAr[i]); // result of cycle is 33 44 55 22 11
    }
    Sort(reinterpret_cast<char*>(&nAr[0]), nTotal, sizeof(int), SwapInt, CmpInt);
for ( int i = 0; i < nTotal; i++)
    {
        printf("%d ",nAr[i]); // result of cycle is 33 44 55 22 11 too =(
    }
}

配列が変更されないのはなぜですか?

デバッガーでは、すべてのポインターが変更され、正しい値が取得されることがわかりますがmain、配列は変更されていません。

4

4 に答える 4

3

ポインターはオブジェクトを指す

コード

int * ptmp = ptmp1;
ptmp1 = ptmp2;
ptmp2 = ptmp;

関数内でいくつかのポインター値をローカルに変更します。それだけです。

2 つのオブジェクトの値を交換するには、それらを参照渡しします。

void swap_values_of( int& a, int& b )
{
    int const original_a = a;
    a = b;
    b = original_a;
}

安全性は劣りますが、ポインター引数を使用して、ポインター自体の代わりにポイントされている値を交換するように注意することもできます。

ただし、学習目的を除き、std::swap代わりに使用してください


求められていませんが...現在のMicrosoft固有の

int _tmain()

ただのスタンダードに

int main()

そうすれば、コードは (より可能性が高い) Linux などでも動作します。

ただのヒント

于 2013-02-23T17:12:12.930 に答える
0

関数は、sSwapIntではなく、いくつかのポインターを交換しますint。これらのポインタはすべて に対してローカルであるためSwapInt、実際の効果はありません。おそらく、ints*ptmp1*ptmp2.

于 2013-02-23T17:11:58.873 に答える
0

実際に行っているのは、ポインターの交換です。あなたがやろうとしているのは、そのポインターが指す値を交換することです。少なくともそれはあなたのプログラムロジックから来ています。したがって、コードは次のようになります。

void SwapInt(void* p1, void* p2) 
{
    int * ptmp1 = static_cast<int*>(p1);
    int * ptmp2 = static_cast<int*>(p2);
    int ptmp = *ptmp1;
    *ptmp1 = *ptmp2;
    *ptmp2 = ptmp;
}
于 2013-02-23T17:18:31.403 に答える
0

これらのように、さまざまな組み合わせを見ることができます.....

#include<iostream>
#include<stdio.h>
#include<malloc.h>
//Call by Address
    void SwapIntAddr(int* ptmp1, int* ptmp2) 
    {
        int ptmp;
        ptmp  = *ptmp1;
        *ptmp1 = *ptmp2;
        *ptmp2 = ptmp;
    }

//Call by Reference

    void SwapIntRef(int& ptmp1, int& ptmp2) 
    {
         int ptmp;
         ptmp  = ptmp1;
         ptmp1 = ptmp2;
         ptmp2 = ptmp;
    }
//Call by Reference but in pointer level
    void SwapPtrRef(int*& ptmp1, int*& ptmp2) 
    {
         int* ptmp;
         ptmp  = ptmp1;
         ptmp1 = ptmp2;
         ptmp2 = ptmp;
    }

//Call by Address but in Pointer level.

    void SwapPtrAddr(int** ptmp1,int** ptmp2) 
    {
        int** ptmp = (int**) malloc(sizeof(int*));
        *ptmp  = *ptmp1;
        *ptmp1 = *ptmp2;
        *ptmp2 = *ptmp;
    }


int main(){
  int a = 3, b= 5;
  int* p1 = &a;
  int* p2 = &b;

  SwapIntAddr(p1,p2);
  printf("%d %d\n",*p1,*p2);

  SwapIntRef(*p1,*p2);
  printf("%d %d\n",*p1,*p2);

  SwapPtrRef(p1,p2);
  printf("%d %d\n",*p1,*p2);

  SwapPtrAddr(&p1,&p2);
  printf("%d %d\n",*p1,*p2);

  return 0;
}
于 2013-02-23T17:19:30.370 に答える