Tôi có nên sử dụng lại các biến?

Tôi có nên sử dụng lại các biến?

Tôi biết rằng nhiều thực tiễn tốt nhất nói rằng bạn không nên làm điều đó, tuy nhiên, sau đó, khi nhà phát triển khác nhau đang gỡ lỗi mã và có 3 biến giống nhau và sự khác biệt duy nhất là chúng được tạo ở những nơi khác nhau trong mã, anh ta có thể bối rối. Kiểm thử đơn vị là một ví dụ tuyệt vời về điều này.

Tuy nhiên, tôi làm biết rằng thực hành tốt nhất là phần lớn thời gian chống lại nó. Ví dụ, họ nói không "ghi đè" tham số phương thức.

Các thực tiễn tốt nhất thậm chí còn chống lại việc vô hiệu hóa các biến trước đó (trong Java có Sonar đưa ra cảnh báo khi bạn gán nullcho biến, rằng bạn không cần phải làm điều đó để gọi trình thu gom rác kể từ Java 6. Bạn không thể luôn kiểm soát cảnh báo nào bị tắt; hầu hết thời gian mặc định được bật.)

Vấn đề của bạn chỉ xuất hiện khi các phương thức của bạn dài và đang thực hiện nhiều tác vụ theo một trình tự. Điều này làm cho mã khó hiểu hơn (và do đó duy trì) mỗi se. Việc sử dụng lại các biến thêm vào đầu yếu tố này là một yếu tố rủi ro bổ sung, làm cho mã trở nên khó theo dõi hơn và dễ bị lỗi hơn.

Thực hành tốt nhất của IMO là sử dụng các phương pháp đủ ngắn, chỉ làm một việc, loại bỏ toàn bộ vấn đề.

Tái sử dụng biến trong một phương thức là một dấu hiệu mạnh mẽ cho thấy bạn nên cấu trúc lại / tách nó.

Vì vậy, câu trả lời của tôi là bạn không nên sử dụng lại chúng, bởi vì nếu bạn làm thế thì việc tái cấu trúc nó sẽ khó hơn nhiều.

Nó phụ thuộc.

Một số biến có thể được tạo chính xác cho mục đích giữ một loại dữ liệu nhất định, có thể thay đổi trong quá trình thực thi hàm. Mã trả về đến với tâm trí ở đây, ví dụ:

void my_function() {
    HRESULT errorcode;
    errorcode = ::SomeWindowsApiFunction();
    if (FAILED(errorcode)) { /* handle error */ }
    errorcode = ::AnotherWindowsApiFunction();
    if (FAILED(errorcode)) { /* handle error */ }
}

Tên biến làm cho nó rất rõ ràng những gì biến này được dự định để lưu trữ. Tôi nghĩ rằng các usecase khác như điều này là có thể, trong đó một biến về mặt khái niệm là một thùng chứa được sử dụng hợp lý bởi các trường hợp khác nhau của những thứ rất giống nhau trong quá trình của một hàm.

Tuy nhiên, nói chung, điều này chỉ nên được thực hiện trong những trường hợp hoàn toàn rõ ràng đối với những người đọc mã có thể. Trong mọi trường hợp, ngoại trừ có thể tối ưu hóa cực kỳ không liên quan đến mức độ dễ đọc của mã, nên sử dụng một biến chỉ vì loại phù hợp.

Tất cả điều này về cơ bản xuất phát từ thực tiễn tốt trong việc đặt tên biến. Tên nên nói cho chính họ. Nếu khó có thể đặt mục đích chính xác cho tất cả các lần sử dụng lại trong một tên ngắn, tốt nhất là chỉ sử dụng các biến khác biệt.

Lý do lớn nhất tôi không sử dụng lại các biến (đặc biệt là trong các thử nghiệm đơn vị) là vì nó đưa ra một đường dẫn mã không cần thiết, một đường dẫn khó kiểm tra và gỡ lỗi. Một bài kiểm tra đơn vị tốt phải độc lập với các bài kiểm tra khác và khi bạn sử dụng lại biến cấp độ lớp (ví dụ) trong một bài kiểm tra đơn vị, bạn phải đảm bảo khẳng định trạng thái của chúng trước mỗi bài kiểm tra. Một thử nghiệm đơn vị tốt cũng cô lập các khuyết tật, vì vậy về mặt lý thuyết, mỗi trường hợp thử nghiệm (phương pháp) chỉ nên xác nhận 1 hành vi cho hệ thống được thử nghiệm. Nếu các phương thức kiểm tra của bạn được viết như thế này, hiếm khi có nhu cầu hoặc lợi ích để sử dụng lại một biến mức phương thức. Cuối cùng, trong các ngôn ngữ hỗ trợ các bao đóng và xử lý không đồng bộ, thực sự rất khó để suy luận về những gì đang xảy ra nếu bạn đang sử dụng lại các biến trong suốt một phương thức.

Bạn nên sử dụng các biến khác nhau. Nếu bạn lo lắng rằng đồng nghiệp của bạn sẽ bị nhầm lẫn, hãy đặt cho họ những cái tên thể hiện rõ vai trò của họ.
Sử dụng lại các biến là một nguồn có thể gây nhầm lẫn trong tương lai; tốt hơn để xóa nó ngay bây giờ.
Trong một số trường hợp, tên biến tương tự có thể được sử dụng lại; ví dụ itrong một vòng lặp đếm đơn giản. Trong những trường hợp này, tất nhiên bạn nên đảm bảo rằng các biến nằm trong phạm vi riêng của chúng.

EDIT: Việc sử dụng lại các biến đôi khi là một dấu hiệu cho thấy Nguyên tắc Trách nhiệm Đơn lẻ bị vi phạm. Kiểm tra xem nếu biến được sử dụng lại được sử dụng trong cùng một vai trò. Nếu có, nó hoàn toàn không thể sử dụng lại (mặc dù vẫn có thể có hai biến khác nhau, để hạn chế phạm vi của các biến đã nói). Nếu nó được sử dụng trong các vai trò khác nhau, bạn có vi phạm SRP trên tay.

Có một tình huống trong đó bạn có thể muốn sử dụng lại một biến bất kể lời khuyên tuyệt vời được đưa ra bởi các câu trả lời khác: khi trình biên dịch của bạn cần một bàn tay trợ giúp.

Trong một số trường hợp, trình biên dịch của bạn có thể không đủ thông minh để nhận ra rằng một biến được cấp phát đăng ký nhất định không còn được sử dụng bởi phần tiếp theo của mã. Do đó, nó sẽ không sử dụng lại đăng ký miễn phí về mặt lý thuyết cho các biến tiếp theo và mã được tạo có thể là tối ưu phụ.

Lưu ý rằng tôi không biết bất kỳ trình biên dịch chính hiện tại nào không nắm bắt được tình huống này một cách chính xác, vì vậy, sẽ không bao giờ, làm điều này trừ khi bạn biết rằng trình biên dịch của bạn đang tạo mã tối ưu phụ. Nếu bạn đang biên dịch cho các hệ thống nhúng đặc biệt với trình biên dịch tùy chỉnh, bạn vẫn có thể gặp phải sự cố này.

Tôi sẽ nói không.

Hãy suy nghĩ về kịch bản này: chương trình của bạn bị sập và bạn cần tìm ra điều gì đã xảy ra bằng cách kiểm tra một bãi chứa cốt lõi ... bạn có thể thấy sự khác biệt không? ;-)

Không, bạn không cải thiện mã mà máy đang chạy (... mã lắp ráp). Để lại việc sử dụng bất cứ bộ nhớ nào mà trình biên dịch sử dụng cho biến cho trình biên dịch. Thông thường, nó sẽ là một đăng ký, và sử dụng lại không mua gì cho bạn. Tập trung vào việc làm cho mã dễ đọc hơn.

Nó phụ thuộc. Trong một ngôn ngữ động, trong đó kiểu nằm trên các giá trị thay vì các biến thì nếu tôi có một đối số được đặt tên tốt cho một hàm, ví dụ, là một chuỗi. Tôi thay đổi thuật toán có nghĩa là nó luôn được sử dụng sau khi được hiểu là số nguyên, sau đó, như một bản nháp đầu tiên tôi có thể làm tương đương với

Scrote = int (Scrote)

Nhưng cuối cùng tôi sẽ tìm cách thay đổi loại được gửi đến hàm và lưu ý sự thay đổi trong loại tham số.

Đầu tiên, hãy nhìn vào môi trường phát triển của bạn. Nếu bạn gặp vấn đề gỡ lỗi vì bạn có năm biến trong các phạm vi khác nhau có tên giống nhau và trình gỡ lỗi không hiển thị cho bạn biến nào trong số các biến này là biến bạn cần, thì đừng sử dụng năm biến có cùng tên. Có hai cách để đạt được điều này: Sử dụng một biến hoặc sử dụng năm tên khác nhau.

Trình gỡ lỗi của bạn cũng có thể gây khó khăn khi gỡ lỗi một hàm có nhiều biến. Nếu một hàm có 20 biến khó gỡ lỗi hơn một biến có 16 biến, thì bạn có thể xem xét thay thế 5 biến bằng một biến. ("Có thể cân nhắc" không giống như "nên luôn luôn").

Sẽ có một biến được sử dụng ở nhiều nơi miễn là biến đó luôn có cùng mục đích. Ví dụ: nếu mười lệnh gọi hàm trả về mã lỗi, được xử lý ngay lập tức cho mỗi cuộc gọi, hãy sử dụng một biến chứ không phải 10. Nhưng không sử dụng cùng một biến cho những thứ hoàn toàn khác nhau. Giống như sử dụng "tên" cho tên khách hàng và 10 dòng sau sử dụng cùng một biến cho tên công ty, điều đó thật tệ và sẽ khiến bạn gặp rắc rối. Tồi tệ hơn, sử dụng "tên khách hàng" cho tên khách hàng và 10 dòng sau sử dụng cùng một biến "tên khách hàng" cho tên công ty.

Điều quan trọng, không có gì là một quy tắc sắt. Cái gì cũng có ưu điểm và nhược điểm. Nếu "thực tiễn tốt nhất" gợi ý một điều, và bạn có lý do để nói rằng đó là một ý tưởng tồi trong tình huống cụ thể của bạn, thì đừng làm điều đó.

Đầu tiên, hãy nhìn vào môi trường phát triển của bạn. Nếu bạn gặp vấn đề gỡ lỗi vì bạn có năm biến trong các phạm vi khác nhau có tên giống nhau và trình gỡ lỗi không hiển thị cho bạn biến nào trong số các biến này là biến bạn cần, thì đừng sử dụng năm biến có cùng tên. Có hai cách để đạt được điều này: Sử dụng một biến hoặc sử dụng năm tên khác nhau.

Trình gỡ lỗi của bạn cũng có thể gây khó khăn khi gỡ lỗi một hàm có nhiều biến. Nếu một hàm có 20 biến khó gỡ lỗi hơn một biến có 16 biến, thì bạn có thể xem xét thay thế 5 biến bằng một biến. ("Có thể cân nhắc" không giống như "nên luôn luôn").

Sẽ có một biến được sử dụng ở nhiều nơi miễn là biến đó luôn có cùng mục đích. Ví dụ: nếu mười lệnh gọi hàm trả về mã lỗi, được xử lý ngay lập tức cho mỗi cuộc gọi, hãy sử dụng một biến và không 10. Có một vấn đề với điều này: Thông thường, trình biên dịch sẽ cho bạn biết khi bạn sử dụng biến chưa được khởi tạo. Nhưng ở đây, nếu bạn đọc mã lỗi sau cuộc gọi 6, nhưng biến không thực sự được thay đổi sau cuộc gọi 5, bạn sẽ nhận được dữ liệu vô dụng mà không cần trình biên dịch cảnh báo cho bạn. Bởi vì biến đã được khởi tạo, với dữ liệu hiện vô dụng.

Điều quan trọng, không có gì là một quy tắc sắt. Cái gì cũng có ưu điểm và nhược điểm. Nếu "thực tiễn tốt nhất" gợi ý một điều, và bạn có lý do để nói rằng đó là một ý tưởng tồi trong tình huống cụ thể của bạn, thì đừng làm điều đó.

Sử dụng các biến toàn cục khi bạn cần duy trì trạng thái hoặc lưu trữ hằng. Bằng cách sử dụng lại các biến, bạn chỉ sử dụng lại tên trỏ đến một vị trí trong bộ nhớ. Tên mô tả về khởi tạo chỉ có thể giúp bạn hiểu rõ hơn (Giả sử Java và không có OCD nguyên thủy).

Trừ khi bạn vào mã obfuscation bằng tay hoặc gây khó chịu cho các nhà phát triển khác.