Go Back   vb.org Archive > vBulletin Article Depository > Read An Article > vBulletin 3 Articles
Reload this Page Coding Syntax
FAQ Community Calendar Today's Posts Search

Reply
Page 1 of 2 1 2
 
Thread Tools
Coding Syntax
Scott MacVicar
Join Date: Oct 2001
Posts: 1,199

PHP, Perl and gaming

Glasgow, Scotland
Show Printable Version Email this Page Subscription
Scott MacVicar Scott MacVicar is offline 08-29-2002, 10:00 PM
Due to the release of vB3 being around the corner (i like being vague ) I just thought I'd post some coding standards that

have been introduced.

Variables
They must have appropriate names to the data they contain if you've just selected data from the user table don't call it

$stuff or anything like that.
When your reffering to any information passed in by the user use $_REQUEST['variablename'] or $_POST['variablename'] instead

of $variablename, some nice register_globals off compliance.

Basic Syntax
Use tabs to indent the line, this should be done everytime you open a new set of parenthesis and will continue until the closing parenthesis.

Parenthesis should be on a new line of there own like below

PHP Code:
if ($_REQUEST['do'] == 'var')
{
    $var = 'test';
    if ($bbuserinfo['usergroupid'] == 6)
    {
        echo 'your an admin';
    }
} 
If you are using if or elseif, make sure there is a space between that and the comparison.

Use singlequotes from now on where possible, you will not be able to do this when you have a variable in the string as this

will not be evaluated with the string. This includes keys in arrays, unless the key is another array you would use the second example in the following code.

PHP Code:
if (empty($array["var"]))
{
    echo 'blah';
}

//CORRECT USAGE

if (empty($array['var']))
{
    echo 'blah';
}

//if you are referring to the key by another array use
if (empty($array["$array2[var]"]))
{
    echo 'blah';
} 
Functions
When reffered to there should be no spaces between the name and the parameters being passed in.

PHP Code:
imaginary_function($var, $rank); 
When passing in parameters to a function ensure that you have spaces between the comma and each variable being passed in.

Depreciated Functions
It was common to see the following in vBulletin 2, the second version is what will be used in vBulletin 3.

PHP Code:
if (isset($action) == 0)
{
    $action = 'modify';
}

//USE THIS INSTEAD

if (empty($_REQUEST['do']))
{
    $_REQUEST['do'] = 'modify';
} 
Expressions
With an if expression where you are comparing two or more items use OR and AND, not || and &&. This is simply due to the fact that its easier for the human to read.

I know i've forgotten stuff and I will add more as I can think of it.

Hopefully this will improve coding in general when your not using vBulletin and writing your own scripts.
Reply With Quote
  #2  
Old 08-31-2002, 03:57 PM
g-force2k2 g-force2k2 is offline
 
Join Date: Mar 2002
Location: Everywhere you wanna be..
Posts: 1,608
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default
wow PPN thanks alot now i can see some light as to why vb3 will be in fact diferent thanks for sharing the very useful information... regards...

g-force2k2
Reply With Quote
  #3  
Old 08-31-2002, 03:59 PM
Scott MacVicar Scott MacVicar is offline
 
Join Date: Oct 2001
Location: Glasgow, Scotland
Posts: 1,199
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default
Everything is more or less coded to a standard, anything thats not will get changed eventually.
Reply With Quote
  #4  
Old 08-31-2002, 04:03 PM
g-force2k2 g-force2k2 is offline
 
Join Date: Mar 2002
Location: Everywhere you wanna be..
Posts: 1,608
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default
i see around all the array stuff

ie ::

PHP Code:
$bbuserinfo['usergroupid'] 
instead of the usual ::

PHP Code:
$bbuserinfo[usergroupid] 
i've read that the first one (that vb3 uses) is the correct way to do it then the second one... can you confirm that this is in fact true?

Also should we not use the second example anymore? regards...

g-force2k2
Reply With Quote
  #5  
Old 08-31-2002, 04:09 PM
Scott MacVicar Scott MacVicar is offline
 
Join Date: Oct 2001
Location: Glasgow, Scotland
Posts: 1,199
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default
Yep vB3 uses the first and it is correct, if you dont have any quotes then its an undefined constant, using the quotes indicates a string.

Added a bit about using 'and' and 'or' instead of || and &&
Reply With Quote
  #6  
Old 09-01-2002, 06:28 PM
g-force2k2 g-force2k2 is offline
 
Join Date: Mar 2002
Location: Everywhere you wanna be..
Posts: 1,608
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default
question PPN?

whats the difference btw using? the positioning of if?

ie ::

PHP Code:
if ($_REQUEST['action'] == "go_here") { 
and

PHP Code:
if($_REQUEST['action'] == "go_here") { 
does that space make it more efficient or something?

And is this $_REQUEST and $_POST vars vb3 creations or are they standard with php? regards...

g-force2k2
Reply With Quote
  #7  
Old 09-01-2002, 09:42 PM
Scott MacVicar Scott MacVicar is offline
 
Join Date: Oct 2001
Location: Glasgow, Scotland
Posts: 1,199
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default
Em just cosmetic with the if, it looks nicer

its the PHP Super Globals http://www.php.net/manual/en/languag...predefined.php for $_POST and $_REQUEST.

Though $_REQUEST is used because its all input from the user.
Reply With Quote
  #8  
Old 09-03-2002, 12:17 PM
Admin's Avatar
Admin Admin is offline
Coder
  
Join Date: Oct 2023
Location: Server
Posts: 1
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default
There are 7 super global arrays in PHP, as of 4.1.0:
$_GET - contains all information that passed in the address bar (or a form with get method)... used to be $HTTP_GET_VARS.
$_POST - contains all information that is passed through a posted form (if the form was using the post method)... used to be $HTTP_POST_VARS.
$_COOKIE - contains all cookie information... used to be $HTTP_COOKIE_VARS.
$_REQUEST - simply a merge of all three arrays above... new.
$_FILES - contains information about uploaded files... used to be $HTTP_POST_FILES.
$_ENV - all environment variables... new AFAIK.
$_SERVER - all server variables... used to be $HTTP_SERVER_VARS.

Note that in vB3 these arrays are NOT super globals, or at least you must not assume they are. You'll see why when you actually see the code in init.php.
Reply With Quote
  #9  
Old 09-28-2002, 04:55 AM
DraX DraX is offline
 
Join Date: Aug 2002
Location: Bear, DE
Posts: 54
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default
This brings up a question... will vB3 still be compatible on servers running older versions of php?

Due to an operating system conflict (damned Sun Microsystems machines), I can't safely upgrade my version of php without the potential of breaking the control panel for the OS....

Is there a function to determine if the version of php supports the super globals, such as $_REQUEST and if not, to define it as a new global using $HTTP_GET_VARS['REQUEST'] (I think that was right... off the top of my head)?
Reply With Quote
  #10  
Old 09-28-2002, 06:19 AM
JulianD's Avatar
JulianD JulianD is offline
 
Join Date: Jan 2002
Posts: 455
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default
Yeah, it's one of my main concerns about vb3. What will be the minimun required PHP version to run vb3?
Reply With Quote
Reply
Page 1 of 2 1 2

« Previous Thread | Next Thread »

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts
BB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Forum Jump


All times are GMT. The time now is 09:11 AM.

Contact Us - Archive - Top

Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2024, vBulletin Solutions Inc.
X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.06007 seconds
  • Memory Usage 2,305KB
  • Queries Executed 23 (?)
More Information
Template Usage:
  • (1)SHOWTHREAD
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (1)ad_showthread_beforeqr
  • (8)bbcode_php
  • (1)footer
  • (1)forumjump
  • (1)forumrules
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (1)modsystem_article
  • (1)navbar
  • (4)navbar_link
  • (120)option
  • (1)pagenav
  • (1)pagenav_curpage
  • (1)pagenav_pagelink
  • (10)post_thanks_box
  • (10)post_thanks_button
  • (1)post_thanks_javascript
  • (1)post_thanks_navbar_search
  • (10)post_thanks_postbit_info
  • (9)postbit
  • (10)postbit_onlinestatus
  • (10)postbit_wrapper
  • (1)spacer_close
  • (1)spacer_open
  • (1)tagbit_wrapper 
Phrase Groups Available:
  • global
  • inlinemod
  • postbit
  • posting
  • reputationlevel
  • showthread
Included Files:
  • ./showthread.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/functions_bigthree.php
  • ./includes/class_postbit.php
  • ./includes/class_bbcode.php
  • ./includes/functions_reputation.php
  • ./includes/functions_post_thanks.php 
Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • showthread_start
  • showthread_getinfo
  • forumjump
  • showthread_post_start
  • showthread_query_postids
  • showthread_query
  • bbcode_fetch_tags
  • bbcode_create
  • showthread_postbit_create
  • postbit_factory
  • postbit_display_start
  • post_thanks_function_post_thanks_off_start
  • post_thanks_function_post_thanks_off_end
  • post_thanks_function_fetch_thanks_start
  • post_thanks_function_fetch_thanks_end
  • post_thanks_function_thanked_already_start
  • post_thanks_function_thanked_already_end
  • fetch_musername
  • postbit_imicons
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • postbit_display_complete
  • post_thanks_function_can_thank_this_post_start
  • pagenav_page
  • pagenav_complete
  • tag_fetchbit_complete
  • forumrules
  • navbits
  • navbits_complete
  • showthread_complete