Normalizing package names according to
[mdsal.git] / binding2 / mdsal-binding2-spec / src / site / asciidoc / binding-2.adoc
index 1e344251cf2dc47ad264d01635e980058d341a49..797750c68cb0ffb438183c592cb3badb36179acb 100644 (file)
@@ -20,6 +20,122 @@ Transfer Object::
 Builder::
   builder
 
+=== YANG Identifiers Mapping
+
+Every non-Java char in identifier is converted to Java char by its unicode name http://docs.oracle.com/javase/specs/jls/se8/html/jls-3.html#jls-3.8
+JAVA SE SPECIFICATIONS - Identifiers. This mapping solves various issues from Binding Specification v1, which led to compilation issues.
+
+There are special types of mapping non-java chars to original identifiers according to specific Java type:
+
+* class, enum, interface
+
+** without special separator
+the first character of identifier, any other first character of identifier part mapped by
+** non-Java char name from unicode and char in identifier behind non-java char name are converting to upper case
+
+ examples:
+ example* - ExampleAsterisk
+ example*example - ExampleAserisksExample
+ \example - ReverseSolidusExample
+ 1example - DigitOneExample
+ example1 - Example1
+ int - IntReservedKeyword
+ con - ConReservedKeyword
+
+* enum value, constant
+** used underscore as special separator
+** converted identifier to upper case
+
+ examples:
+ example* - EXAMPLE_ASTERISK
+ example*example - EXAMPLE_ASTERISK_EXAMPLE
+ \example - REVERSE_SOLIDUS_EXAMPLE
+ 1example - DIGIT_ONE_EXAMPLE
+ example1 - EXAMPLE1
+ int - INT_RESERVED_KEYWORD
+ con - CON_RESERVED_KEYWORD
+
+* method, variable
+** without special separator
+** the first character of identifier is converting to lower case
+** any other first character of identifier part mapped by non-Java char name from unicode and char in identifier behind non-java char name are converting to upper case
+
+ examples:
+ example* - exampleAsterisk
+ example*example - exampleAserisksExample
+ \example - reverseSolidusExample
+ 1example - digitOneExample
+ example1 - example1
+ int - intReservedKeyword
+ con - conReservedKeyword
+
+* package - full package name - https://docs.oracle.com/javase/tutorial/java/package/namingpkgs.html
+** parts of package name are separated by dots
+** parts of package name are converting to lower case
+** if parts of package name are reserved Java or Windows keywords, such as 'int' the suggested convention is to add an underscore to keyword
+** dash is parsed as underscore according to https://docs.oracle.com/javase/tutorial/java/package/namingpkgs.html
+
+ examples:
+ org.example* - org.exampleasterisk
+ org.example*example - org.exampleasteriskexample
+ org.example - org.reversesolidusexample
+ org.1example - org.digitoneexample
+ org.example1 - org.example1
+ org.int - org.int_
+ org.con - org.con_
+ org.foo-cont - org.foo_cont
+
+==== Special case - '-' in identifiers
+There is special case in CLASS, INTERFACE, ENUM, ENUM VALUE, CONSTANT, METHOD and VARIABLE if
+identifier contains single dash - then the converter ignores the single dash in the way of the
+non-java chars. In other way, if dash is the first or the last char in the identifier or there is
+more dashes in a row in the identifier, then these dashes are converted as non-java chars.
+
+Example:
+
+* class, enum, interface
+
+ foo-cont - FooCont
+ foo--cont - FooHyphenMinusHyphenMinusCont
+ -foo - HyphenMinusFoo
+ foo- - FooHyphenMinus
+
+* enum value, constant
+
+ foo-cont - FOO_CONT
+ foo--cont - FOO_HYPHEN_MINUS_HYPHEN_MINUS_CONT
+ -foo - HYPHEN_MINUS_FOO
+ foo- - FOO_HYPHEN_MINUS
+
+* method, variable
+
+ foo-cont - fooCont
+ foo--cont - fooHyphenMinusHyphenMinusCont
+ -foo - hyphenMinusFoo
+ foo- - fooHyphenMinus
+
+==== Special case - same class (or enum or interface) names with different camel cases
+Next special case talks about normalizing class name which already exists in package - but with
+different camel cases (foo, Foo, fOo, ...). To every next classes with same names will by added
+their actual rank (serial number), except the first one. This working for CLASS, ENUM and
+INTEFACE java identifiers. If there exist the same ENUM VALUES in ENUM (with different camel
+cases), then it's parsed with same logic like CLASSES, ENUMS and INTERFACES but according to list
+of pairs of their ENUM parent. Example:
+
+* class, enum, interface
+
+ package name org.example, class (or interface or enum) Foo - normalized to Foo
+ package name org.example, class (or interface or enum) fOo - normalized to Foo1
+
+* enum value
+
+ type enumeration {
+     enum foo;
+     enum Foo;
+ }
+ YANG enum values will be mapped to 'FOO' and 'FOO_1' Java enum values.
+
+
 === Binding Specification v2 Concepts
 
 <<Instantiable>>::
@@ -30,7 +146,7 @@ Builder::
   is not used directly, but rather via <<TreeChildNode>>. See <<instantiated-data-tree-rules>>
   for more information.
 <<TreeRoot>>::
-  Represents virtual root of instantiated data tree node.
+  Represents virtual root of instantiated data tree.
 <<TreeChildNode>>::
   Represents node, which is part of instantiated data tree and is not root of
   data tree.
@@ -102,8 +218,7 @@ YANG namespace::
 Binding namespace::
   Naming space introduced by Binding Specification for additional properties
   and functionality of Binding Specification. This namespaces needs to be separate
-  from YANG namespaces in order to not have naming conflict with YANG-derived
-  names.
+  from YANG namespaces in order to not have naming conflict with YANG-derived.
 
 
 Binding Specification v2 uses following namespaces:
@@ -351,15 +466,22 @@ container example-outter {
 [source,yang]
 ----
 list foo {
-    key identifier;
+    key identifier key fookey;
     leaf identifier {
         type union {
             type string;
         }
     }
+
+    leaf key {
+        type string;
+    }
+
+    leaf fookey {
+        type string;
+    }
 }
 ----
-
 [uml, file="list-Keyed.png"]
 --
 set namespaceSeparator none
@@ -370,10 +492,18 @@ interface data.Foo {
 interface key.foo.FooIdentifier {
 }
 
+interface key.foo.FooKey {
+}
+
+interface key.foo.FooFooKey {
+}
+
 interface type.foo.identifier.IdentifierUnion {
 }
 
 data.Foo o- key.foo.FooIdentifier
+data.Foo o- key.foo.FooKey
+data.Foo o- key.foo.FooFooKey
 key.foo.FooIdentifier o- type.foo.identifier.IdentifierUnion
 --
 
@@ -423,7 +553,7 @@ data.top.base.Foo -u-|> Case
 data.top.base.Bar -u-|> Case
 
 data.top.base.Foo -u-|> data.top.Base
-data.top.base.Bar -u-|> data.top.base.Foo
+data.top.base.Bar -u-|> data.top.Base
 
 data.Top o- data.top.Base
 data.top.base.Foo o- data.top.base.foo.Foo
@@ -511,13 +641,13 @@ FIXME: Here should be Augmentable & Instantied & ChildDataNode
 === `grouping` Statement
 
 * `grouping` statement is represented by `interface`
-** class name is generated according to <<class-naming>> with suffix `Grouping`
+** interface name is generated according to <<class-naming>> with suffix `Grouping`
 * Representations of `grouping` statements are generated into <<grouping-namespace>>
-* data schema nodes under grouping are represented by `interface` and are generated
-  into <<grouping-namespace>>
+* schema nodes under grouping are represented by `interface` and are generated
+  into <<grouping-namespace>> + name of grouping
 ** getters (accessors) from parent nodes are generated according to <<accessor-rules>>
 ** class name is generated according to <<class-naming>> with suffix `Data`
-** data schema nodes does not follow <<instantiated-data-tree-rules>>, these interfaces
+** schema nodes does not follow <<instantiated-data-tree-rules>>, these interfaces
    are used only in instantiated data tree.
 
 .Simple Grouping
@@ -550,7 +680,16 @@ grp.SimpleGrouping o- grp.simple.FooData
 
 Representations of data node substatements are generated according to rules
 described in <<data-node-rules>> with following changes:
-
+////
+MS: proposed interface names:
+case - <NodeName>Case
+choice - <<NodeName>Choice
+container, list - <NodeName>
+////
+////
+MC: I would keep Data suffix, but idea about distinguishing cases and choices
+is to think about
+////
 * Interface names for `case`, `choice`, `container` and `list`, is suffixed by
   `Data` suffix, in order to not conflict with same named groupings inside same
   package
@@ -604,7 +743,6 @@ container top {
   uses simple;
 }
 ----
-
 [uml, file="grouping2.png"]
 --
 set namespaceSeparator none
@@ -637,9 +775,7 @@ are based on <<instantiated-data-tree-rules>>
 .YANG Snippet
 [source, yang]
 ----
-set namespaceSeparator none
 grouping with-inner {
-
   grouping inner {
     container cont;
   }
@@ -807,9 +943,8 @@ augment statement is defined
   augmentation is generated, child data nodes are generated by rules for
   <<instantiated-data-node-rules>>.
   See <<augment-other-module>> for details & examples.
-
-`augment` statement targets only instantiated data nodes, so child data node
-representation are always ge
+`augment` statement targets only instantiated data nodes, so child data nodes
+representation is always generated.
 
 [[augment-same-module]]
 ==== Augmentation target in same module
@@ -1071,6 +1206,6 @@ grouping nodes {
 }
 
 ----
-grouping key.grp.nodes.node.nodeidentifier
+grouping key.grp.nodes.node.<nodeidentifier>
 
-instantiated key.data.nodes.node.nodeidentifier
\ No newline at end of file
+instantiated key.data.nodes.node.<nodeidentifier>