Skip to content

Conversation

jimmytty
Copy link
Contributor

I have removed all "error handling" test cases.

The difficulty was set to 8. This exercise need looping through each character from strands to compare and count differences. (my own opinion) Loop it's not an easy feature/concept for SQL, it's need a recursive cte.

However,
From all 77 active tracks, only the tracks: cobol, rust and vimscript have set exercise difficulty above "easy". Those tracks set difficulty to 4 (medium).

I can change the difficulty if you want and I can also introduce the "error handling" tests. But in this case, I'll need an opinion on whether to use the result column as json object or set invalid/errors as -1, or even something else.

@IsaacG
Copy link
Member

IsaacG commented Aug 10, 2025

Being hard where other tracks mark this as easy is fine. Other languages have loops.
I'd love some input from @exercism/sqlite on whether it makes sense to skip those tests or not. Unequal lengths feel more like part of the core problem to me for this exercise.

If we do skip the tests, we should include a comment with the skips.

@jimmytty
Copy link
Contributor Author

🤔 I was thinking about the @glennj option with an extra column for errors. I could implement all error handling tests cases without use a json object.

@IsaacG
Copy link
Member

IsaacG commented Aug 17, 2025

That sounds like a good option to me

@jimmytty
Copy link
Contributor Author

About the expected_result and expected_error columns, using another approach, I can also use one single column, expected and a json object to represent both outputs.

@IsaacG IsaacG changed the title * add new practice exercise: hamming Add new practice exercise: hamming Aug 20, 2025
@IsaacG IsaacG added x:action/create Work on something from scratch x:size/large Large amount of work x:rep/large Large amount of reputation x:module/practice-exercise Work on Practice Exercises labels Aug 20, 2025
@IsaacG IsaacG merged commit 7be3423 into exercism:main Aug 20, 2025
2 checks passed
@jimmytty jimmytty deleted the hamming branch August 29, 2025 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:action/create Work on something from scratch x:module/practice-exercise Work on Practice Exercises x:rep/large Large amount of reputation x:size/large Large amount of work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants