Skip to content

Conversation

@harusamer
Copy link

@harusamer harusamer commented Oct 9, 2022

ชื่อ Yingyos
ทีมมี 1 คน

https://github.com/harusamer/hackathon-season2

JSON File
Link

Visualization
Link

@harusamer harusamer changed the title Test Finish Oct 9, 2022
@annibuliful
Copy link
Collaborator

แบบนี้ผิดนะครับ รบกวนไปแก้ด้วย ส่งแค่ Github Repo มาครับ ไม่ใช่เปิด PR ด้วย branch ตัวเองครับ

Screen Shot 2565-10-09 at 11 05 03

@annibuliful
Copy link
Collaborator

Screen Shot 2565-10-09 at 11 12 20

@harusamer
Copy link
Author

ชื่อ Yingyos
ทีม 1 คนครับ

@annibuliful
Copy link
Collaborator

ไม่ต้องใส่มาทั้งทั้งโค๊ดครับ ใส่แค่ repo link ก็พอ

@harusamer
Copy link
Author

แบบนี้ผิดนะครับ รบกวนไปแก้ด้วย ส่งแค่ Github Repo มาครับ ไม่ใช่เปิด PR ด้วย branch ตัวเองครับ

Screen Shot 2565-10-09 at 11 05 03

อ้อ เห็นใน Facebook แจ้งว่าให้เปิด PR มาครับ

@annibuliful
Copy link
Collaborator

เหมือนยังขาดข้อนี้ไปนะครับ ผมเห็นแต่ CSV ยังไม่เป็นที่เป็น JSON

Screen Shot 2565-10-09 at 11 22 23

@harusamer
Copy link
Author

ไม่ต้องใส่มาทั้งทั้งโค๊ดครับ ใส่แค่ repo link ก็พอ

แบบนี้ใช่มั้ยครับ
https://github.com/harusamer/hackathon-season2

@harusamer
Copy link
Author

เหมือนยังขาดข้อนี้ไปนะครับ ผมเห็นแต่ CSV ยังไม่เป็นที่เป็น JSON

Screen Shot 2565-10-09 at 11 22 23

มีไฟล์ชื่อ devclub.json อยู่ด้วยครับ

@annibuliful
Copy link
Collaborator

Screen Shot 2565-10-09 at 11 25 36

เหมือนไม่มี data เลยครับ

@annibuliful
Copy link
Collaborator

แล้วก็รบกวนแปะรูปภาพ visualization หรือมาเป็นวิดีโอก็ได้ครับ

@annibuliful
Copy link
Collaborator

ผมเพิ่ม visualization ให้แล้วครับ ไม่เป็นไร เดี๋ยวทางเราจะตรวจสอบความถูกต้องอีกทีครับ ขอบคุณครับ

@harusamer
Copy link
Author

Screen Shot 2565-10-09 at 11 25 36

เหมือนไม่มี data เลยครับ

ขออภัยครับ ผมเพิ่มไฟล์ devclub.json ไปใหม่แล้วครับ

+",'"+ (record \ "REGION").text + "'"
+ ")")
}
val allowPosition = Set("Airhostess", "Pilot", "Steward")
Copy link
Collaborator

Choose a reason for hiding this comment

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

It should be plural

Suggested change
val allowPosition = Set("Airhostess", "Pilot", "Steward")
val allowPositions = Set("Airhostess", "Pilot", "Steward")

println(" Read SQLite ")
var connection = java.sql.DriverManager.getConnection("jdbc:sqlite:result.sqlite");
var statement = connection.createStatement();
var result = statement.execute("""CREATE TABLE DEV_CLUB_OLD (
Copy link
Collaborator

Choose a reason for hiding this comment

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

it should be explicit variable

Suggested change
var result = statement.execute("""CREATE TABLE DEV_CLUB_OLD (
var createdTableResult = statement.execute("""CREATE TABLE DEV_CLUB_OLD (

+ ")")
}
}
var rs1 = statement.executeQuery("SELECT * FROM DEV_CLUB")
Copy link
Collaborator

Choose a reason for hiding this comment

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

it should be explicit variable

Suggested change
var rs1 = statement.executeQuery("SELECT * FROM DEV_CLUB")
var exployeeRecords = statement.executeQuery("SELECT * FROM DEV_CLUB")

}
}
var rs1 = statement.executeQuery("SELECT * FROM DEV_CLUB")
val csv2File = new PrintWriter("devclub.csv")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
val csv2File = new PrintWriter("devclub.csv")
val csvToFile = new PrintWriter("devclub.csv")

@harusamer
Copy link
Author

harusamer commented Oct 19, 2022

รบกวนสอบถามครับ

การแจ้งยืนยันตัวตนที่บอกว่าแจ้งทาง Admin Page พอดีผมใช้ facebook ไม่ค่อยเป็นเลยอยากสอบถามวิธีหน่อยครับ

แล้วถ้ารอบที่ไปแข่งที่เชียงใหม่ อยากเพิ่มสมาชิกมาอีก 2 คนทำได้มั้ยครับ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants