Add solution for Challenge 2 by Kosench

[Kosench]

Challenge 2 Solution

Submitted by: @Kosench
Challenge: Challenge 2

Description

This PR contains my solution for Challenge 2.

Changes

• Added solution file to challenge-2/submissions/Kosench/solution-template.go

Testing

• Solution passes all test cases
• Code follows Go best practices

Thank you for reviewing my submission! 🚀

[coderabbitai]

Walkthrough

Introduces a new Go program that reads a line from standard input, reverses it using a rune-aware ReverseString function, and prints the result. The main function uses bufio.Scanner to read input, applies ReverseString, and outputs the reversed string.

Changes

Cohort / File(s)	Summary of changes
Go solution implementation challenge-2/submissions/Kosench/solution-template.go	Added a main program that reads stdin, reverses the string via ReverseString(s string) string (rune-slice reversal), and prints the output.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant Stdin as Stdin
  participant Main as main()
  participant Reverse as ReverseString()
  participant Stdout as Stdout

  User->>Stdin: Enter line
  Main->>Stdin: bufio.Scanner.Scan()
  alt input available
    Main->>Stdin: scanner.Text()
    Main->>Reverse: ReverseString(s)
    Reverse-->>Main: reversed string
    Main->>Stdout: print reversed string
  else no input
    Main-->>Stdout: (no output)
  end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~7 minutes

Pre-merge checks

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title “Add solution for Challenge 2 by Kosench” directly summarizes the main change of the pull request by indicating that a solution for Challenge 2 is being added by the author, making it clear and concise without extraneous details.
Description Check	✅ Passed	The pull request description accurately describes that this submission contains the author’s solution for Challenge 2, details the added file path, and notes successful testing and adherence to Go best practices, making it clearly related to the changeset.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

challenge-2/submissions/Kosench/solution-template.go (1)

9-21: Consider checking for scanner errors.

The code doesn't verify scanner.Err() when Scan() returns false. Go best practices recommend checking for I/O errors to distinguish between EOF and actual errors.

Apply this diff to add error handling:

 func main() {
 	// Read input from standard input
 	scanner := bufio.NewScanner(os.Stdin)
 	if scanner.Scan() {
 		input := scanner.Text()
 
 		// Call the ReverseString function
 		output := ReverseString(input)
 
 		// Print the result
 		fmt.Println(output)
 	}
+	if err := scanner.Err(); err != nil {
+		fmt.Fprintf(os.Stderr, "Error reading input: %v\n", err)
+		os.Exit(1)
+	}
 }

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 54d23f2 and 4064678.

📒 Files selected for processing (1)

• challenge-2/submissions/Kosench/solution-template.go (1 hunks)

🔇 Additional comments (2)

challenge-2/submissions/Kosench/solution-template.go (2)

1-7: LGTM!

The imports are appropriate for the task. No issues here.

---

24-30: Excellent Unicode handling!

The function correctly handles Unicode characters by converting to runes before reversing. The two-pointer swap technique is efficient and idiomatic Go.