kylim0102 / Kiosk-Project

0 stars 0 forks source link

Server Clean Up Code #6

Open donghoonleeeeeee opened 1 month ago

donghoonleeeeeee commented 1 month ago
  1. oGlobal.cs 최하단에 위치한 enum enWeek, enWeek_Han의 참조가 0이여서 삭제해도 되는지 여부 확인바랍니다.

  2. oGlobal.cs MySqlConnection(DB_Connection())에서 전역 데이터베이스를 연결 할 때 static으로 선언되어있는데 그럼 다른 페이지에서 사용할 때도 static으로 메서드를 만들어야 할텐데 그 부분에서 문제가 생겨 가끔 UI 확인 중에서 DB Connection 에러가 발생하는게 아닌가 싶습니다. 2-1. 또한 DBConnection.Open할 때 전역 데이터베이스 연결 할 때 애초에 If(DBConnection.state = ConnectionState.Closed)로 예외처리를 한번에 해버리고 후에 사용하는게 어떨가 싶습니다. 아마 이 부분은 Client에서도 DB를 연결할 때 참고해야 할 부분인듯하여 작성합니다.

해당 내용들의 최종 수정은 내일 오후에 작업하겠습니다.

wkkjhhhhh commented 1 month ago
  1. 삭제해도 됩니다 enum을 쓴게 차트 에서 임의로 요일별로 뽑아내고 싶어서 만들었던거라

  2. 정적으로 값을 직접 입력하는 DB_Connecion() 메소드는 static으로 만드는 게 좋을거 같고 그 밑에 GetConnection 은 static 메소드로 만들 필요는 없을것 같네요. 각 페이지에 GetConnection 메소드를 불러오는 곳에 static을 빼주는 귀찮음이 생기겠지만 수정하는게 좋을듯 하네요

2-1. 현재 GetConnection 메소드 안에 if 문이 있는데 그 조건문을 바로 DBConnection.state = ConnectionState.Closed 로 넣어주면 되겠네요 Client 부분도 그렇게 만들도록 하겠습니다

donghoonleeeeeee commented 1 month ago

참고로 어제 확인해본 바로는 Static으로 con을 연결할 경우에 그 con을 사용하는 다른 private에서 동일하게 static을 넣어줘야합니다 만약 private에서 별도로 con을 열고 닫고 관리 할경우에는 그냥 void로 열면 됩니다. 그래서 client에서 db 작업할때 이 부분을 고려하면 될 듯 합니다