refactor(map_loader): split to member functions

[takayuki5168]

Signed-off-by: Takayuki Murooka takayuki5168@gmail.com

Description

small refactoring for #1942

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

• There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

[codecov]

Codecov Report

Base: 10.44% // Head: 10.44% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (40fdea4) compared to base (93cc9bd).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1941      +/-   ##
==========================================
- Coverage   10.44%   10.44%   -0.01%     
==========================================
  Files        1233     1233              
  Lines       88034    88058      +24     
  Branches    20501    20501              
==========================================
  Hits         9197     9197              
- Misses      69328    69352      +24     
  Partials     9509     9509              

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <0.00%> (?)
total	10.43% <0.00%> (ø)		Carriedforward from 93cc9bd

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files	Coverage Δ
...c/lanelet2_map_loader/lanelet2_map_loader_node.cpp	0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[takayuki5168]

@mitsudome-r @TakaHoribe @kenji-miyake Could you review?

The reason to split the implementation to some member functions is as follows. I will create a script which uses map_loader as a library and calls these member functions.
#1942

[kenji-miyake]

For future reviews, it's better to split these kinds of changes (removing this, use const, etc.) into a separate commit.

[takayuki5168]

@kenji-miyake Thank you always for your quick review and also further review.