Question

The purpose of the function below is to return a string having inserted the argument value between two stars.

star-name: func [name /local stars] [
    stars: "**"
    insert next stars name
    stars
]
print star-name "test" ;*test*
print star-name "this" ;*thistest*, but what I really want is *this*  

The second time I call the function, the argument of the first call still remains inserted. I know the answer is to use copy "**". My question is, doesn't it reassign the stars variable to "**" every time the function is called?

Was it helpful?

Solution

In the case of the function there is just one "**" string definition. That definition is used by Rebol load function just once, since load is run just once to translate the code to Rebol internal form - a block. It is true that the assignment occurs twice if you call the function twice, but the assignment does not create anything, it just makes the variable refer to the same string again.

In your comment you should notice that you actually have two "**" string definitions leading to two strings being created by load. If you use

code: [stars: "**" insert next stars something]
something: "this"
do code
something: "that"
do code

you will notice that there is just one string definition and while you do not have any function the behaviour is the same as it was when a function was used.

OTHER TIPS

If you use a set-word on a series, then the default behavior is to allocate the memory for that series just the once. This allows you to use that as a static variable that persists between function calls as you have found.

If you don't want that behavior, then you need to explicitly copy the series to create a new series each time.

This is another way you can do this as the stars local is not required

star-name: func [ name ][
  rejoin [ "*" name "*" ]
]
Licensed under: CC-BY-SA with attribution
Not affiliated with StackOverflow
scroll top