[rac] Format an address for display.

third_party/libaddressinput/chromium/cpp/src/address_data.cc

 // Copyright (C) 2013 Google Inc.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 // http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
 #include <libaddressinput/address_data.h>
 
 #include <libaddressinput/address_field.h>
 
 #include <cassert>
+#include <cstddef>
 #include <string>
+#include <vector>
+
+#include "region_data_constants.h"
+#include "rule.h"
 
 namespace i18n {
 namespace addressinput {
 
+void AddressData::BuildDisplayLines(std::vector<std::string>* lines) const {
+  assert(lines != NULL);
+  lines->clear();
+
+  Rule rule;
+  rule.CopyFrom(Rule::GetDefault());
+  rule.ParseSerializedRule(RegionDataConstants::GetRegionData(country_code));
+
+  for (size_t i = 0; i < rule.GetFormat().size(); ++i) {
+    std::string line;
+    for (size_t j = 0; j < rule.GetFormat()[i].size(); ++j) {
+      const FormatElement& element = rule.GetFormat()[i][j];

[Evan Stade, 2014/01/14 18:05:22]

rule.GetFormat()[i] is used enough to get a local variable imo (like |element|
here)

[please use gerrit instead, 2014/01/14 23:21:12]

Done.

+      switch (element.type) {
+        case FormatElement::LITERAL:
+          line += element.literal;
+          break;
+
+        case FormatElement::FIELD:
+          if (element.field == STREET_ADDRESS) {
+            // A street address is always surrounded by either newlines or
+            // spaces.
+            if (rule.GetFormat()[i].size() == 1) {
+              lines->insert(lines->end(),
+                            address_lines.begin(),
+                            address_lines.end());
+            } else {
+              for (size_t k = 0; k < address_lines.size(); ++k) {

[Evan Stade, 2014/01/14 18:05:22]

I don't understand why street address line 2 might ever be on the same line as
street address line 1

[please use gerrit instead, 2014/01/14 23:21:12]

Take the https://i18napis.appspot.com/ssl-address/data/CI format for example:

"%N%n%O%n%X %A %C %X"

This parses into:

{
    {RECIPIENT},
    {ORGANIZATION},
    {SORTING-CODE, " ", STREET-ADDRESS, " ", LOCALITY, " ", SORTING-CODE}
}

[Evan Stade, 2014/01/14 23:45:36]

I think that would imply there's only one street address line, so you don't need
to loop here. I do think you can combine the size() == 1 case and this loop.

[please use gerrit instead, 2014/01/15 00:05:47]

Combined the cases and simplified the code. If Chrome provides only one street
address line for Ivory Coast, everything will look good.

+                line += address_lines[k];
+                if (k < address_lines.size() - 1) {
+                  line += " ";
+                }
+              }
+            }
+          } else {
+            const std::string& field = GetField(element.field);
+            if (!field.empty()) {
+              line += field;
+            }
+          }
+          break;
+
+        default:
+          assert(false);
+          break;
+      }
+    }
+
+    if (!line.empty()) {
+      lines->push_back(line);
+    }
+  }
+}
+
 const std::string& AddressData::GetField(AddressField field) const {

[Evan Stade, 2014/01/14 18:05:22]

nit: should be called GetFieldValue

[please use gerrit instead, 2014/01/14 23:21:12]

Done.

   switch (field) {
     case COUNTRY:
       return country_code;
     case ADMIN_AREA:
       return administrative_area;
     case LOCALITY:
       return locality;
     case DEPENDENT_LOCALITY:
       return dependent_locality;
     case SORTING_CODE:
       return sorting_code;
     case POSTAL_CODE:
       return postal_code;
     case ORGANIZATION:
       return organization;
     case RECIPIENT:
       return recipient;
     default:
       assert(false);
       return recipient;
   }
 }
 
 }  // namespace addressinput
 }  // namespace i18n