[[TranslatedPages(revision=38)]]
= Richtlijnen voor ontwikkelen =
[[PageOutline(2-2,Inhoud,inline)]]

== Hoe uw code er uit zou moeten zien ==

 * Zorg ervoor dat de code compatibel is met Java 11
 * **Documenteer** uw code met behulp van commentaar op de regel en javadoc. Vele mensen zullen u daarvoor danken :)
 * Probeer publieke velden te vermijden
 * JOSM heeft heel veel hulpmethoden in de klassen `Utils`, `GuiUtils`, `Geometry` .... Gebruik ze als u ze nodig hebt.
 * Controleer parameters. U kunt `Objects.requireNonNull` gebruiken.
 * Schrijf niet voor de uitvoering - schrijf voor leesbaarheid. Gebruik `Stream`s, `Function`s en andere objecten van Java 8+ als dat de code beter leesbaar maakt.

=== Threading / Vergrendeling ===

 * JOSM gebruikt verschillende mechanismen om te vergrendelen, afhankelijk van het object.
 * De gegevenssets worden beschermd door een RW-vergrendeling. Sommige methoden vergrendelen niet automatisch om redenen van uitvoering. Zorg er voor de vergrendelingen aan te roepen die voor uw wijzigingen nodig zijn.
 * Componenten van de GUI zouden alleen moeten worden aangepast in de EDT-thread
 * Gebruik bij voorkeur `SwingUtils.invokeLater` indien u iets moet uitvoeren in de UI-thread
 * Veel listeners worden al uitgevoerd in de EDT-thread (laagwijzigingen) of hebben een centrale beheerder die u toestaat listeners te registreren die worden uitgevoerd in de EDT (wijzigingen in gegevensset, wijzigingen in selectie).

== Hoe uw opmaak eruit zou moeten zien ==

 * zorg ervoor dat er geen witruimte achter staat
 * gebruik niet meerdere lege regels achter elkaar
 * JOSM gebruikt 4 tekens voor inspringen en geen tabstops (Als u Notepad++ gebruikt kunt u de standaard inspringing wijzigen in de "Bewerken" -> "Witruimte-bewerkingen" -> selecteer "Tabs omzetten in spaties" (dit is permanent) of met de knop "Instellingen insprong wijzigen" in de werkbalk door het vinkje te verwijderen bij "Tabs gebruiken" (dit is een tijdelijke instelling).)
 * voeg gekrulde haken toe voor elke `if`, tenzij die wordt gevolgd door een `return` (of misschien een `throw`)
 * U zou **checkstyle** moeten gebruiken vóór patch/commit: `ant checkstyle` en controleer `checkstyle-josm.xml`; indien u vindt dat het uitvoeren van checkstyle te lang duurt voor alle bestanden, voer het dan alleen uit op de gewijzigde bestanden:
	
{{{
  #!bash
    # 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
}}}

== Hoe uw javadoc er uit zou moeten zien ==
 * De [https://www.oracle.com/technetwork/java/javase/documentation/index-137868.html#styleguide Oracle Javadoc style guide] wordt gebruikt als gids voor de basis
 * `@since` wordt gebruikt voor publieke klassen en methoden (zichtbaar voor de ontwikkelaars van invoegtoepassingen) met de revisie van JOSM waarin het element werd geïntroduceerd. Voorbeeld: `@since 5408`
   * `@since` wordt bijgewerkt / toegevoegd als een handtekening van een publieke methode wijzigt of indien een klasse een andere naam krijgt
   * `@since` kan worden weggelaten voor publieke methoden en velden die tegelijkertijd met de klasse worden geïntroduceerd, vooropgesteld dat zij niet zijn gewijzigd en de klasse juist is gedocumenteerd.
   * Er kunnen meerdere tags `@since` zijn, bijv. voor het toevoegen van interfaces aan een klasse. De reden voor deze tags zou moeten worden toegevoegd. Voorbeeld: `@since 12345 @FunctionalIterface werd toegevoegd`
   * Indien u een patch indient en u weet de revisie niet, voeg dan toch `@since xxx` toe. Het kan dan worden vervangen bij het samenvoegen van uw patch.
 * `@throws` heeft de voorkeur boven `@exception`
* controleer uw wijzigingen vóór patch/commit door het genereren van javadoc: `ant javadoc`, blader door de uitvoerberichten; indien u vindt dat het uitvoeren van javadoc te lang duurt voor alle bestanden, voer het dan alleen uit op de gewijzigde bestanden:

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

=== Eclipse configureren ===
[[Image(wiki:DevelopersGuide/StyleGuide:styleguide_compiler_16.png,700px)]]

[[Image(wiki:DevelopersGuide/StyleGuide:ss1.png,700px)]]

[[Image(wiki:DevelopersGuide/StyleGuide:ss2.png,700px)]]

[[Image(wiki:DevelopersGuide/StyleGuide:ss3.png,700px)]]


== Internationalisatie ==

 * zorg er voor dat `tr(...)` gebruikt wordt voor alle gelokaliseerde tekenreeksen
   {{{
   #!java
   import import static org.openstreetmap.josm.tools.I18n.tr;

   // gebruik tr(...) voor berichten van uitzonderingen
   //
   throw new Exception(tr("foutenbericht altijd in tr()"));

   // gebruik tr(...) voor labels, titel, teksten van helptips en soortgelijke 
   //
   new JLabel(tr("Label altijd in tr()"));

   // etc.
   }}}

 * assembleer gelokaliseerde berichten NOOIT met `+`. Gebruik in plaats
   daarvan tijdelijke plaatsaanduidingen voor opmaak.

   '''NIET'''
   `new JLabel(tr("Mijn label " + labelId));`

   '''MAAR'''
   `new JLabel(tr("Mijn label {0}",labelId));`

   Enige uitzondering: `+` kan worden gebruikt om lange regels van niet-variabele teksten af te breken.
   De plaatsvervangers zijn verplicht in eenvoudige vertalingen.

 * Bij het gebruiken van een apostrof in de tekenreeks voor de bron, moet die zijn geëscapet door een andere apostrof (Zoals de backslash in C):[[BR]]
   {{{#!java
   new JButton(tr("Don''t press me more than {0} times!", n))
   }}}

 * Een context voor een vertaling kan direct worden ingesteld met `trc(...)`. Aanvullende hints voor vertalers worden gegeven in opmerkingen voor Java bij de functie:
   {{{#!java
   /* I18n: huisnummer, straat als parameter; plaats nummer eerst voor zichtbaarheid */
   msg = tr("House number {0} at {1}", s, t);
   }}}

 * Gebruik `trn(...)` om vertalers de taalspecifieke meervoudsvorm te laten kiezen:
   {{{#!java
   msg = trn("Object deleted", "Objects deleted", del.size();
   
   // of met plaatsvervangers:
   //
   new JButton(trn(/* I18n: keer nodig, een naam als parameter */
                       "Press {1} {0} times!", n, n, someName))

   // De Engelse enkelvoud tekenreeks voor de bron moet worden opgegeven voor identificatie
   // zelfs als dat logisch gezien ongeldig is en niet zal voorkomen. Voor consistentie
   // zou de plaatsvervanger voor het nummer erin moeten zijn ingesteld.
   //
   msg = trn("Combine {0} way", "Combine {0} ways", n, n);
   }}}
   In segmenten voor meervoud is een plaatsvervanger niet verplicht voor vertalers.

----
Terug naar [wikitr:/Help/DevelopersGuide Developers Guide]