Skip to content

Conversation

@hageboeck
Copy link
Contributor

@hageboeck hageboeck commented Oct 29, 2025

This exercise is used both in the essentials and the advanced course.

  • In the essentials course, we cannot require students to work with hidden friends and <=>.
  • In the advanced course, it is run on the first day, where hidden friends are discussed, but <=> is discussed on the third day.

Here, all mentions of <=> are removed (there is a dedicated exercise on
the third day of the advanced course), and the mention of hidden friends now
refers only to the advanced course.
I'm unsure if we should provide an extra solution for the essentials course that
doesn't use the friend keyword anywhere?

Furthermore, the diff between the original file and the solution was minimised, so
viewing the solution with a diff program is hopefully helpful for students.

Lastly, the syntax of the CHECK macro has been simplified by removing its first argument.
This will hopefully help a bit to focus on the operators.

@hageboeck hageboeck requested review from chavid and sponce October 29, 2025 16:21
@hageboeck hageboeck self-assigned this Oct 29, 2025
The first argument of the check macro is unnecessary if all checks are
printed using the same width. This makes the code easier to read.
Using the variadic macro approach from the previous commit, the solution
could also be simplified.
This exercise is used both in the essentials and the advanced course. We
can therefore not require students to work with hidden friends and <=>.
Here, all mentions of <=> are removed (there is a dedicated exercise on
the third day of the advanced course), and the tasks are rephrased such
that students can work on this exercise also in the essentials course.

The diff between the original file and the solution is minimised, so
viewing the solution with a diff program hopefully helps the students.
@hageboeck hageboeck force-pushed the operatorExercise branch 2 times, most recently from 626e9a9 to 658a5b9 Compare October 29, 2025 16:34
@hageboeck hageboeck merged commit cd54393 into hsf-training:master Nov 4, 2025
155 of 175 checks passed
@hageboeck hageboeck deleted the operatorExercise branch November 4, 2025 09:12
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.

2 participants