Add solution for Challenge 1 by berkkaradalan

[berkkaradalan]

Challenge 1 Solution

Submitted by: @berkkaradalan
Challenge: Challenge 1

Description

This PR contains my solution for Challenge 1.

Changes

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

Testing

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

Thank you for reviewing my submission! 🚀

[coderabbitai]

Walkthrough

Adds a new Go program for Challenge 1 that reads two comma-separated integers from stdin, exposes an exported Sum(a int, b int) int function returning a+b, and prints the result; includes simple input error handling.

Changes

Cohort / File(s)	Change Summary
Challenge 1 solution (new file) challenge-1/submissions/berkkaradalan/solution-template.go	Added new Go program: main reads two ints with fmt.Scanf("%d, %d", &a, &b), handles read errors, calls exported Sum(a int, b int) int which returns a + b, and prints the result.

Sequence Diagram(s)

sequenceDiagram
    participant User as User (stdin)
    participant Main as main()
    participant Sum as Sum(a,b)

    Note over User,Main: Program start — reads input
    User->>Main: provide "int, int"
    Main->>Main: parse input (fmt.Scanf)
    alt parse success
        Main->>Sum: call Sum(a,b)
        Sum-->>Main: return a+b
        Main->>User: print sum
    else parse error
        Main->>User: print error and exit
    end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Single small file, trivial arithmetic and basic I/O.
• Review attention: correctness of scanf format and exported function signature.

Possibly related PRs

• Add solution for Challenge 1 by Kosench #559 — Adds a near-identical Go solution for Challenge 1 with the same exported Sum(a int, b int) int and similar main.
• Add solution for Challenge 1 by Dhar01 #737 — Introduces matching Sum function and main reading two ints with the same fmt.Scanf format.
• Add solution for Challenge 1 by ansmonjol #751 — Adds an equivalent solution-template.go for Challenge 1 with identical input/print and Sum implementation.

Pre-merge checks

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: adding a solution for Challenge 1 by a specific contributor.
Description check	✅ Passed	The description is directly related to the changeset, providing context about the Challenge 1 submission, file location, and testing checklist.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aec20ff and b342b0a.

📒 Files selected for processing (1)

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

🚧 Files skipped from review as they are similar to previous changes (1)

• challenge-1/submissions/berkkaradalan/solution-template.go

---

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: 1

🧹 Nitpick comments (2)

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

11-14: Consider writing errors to stderr.

Error messages are currently written to stdout. For better practice, error output should go to stderr.

Apply this diff:

 	if err != nil {
-		fmt.Println("Error reading input:", err)
+		fmt.Fprintln(os.Stderr, "Error reading input:", err)
 		return
 	}

You'll also need to import os:

 import (
 	"fmt"
+	"os"
 )

---

22-22: Consider using grouped parameter syntax.

Go allows grouping parameters of the same type for more concise syntax.

Apply this diff:

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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ca8ea29 and aec20ff.

📒 Files selected for processing (1)

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

🔇 Additional comments (1)

challenge-1/submissions/berkkaradalan/solution-template.go (1)

10-10: Input format is correct and intentional.

The test file (challenge-1/solution-template_test.go) explicitly validates comma-separated input with test cases like "2, 3", "0, 0", "-2, -3", confirming that the format string "%d, %d" is the intended specification for Challenge 1. All submissions and the root template consistently use this format, indicating it is the required standard.

Likely an incorrect or invalid review comment.

[berkkaradalan]

fix: todo comment line removed