Registered Count on Header

[Glee]

Ok, i started to learn PHP and MySQL i created a little script for people that don't have the Registered Count on the the Header Here we go

First open inc/class.core.php

find

public static function UsersOnline()
	{
		$online = mysql_fetch_array(mysql_query("SELECT * FROM server_status"));
		return intval($online['users_online']);
	}

Under it add

 public static function UsersRegistered()
	{
		$register = mysql_query("SELECT * FROM users");  
                return intval ($users = mysql_num_rows($register));
	}

Save it and Done with that

Now Open header.php

find

<div class="OnlineCount"><?php echo $core->UsersOnline(); ?> <?php echo $lang['users_online']; ?></div>

Under it add this

<div class="RegisterCount"><?php echo $core->UsersRegistered(); ?> <?php echo $lang['registered']; ?></div>

Also for got Open header.php again

find

function RefreshUserCount()
{
var html = $.ajax({
url: "online.php",
async: false
}).responseText;
$('.OnlineCount').fadeOut(200, function () {
$('.OnlineCount').css("display", "none")
$('.OnlineCount').html(html + ' <?php echo $lang['users_online']; ?>')
$('.OnlineCount').fadeIn(200)
});
setTimeout("RefreshUserCount()",120000);
}

Under it add

function UsersRegistered()
{
var html = $.ajax({
url: "registered.php",
async: false
}).responseText;
$('.RegisterCount').fadeOut(200, function () {
$('.RegisterCount').css("display", "none")
$('.RegisterCount').html(html + ' <?php echo $lang['users_online']; ?>')
$('.RegisterCount').fadeIn(200)
});
setTimeout("UsersRegistered()",120000);
}

Now open lang/en.php

find

'users_online' => 'Users Online'

under it add

'registered' => 'Users Registered'

Then after that all done Create and File called registered.php and place this in it

<?php
define('USERNAME_REQUIRED', FALSE);
define('ACCOUNT_REQUIRED', FALSE);
include('global.php');
echo $core->UsersRegistered();
?>

Then your Done!!!

Screenie:

UPDATED

For the Registered to show on index page open index.php

fine

<?php echo $core->UsersOnline(); ?> <?php echo $lang['users_online']; ?>

beside it add

<br /> <?php echo $core->UsersRegistered(); ?> <?php echo $lang['registered']; ?>

Make sure you have the <br />

 

[Jonteh]

Simple but good for a beginner. Well done

 

[Glee]

Thank You

 

[leenster]

Good job. Pretty much the same way i did it.... U even used the lang file...

 

[Glee]

Yepper

 

[Putako]

Great job man You out-did yourself.

 

[Glee]

Thank's

 

[Someuser]

*cough* MySQL overload *cough*

 

[TJP]

Why did you bother to make it use more resources then it needed to. You could of easily done something like this as your functions which a) use less resources and b) take less time to execute (seeing as PHP is an interpreted language).

NOTE: Not tested, but you should get the idea.

// Code inside total registered.

return mysql_query("SELECT DISTINCT COUNT(*) FROM users;");

// Code inside total online.

return mysql_query("SELECT DISTINCT COUNT(*) FROM users AS u WHERE u.online = '1';");

Hopefully you'll learn from this project so the next one is better. Have fun.

 

[Glee]

Why did you bother to make it use more resources then it needed to. You could of easily done something like this as your functions which a) use less resources and b) take less time to execute (seeing as PHP is an interpreted language).

NOTE: Not tested, but you should get the idea.

// Code inside total registered.

return mysql_query("SELECT DISTINCT COUNT(*) FROM users;");

// Code inside total online.

return mysql_query("SELECT DISTINCT COUNT(*) FROM users AS u WHERE u.online = '1';");

Hopefully you'll learn from this project so the next one is better. Have fun.

Putting this on the header.php screw's up the CMS

 

[TJP]

Putting this on the header.php screw's up the CMS

I meant it as put it inside the two public static functions, as oppose to what you currently have.

 

[Matthew]

NOTE: Not tested, but you should get the idea.

// Code inside total registered.

return mysql_query("SELECT DISTINCT COUNT(*) FROM users;");

// Code inside total online.

return mysql_query("SELECT DISTINCT COUNT(*) FROM users AS u WHERE u.online = '1';");

Hopefully you'll learn from this project so the next one is better. Have fun.

His code is fine. Yours is making MySQL do even more work doing that. Why do you over-engineer things that shouldn't be ?

Even though this is simple it's a start and a good effort. Congrats!

 

[TJP]

His code is fine. Yours is making MySQL do even more work doing that. Why do you over-engineer things that shouldn't be ?

LOL and you're trying to tell me you have this amazing "mystery" work on your computer when you say something as stupid as that. Go back to w3schools rock star .

 

[Glee]

Please Do not Flame my Post Thank You

 

[TJP]

Please Do not Flame my Post Thank You

I wasn't, I am simply just providing some feedback.

 

[c0mma]

It's a ok release, very simple, but good job! Sad I don't use PhoenixPHP anymore.

 

[Matthew]

LOL and you're trying to tell me you have this amazing "mystery" work on your computer when you say something as stupid as that. Go back to w3schools rock star .

This guy has just released something so basic and he's new to coding. But it's better than anything you've ever contributed. And why would I need to do that ? Prove my work isn't on my computer