search

hajubal / pickupcoins

0 stars 0 forks source link

crawler interface에 site 정보를 가져오는 행위 명시할 필요가 있나? #3

Closed hajubal closed 1 year ago

hajubal commented 1 year ago

https://github.com/hajubal/pickupcoins/blob/864d199323c9575807e11ca3afc0af0c9cb38522/src/main/java/me/synology/hajubal/coins/crawler/WebCrawler.java#L14

site 정보는 주입 받아야 하는게 맞아 보임. 이것은 crawling에 대한 interface 이므로

hajubal commented 1 year ago

사이트 인터페이스에 사이트 정보를 가져오는 함수정의는 필요없는게 맞지만 사이트 정보를 주입 받을 필요는 없음. 초기화 시에 저장된 값으로 db에서 조회해 오니까. 아님?

hajubal commented 1 year ago

근데 애초에 초기화 데이터를 코드에서 관리가 되어야 하는게 맞는 건가? init.sql 파일 같은 걸로 초기화 데이터를 밀어 넣어야 하지 않을까?

hajubal commented 1 year ago

근데 애초에 초기화 데이터를 코드에서 관리가 되어야 하는게 맞는 건가? init.sql 파일 같은 걸로 초기화 데이터를 밀어 넣어야 하지 않을까?

사이트 테이블에 생성일과 같은 데이터 때문에 코드에서 넣으려고 했는데. h2, mysql 둘다 "now()"로 현재 시간이 입력된다. 사이트 초기화 데이터는 data.sql 파일로 입력하는걸로 변경해 보겠다. 다만 사용자 쿠키 데이터는 data.sql 파일에 넣을 수가 없어서 일단 코드로 유지한다.

hajubal commented 1 year ago

근데 애초에 초기화 데이터를 코드에서 관리가 되어야 하는게 맞는 건가? init.sql 파일 같은 걸로 초기화 데이터를 밀어 넣어야 하지 않을까?

사이트 테이블에 생성일과 같은 데이터 때문에 코드에서 넣으려고 했는데. h2, mysql 둘다 "now()"로 현재 시간이 입력된다. 사이트 초기화 데이터는 data.sql 파일로 입력하는걸로 변경해 보겠다. 다만 사용자 쿠키 데이터는 data.sql 파일에 넣을 수가 없어서 일단 코드로 유지한다.

확인 결과 data.sql 은 서버 시작할 때 마다 실행되니 데이터가 계속 추가되는 문제가 있어 사용할 수 없다. 일단 기존대로 코드로 데이터를 초기화 한다.

hajubal commented 1 year ago

작업완료되었다.