Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Miro] 학생 성적 관리 시스템 #3

Open
wants to merge 4 commits into
base: 1_Miro
Choose a base branch
from

Conversation

longlivedrgn
Copy link

@longlivedrgn longlivedrgn commented Dec 20, 2022

Summary

  • 이름이 존재하는지, input값의 index가 무엇인지 알려주는 함수를 생성했습니다.
  • 전체를 while문으로 감싸 input이 'X'가 아닐 경우 계속해서 호출되도록 코드를 작성했습니다.

커밋한 내용

  • 함수의 이름을 [Swift API Design Guidelines]에 맞게 변경해보았습니다.

What to review?

  • 다양한 함수를 사용하였는데, 서로 합칠 수 있는 함수가 있는 review 부탁드립니다.
  1. checkName
  2. checkIndex
  3. calculateScore
  4. sumScore
  5. macro

특히 'checkName', 'checkIndex' 위주로 확인부탁드리겠습니다.

func checkName(temp: [(String,String,String)], findingA: String, findingB: String) -> Bool {
    let count = temp.count
    var find = false
    for i in 0...(count-1){
        if temp[i].0 == findingA && temp[i].1 == findingB{
            find = true
        } else {
            continue
        }
    }
    return find
}
func checkIndex(temp: [(String,String,String)], findingA: String, findingB: String) -> Int{
    let count = temp.count
    var index = 0
    for i in 0...(count-1){
        if temp[i].0 == findingA && temp[i].1 == findingB {
            break
        } else {
            index += 1
        }
    }
    return index
}
  • 함수의 이름을 변경하였는데, naming에서 아직 부족한 부분이 있는 지 확인 부탁드리겠습니다.
  • input 값을 나누는 더 효과적인 알고리즘이 있는 지 확인부탁드립니다.

@longlivedrgn longlivedrgn changed the title Pulll request practice [Miro] First pull request Dec 20, 2022
@longlivedrgn longlivedrgn changed the title [Miro] First pull request [Miro] 학생 성적 관리 시스템 Dec 20, 2022
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.

1 participant