Có phải mọi số trong mã đều được coi là số ma thuật của người Viking không?

Vì vậy, mỗi số trong mã mà chúng tôi đang gửi đến một phương thức làm đối số có được coi là Số ma thuật không? Đối với tôi, nó không nên. Tôi nghĩ rằng nếu một số nào đó giả sử nó là độ dài tối thiểu của tên người dùng và chúng tôi bắt đầu sử dụng "6" trong mã ... thì vâng, chúng tôi có vấn đề về bảo trì và ở đây "6" là một con số kỳ diệu .... nhưng nếu chúng ta đang gọi một phương thức mà một trong các đối số của nó chấp nhận một số nguyên, ví dụ như là thành viên thứ i của một bộ sưu tập và sau đó chúng ta chuyển "0" cho lệnh gọi phương thức đó, trong trường hợp này tôi không thấy "0" là một phép thuật con số. Bạn nghĩ sao?

Nếu ý nghĩa của con số rất rõ ràng trong ngữ cảnh, tôi không nghĩ đó là vấn đề "số ma thuật".

Ví dụ: Giả sử bạn đang cố gắng lấy chuỗi con của chuỗi, từ đầu đến một số mã thông báo và mã trông như thế này (ngôn ngữ và thư viện tưởng tượng):

s := substring(big_string, 0, findFirstOccurence(SOME_TOKEN, big_string));

Trong bối cảnh này, ý nghĩa của số 0 là đủ rõ ràng. Tôi cho rằng bạn có thể xác định START_OF_SUBSTRINGvà đặt nó thành 0, nhưng trong trường hợp này tôi nghĩ rằng nó sẽ là quá mức cần thiết (mặc dù đó sẽ là cách tiếp cận chính xác nếu bạn biết rằng bắt đầu chuỗi con của bạn có thể không phải là 0, nhưng điều đó phụ thuộc vào chi tiết cụ thể của hoàn cảnh của bạn).

Một ví dụ khác có thể là nếu bạn đang cố xác định xem một số chẵn hay lẻ. Viết:

isEven := x % 2;

không lạ lùng như:

TWO := 2;
isEven := x % TWO;

Kiểm tra số âm như

MINUS_ONE := -1;
isNegativeInt := i <= MINUS_ONE;

cũng cảm thấy kỳ lạ với tôi, tôi muốn nhìn thấy nhiều hơn

isNegativeInt := i <= -1;

bool hasApples = apples > 0;

Rõ ràng là không có nghĩa là vắng mặt. Tôi thấy 0 dễ hiểu hơn một biến có tên "absValue".

for(int i=0; i < arr.length; i++)

Rõ ràng 0 là vị trí bắt đầu. Tôi sẽ bị nhầm lẫn bởi một biến có tên là "FirstPocation". Một biến như vậy sẽ khiến tôi tự hỏi nếu vị trí bắt đầu có thể thay đổi.

Tôi sẽ đề xuất ba yếu tố chính trong việc quyết định liệu một cái gì đó có nên là một tuyên bố liên tục hay không:

1. Là số một cái gì đó là chính xác và chính xác đại diện
2. Có bất kỳ kịch bản hợp lý nào theo đó giá trị sẽ phải thay đổi, nhưng mã sẽ không phải viết lại
3. Ai đó nhìn thấy số có thể nhận ra nó nhanh hơn hoặc nhanh hơn so với người nhìn thấy hằng số có tên

Một cái gì đó như pi có lẽ nên được viết dưới dạng hằng số được đặt tên, chứ không phải là một chữ số, vì một chữ số có khả năng dài dòng không cần thiết, không cần thiết hoặc cả hai. Một cái gì đó như số lượng vị trí trong bộ đệm có thể là một hằng số được đặt tên (mặc dù xem ghi chú bên dưới) để cho phép khả năng mở rộng bộ đệm mà không phải sửa đổi tất cả mã sử dụng nó. Những thứ như các số "4", "28" và "29" trong câu lệnh if ((year % 4)==0) FebruaryDays = 29; else FebruaryDays = 28;có lẽ không nên được đặt tên là hằng số, vì biểu thức gần như chắc chắn dễ đọc hơn if ((year % YearsBetweenLeapYears)==0) FebruaryDays = FebruaryDaysInLeapYear; else FebruaryDays = FebruaryDaysInNonLeapYear;. Lưu ý rằng những người duy trì tiêu chuẩn đã chỉ ra rằng độ dài của tháng 2 năm 2100 trong năm đó sẽ không phù hợp với công thức trên, cản trở xử lý chính xác những ngày như vậy (tức là mã sẽ không bị vấp bởi tràn số nguyên hoặc các vấn đề khác như vậy).

Một cảnh báo quan trọng với quy tắc số 2 là trong một số trường hợp, mã có thể dựa vào các số được mã hóa cứng theo cách không thể được biểu diễn bằng hằng số có tên. Ví dụ, một phương pháp tính toán một sản phẩm chéo của hai vectơ được truyền dưới dạng các tham số rời rạc sẽ chỉ có ý nghĩa khi được sử dụng trên các vectơ ba chiều. Số lượng kích thước yêu cầu không phải là một giá trị có thể được thay đổi một cách có ý nghĩa mà không cần viết lại hoàn toàn thói quen. Ngay cả khi người ta thấy trước một nhu cầu có thể để tính toán sản phẩm chéo của ba vectơ 4 chiều, sử dụng hằng số có tên cho giá trị "3" sẽ giúp làm thỏa mãn nhu cầu đó dễ dàng hơn.

Điều này, như tất cả các nguyên tắc, là một vấn đề mức độ. Nói chung, số chữ trong mã nguồn càng nghi ngờ thì chúng càng lớn. Độ dài tối đa như 10 hoặc địa chỉ bộ nhớ như 0x587FB0 rõ ràng là thực tiễn xấu - gần như chắc chắn rằng sớm muộn bạn sẽ phải lặp lại các giá trị này nhiều lần, tạo ra nguy cơ không tương thích và lỗi tinh vi được đưa ra ở những nơi không có đã thay đổi

0 nằm ở đầu kia của thang đo; nó vẫn còn nghi ngờ nhưng không nhiều lắm Bạn có đang sử dụng 0 làm giá trị sentinel không? Sau đó, có lẽ bạn nên sử dụng hằng số tượng trưng thay vì chỉ vì hằng số có thể giải thích ý nghĩa của nó. Đây có phải là một thỏa thuận văn hóa cực kỳ cố thủ như "0 có nghĩa là hoàn thành thành công" không? Điều đó có thể ổn. Có nghĩa là "mục đầu tiên trong bộ sưu tập"? Điều đó có thể vô hại, nhưng nếu có một phương pháp thay thế như first()tôi có thể thích điều đó hơn.

Mỗi con số không tên không rõ ràng ngay lập tức từ ngữ cảnh là một con số kỳ diệu. Thật là hơi ngớ ngẩn khi định nghĩa những con số có ý nghĩa rõ ràng ngay lập tức từ bối cảnh.

Trong django (khung web python), tôi có thể định nghĩa một số trường cơ sở dữ liệu với một số nguyên như:

firstname = models.CharField(max_length=40)
middlename = models.CharField(max_length=40)
lastname =  models.CharField(max_length=40) 

đó là rõ ràng hơn (và thực hành được đề nghị ) hơn nói

MAX_LENGTH_NAME = 40
...
firstname = models.CharField(max_length=MAX_LENGTH_NAME)
middlename = models.CharField(max_length=MAX_LENGTH_NAME)
lastname =  models.CharField(max_length=MAX_LENGTH_NAME) 

vì tôi không bao giờ cần phải thay đổi độ dài (và luôn có thể so sánh với max_lengthtrường). Nếu tôi cần thay đổi độ dài của trường sau khi triển khai ứng dụng ban đầu, tôi cần thay đổi nó ở chính xác một vị trí trên mỗi trường trong mã django của mình, sau đó viết thêm một di chuyển để thay đổi lược đồ của DB. Nếu tôi cần tham chiếu max_lengthđến một trường được xác định của một loại đối tượng, tôi có thể thực hiện trực tiếp - nếu các trường đó đang xác định một Personlớp, tôi có thể sử dụng Person._meta.get_field('firstname').max_lengthđể có đượcmax_lengthđang được sử dụng (được xác định ở một nơi). Việc 40 cùng được sử dụng cho nhiều lĩnh vực là không liên quan vì tôi có thể muốn thay đổi chúng một cách độc lập. Độ dài của tên đầu tiên không bao giờ nên phụ thuộc vào độ dài của middlename hoặc họ; chúng là những giá trị riêng biệt và có thể thay đổi độc lập.

Thông thường các chỉ số mảng có thể sử dụng các số chưa được đặt tên; giống như nếu tôi có một tệp dữ liệu CSV mà tôi muốn đặt vào một từ điển python, với phần tử đầu tiên trong hàng là từ điển keytôi sẽ viết:

mydict = {}
for row in csv.reader(f):
    mydict[row[0]] = row[1:]

Chắc chắn tôi có thể đặt tên index_column = 0và làm một cái gì đó như:

index_col = 0
mydict = {}
for row in csv.reader(f):
    mydict[row[index_col]] = row[:index_col] + row[index_col+1:]

hoặc tệ hơn là xác định after_index_col = index_col + 1để loại bỏ index_col+1, nhưng điều đó không làm cho mã rõ ràng hơn trong quan điểm của tôi. Ngoài ra, nếu tôi index_colđặt tên, tốt hơn tôi nên làm cho mã hoạt động ngay cả khi cột không bằng 0 (do đó là row[:index_col] +một phần).