Problem exchanging values using pointers.

[HottoDogu]

hi, i've been having trouble with relocating a value in a vector when using pointers as passage in the 'exchange' function.

in this exercise i need to read n values for a vector, then pass a new value to the function as a pointer, and order it inside the vector.

#include <iostream>
using namespace std;

void troca (int*, int*, int);

int main () {
 int n, i, tamanho, vet[10], *ptrvalor, valor;
   cin >> n;
 
 for (i=0;i<n;i++) {
   cin >> vet[i];
 } 
 
 cin >> valor;
 ptrvalor = &valor;
 tamanho = sizeof(vet)/sizeof(int);
 troca (vet, ptrvalor, tamanho);
 
 for (int j=0;j<n+1;j++) {
   cout << vet[j] << "";
 } cout << endl;
 
return 0;
}

void troca (int* vet, int* ptrvalor, int tamanho) {
  int aux;
    
  for (int i=(tamanho-1); i>0; i--) {
    if (*ptrvalor > vet[i]) {
      aux = vet[i];
      vet[i] = *ptrvalor;
      vet[i+1] = aux;      
    }
  }
}

i don't understand what i'm doing wrong in the passage, as the code compiles.
if i input, say, 'n' as 5, then fill the vector with '1,2,3,5,6' and the value as '4';

it should output: '1,2,3,4,5,6'

but the result i get is: '1,2,4,5,6,4'

please help me find my flawed logic, or point me to a thread where a similar problem has been solved, as i couldn't find any. thanks.

[jonnin]

is 4 > 6? no, skip if body.
is 4 > 5? no, skip if body.
is 4 > 3? yes.
aux = 3
vet[I] = 4 (list is now 1,2,4,5,6) aux is 3
… keep going. what happens? Is what happened correct?
just watch what it does and you will see that the logic isn't right.

then do the problem by hand, and try to code what you did.