Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Roman Numerals] add practice exercise #310

Closed
wants to merge 2 commits into from

Conversation

colinleach
Copy link
Contributor

@colinleach colinleach commented Jan 22, 2024

The performance stuff for reverse-string is going badly, so I did this in the interim. It's for week [02-06]. Only the exercise, no Deep Dive stuff.

It works, but the example implementation may not be the most idiomatic R. A while loop inside a for loop is standard across most languages (except the ones relying heavily on pattern matching), but I'll be very happy to be shown that I'm missing something R-specific.

Feel free to replace with something better if you wish: we can use all the implementations for approaches later. It certainly isn't vectorized, but I couldn't see how to make that work within the function.

@colinleach
Copy link
Contributor Author

Scrap this. I'm not in best health (again) today, and failed to create a separate branch. One of those days when I seem to have forgotten how to code in any language, despite doing it for nearly 50 years...

@colinleach colinleach closed this Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant