Private Messaging System

[Jam-ez]

Hi, :)

I'm in need of a private messaging system that comes with a login. Or perhaps an existing user system with a messaging system built in.

Any ideas?

[Calon]

I could easily build one for you if you wanted. (free because I'm kind)

[Jam-ez]

Well if you would I'd appreciate it. :)

All it needs is a login, inbox, outbox, send message, sent messages, nothing too fancy. An admin panel is optional (for deleting/viewing all mail).

Thanks a lot! :O

[jackass]

I could do with one of these really. :P

[Calon]

Started coding it now. :)

[Calon]

http://rapidshare.com/files/170615372/pm.zip

Didn't have the time to finish it, sorry.

[Jam-ez]

How far is it, and is anyone else up for the challenge?

[Jackboy]

Just looking at a small part of your code.


{
public function mysql_connect( $hostname, $username, $password, $database )
{
$db_connect = mysql_connect( $hostname, $username, $password );
$sr_connect = mysql_select_db( $database );
}Why have you used variable names. You can just have


{
public function mysql_connect( $hostname, $username, $password, $database )
{
mysql_connect( $hostname, $username, $password );
mysql_select_db( $database );
}You don't actually need to define variables unless you plan to use them later or w/e.

But nice of you for trying :) Also why have you put the public function as an actual reserved function? Im confused. Well have fun learning PHP :D

[Calon]

Just looking at a small part of your code.


{
public function mysql_connect( $hostname, $username, $password, $database )
{
$db_connect = mysql_connect( $hostname, $username, $password );
$sr_connect = mysql_select_db( $database );
}Why have you used variable names. You can just have


{
public function mysql_connect( $hostname, $username, $password, $database )
{
mysql_connect( $hostname, $username, $password );
mysql_select_db( $database );
}You don't actually need to define variables unless you plan to use them later or w/e.

But nice of you for trying :) Also why have you put the public function as an actual reserved function? Im confused. Well have fun learning PHP :D
public is a keyword, so the function in the class is publicly usable, you should read up on it ^_^

[Jackboy]

public is a keyword, so the function in the class is publicly usable, you should read up on it ^_^

Well i said what i was trying to say wrong :P

I was trying to say, whats the point in using public function mysql_connect when mysql_connect is already useable without that silly function there...

[Excellent2]

Lol Calon there was really no need to try being fancy :P

[Calon]

Lol Calon there was really no need to try being fancy :P
No, there is because MySQL doesn't have a function which connects to both the database and the server, hence why I created a custom function for it.

[Jackboy]

No, there is because MySQL doesn't have a function which connects to both the database and the server, hence why I created a custom function for it.

Oh dear. I really can't be bothered to argue.

I just wouldn't have done it that way, tis my opinion :P

[Calon]

Oh dear. I really can't be bothered to argue.

I just wouldn't have done it that way, tis my opinion :P
To be honest, we all have our different ways of coding. And it is your opinion, I couldn't be bothered using two functions which could easily be done in one.

[Source]

Calon, creating a class for something with a single purpose seems extremely pointless in a variety of ways to me - you could of done it nice and simply with a functions.php and a config.php (with the connection in the config.php). Seems as though you try todo coding methods which are meant to impress people, which in the long run is making more hassle.

[Jam-ez]

Well, whatever. Is anyone else up for the challenge or does anyone know of any good systems?

[Hypertext]

Unless he's making a MySQL wrapper, like if he's going to use MySQLi, there isn't much point putting it in a class.

[Jam-ez]

Yabberer has done this for me. Thanks to him!

[Calon]

Unless he's making a MySQL wrapper, like if he's going to use MySQLi, there isn't much point putting it in a class.


Calon, creating a class for something with a single purpose seems extremely pointless in a variety of ways to me - you could of done it nice and simply with a functions.php and a config.php (with the connection in the config.php). Seems as though you try todo coding methods which are meant to impress people, which in the long run is making more hassle.

I'm not trying to impress anyone, I just found it easier too do it my way as "code is art" - quoted from w3c. And I try keep my code neat, and I found using one function instead of using two might be a better method. Clearly, proven wrong.

[L?KE]

Ahh, give him a break :P At least he admits he's wrong.

[Protege]

I think no one should of coded it for him, I think the best method was to help him make it himself - at least he could of learnt something and had a working PM system.