what's the difference between these two snippets of ruby?

[Mr. Bill]

Sorry I couldn't help the line wrapping. Thanks in advance.

parent_path =
File.expand_path(config['parent'],
File.dirname(yaml_path)).gsub('${project_dir}',
PROJECT_ROOT)


config['parent'].gsub!("${project_dir}", PROJECT_ROOT)
parent_path = File.expand_path(config['parent'],
File.dirname(yaml_path))

 

[Josh Cheek]

[Note: parts of this message were removed to make it a legal post.]

Sorry I couldn't help the line wrapping. Thanks in advance.

parent_path =
File.expand_path(config['parent'],
File.dirname(yaml_path)).gsub('${project_dir}',
PROJECT_ROOT)


config['parent'].gsub!("${project_dir}", PROJECT_ROOT)
parent_path = File.expand_path(config['parent'],
File.dirname(yaml_path))

The second version modifies config['parent']

If you want the second version, which is much easier to understand, imo. But
don't want the side effect of changing the config hash, then use gsub
without the bang, and assign it to a local variable:

parent = config['parent'].gsub("${project_dir}", PROJECT_ROOT)
parent_path = File.expand_path( parent , File.dirname(yaml_path) )

 

[Mr. Bill]

Got it. Thank you much.


Josh Cheek wrote in post #975146:

File.dirname(yaml_path))

The second version modifies config['parent']

If you want the second version, which is much easier to understand, imo.
But
don't want the side effect of changing the config hash, then use gsub
without the bang, and assign it to a local variable:

parent = config['parent'].gsub("${project_dir}", PROJECT_ROOT)
parent_path = File.expand_path( parent , File.dirname(yaml_path) )