[1578] | 1 | <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> |
---|
| 2 | |
---|
| 3 | <html> |
---|
| 4 | <head> |
---|
| 5 | <title>Perl coding standard for FCM</title> |
---|
| 6 | <meta name="author" content="FCM development team"> |
---|
| 7 | <meta name="descriptions" content="Perl coding standard for FCM"> |
---|
| 8 | <meta name="keywords" content="Perl, coding standard, FCM"> |
---|
| 9 | <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> |
---|
| 10 | <link rel="StyleSheet" type="text/css" href="style.css"> |
---|
| 11 | <style type="text/css"> |
---|
| 12 | <!-- |
---|
| 13 | li, th, td { |
---|
| 14 | padding: 3px; |
---|
| 15 | } |
---|
| 16 | td { |
---|
| 17 | vertical-align: top; |
---|
| 18 | } |
---|
| 19 | th.r { |
---|
| 20 | text-align: right; |
---|
| 21 | background-color: transparent; |
---|
| 22 | padding-right: 5px; |
---|
| 23 | } |
---|
| 24 | th.l { |
---|
| 25 | text-align: left; |
---|
| 26 | background-color: transparent; |
---|
| 27 | } |
---|
| 28 | --> |
---|
| 29 | </style> |
---|
| 30 | </head> |
---|
| 31 | |
---|
| 32 | <body> |
---|
| 33 | <p align="right"><img src="logo.png" alt="Met Office logo" width="85" |
---|
| 34 | height="85"></p> |
---|
| 35 | |
---|
| 36 | <h1>Perl coding standard for FCM</h1> |
---|
| 37 | |
---|
| 38 | <p align="center">Last updated: 28 November 2006</a> |
---|
| 39 | |
---|
| 40 | <p align="center">Met Office<br> |
---|
| 41 | FitzRoy Road, Exeter<br> |
---|
| 42 | Devon, EX1 3PB<br> |
---|
| 43 | United Kingdom</p> |
---|
| 44 | |
---|
| 45 | <p align="center">© Crown copyright 2005-6. All rights reserved.</p> |
---|
| 46 | |
---|
| 47 | <p align="center">Questions regarding this document or permissions to quote |
---|
| 48 | from it should be directed to the <a href= |
---|
| 49 | "mailto:iprmanager@metoffice.gov.uk">IPR Manager</a>.</p> |
---|
| 50 | |
---|
| 51 | <script type="text/javascript"> |
---|
| 52 | <!-- |
---|
| 53 | var out = 'For printing, please use the ' |
---|
| 54 | out += '<a href="fcm-perl-standard.pdf">PDF<\/a> version of the document.' |
---|
| 55 | document.write ('<p align="center">') |
---|
| 56 | document.write (out) |
---|
| 57 | document.write ('<\/p>') |
---|
| 58 | //--> |
---|
| 59 | </script> |
---|
| 60 | |
---|
| 61 | <h2>Contents</h2> |
---|
| 62 | |
---|
| 63 | <ul> |
---|
| 64 | <li>Main contents: |
---|
| 65 | |
---|
| 66 | <ol> |
---|
| 67 | <li><a href="#1">Introduction</a></li> |
---|
| 68 | |
---|
| 69 | <li><a href="#2">Online Perl style guides</a></li> |
---|
| 70 | |
---|
| 71 | <li><a href="#3">Coding standard</a></li> |
---|
| 72 | </ol> |
---|
| 73 | </li> |
---|
| 74 | </ul><a name="1"></a> |
---|
| 75 | |
---|
| 76 | <h2>1. Introduction</h2> |
---|
| 77 | |
---|
| 78 | <p>Perl (Practical Extraction and Report Language) is gaining a lot of |
---|
| 79 | popularity at the Met Office for developing applications that are |
---|
| 80 | traditionally programmed using shell script. For example, it is currently our |
---|
| 81 | choice of language for developing the in-house components of the FCM |
---|
| 82 | system.</p> |
---|
| 83 | |
---|
| 84 | <p>Perl is a very powerful general purpose scripting tool that is free and |
---|
| 85 | portable across a huge variety of platforms including many non-Unix systems. |
---|
| 86 | Replacing a shell script with an equivalent Perl program often results in a |
---|
| 87 | massive reduction in runtime - using cleaner syntax and algorithm.</p> |
---|
| 88 | |
---|
| 89 | <p>Perl is a language with a rich set of "grammar". To most people, the first |
---|
| 90 | impression of a piece of code written in Perl is that it is very ugly. Its |
---|
| 91 | lines are full of punctuation junks that nobody can hope to understand. This |
---|
| 92 | is often caused by poorly written programs coupled with little and often |
---|
| 93 | inadequte documentations. To improve readability and to reduce the overheads |
---|
| 94 | of maintenance, it is important for Perl programmers to write their code in a |
---|
| 95 | nice and consistent way.</p> |
---|
| 96 | |
---|
| 97 | <p>The aim of this document is to propose ideas on how the Perl programming |
---|
| 98 | language should be used with FCM.</p> |
---|
| 99 | |
---|
| 100 | <a name="2"></a><h2>2. Online Perl style guides</h2> |
---|
| 101 | |
---|
| 102 | <p>There are many Perl style guides available online. Some are listed |
---|
| 103 | below:</p> |
---|
| 104 | |
---|
| 105 | <ul> |
---|
| 106 | <li>the manpages <code>perlstyle</code> and <code>perlmodstyle</code></li> |
---|
| 107 | |
---|
| 108 | <li><a href= |
---|
| 109 | "http://www.kulnet.kuleuven.ac.be/perlcourse/perlingo.html">Perlingo</a></li> |
---|
| 110 | |
---|
| 111 | <li><a href="http://ali.as/devel/code.html">Perl Style Guide for Large Scale |
---|
| 112 | Object Oriented Development</a></li> |
---|
| 113 | |
---|
| 114 | <li><a href="http://perltidy.sourceforge.net/">Perltidy</a></li> |
---|
| 115 | |
---|
| 116 | <li><a href= |
---|
| 117 | "http://perl.apache.org/docs/2.0/devel/core/coding_style.pdf">mod_perl |
---|
| 118 | Coding Style Guide</a> (PDF)</li> |
---|
| 119 | </ul> |
---|
| 120 | |
---|
| 121 | <p>It is worth noting that the ideas in some of the Perl style guides may |
---|
| 122 | conflict with each other - as they are targeted to different people. However, |
---|
| 123 | there is a common theme to most of the best practices. The following is a |
---|
| 124 | summary:</p> |
---|
| 125 | |
---|
| 126 | <ul> |
---|
| 127 | <li>A source file, whether it is a top-level script or a module, should |
---|
| 128 | contain a header block with information of the source file, |
---|
| 129 | some general description of the code as well as standard pragmas or |
---|
| 130 | options for running the Perl interpreter.</li> |
---|
| 131 | |
---|
| 132 | <li>Each function should be preceded by a comment block or a block of |
---|
| 133 | POD to specify a synopsis for the defined function.</li> |
---|
| 134 | |
---|
| 135 | <li>Avoid using names of built-in functions to name your new functions.</li> |
---|
| 136 | |
---|
| 137 | <li>If the parameter list of a function is becoming too long, e.g. more |
---|
| 138 | than 3 arguments, you may want to implement the argument list with a hash, |
---|
| 139 | so that all the arguments are named.</li> |
---|
| 140 | |
---|
| 141 | <li>Indent to an appropriate number of spaces/tabs for code blocks.</li> |
---|
| 142 | |
---|
| 143 | <li>Open curly brackets should be on the same line as the keyword.</li> |
---|
| 144 | |
---|
| 145 | <li>Close curly brackets should line up with the keywords that started |
---|
| 146 | the block.</li> |
---|
| 147 | |
---|
| 148 | <li>Insert a blank line between chunks of code that do different |
---|
| 149 | things.</li> |
---|
| 150 | |
---|
| 151 | <li>Use space between tokens and operators to improve readability. |
---|
| 152 | Use space after commas. No space before commas and semi-colons.</li> |
---|
| 153 | |
---|
| 154 | <li>Line up corresponding items vertically.</li> |
---|
| 155 | |
---|
| 156 | <li>Use parentheses only if necessary.</li> |
---|
| 157 | |
---|
| 158 | <li>Restrict line length to e.g. 80 characters.</li> |
---|
| 159 | |
---|
| 160 | <li>Be consistent with style.</li> |
---|
| 161 | |
---|
| 162 | <li>Use the "strict" and "warnings" pragmas.</li> |
---|
| 163 | |
---|
| 164 | <li>options and arguments should be processed as "soon" as possible, |
---|
| 165 | preferably at the beginning of a source file or a function.</li> |
---|
| 166 | |
---|
| 167 | <li>Use the "__END__" directive at the end of a Perl script.</li> |
---|
| 168 | |
---|
| 169 | <li>-more later-</li> |
---|
| 170 | </ul> |
---|
| 171 | |
---|
| 172 | <a name="3"></a><h2>3. Coding standard</h2> |
---|
| 173 | |
---|
| 174 | <p>The main question is: what should we include in our coding standard? |
---|
| 175 | Although we would like to recommend using the best practices described in |
---|
| 176 | the last section, we would not want to impose any restriction, as Perl is a |
---|
| 177 | language designed to do things in many different ways. The only thing we |
---|
| 178 | would like to impose is a header block in each source file, and a header |
---|
| 179 | comment block for each function in a source file.</p> |
---|
| 180 | |
---|
| 181 | <p>For a Perl executable, the header block of the source file should contain |
---|
| 182 | the following:</p> |
---|
| 183 | |
---|
| 184 | <ul> |
---|
| 185 | <li>the first line should be <tt>"#!/usr/bin/perl"</tt></li> |
---|
| 186 | |
---|
| 187 | <li>the name of the executable</li> |
---|
| 188 | |
---|
| 189 | <li>a synopsis and/or a description or what the executable does</li> |
---|
| 190 | |
---|
| 191 | <li>an explanation of the options and arguments (either in separate |
---|
| 192 | sections or described within the "description" section)</li> |
---|
| 193 | |
---|
| 194 | <li>copyright and owner information</li> |
---|
| 195 | |
---|
| 196 | <li>no history or revision information, (as we would prefer using the |
---|
| 197 | Subversion log message to record such information)</li> |
---|
| 198 | |
---|
| 199 | <li>use "strict" and use "warnings" statements</li> |
---|
| 200 | |
---|
| 201 | <li>use the "__END__" directive at the end of a Perl script.</li> |
---|
| 202 | </ul> |
---|
| 203 | |
---|
| 204 | <table summary="executable header block example" border="1" width="100%"> |
---|
| 205 | <tr> |
---|
| 206 | <th>Executable header block example</th> |
---|
| 207 | </tr> |
---|
| 208 | |
---|
| 209 | <tr> |
---|
| 210 | <td> |
---|
| 211 | <pre> |
---|
| 212 | #!/usr/bin/perl |
---|
| 213 | # ------------------------------------------------------------------------------ |
---|
| 214 | # NAME |
---|
| 215 | # example.pl |
---|
| 216 | # |
---|
| 217 | # SYNOPSIS |
---|
| 218 | # example.pl [options] args |
---|
| 219 | # |
---|
| 220 | # DESCRIPTION |
---|
| 221 | # This is a header example, and so on and so forth. |
---|
| 222 | # |
---|
| 223 | # OPTIONS |
---|
| 224 | # -b [--bar] - this option does what "bar" does. |
---|
| 225 | # -f [--foo] - this option does what "foo" does. |
---|
| 226 | # |
---|
| 227 | # ARGUMENTS |
---|
| 228 | # args - description of each argument. |
---|
| 229 | # |
---|
| 230 | # SEE ALSO |
---|
| 231 | # list of relevant commands, modules and documents |
---|
| 232 | # |
---|
| 233 | # COPYRIGHT |
---|
| 234 | # (C) Crown copyright Met Office. All rights reserved. |
---|
| 235 | # For further details please refer to the file COPYRIGHT.txt |
---|
| 236 | # which you should have received as part of this distribution. |
---|
| 237 | # ------------------------------------------------------------------------------ |
---|
| 238 | |
---|
| 239 | # Standard pragmas |
---|
| 240 | use strict; |
---|
| 241 | use warnings; |
---|
| 242 | |
---|
| 243 | # Standard modules |
---|
| 244 | use Getopt::Long; |
---|
| 245 | |
---|
| 246 | # ------------------------------------------------------------------------------ |
---|
| 247 | |
---|
| 248 | # Process options |
---|
| 249 | my ($foo, $bar); |
---|
| 250 | GetOptions ( |
---|
| 251 | 'foo|f' => \$foo, |
---|
| 252 | 'bar|b' => \$bar, |
---|
| 253 | ); |
---|
| 254 | |
---|
| 255 | # Process arguments |
---|
| 256 | my $arg = shift @ARGV; |
---|
| 257 | |
---|
| 258 | # Do something... |
---|
| 259 | print 'This is an example: ', $arg, "\n"; |
---|
| 260 | print 'FOO', "\n" if $foo; |
---|
| 261 | print 'BAR', "\n" if $bar; |
---|
| 262 | |
---|
| 263 | __END__ |
---|
| 264 | </pre> |
---|
| 265 | </td> |
---|
| 266 | </tr> |
---|
| 267 | </table> |
---|
| 268 | |
---|
| 269 | <p>For a Perl module, the header block of the source file should contain |
---|
| 270 | the following:</p> |
---|
| 271 | |
---|
| 272 | <ul> |
---|
| 273 | <li>the first line should be <tt>"#!/usr/bin/perl"</tt></li> |
---|
| 274 | |
---|
| 275 | <li>the name of the module</li> |
---|
| 276 | |
---|
| 277 | <li>a synopsis and/or a description or what the module does</li> |
---|
| 278 | |
---|
| 279 | <li>copyright and owner information</li> |
---|
| 280 | |
---|
| 281 | <li>no history or revision information</li> |
---|
| 282 | |
---|
| 283 | <li>use "strict" and use "warnings" statements</li> |
---|
| 284 | |
---|
| 285 | <li>a list of exports (do not export if you are writing an OO module)</li> |
---|
| 286 | |
---|
| 287 | <li>use the "__END__" directive at the end of a Perl script, and remember |
---|
| 288 | to put 1 before the end of the module, so that it will return true.</li> |
---|
| 289 | </ul> |
---|
| 290 | |
---|
| 291 | <table summary="module header block example" border="1" width="100%"> |
---|
| 292 | <tr> |
---|
| 293 | <th>Module header block example</th> |
---|
| 294 | </tr> |
---|
| 295 | |
---|
| 296 | <tr> |
---|
| 297 | <td> |
---|
| 298 | <pre> |
---|
| 299 | #!/usr/bin/perl |
---|
| 300 | # ------------------------------------------------------------------------------ |
---|
| 301 | # NAME |
---|
| 302 | # Metoffice::Example |
---|
| 303 | # |
---|
| 304 | # DESCRIPTION |
---|
| 305 | # This is a header example, and so on and so forth. |
---|
| 306 | # |
---|
| 307 | # SEE ALSO |
---|
| 308 | # list of relevant commands, modules and documents |
---|
| 309 | # |
---|
| 310 | # COPYRIGHT |
---|
| 311 | # (C) Crown copyright Met Office. All rights reserved. |
---|
| 312 | # For further details please refer to the file COPYRIGHT.txt |
---|
| 313 | # which you should have received as part of this distribution. |
---|
| 314 | # ------------------------------------------------------------------------------ |
---|
| 315 | |
---|
| 316 | package Metoffice::Example; |
---|
| 317 | |
---|
| 318 | # Standard pragmas |
---|
| 319 | use strict; |
---|
| 320 | use warnings; |
---|
| 321 | |
---|
| 322 | # Exports |
---|
| 323 | our (@ISA, @EXPORT, @EXPORT_OK); |
---|
| 324 | require Exporter; |
---|
| 325 | @ISA = qw(Exporter); |
---|
| 326 | @EXPORT = qw( |
---|
| 327 | foo |
---|
| 328 | bar |
---|
| 329 | ); |
---|
| 330 | |
---|
| 331 | # ------------------------------------------------------------------------------ |
---|
| 332 | # ... some more Perl ... |
---|
| 333 | # ------------------------------------------------------------------------------ |
---|
| 334 | |
---|
| 335 | 1; |
---|
| 336 | |
---|
| 337 | __END__ |
---|
| 338 | </pre> |
---|
| 339 | </td> |
---|
| 340 | </tr> |
---|
| 341 | </table> |
---|
| 342 | |
---|
| 343 | <p>The header of a function (or "method" for OO code) should have the |
---|
| 344 | following:</p> |
---|
| 345 | |
---|
| 346 | <ul> |
---|
| 347 | <li>a synopsis of the function's calling interfaces.</li> |
---|
| 348 | |
---|
| 349 | <li>a description of what the function does and what it returns, and if |
---|
| 350 | appropriate, what each argument represents</li> |
---|
| 351 | </ul> |
---|
| 352 | |
---|
| 353 | <table summary="function header block example" border="1" width="100%"> |
---|
| 354 | <tr> |
---|
| 355 | <th>Function header block example</th> |
---|
| 356 | </tr> |
---|
| 357 | |
---|
| 358 | <tr> |
---|
| 359 | <td> |
---|
| 360 | <pre> |
---|
| 361 | # ... Something before the function ... |
---|
| 362 | # |
---|
| 363 | # ------------------------------------------------------------------------------ |
---|
| 364 | # SYNOPSIS |
---|
| 365 | # $result = &print_hello; |
---|
| 366 | # $result = &print_hello ($arg); |
---|
| 367 | # |
---|
| 368 | # DESCRIPTION |
---|
| 369 | # Print the word "hello" to standard output. If no argument is specified, |
---|
| 370 | # the word "world" will be printed after the word "hello". Otherwise, the |
---|
| 371 | # word specified by the argument $arg will follow "hello". The function |
---|
| 372 | # returns true on success. |
---|
| 373 | # |
---|
| 374 | # ARGUMENTS |
---|
| 375 | # $arg - optional, describe $arg if it has not been done in the above section |
---|
| 376 | # ------------------------------------------------------------------------------ |
---|
| 377 | |
---|
| 378 | sub print_hello { |
---|
| 379 | my ($arg) = @_; |
---|
| 380 | $arg = defined ($arg) ? $arg : 'world'; |
---|
| 381 | my $result = print 'hello ', $arg, "\n"; |
---|
| 382 | return $result; |
---|
| 383 | } |
---|
| 384 | |
---|
| 385 | # ------------------------------------------------------------------------------ |
---|
| 386 | # |
---|
| 387 | # ... Something after the function ... |
---|
| 388 | </pre> |
---|
| 389 | </td> |
---|
| 390 | </tr> |
---|
| 391 | </table> |
---|
| 392 | |
---|
| 393 | <script type="text/javascript" src="maintain.js"> |
---|
| 394 | </script> |
---|
| 395 | </body> |
---|
| 396 | </html> |
---|