đang Redundant xây dựng

Question

Mã egregiously dư thừa nhất xây dựng Tôi thường thấy bao gồm việc sử dụng mã chuỗi

if (condition) 
    return true; 
else 
    return false; 

thay vì chỉ đơn giản là viết

return (condition); 

Tôi đã nhìn thấy lỗi người mới bắt đầu này trong tất cả các loại ngôn ngữ : từ Pascal và C sang PHP và Java. Bạn sẽ gắn cờ các cấu trúc khác như thế nào trong đánh giá mã?

Answer 1

Trở vô ích ở cuối:

// stuff 
    return; 
} 

Answer 2

Sử dụng ToString trên một chuỗi

Answer 3

if (condition == true) 
{ 
    ... 
} 

thay vì

if (condition) 
{ 
    ... 
} 

Edit:

hoặc thậm chí tồi tệ hơn và biến xung quanh kiểm tra điều kiện:

if (condition == false) 
{ 
    ... 
} 

mà có thể dễ dàng đọc như

if (condition) then ... 

Answer 4

if (foo == true) 
{ 
    do stuff 
} 

Tôi luôn nói với các nhà phát triển nào đó mà nó phải được

if ((foo == true) == true) 
{ 
    do stuff 
} 

nhưng anh chưa nhận được gợi ý.

Answer 5

void myfunction() { 
    if(condition) { 
    // Do some stuff 
    if(othercond) { 
     // Do more stuff 
    } 
    } 
} 

thay vì

void myfunction() { 
    if(!condition) 
    return; 

    // Do some stuff 

    if(!othercond) 
    return; 

    // Do more stuff 
} 

Answer 6

Tôi đã từng có một anh chàng liên tục đã làm điều này:

bool a; 
bool b; 
... 
if (a == true) 
    b = true; 
else 
    b = false; 

Answer 7

Đưa một tuyên bố exit như tuyên bố đầu tiên trong một chức năng để vô hiệu hóa việc thực hiện các chức năng đó , thay vì một trong các tùy chọn sau:

• hoàn toàn loại bỏ các chức năng
• Bình luận các chức năng cơ thể
• Giữ chức năng nhưng xóa tất cả các mã

Sử dụng exit như tuyên bố đầu tiên làm cho nó rất khó để phát hiện, bạn có thể dễ dàng đọc qua nó.

Answer 8

Tuyên bố riêng rẽ với nhiệm vụ bằng các ngôn ngữ khác ngoài C:

int foo; 
foo = GetFoo(); 

Answer 9

Sử dụng ý kiến ​​thay vì kiểm soát nguồn:
-Commenting ra hoặc đổi tên chức năng thay vì xóa chúng và tin tưởng rằng kiểm soát nguồn có thể nhận được chúng trở lại cho nếu cần.
-Thêm nhận xét như "Thay đổi RWF" thay vì chỉ thực hiện thay đổi và cho phép kiểm soát nguồn gán đổ lỗi.

Answer 10

Somewhere I đã phát hiện điều này, mà tôi tìm thấy là đỉnh cao của sự dư thừa boolean:

return (test == 1)? ((test == 0) ? 0 : 1) : ((test == 0) ? 0 : 1); 

:-)

Answer 11

Tôi thường chạy vào như sau:

function foo() { 
    if (something) { 
     return; 
    } else { 
     do_something(); 
    } 
} 

Nhưng nó không giúp nói với họ rằng người khác là vô ích ở đây. Nó phải là một trong hai

function foo() { 
    if (something) { 
     return; 
    } 
    do_something(); 
} 

hoặc - tùy thuộc vào độ dài của kiểm tra được thực hiện trước khi do_something():

function foo() { 
    if (!something) { 
     do_something(); 
    } 
} 

Answer 12

Sử dụng một mảng khi bạn muốn hành vi thiết lập. Bạn cần phải kiểm tra mọi thứ để đảm bảo rằng nó không nằm trong mảng trước khi bạn chèn nó, điều này làm cho mã của bạn dài hơn và chậm hơn.

Answer 13

Mã dự phòng không phải là lỗi. Nhưng nếu bạn thực sự cố gắng lưu mọi ký tự

return (condition); 

cũng thừa. Bạn có thể viết:

return condition; 

Answer 14

Sợ null (điều này cũng có thể dẫn đến những vấn đề nghiêm trọng):

if (name != null) 
    person.Name = name; 

Redundant nếu nhân (không sử dụng khác):

kiểm tra

if (!IsPostback) 
{ 
    // do something 
} 
if (IsPostback) 
{ 
    // do something else 
} 

Redundant (Split không bao giờ trả về null):

string[] words = sentence.Split(' '); 
if (words != null) 

Thông tin thêm về kiểm tra (kiểm tra thứ hai là không cần thiết nếu bạn đang đi để lặp)

if (myArray != null && myArray.Length > 0) 
    foreach (string s in myArray) 

Và yêu thích của tôi cho ASP.NET: rải rác DataBind s trên tất cả các mã để làm cho trang render.

Answer 15

Sao chép dán dự phòng:

if (x > 0) 
{ 
    // a lot of code to calculate z 
    y = x + z; 
} 
else 
{ 
    // a lot of code to calculate z 
    y = x - z; 
} 

thay vì

if (x > 0) 
    y = x + CalcZ(x); 
else 
    y = x - CalcZ(x); 

hoặc thậm chí tốt hơn (hoặc khó hiểu hơn)

y = x + (x > 0 ? 1 : -1) * CalcZ(x) 

Answer 16

Phân bổ các yếu tố trên heap thay vì ngăn xếp.

{ 
    char buff = malloc(1024); 
    /* ... */ 
    free(buff); 
} 

thay vì

{ 
    char buff[1024]; 
    /* ... */ 
} 

hoặc

{  
    struct foo *x = (struct foo *)malloc(sizeof(struct foo)); 
    x->a = ...; 
    bar(x); 
    free(x); 
} 

thay vì

{ 
    struct foo x; 
    x.a = ...; 
    bar(&x); 
} 

Answer 17

Từ nhận xét mã cơn ác mộng .....

char s[100]; 

Tiếp theo

memset(s,0,100); 

Tiếp theo

s[strlen(s)] = 0; 

với rất nhiều khó chịu

if (strcmp(s, "1") == 0) 

rải rác về mã.

Answer 18

Redundant ToString() lời gọi:

const int foo = 5; 
Console.WriteLine("Number of Items: " + foo.ToString()); 

không cần thiết chuỗi định dạng:

const int foo = 5; 
Console.WriteLine("Number of Items: {0}", foo); 

Answer 19

Phổ biến nhất dự phòng đang xây dựng tôi thấy là mã mà không bao giờ được gọi từ bất cứ nơi nào trong chương trình.

Khác là các mẫu thiết kế được sử dụng khi không có điểm trong việc sử dụng chúng. Ví dụ, viết "new BobFactory(). CreateBob()" ở khắp mọi nơi, thay vì chỉ viết "new Bob()".

Xóa mã không sử dụng và không cần thiết có thể cải thiện đáng kể chất lượng của hệ thống và khả năng duy trì của hệ thống. Những lợi ích thường gây sửng sốt cho các nhóm chưa bao giờ xem xét xóa mã không cần thiết khỏi hệ thống của họ. Tôi đã từng thực hiện đánh giá mã bằng cách ngồi với một nhóm và xóa hơn một nửa mã trong dự án của họ mà không thay đổi chức năng của hệ thống của họ. Tôi nghĩ họ sẽ bị xúc phạm nhưng họ thường hỏi tôi về lời khuyên thiết kế và phản hồi sau đó.