Ruby: ¿cómo puedo copiar esta matriz?

Question

(Seguimiento de mi pregunta anterior, Ruby: how can I copy a variable without pointing to the same object?)

Estoy escribiendo un programa simple de Ruby para hacer algunas sustituciones en un archivo .svg. El primer paso es extraer información del archivo y colocarlo en una matriz. Para no leer el archivo del disco cada vez que se llama a esta función, intento utilizar el patrón de diseño de memoria: use un resultado en caché en cada llamada después de la primera.

Para hacer esto, estoy usando una variable global, definida justo antes de la función. Pero a pesar de que I .dup es variable a una local antes de devolver la variable local, la función que llama a esta aún está modificando la variable global.

Aquí está mi código real:

#memoize to keep from having to read original file on each pass 
$svg_filedata_cache = [] #the global variable 
def svg_filedata(filename) 
    if $svg_filedata_cache.empty? 
     File.open(filename, "r"){|f| $svg_filedata_cache = f.readlines} 
    end 
    svg_filedata_cache = $svg_filedata_cache.dup #try to copy it 
    return svg_filedata_cache #SHOULD point to a different object (but doesn't) 
end 

Dos preguntas (respuesta o ambas):

1. ¿Por qué otras funciones, que tienen la sesión y modificar el valor devuelto aquí, también afectan a la variable global, aunque utilicé .dup para copiarlo?
2. Soy nuevo en Ruby y estoy seguro de que esta no es la forma más Rubyesque de hacer esto (y de todos modos no me gustan las variables globales). ¿Puedes sugerir una mejor estrategia?

Answer 1

La modificación de la matriz duplicada no afectará al original. Sin embargo, las modificaciones en las cadenas dentro de la matriz serán visibles globalmente porque la matriz global y la matriz duplicada aún contienen referencias a las mismas cadenas (dup no realiza una copia profunda).

O bien realice una copia profunda (svg_filedata_cache = $svg_filedata_cache.map {|line| line.dup}) o simplemente evite las operaciones de mutación en las cadenas.

Answer 2

Mejora el código un poco:

$svg_filedata_cache = [] #the global variable 
def svg_filedata(filename) 
    # Use ||= for memoiziation 
    $svg_filedata_cache ||= File.open(filename, "r"){|f| $svg_filedata_cache = f.readlines} 
    $svg_filedata_cache.dup #shallow copying 
end 

Actualización: un truco simple de hacer una copia completa en general es:

def deep_copy(obj) 
    Marshal.load(Marshal.dump(obj)) 
end 

Answer 3

lo global es probable que no se modificó, pero los elementos que esto y tu referencia .dup están cambiando. Para hacerlo más rubí canónico, deshacerse de lo global, usar una clase y leer el archivo en la función initialize. (El constructor). Haga que la matriz sea una variable de instancia con @v.