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

[브루트포스] 9월 13일 #4

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

[브루트포스] 9월 13일 #4

wants to merge 10 commits into from

Conversation

rahaaaiii
Copy link
Collaborator

@rahaaaiii rahaaaiii commented Sep 13, 2023

인적사항

학번 : 2271092
이름 : 이한선

과제제출

기존 제출 : 11723, 1436, 1063

error:invalid argumnet
int i=N-count*9보다 작은 경우일경우 stoi함수에서 접근할 때 invalid_argumnet runtime error 발생, N이 (9*자리수)보다 크냐 작냐에따라 실행되는 계산법 다르게 해서 해결
Copy link
Member

@Goldchae Goldchae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

브루트 포스- 구현 문제 리뷰 완료!
p3. 열심히 잘 작성해주신 것 같습니다! 수고하셨어요!

s = kd[0];

if (s == 'A') {
if (n == '1') {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p3. 모든 경우에 기물이 체스판을 벗어나는 것이 조건 위배죠! 움직임 조건별로 정리하는 것보다 전부 하나의 함수로 만들어서 처리하셨어도 좋을 것 같네요!

Comment on lines +67 to +81
else if(dis=="RT"){
s++;
n++;
}
else if(dis=="LT"){
s--;
n++;
}
else if(dis=="RB"){
s++;
n--;
}
else if(dis=="LB"){
s--;
n--;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p3. 두개의 문자로 이루어진 방향표시는 각각 알파벳 두 개의 방향표시로 나누어 생각할 수 있죠! 코드를 줄일 방법은 없을까요?

return dis;
}

int main()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

가벼운 main 함수 좋습니다!

Copy link
Contributor

@kwakrhkr59 kwakrhkr59 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

브루트포스 필수 알고리즘 코드 리뷰 완료
1436(P3) 안녕하세요 한선님~ 필수 알고리즘 문제 코드리뷰 완료되셨습니다! 몇 가지 코멘트 남겨드렸으니 확인해보시고 질문 있으시면 언제든 리뷰어로 호출해주세요😁 수고 많으셨습니다~

}

void toggle(int& ans, int value){
ans^=(1<<value);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

비트연산자 활용 아주 좋습니다👍👍

}

void all(int& ans){
ans=(1<<21)-1;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

의미를 명확히 하기 위해서 const int ALL = (1 << 21) - 1로 선언한 후 ans = ALL등으로 사용해주는 건 어떨까요?

while(count!=N){
s=to_string(n);
for(int i=0; i<s.size()-2; i++){
if(s[i]=='6' && s[i+1]=='6' && s[i+2]=='6'){//string vector에서 문자 비교하고 싶을 때 "" 쓰면 pointer error 발생!!
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

n을 string으로 바꾸지 않아도 666을 찾을 수 있어요! 한 정수가 주어져있울 때 각 자릿수를 접근하고 싶을 때 이 숫자를 10씩 나눠가며 나머지를 확인했던 거 기억하시나요? 여기서도 마찬가지로, 대신 666이라는 세자리 숫자를 확인해야 하니 1000으로 나눴을 때의 나머지를 확인하면 666의 존재를 찾을 수 있어요! 샘플 코드에는 이 방식으로 풀이가 되어 있으니 한 번 확인해보셔도 좋을 것 같습니다😁


int main()
{
int N;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P3. 변수의 표기법을 통일해주세요! 변수에서는 스네이크 표기법을 함수에서는 카멜 표기법을 사용하기로 정하고 변수에서 다른 표기법을 사용하실 경우 혼돈이 올 수 있어요😂 알튜비튜에서는 변수명에는 스네이크 표기법을, 상수명에는 대문자를, 함수명에는 카멜 표기법을 사용하도록 권고드리고 있습니다😊

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.

3 participants