Có phải là thực tế xấu để lưu trữ các giá trị nhất định như chuỗi?

Đó là một tiêu đề rất mơ hồ nhưng tôi không thể nghĩ ra cách nào tốt hơn để diễn đạt nó. Nhưng, chỉ là một ví dụ, hãy nghĩ về hướng mà một nhân vật trong trò chơi đang di chuyển. Nó chỉ cảm thấy sai lầm khi sử dụng một chuỗi và sau đó làm những việc như thế if(character.direction == "left"). Dường như với tôi rằng nó để lại quá nhiều chỗ cho sai sót ngớ ngẩn, như vô tình sử dụng Lefthoặc lhoặc bất cứ điều gì thay vì left. Những nghi ngờ của tôi có đúng không? Nếu vậy, cách ưa thích để đạt được một cái gì đó như thế này là gì?

Nếu ngôn ngữ bạn đang sử dụng hỗ trợ việc sử dụng enum, tôi sẽ sử dụng chúng. Nó cho phép bạn giới hạn số lượng tùy chọn có sẵn cho một loại nhất định. ví dụ: trong Java:

public enum Direction {
    LEFT,
    RIGHT,
    UP,
    DOWN
}

Đó là cách thực hành khủng khiếp để sử dụng các chuỗi ký tự (hoặc số ma thuật) trong mã.

Enums là tốt, hoặc ít nhất là sử dụng hằng số (enums tất nhiên chỉ là một loại bao bọc cho một hoặc nhiều hằng số liên quan).

const string LEFT = "left"
if (someThing == LEFT) { doStuff(); }

Công tắc đơn giản này có rất nhiều lợi thế tôi thậm chí không biết bắt đầu giải thích chúng ở đâu (ít nhất là không có thêm cà phê). Đọc các số ma thuật, đó là khái niệm chính xác tương tự, chỉ áp dụng cho một giá trị số thay vì giá trị chuỗi.

Đây là một giới thiệu nhanh, tôi chắc chắn còn hàng trăm: /programming/47882/what-is-a-magic-number-and-why-is-it-bad

Câu trả lời ngắn: Có, các chuỗi không lý tưởng cho bất kỳ tác vụ nào ngoài việc lưu trữ và truy cập một chuỗi các ký tự văn bản và ngay cả khi các bit cơ bản của trừu tượng là các chuỗi, có những lợi ích khi gọi chúng là biến hoặc hằng .

Câu trả lời dài: hầu hết các ngôn ngữ cung cấp các loại gần với miền của vấn đề của bạn và ngay cả khi chúng không có, chúng có thể có một số phương pháp mà bạn có thể xác định leftlà một thực thể khác với right(v.v.). Biến nó thành một chuỗi bằng cách sử dụng "left"chỉ đơn giản là bị mất ngôn ngữ và chức năng hữu ích của IDE như kiểm tra lỗi. Ngay cả khi bạn phải sử dụng

left = "left";

hoặc một cái gì đó tương đương, nó có lợi thế khi sử dụng chuỗi khi bạn tham chiếu nó qua mã của bạn. Ví dụ,

if (player.direction == Left)

sẽ được phát hiện là lỗi thời gian biên dịch (trường hợp tốt nhất, java và tương tự) hoặc bị đánh dấu bởi bất kỳ IDE nào có giá trị bit của nó là sai (trường hợp xấu nhất, cơ bản và như vậy).

Ngoài ra, có khái niệm leftlà một thực thể có thể tham chiếu sẽ giúp bạn điều chỉnh bất kỳ hướng nào bạn quyết định đi với loại hướng. Bằng cách sử dụng "left", hướng được cam kết là a String. Nếu bạn tìm thấy hoặc tạo ra một sự trừu tượng hóa tốt hơn cho loại, bạn cần phải đi tìm toàn bộ phần thân mã của mình thay đổi mọi trường hợp "left". Một hằng hoặc biến sẽ không yêu cầu bất kỳ thay đổi nếu loại được xây dựng.

Loại bạn thực sự muốn là đặc biệt cho miền của bạn. Mã của bạn có kế hoạch để làm gì với của bạn left? Có lẽ bạn sẽ muốn phản chiếu trục x của một kết cấu hoặc sprite hướng về bên trái hoặc di chuyển về phía trước; nếu vậy, bạn có thể muốn tạo leftmột đối tượng bằng một thuộc tính hoặc phương thức phản ánh hành vi đó, với rightđối tượng của bạn có kết quả không được nhân đôi ngược lại. Bạn có thể thực hiện logic đó trong một đối tượng đại diện cho các họa tiết hoặc kết cấu, trong trường hợp đó bạn sẽ phải chịu đựng khi sử dụng "left"thay vì các leftlý do đã nêu ở trên.

Trong Java (và có lẽ các ngôn ngữ khác mà tôi chưa biết), enumlà một lựa chọn tốt vì trong Java, enumcũng hữu ích final classvới một tập hợp hữu hạn. Bạn có thể xác định các hành vi nếu bạn cần chúng và bạn có thể chuyển sang một lớp mở mà không gặp rắc rối nào bên ngoài tệp khai báo enum, nhưng nhiều ngôn ngữ xem enumlà một kiểu nguyên thủy hoặc yêu cầu cú pháp đặc biệt để sử dụng. Điều đó làm rò rỉ enummui xe thành mã không có kinh doanh với nó và có thể cần phải được sửa chữa sau này. Hãy chắc chắn rằng bạn hiểu khái niệm ngôn ngữ của enumbạn trước khi bạn xem xét tùy chọn.

Bạn đã không chỉ định ngôn ngữ của mình, nhưng để đưa ra câu trả lời chung chung, bạn nên sử dụng các chuỗi khi bạn thực sự cần văn bản, không phải để thể hiện một cái gì đó. Ví dụ bạn đưa ra chẳng hạn sẽ không được chấp nhận trong phần mềm sản xuất.

Mỗi ngôn ngữ có các loại dữ liệu cơ bản khác nhau và bạn nên sử dụng một loại phù hợp nhất cho nhiệm vụ. Để sử dụng ví dụ của bạn, bạn có thể sử dụng các hằng số để biểu diễn các trạng thái khác nhau. Vì vậy, bên trái có thể có giá trị bằng 0 và bên phải sẽ là một. Bên cạnh lý do bạn đã đề cập, các giá trị số chiếm ít không gian (bộ nhớ) và việc so sánh các số luôn nhanh hơn so với nhiều chuỗi ký tự.

Như đã đề xuất, hãy sử dụng bảng liệt kê nếu ngôn ngữ của bạn cho phép. Trong ví dụ cụ thể của bạn, rõ ràng đó là sự lựa chọn tốt hơn.

Chỉ cần sử dụng kiểu dữ liệu có ý nghĩa trong tình huống của bạn.

Việc sử dụng stringloại như một giao tiếp nội bộ / ứng dụng cơ bản không phải là một vấn đề. Trên thực tế, đôi khi nên sử dụng các chuỗi: Nếu bạn có API công khai, có thể mong muốn các giá trị đi vào và ra khỏi API có thể đọc được. Nó giúp mọi người sử dụng API của bạn để tìm hiểu và gỡ lỗi dễ dàng hơn.

Vấn đề thực sự với mã của bạn nằm ở việc lặp lại giá trị.

Đừng làm điều này:

if (direction == 'left') {
  goLeft();
}

Nếu bạn mắc lỗi đánh máy ở một trong những điều kiện này hoặc cách sử dụng "trái" khác ở đâu đó, bạn có thể không thể thực hiện một cách hiệu quả một tìm kiếm trên toàn bộ cơ sở mã cho nó, vì bạn đã nhầm nó!

Thay vào đó, mã theo bảng liệt kê hoặc hằng số - tùy thuộc vào loại nào hỗ trợ loại dữ liệu bạn cần trong (các) ngôn ngữ bạn đang sử dụng.

Điều đó có nghĩa là LEFTnên được chỉ định một lần và chỉ một lần trong ứng dụng của bạn. Và, phần còn lại của mã của bạn sẽ tận dụng các enum hoặc hằng số đó:

const string LEFT = "left";

if (direction == LEFT) {
  goLeft();
}

Điều này cũng cho phép bạn dễ dàng thay đổi loại dữ liệu bạn sử dụng cho chỉ đường của mình sau này, nếu bạn chọn.

Có phải là thực hành xấu?

Có thể là có, nhưng nó phụ thuộc vào chính xác những gì bạn đang làm, và cũng phụ thuộc vào ngôn ngữ lập trình bạn đang sử dụng.

Trong ví dụ của bạn, chúng ta có thể suy ra rằng có một tập hợp nhỏ các giá trị cố định mà một hướng có thể thực hiện. Trong trường hợp này, không có lợi thế trong việc sử dụng một chuỗi và một số nhược điểm nhất định. Ví dụ này:

• Một enumloại sẽ thích hợp hơn, nếu ngôn ngữ lập trình của bạn hỗ trợ điều này.
• Mặt khác, các hằng số nguyên được đặt tên là con đường để đi, với một định nghĩa duy nhất cho các hằng số.

Nhưng câu trả lời có thể khác nếu tập hợp các giá trị có thể thay đổi động hoặc cần phát triển theo thời gian. Trong hầu hết các ngôn ngữ, việc thay đổi một enumloại (ví dụ để thêm hoặc xóa giá trị) yêu cầu biên dịch lại hoàn chỉnh. (Ví dụ: loại bỏ một giá trị enum trong Java sẽ phá vỡ tính tương thích nhị phân.) Và áp dụng tương tự cho các hằng số nguyên được đặt tên.

Trong các tình huống như thế này, một giải pháp khác có thể được yêu cầu và sử dụng chuỗi là một trong các tùy chọn. (Và bạn có thể kết hợp chuỗi ký tự chuỗi và hằng số được đặt tên ... nếu kịch bản đòi hỏi một lõi cố định với các phần mở rộng động.)

Nếu bạn đang triển khai trong Java, character.direction == "left"thì thực tế không tốt vì một lý do khác. Bạn không nên sử dụng ==để so sánh các chuỗi, bởi vì nó đang kiểm tra danh tính đối tượng hơn là bình đẳng chuỗi. (Nó sẽ hoạt động nếu bạn có thể đảm bảo rằng tất cả các phiên bản của "left"chuỗi đã được thực tập, nhưng điều đó đòi hỏi phải phân tích toàn bộ ứng dụng.)

Theo đề xuất, bạn nên sử dụng Enums. Tuy nhiên, chỉ sử dụng Enum để thay thế cho Strigns là không đủ IMHO. Trong ví dụ particualr của bạn, vận tốc của người chơi thực sự phải là một vectơ bằng phương pháp vật lý:

class Vector {
  final double x;
  final double y;
}

Vectơ này sau đó nên được sử dụng để cập nhật vị trí người chơi mỗi tích tắc.

Sau đó, bạn có thể kết hợp điều đó với một enum để dễ đọc hơn:

enum Direction {
  UP(new Vector(0, 1)),
  RIGHT(new Vector(1, 0)),
  DOWN(new Vector(0, -1)),
  LEFT(new Vector(-1, 0));

  private Vector direction;

  Direction(Vector v) {
    this.direction = v;
  }

  public Vector getVector() { return this.direction; }
}