LeaveMessage()

[fragman85]

This is my first post in the Code Gallery, I am looking for good critique to improve my coding/styling etc.

If you like my code, feel free to use/edit it.

Also I think this is quite a good example for using DataBase NPCs :o

So I made this 4 functions:

LeaveMessage(Account, MSG, DataBase)

Will create/add a Message to a Database as an array (named as the Account of the Receiver.)

CheckMessage(Account, DataBase)

Will return how many Messages are saved for an Account in a DataBase.

GetMessage(Account, Database, ID)

Will return a Message. If ID is not provided, the first one.

DeleteMessage(Account, Database, ID)

Will delete a Message, if ID is not provided the first one.

PHP Code:

function LeaveMessage(Account, MSG, DataBase) {
  if (Account && MSG && DataBase) {
    // Edited:
      findNPC(DataBase).this.(@ Account).add(MSG);
      return true;
  }
  else {
    echo("LeaveMessage(Account, MSG, DataBase)!");
    return false;
  }
}
function CheckMessage(Account, DataBase) {
  if (Account && DataBase) {
    return findNPC(DataBase).(@ Account).size();
  }
  else {
    echo("CheckMessage(Account, Database!");
    return false;
  }
}
function GetMessage(Account, DataBase, ID) {
  if (Account && DataBase) {
    if (ID > 0) {
      return findNPC(DataBase).(@ Account)[ID - 1];
    }
    else {
      return findNPC(DataBase).(@ Account)[0];
    }
  }
  else {
    echo("GetMessage(Account, DataBase, ID)!");
    return false;
  }
}
function DeleteMessage(Account, DataBase, ID) {
  if (Account && DataBase) {
    if (ID > 0) {
      findNPC(DataBase).(@ Account).delete(ID - 1);
      return true;
    }
    else {
      findNPC(DataBase).(@ Account).delete(0);
      return true;
    }
  }
  else {
    echo("DeleteMessage(Account, DataBase, ID)!");
    return false;
  }
} 


As I said above, I'm looking for critique ;D