TIL 23-04-24 팀원과의 알고리즘 코드리뷰 - 양꼬치

1. 팀원과의 알고리즘 코드리뷰 - 양꼬치

 문제점

def solution(n, k):
    YangKkochi = n * 12000
    freeCoke = str(n)[:-1]
    coke = (k - int(freeCoke)) * 2000
    answer = YangKkochi + coke
    return answer

반례 찾기

 시도해 본 것들

문제가 생기는 부분 찾기 1

freeCoke = str(n)[:-1]
coke = (k - int(freeCoke)) * 2000

이 부분에서 n이 9이하인 경우 freeCoke가 빈값으로 남아서 int로 형 변환시 에러 발생 

if n < 10:
        freeCoke = 0
    else:
        freeCoke = str(n)[:-1]

 

if문 추가 

def solution(n, k):
    YangKkochi = n * 12000
    if n < 10:
        freeCoke = 0
    else:
        freeCoke = str(n)[:-1]

 해결 방법

def solution(n, k):
    YangKkochi = n * 12000
    if n < 10:
        freeCoke = 0
    else:
        freeCoke = str(n)[:-1]
    
    coke = (k - int(freeCoke)) * 2000
    answer = YangKkochi + coke
    return answer

 알게 된 점

[-1]같이 한글자를 지정해서 가져오는 게 아닌 [:-1]처럼 여러 문자를 가져오는 경우 값이 1개밖에 없을 때 에러가 발생함을 알게됨

 

 

1. 팀원과의 알고리즘 코드리뷰 - 피자 나눠먹기(1)

 문제점

def solution(n):
    answer = 0
    
    if n // 7 == 0:
        answer = 1
    elif n // 7 == 1:
        answer = 1
    elif (n % 7) == True:
        answer = (n // 7) + 1
    
    return answer

반례 찾기 

 시도해 본 것들

문제가 생기는 부분 찾기 1

elif n // 7 == 1:
        answer = 1

이 코드가 n이 8~13일 때 answer가 2가 되어야 하지만 answer를 1로 출력함 코드 제거

 

문제가 생기는 부분 찾기 2

  elif (n % 7) == True:
        answer = (n // 7) + 1

elif의 조건문 설정에서 문제 발생

 

코드 테스트 하기

if 7 % 7 == True:
    print('True')
else:
    print('False')

이 코드를 7,8,9,10으로 실행한 결과 나머지가 1인 경우만 True 

 

if n % 7:

n % 7이 0이 아닌경우 실행하게 변경 

 해결 방법

def solution(n):
    answer = 0    
    if n % 7:
        answer = (n // 7) + 1
    else:
        answer = n//7
    return answer

 알게 된 점

if문에서 == True를 주게 되면 값이 있는 경우가 아니라 1, True과 같은 경우 실행된다.

 

 

2. 백준 알고리즘 - 그룹 단어 체커 

 문제점

문제
그룹 단어란 단어에 존재하는 모든 문자에 대해서, 
각 문자가 연속해서 나타나는 경우만을 말한다. 
예를 들면, ccazzzzbb는 c, a, z, b가 모두 연속해서 나타나고, 
kin도 k, i, n이 연속해서 나타나기 때문에 그룹 단어이지만, 
aabbbccb는 b가 떨어져서 나타나기 때문에 그룹 단어가 아니다.

단어 N개를 입력으로 받아 그룹 단어의 개수를 출력하는 프로그램을 작성하시오.

입력
첫째 줄에 단어의 개수 N이 들어온다. N은 100보다 작거나 같은 자연수이다. 
둘째 줄부터 N개의 줄에 단어가 들어온다. 
단어는 알파벳 소문자로만 되어있고 중복되지 않으며, 길이는 최대 100이다.

출력
첫째 줄에 그룹 단어의 개수를 출력한다.

# 이어서 나오는 것은 괜찮지만, 한 번 떨어졌다가 재 등장은 불가

# while문으로 반복 시도

# 딕셔너리로 만들어서 단어가 바뀔 때 딕셔너리의 값을 체크?

 시도해 본 것들

1개의 단어만 주어진 경우 그룹 단어 여부 판단하기 

count = 0
word_dict = {}
word = 'happy'
for w in range(len(word)):
    word_dict[word[w]] = 1
    # 단어가 바뀌는 구간
    if w != 0 and word[w] != word[w-1]:
        # 단어가 첫 등장이 아니면
        if word_dict[word[w]] != 1:
            break
    # for문의 마지막일 때 count 증가
    if w == len(word)-1:
        count += 1
print(count)

print로 테스트한 결과 word_dict의 값은 무조건 1로 설정되어 있어서 두 번째 if문 진입 X

 

dict 값 변경을 위한 조건문 추가하기 

        if word[w] not in word_dict:
            word_dict[word[w]] = 1
        else:
            word_dict[word[w]] += 1

이렇게 변경 후 count 테스트 성공

 

단어가 여러 개인 경우로 바꾸기

a = int(input())
count = 0
while a > 0:
    a -= 1
    word = input()
    word_dict = {}
    for w in range(len(word)):
        if word[w] not in word_dict:
            word_dict[word[w]] = 1
        else:
            word_dict[word[w]] += 1
        # 단어가 바뀌는 구간
        if w != 0 and word[w] != word[w-1]:
            # 단어가 첫 등장이 아니면
            if word_dict[word[w]] != 1:
                break
        # for문의 마지막일 때 count 증가
        if w == len(word)-1:
            count += 1
print(count)

 while문을 통한 반복과 dict 초기화만 해주면 완성
제출은 총과했으나 조건문의 수를 줄이고 싶어 짐

count 로직 바꾸기

word_list = []
    word = input()
    word_list.append(word)

            if word_dict[word[w]] != 1:
                word_list.pop()
                break
print(len(word_list))

빈 리스트를 선언하고 append로 추가한 뒤 브레이크를 하는 경우 pop으로 제거, print를 리스트의 길이로 출력

 해결 방법

a = int(input())
word_list = []
while a > 0:
    a -= 1
    word = input()
    word_list.append(word)
    word_dict = {}
    for w in range(len(word)):
        if word[w] not in word_dict:
            word_dict[word[w]] = 1
        else:
            word_dict[word[w]] += 1
        # 단어가 바뀌는 구간
        if w != 0 and word[w] != word[w-1]:
            # 단어가 첫 등장이 아니면
            if word_dict[word[w]] != 1:
                word_list.pop()
                break
print(len(word_list))

 알게 된 점

어려울 것 같아서 풀지 못하고 있었는데 가능한 부분부터 하나씩 만들어가니 시간은 꽤 걸렸지만, 어렵다는 느낌을 받지는 않았다.