Moodleコーディングガイドライン

協力的なプロジェクトを堅牢にするためには、一貫性および安定性が必要です。

このガイドラインは、Moodleコードが目指すゴールを提示するためにあります。 間違いなく、いくつかの古いコードは少数の場所で目標基準に達していませんが、最終的に修正します。 すべての新しいコードは、これらの基準を可能な限り確実に順守してください。

一般ルール

  1. すべてのコードファイルは、.php拡張子を使用してください。
  2. すべてのテンプレートファイルは、.html拡張子を使用してください。
  3. すべてのテキストファイルは、Unixスタイルのテキストフォーマットを使用してください。 (ほとんどのテキストエディタには、このオプションがあります。)
  4. すべてのphpタグは、<?php ?> のように「フルタグ」を使用してください ... <? ?> のように「ショートタグ」ではなく。
  5. すべての著作権情報は保持してください。必要であれば、あなた独自の情報を追加することができます。
  6. 各ファイルには、config.phpをインクルードしてください。
  7. 各ファイルでは、has_capability() または required_capability() を使用してユーザが正しく認証されているか確認してください。
  8. データベースへのアクセスは、可能な場合は常に関数 lib/datalib.php を使用してください - この関数により幅広いデータベースの互換性を実現します。この関数を使用することにより、ほとんどのことが実現できることに気づくでしょう。SQLコードを書く必要がある場合、次のことを確認してください: クロスプラットフォームであること、あなたのコードが特定の関数 (通常、lib.phpファイル) に限定されていること、明確に注釈がなされていること。
  9. $CFG、$SESSION、$THEME および $USER のような標準的なもの以外、グローバル変数を作成または使用しないでください。
  10. すべての変数は使用する前に初期化するか、少なくとも isset() または empty() を使用して存在チェックを行ってください。
  11. すべてのストリングは翻訳できるようにしてください - 「lang/en」内のファイルに新しい簡潔な英語の小文字でテキストを作成して、あなたのコードから get_string() または print_string() を使用して取り出してください。
  12. すべてのヘルプファイルは翻訳できるようにしてください - 「en/help」ディレクトリ内に新しいテキストを作成して、helpbutton() でコールしてください。

    ヘルプファイルを更新したい場合は:

  13. ブラウザから (GET または POST 経由で) 入ってくるデータは、(PHPの設定に関わらず) 安全にデータベースにインサートできるよう、自動的に magic_quotes が適用されます。他のすべての生データ (ファイルまたはデータベース) は、データベースにインサートする前に addslashes() でエスケープしてください。
  14. 重要: Moodle内のすべてのテキスト、特にユーザによって入力されるものは format_text() 関数を使用してプリントしてください。これによりテキストは、正確にフィルタおよびクリーニングされます。

 

コーディングスタイル

他のスタイルを使用してきた場合、あなたのスタイルを変更することは少々面倒なことであると私は理解しています。しかし、後で人々が雑多なスタイルが使用されたMoodleコードの意味を理解する面倒さと、この面倒さのバランスを取る必要があります。人々が使用するスタイルには、明らかに良い部分もありますが、現在のスタイルはこのスタイルですので、どうか従ってください。

  1. インデント は連続した4つのスペースにしてください。タブは使用しないでください。
  2. 変数名 は常に読みやすく、意味のある英語小文字にしてください。本当に1語以上の単語が必要な場合、単語を連結して可能な限り短くしてください。配列オブジェクト名には、複数形を使用してください。

    良い: $quiz
    良い: $errorstring
    良い: $assignments (for an array of objects)
    良い: $i (but only in little loops)

    悪い: $Quiz
    悪い: $aReallyLongVariableNameWithoutAGoodReason
    悪い: $error_string

  3. 定数 は常に大文字を使用して、常にモジュール名で始めてください。単語はアンダースコア (_) で別けてください。

    define("FORUM_MODE_FLATOLDEST", 1);

  4. 関数名 は単純な英語の小文字の単語を使用して、モジュール間のコンフリクトを避けるため、モジュール名で始めてください。単語はアンダースコア (_) で別けてください。可能でしたら、パラメータは常に適切なデフォルト値を持つようにしてください。関数名および続く (大括弧) の間にはスペースを入れないことに注意してください。

    function forum_set_display_mode($mode=0) {
        global
    $USER, $CFG;

        if (
    $mode) {
            
    $USER->mode = $mode;
        } else if (empty(
    $USER->mode)) {
            
    $USER->mode = $CFG->forum_displaymode;
        }
    }

  5. ブロック は (1行のみでも) 常に丸括弧で囲んでください。Moodleは、このようなスタイルを使用します。

    if ($quiz->attempts) {
        if (
    $numattempts > $quiz->attempts) {
            
    error($strtoomanyattempts, "view.php?id=$cm->id");
        }
    }

  6. 文字列 は処理速度を増すため、可能であればシングル・クオテーションで定義してください。

    $var = 'some text without any variables';
    $var = "with special characters like a new line \n";
    $var = 'a very, very long string with a '.$single.' variable in it';
    $var = "some $text with $many variables $within it";

  7. コメント は可能な限り実用的に追加し、コードの流れ、関数および変数の目的を説明してください。

    /**
    * The description should be first, with asterisks laid out exactly
    * like this example. If you want to refer to a another function,
    * do it like this: {@link clean_param()}. Then, add descriptions
    * for each parameter as follows.
    *
    * @param int $postid The PHP type is followed by the variable name
    * @param array $scale The PHP type is followed by the variable name
    * @param array $ratings The PHP type is followed by the variable name
    * @return mixed
    */

    function forum_get_ratings_mean($postid, $scale, $ratings=NULL) {
        if (!$ratings) {
            
    $ratings = array();     // Initialize the empty array
            if (
    $rates = get_records("forum_ratings", "post", $postid)) {
                
    // Process each rating in turn
                foreach (
    $rates as $rate) {
    ....etc

  8. スペースは自由に使用してください - 明快さを得るために、物事を少し広げることを恐れないでください。一般的に、大括弧と標準的な命令文の間には1つのスペースを入れますが、大括弧と変数および関数の間にはスペースを入れません。

    foreach ($objects as $key => $thing) {
        process($thing);
    }

    if ($x == $y) {
        $a = $b;
    } else if (
    $x == $z) {
        $a = $c;
    } else {
        $a = $d;
    }

  9. オブジェクトのコピーを作成する場合、常にphp5 clone() 関数 (そうでなければ最初のオブジェクトの参照で終わります) を使用してください。Moodleは、これがphp4でも間違いなく動作するようにします。

    良い: $quiz
    悪い: $b = $a;
    良い: $b = clone($a);


    あなたがコピーしたいものがオブジェクトではない (例 オブジェクトの配列) 場合、代わりにfullclone() 関数を使用してください。

 

データベース構造

  1. すべてのテーブルは、プライマリインデックスとして auto-incrementing id フィールド (INT10) を持つ必要があります。
  2. 各モジュールのインスタンスを含むメインテーブルは、モジュールと同じ名称を持ち (例 widget)、次の最小限のフィールドを含む必要があります:
  3. モジュールと関連する他のテーブルで「things」に関する情報を含むものは widget_things という名称を付けてください (複数形に注意してください)。
  4. カラムの名称は、変数名と同じルールに従って簡単かつ短くつけてください。
  5. 可能であれば、他のテーブルのidフィールドへの参照を含むカラム (例 widget) は、widgetidと呼ばれるべきです。(この仕様は、やや新しいもので、他のいくつかの古いテーブルには適用されていないことに注意してください。)
  6. 論理値 (Boolean) フィールドは、必要に応じて後で値を拡張できるよう、0または1を含む整数フィールド (例 INT4) を使用してください。
  7. ほとんどのフィールドは、PHP time() 関数から現在のタイムスタンプで更新される timemodified フィールド (INT10) を持つ必要があります。

 

セキュリティ関連 (およびフォームおよびURLデータの取り扱い)

  1. 「register_globals」を信頼しないでください。すべての変数は、すべてのコードファイルで適切に初期化してください。変数がどこから来たか明らかでなければなりません。
  2. 空でもすべての配列およびオブジェクトを初期化してください。$a = array() または $obj = new stdClass();
  3. optional_variable() 関数は使用しないでください。代わりに optional_param() 関数を使用してください。あなたが必要なデータタイプのために正しい PARAM_XXXX 値を選択してください。変数の任意の値をチェックおよびセットする場合、set_default() 関数を使用してください。
  4. require_variable() 関数を使用しないでください。代わりに required_param() 関数を使用してください。あなたが必要なデータタイプのために正しい PARAM_XXXX 値を選択してください。
  5. 気をつけて data_submitted() を使用してください。データは使用される前に、さらにきれいにする必要があります。
  6. $_GET$_POST または $_REQUEST を使用しないでください。必要に応じて、適切な required_param() または optional_param() を使用してください。
  7. if (isset($_GET['something'])) のようなものを使用して動作をチェックしないでください。例えば $something = optional_param('something',-1,PARAM_INT) を使用して、適切な値の幅に入っているかテストしてください。例 if ($something>=0) {...
  8. 簡単に探すことができるよう、可能な限りすべての required_param()optional_param() および他の変数を各ファイルの最初で初期化してください。
  9. フォーム処理ルーチンをアタックから保護するために「sesskey」メカニズムを使用してください。基本的な使用例: フォームが生成されるときに、include <input type="hidden" name="sesskey" value="<?php echo sesskey(); ?>" /> と使用します。フォームの処理を行うときは、 if (!confirm_sesskey()) {error('Bad Session Key');} でチェックしてください。
  10. required_param() または optional_param() の適切な使用で「きれいにされていない」場合、すべてのファイルは clean_filename() 関数を使用して「きれいにする」必要があります。
  11. データベースから読み込んだデータは、更新する前に addslashes() を適用してください。データオブジェクト全体は、addslashes_object() で一度に処理することができます。
  12. 可能な限り、データベースに保存するデータは、GETデータ (例 URLラインからのデータ) とは対照的な POSTデータ (method="POST" によるフォームデータ) を使用してください。
  13. 避けることができるなら $_SERVER からのデータを使用しないでください。これは移植性に問題があります。
  14. 他で行っていない場合、データベースに書き込むすべてのデータを適切なデータタイプのPARAM_XXXXを使用して clean_param() 関数に通してください。
  15. あなたがカスタムSQLコードを記述する場合、間違いなく正しいことを確認してください。特に値の周囲のクォート漏れに気をつけてください。SQLインジェクションの脆弱性 (exploit) となる可能性があります。
  16. すべてのファイルで使用されるすべてのデータ (特にデータベースに書き込む部分) をチェックしてください。他でチェックされることを期待したり、信頼しないでください。
  17. インクルードされるコードのブロックは、明確なPHP構造を含むべきです (例 クラス宣言、関数定義等) - 連続したコードのブロックは、初期化なしの変数使用を奨励してしまいます。

Moodleドキュメンテーション

Version: $Id$