Tôi còn khá mới đối với công nghệ phần mềm và vì vậy, như một bài tập học tập, tôi đã viết một ván cờ. Bạn tôi đã xem nó và chỉ ra rằng mã của tôi trông giống như
for (int i = 0; i < 8; i++){
for (int j = 0; j < 8; j++){
trong khi anh ta khăng khăng rằng nó nên
for (int i = 0; i < CHESS_CONST; i++){
for (int j = 0; j < CHESS_CONST; j++){
với một số tên biểu tượng tốt hơn mà tôi không thể bận tâm để nghĩ ngay bây giờ.
Bây giờ tất nhiên tôi biết là thường tránh sử dụng số ma thuật, nhưng tôi cảm thấy như từ
- con số này sẽ không bao giờ thay đổi;
- cái tên không thể được mô tả bằng mọi cách vì số này được sử dụng ở rất nhiều nơi trong suốt mã; và
- Bất cứ ai đi qua mã nguồn cho một chương trình cờ vua đều phải biết đủ về cờ vua để biết 8 là để làm gì,
thực sự không cần một hằng số tượng trưng.
Vậy các bạn nghĩ sao? Đây có phải là quá mức cần thiết, hay tôi chỉ nên đi theo quy ước và sử dụng một biểu tượng?
46
CHESS_CONST tệ hơn nhiều so với việc chỉ sử dụng số 8, nhưng một hằng số có tên mô tả sẽ là một sự cải tiến. Bạn nói rằng bất cứ ai cũng nên biết 8 là viết tắt của mã, nhưng điều này không đúng. Một số nguyên không có ngữ cảnh có thể có nghĩa là bất kỳ số lượng của sự vật, như một số di chuyển, số lượng các mảnh trên bảng và như vậy. Một tên mô tả cho hằng số sẽ làm cho ý định rõ ràng và do đó mã dễ hiểu hơn.
—
JacquesB
Cá nhân, những gì tôi thấy chói tai hơn nhiều so với số ma thuật là tên
—
Jörg W Mittag
ivà jcho các biến vòng lặp. Cả đời tôi không thể tìm ra cái nào được cho là đại diện cho thứ hạng và cái nào được cho là đại diện cho tập tin. Xếp hạng phạm vi từ 1..8 và các phạm vi từ a..h, nhưng trong trường hợp của bạn, cả hai ivà jphạm vi từ 0..7, vì vậy điều đó không giúp tôi biết đó là cái nào. Có một số cuộc khủng hoảng thiếu thư quốc tế mà tôi không biết, hoặc có gì sai khi đổi tên chúng thành rankvà file?
Chơi người ủng hộ của quỷ trong một giây; Hãy tưởng tượng đọc mã cờ của ai đó nơi họ đã sử dụng số ma thuật 8. Bạn có thể cho rằng bạn biết nó dùng để làm gì không? Làm thế nào bạn có thể chắc chắn 100%? Có khả năng nào nó có thể có nghĩa gì khác không? Nó sẽ không đẹp hơn một chút nếu bạn thậm chí không phải đưa ra một giả định? Bạn có thể dành bao nhiêu thời gian để theo dõi mã để tìm hiểu xem giả định của bạn có đúng không? Thay vào đó, bạn sẽ mất ít thời gian hơn nếu mã đã được ghi lại nhiều hơn bằng cách sử dụng một cái tên sâu sắc, có ý nghĩa?
—
Ben Cottrell
@JacquesB: Thật vậy. Có 8 cấp bậc, 8 tập tin, 8 con tốt. Một số động cơ cờ vua sử dụng hệ thống điểm trong đó một số quân cờ, một số trường nhất định và một số nước đi nhất định có các điểm gắn liền với chúng và động cơ chọn nước đi có số điểm cao nhất. 8 có thể là một số điểm như vậy. 8 có thể là độ sâu tìm kiếm mặc định. Nó có thể là khá nhiều bất cứ điều gì.
—
Jörg W Mittag
Tôi sẽ xem xét sử dụng một vòng lặp foreach, ví dụ
—
CodeInChaos
foreach(var rank in Ranks). Tôi cũng sẽ xem xét việc hợp nhất cả hai vòng lặp thành một trong đó mỗi phần tử là một tuple (xếp hạng, tệp).