C++: Program not outputting function return value

Question

I can't figure out why the return value form the "deleteEntry" function is not being output in the last part of my program. It jumps from "Now your list has 5 names again!" to asking if you want to do it again without showing the list without the name you asked it to delete. I know it did it correctly when I tried earlier, but I'm not sure if I changed something without paying attention or if I'm doing something that makes the program do things randomly.

Here's the source code:

#include <iostream>
#include <string>
using namespace std;

int search(string* dynamicArray, int size, string entrytoDelete);

string* addEntry(string *dynamicArray, int&size, string newEntry);

string* deleteEntry(string *dynamicArray, int &size, string entrytoDelete);

int main()
{
  char answer;

  do
    {

      string* name;
      string name2;
      int size = 5;
      int x;
      name = new string[5];

      cout << "\nEnter 5 names:\n";
      for(x = 0; x < size; x++)
    {
      getline(cin, name[x]);
    }
      cout << endl;
      cout << "You entered: \n";
      for(x = 0; x< size; x++)
    {
      cout << *(name + x) << endl;
    }

      cout << "\nEnter another name: \n";
      getline(cin, name2);
      name = addEntry(name, size, name2);
      cout << "\nYour list now has an extra name!\n";
      for(x = 0; x < size; x++)
    cout << x << ": " << name[x] << endl;

      cout<< "\nPick one name to delete: \n";
      getline(cin, name2);
      name = deleteEntry(name, size, name2);
      cout << "\nNow your list has 5 names again!\n";
      for(x = 0; x < size; x++)
    cout << x << ": " << name[x] << endl;

      cout << "\nWould you like to try again? (Y/N)\n";
      cin >> answer;
    }while(answer == 'y' || answer == 'Y');

  cout << "Goodbye.\n";

  return 0;
}


string* addEntry(string *dynamicArray, int&size, string newEntry)
{ 
  string *new_Large = new string[size + 1];

  for(int x = 0; x < size; x++)
    {
      new_Large[x] = *(dynamicArray + x);
    }

  new_Large[size] = newEntry;
  size++;

  delete [] dynamicArray;


  return new_Large;
}

string* deleteEntry(string *dynamicArray, int &size, string entrytoDelete)
{
  int toDelete;
  toDelete = search(dynamicArray, size, entrytoDelete);

  if(toDelete >= 0)
    {
      string *new_Small = new string[size - 1];

      for(int x = 0; x < (size - 1); x++)
    {
      new_Small[x] = *(dynamicArray + x);
      if(dynamicArray[x] != entrytoDelete)
         new_Small[x] = dynamicArray[x];
    }
      size -= size;
      return new_Small;
    }
  else 
    {
      cout << entrytoDelete << " does not exist.\n";
      return dynamicArray;
    }
}

int search(string* dynamicArray, int size, string entrytoDelete)
{
  int toDelete = -1;
  for(int x = 0; x < size; x++)
    {
      if(*(dynamicArray + x) == entrytoDelete)
    {
      toDelete = x;
    }
    }
      return toDelete;
}

I'd really appreciate any help.

Answer 1

As other mentioned, this code has memory leaks. I can see many flaws in the code. But, to suggest a small change to produce what you want, you may start with the following changes. See the comments in the code:

#include <iostream>
#include <string>
using namespace std;

int search(string* dynamicArray, int size, string entrytoDelete);

string* addEntry(string *dynamicArray, int&size, string newEntry);

string* deleteEntry(string *dynamicArray, int &size, string entrytoDelete);

int main()
{
  char answer;

  do
    {

      string* name;
      string name2;
      int size = 5;
      int x;
      name = new string[5];

      cout << "\nEnter 5 names:\n";
      for(x = 0; x < size; x++)
    {
      getline(cin, name[x]);
    }
      cout << endl;
      cout << "You entered: \n";
      for(x = 0; x< size; x++)
    {
      cout << *(name + x) << endl;
    }

      cout << "\nEnter another name: \n";
      getline(cin, name2);
      name = addEntry(name, size, name2);
      cout << "\nYour list now has an extra name!\n";
      for(x = 0; x < size; x++)
    cout << x << ": " << name[x] << endl;

      cout<< "\nPick one name to delete: \n";
      getline(cin, name2);
      name = deleteEntry(name, size, name2);
      cout << "\nNow your list has 5 names again!\n";
      for(x = 0; x < size; x++)
    cout << x << ": " << name[x] << endl;

      cout << "\nWould you like to try again? (Y/N)\n";
      cin >> answer;
    }while(answer == 'y' || answer == 'Y');

  cout << "Goodbye.\n";

  return 0;
}


string* addEntry(string *dynamicArray, int&size, string newEntry)
{ 
  string *new_Large = new string[size + 1];

  for(int x = 0; x < size; x++)
    {
      new_Large[x] = *(dynamicArray + x);
    }

  new_Large[size] = newEntry;
  size++;

  delete [] dynamicArray;


  return new_Large;
}

string* deleteEntry(string *dynamicArray, int &size, string entrytoDelete)
{
  int toDelete;
  toDelete = search(dynamicArray, size, entrytoDelete);

  if(toDelete >= 0)
    {
      string *new_Small = new string[size - 1];

      for(int x = 0, y = 0; x < size; x++)   //<- start of changes
    {                                        //
      // new_Small[x] = *(dynamicArray + x); //<- remove this
      if(dynamicArray[x] != entrytoDelete)   //
         new_Small[y++] = dynamicArray[x];   //<- x to y++
    }                                        //
      --size;                                //<- end of changes
      return new_Small;
    }
  else 
    {
      cout << entrytoDelete << " does not exist.\n";
      return dynamicArray;
    }
}

int search(string* dynamicArray, int size, string entrytoDelete)
{
  int toDelete = -1;
  for(int x = 0; x < size; x++)
    {
      if(*(dynamicArray + x) == entrytoDelete)
    {
      toDelete = x;
    }
    }
      return toDelete;
}

Answer 2

In your deleteEntry:

size -= size;

This will always set size = 0. I think you're looking to do:

size--;