Translation incomplete

開発ガイドライン

コードの外観

• コードがJava 8と互換性があることを確認してください
• インラインコメントとjavadocを使用してコードをドキュメント化します。多くの人があなたに感謝します :)
• パブリックフィールドを避けるようにしてください
• JOSMは Utils、 GuiUtils、 Geometry ...クラスに多くのヘルパーメソッドを持っています。必要に応じて使用してください
• パラメータを確認してください。Objects.requireNonNullを使うことができます。
• パフォーマンスのために記述しないでください - 読みやすいように記述してください。Stream、 Functionや他のJava 8の機能を使うと、コードを読みやすくします。

Threading / Locking

• JOSMは、オブジェクトに応じてさまざまなロック機構を使用します。
• データセットはRWロックによって保護されています。パフォーマンス上の理由から自動的にロックされないメソッドもあります。変更に必要なロックを取得してください。
• GUIコンポーネントはEDTスレッドでのみ変更する必要があります
• UIスレッドで何かを実行する必要がある場合は SwingUtils.invokeLaterを使用することをお勧めします
• Many listeners already run in the EDT thread (layer changes) or have a central manager that allows you to register listeners that run in EDT (dataset changes, selection changes).

How your formatting should look like

• make sure there is no trailing white space
• don't use multiple consecutive empty lines
• JOSM uses 4 characters indentation and no tab stops. If you use Notepad++ you can change the default indentation in the "Preferences" -> "Language" -> "Tab Settings" -> check "Replace by spaces" (this is permanent) or with the "change indentation settings" button in the toolbar (this is a temopary setting and requires the plugin "Customize Toolbar").
• add curly brackets for each if, unless it is followed by return (or maybe throw)
• You should use checkstyle before patch/commit: ant checkstyle and check checkstyle-josm.xml; if you find running checkstyle slow for all files, run for changed files only:

    # make sure build2 dir exists, if it does not run 'ant checkstyle-compile' before
  
    svn diff --summarize | awk '{ print $2 }' | grep "^[a-z]" | xargs \
      java -classpath "build2:tools/checkstyle/checkstyle-all.jar" \
        com.puppycrawl.tools.checkstyle.Main -c tools/checkstyle/josm_checks.xml
    # or
    git diff --name-only | xargs \
      java -classpath "build2:tools/checkstyle/checkstyle-all.jar" \
        com.puppycrawl.tools.checkstyle.Main -c tools/checkstyle/josm_checks.xml
  

How your javadoc should look like

• The ​Oracle Javadoc style guide is used as the base guide
• @since is used for public classes and methods (visible to plugin developers) with the JOSM revision that introduced the element. Example: @since 5408
  • @since is updated / added when a public method signature changes or if a class is renamed.
  • @since can be omitted for public methods and fields introduced at the same time as the class, provided they do not have changed and the class is correctly documented.
  • There can be multiple @since tags, e.g. for adding interfaces to a class. The reason for those tags should be added Example: @since 12345 @FunctionalIterface was added
  • If you submit a patch and don't know the revision, add @since xxx any way. It can then be replaced when merging your patch.
• @throws is preferred to @exception
• check your changes before patch/commit by generating javadoc: ant javadoc, browse output messages; if you find running javadoc slow for all files, run for changed files only:

    svn diff --summarize | awk '{ print $2 }' | xargs javadoc -d javadoc
    # or
    git diff --name-only | xargs javadoc -d javadoc

Eclipseの設定

国際化

• ローカライズされたすべての文字列にtr(...)を使用するようにしてください

  import import static org.openstreetmap.josm.tools.I18n.tr;
  
  // use tr(...) for exception messages
  //
  throw new Exception(tr("error message always in tr()"));
  
  // use tr(...) for labels, title, tooltip texts and the like
  //
  new JLabel(tr("Label always in tr()"));
  
  // etc.
  

• +でローカライズされたメッセージを決して組み立てることはできません。代わりに、フォーマットプレースホルダを使用してください。

DONT new JLabel(tr("My Label " + labelId));

DO new JLabel(tr("My Label {0}",labelId));

Only exception: + can be used to break long lines of non-variable texts.

• When using apostrophe, the following rules apply:

For all tr the apostrophe is special. (Like backslash in C)
It needs to be escaped by another apostrophe:

new JButton(tr("Don''t press me more than {0} times!", n))

---

Developers Guide (en)へ戻る