Add solution for Challenge 1 by LouisChen-TW

[LouisChen-TW]

Challenge 1 Solution

Submitted by: @LouisChen-TW
Challenge: Challenge 1

Description

This PR contains my solution for Challenge 1.

Changes

• Added solution file to challenge-1/submissions/LouisChen-TW/solution-template.go

Testing

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

Thank you for reviewing my submission! 🚀

[coderabbitai]

Walkthrough

A new Go solution file is introduced in the challenge-1 submission directory with a command-line program that reads two comma-separated integers from stdin, handles input parsing errors, invokes a Sum function to add the integers, and prints the result to stdout.

Changes

Cohort / File(s)	Summary
Challenge 1 Sum Solution challenge-1/submissions/LouisChen-TW/solution-template.go	New Go file implementing a CLI that parses two comma-separated integers from standard input, calls the Sum function to compute their sum, and outputs the result. Includes error handling for malformed input. Exports Sum(a int, b int) int function.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5–10 minutes

• Input parsing and error handling logic should be verified for correctness
• Verify the Sum function implementation and its exported API

Possibly related PRs

• Add solution for Challenge 2 by ymonn #739: Adds an identical challenge-1/.../solution-template.go with exported Sum(a int, b int) int and the same CLI flow pattern.
• Add solution for Challenge 1 by liquiid727 #846: Introduces a nearly identical Go solution file with the same exported Sum function and main routine that reads two integers and prints their sum.
• Add solution for Challenge 1 by RP-Guruh #864: Adds an identical challenge-1 solution file with the same Sum(a int, b int) int function and CLI input/error handling logic.

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 clearly identifies the pull request as adding a solution for Challenge 1 by a specific contributor, which directly relates to the changeset.
Description check	✅ Passed	The description is directly related to the changeset, explaining that it contains a Challenge 1 solution with appropriate details about the submission and testing.

---

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 (2)

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

7-14: Consider exiting with a non-zero status code on error.

When input reading fails, the program currently returns from main, which exits with status code 0 (success). For better CLI behavior, consider using os.Exit(1) to signal an error condition to the calling process.

Apply this diff to improve error handling:

 import (
 	"fmt"
+	"os"
 )

 	_, err := fmt.Scanf("%d, %d", &a, &b)
 	if err != nil {
 		fmt.Println("Error reading input:", err)
-		return
+		os.Exit(1)
 	}

---

21-24: Format the return statement according to Go conventions.

The expression a+b should have spaces around the + operator. Running gofmt on this file would automatically format it as a + b, which is the standard Go style.

Apply this diff:

 func Sum(a int, b int) int {
-	return a+b
+	return a + b
 }

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cbe1b92 and 3ce3180.

📒 Files selected for processing (1)

• challenge-1/submissions/LouisChen-TW/solution-template.go (1 hunks)

🔇 Additional comments (2)

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

1-5: LGTM!

The package declaration and imports are clean and appropriate for a CLI program.

---

16-18: LGTM!

The Sum function is called correctly and the result is printed as expected.