Tại sao ReSharper muốn sử dụng 'var' cho mọi thứ?

Tôi mới bắt đầu sử dụng ReSharper với Visual Studio (sau nhiều khuyến nghị về SO). Để dùng thử, tôi đã mở một dự án ASP.NET MVC gần đây. Một trong những điều đầu tiên và thường xuyên nhất mà tôi nhận thấy nó gợi ý là thay đổi hầu hết / tất cả các tuyên bố rõ ràng của tôi để varthay thế. Ví dụ:

//From This:
MyObject foo = DB.MyObjects.SingleOrDefault(w => w.Id == 1);
//To This:
var foo = DB.MyObjects.SingleOrDefault(w => w.Id == 1);

và như vậy, thậm chí với các loại đơn giản như int, boolvv

Tại sao điều này được khuyến khích? Tôi không đến từ nền tảng khoa học máy tính hoặc .NET, đã "rơi vào" sự phát triển .NET gần đây, vì vậy tôi thực sự muốn hiểu những gì đang diễn ra và liệu nó có lợi hay không.

Một lý do là cải thiện khả năng đọc. Cái nào tốt hơn?

Dictionary<int, MyLongNamedObject> dictionary = new Dictionary<int, MyLongNamedObject>();

hoặc là

var dictionary = new Dictionary<int, MyLongNamedObject>();

Những gì ReSharper gợi ý rõ ràng là quá lạm dụng từ khóa var. Bạn có thể sử dụng nó trong đó loại rõ ràng:

var obj = new SomeObject();

Nếu loại không rõ ràng, bạn nên viết nó ra:

SomeObject obj = DB.SomeClass.GetObject(42);

Cá nhân tôi thích tắt đề nghị này. Sử dụng varthường có thể cải thiện khả năng đọc; nhưng như bạn đã đề cập, đôi khi nó làm giảm nó (với các loại đơn giản hoặc khi loại kết quả bị che khuất ).

Tôi thích chọn khi tôi sử dụng varvà khi tôi không. Nhưng một lần nữa, đó chỉ là tôi.

varcó thể tăng khả năng đọc mã trong khi giảm mức độ hiểu mã ngay lập tức. Tương tự, nó có thể làm giảm khả năng đọc mã cho các tình huống khác. Đôi khi việc sử dụng nó là trung tính. Các biện pháp dễ đọc để hiểu không tỷ lệ thuận mà phụ thuộc vào tình huống. Đôi khi cả hai được tăng hoặc giảm cùng nhau.

Yếu tố là những gì varđang được áp dụng và mức độ mục tiêu hỗ trợ việc loại bỏ ngay lập tức loại dữ liệu của nó cho người đọc hoặc nếu thông tin loại của nó là cần thiết để hiểu được phần chương trình trong tay.

Ví dụ, việc đặt tên xấu có thể dẫn đến vargiảm khả năng hiểu mã. Đây không phải varlà lỗi của:

var value1 = GetNotObviousValue(); //What's the data type? 
//vs. 
var value2 = Math.Abs(-3); // Obviously a numeric data type. 

Đôi khi nó không có ý nghĩa để sử dụng varcho các loại dữ liệu đơn giản khi mã dễ đọc hơn:

var num = GetNumber(); // But what type of number?
// vs. 
double num = GetNumber(); // I see, it's a double type. 

Đôi khi varcó thể hữu ích để ẩn thông tin loại dữ liệu mà bạn không nhất thiết phải quan tâm để thấy sự phức tạp của:

    IEnumerable<KeyValuePair<string,List<Dictionary<int,bool>>>> q = from t in d where t.Key == null select t; // OMG! 
    //vs. 
    var q = from t in d where t.Key == null select t;

    // I simply want the first string, so the last version seems fine.  
    q.First().Key; 

Bạn phải sử dụng varkhi có một loại ẩn danh vì không có tên loại để gọi nó bằng cách:

var o = new { Num=3, Name="" };

Khi bạn có Visual Studio Intellisense cung cấp thông tin loại bất chấp var, sau đó bạn cần ít dựa vào sự hiểu biết của mình thông qua việc đọc mã nghiêm ngặt mà không cần trợ giúp. Có lẽ khôn ngoan khi cho rằng không phải ai cũng có thể có hoặc sử dụng Intellisense.

Tóm lại dựa trên các ví dụ trên, tôi đề nghị áp dụng carte blush varkhông phải là một ý tưởng hay vì hầu hết mọi thứ đều được thực hiện tốt nhất trong chừng mực và dựa trên hoàn cảnh trong tay như được hiển thị ở đây.

Tại sao Resharper sử dụng tất cả trên mặc định? Tôi đề nghị cho dễ, bởi vì nó không thể phân tích các sắc thái của tình huống để quyết định khi nào tốt nhất không sử dụng nó.

Trong ReSharper (8.02, nhưng có thể là các phiên bản khác), tùy chọn "Sử dụng khai báo biến cục bộ được gõ ngầm" có thể được điều chỉnh theo sở thích của bạn , bất cứ điều gì có thể, bằng cách trước tiên mở menu tùy chọn cho ReSharper:

Sau đó, trong phần "Kiểm tra mã" bằng cách điều chỉnh "Mức độ kiểm tra" của ngôn ngữ bạn đã chọn, trong trường hợp của tôi c #:

Như bạn có thể thấy, có các tùy chọn để điều chỉnh tất cả các đề xuất mà ReSharper đưa ra. Hy vọng điều này sẽ giúp những người như tôi đã có chiến lược sử dụng 'var' và chỉ muốn ReSharper tôn trọng nó :)

Tôi ngạc nhiên khi không ai đề cập rằng việc thay đổi loại đối tượng được khởi tạo cũng dễ dàng hơn, bởi vì

AVeryLongTypeName myVariable = new AVeryLongTypeName( arguments );

là một hình thức lặp lại . Nếu tôi muốn thay đổi AVeryLongTypeNamethành một trong các lớp dẫn xuất của nó, tôi chỉ cần thay đổi điều này một lần khi sử dụng varvà vẫn có thể truy cập các phương thức của các lớp con.

Bên cạnh đó, khả năng đọc được cải thiện là một điểm quan trọng, nhưng như những người khác nói, var không nên bị lạm dụng, vì vậy tôi nghĩ tắt gợi ý trong Resharper là hoàn toàn ổn.

'var' là về việc rõ ràng

Cuộc tranh luận chính về việc có nên sử dụng vartừ khóa hay không là về mức độ dễ đọc của mã đối với bạn và các nhà phát triển khác.

Giống như bạn đang viết một câu chuyện, không có câu trả lời đúng. Nhưng hãy xem xét một số ví dụ về điều này bằng tiếng Anh.

Jake nói xin chào với Bill. Anh không thích anh nên anh quay lại và đi con đường khác.

Ai đã đi con đường khác? Jake hay Bill? Trong trường hợp này, sử dụng tên "Jake" và "Bill" giống như sử dụng tên loại. Và "Anh ấy" và "anh ấy" giống như sử dụng vartừ khóa. Trong trường hợp này nó có thể giúp được cụ thể hơn. Ví dụ sau đây rõ ràng hơn nhiều.

Jake nói xin chào với Bill. Jake không thích Bill nên anh quay lại và đi theo con đường khác.

Trong trường hợp này được cụ thể hơn làm cho câu rõ ràng hơn. Nhưng điều đó không phải lúc nào cũng đúng. Trong một số trường hợp cụ thể làm cho nó khó đọc hơn.

Bill thích sách, vì vậy Bill đã đến thư viện và Bill lấy ra một cuốn sách mà Bill luôn thích.

Trong trường hợp này, việc đọc câu sẽ dễ dàng hơn nếu chúng ta sử dụng "anh ấy" và trong một số trường hợp bỏ tên của anh ấy lại với nhau, tương đương với việc sử dụng vartừ khóa.

Bill thích sách, vì vậy anh đến thư viện và lấy ra một cuốn sách mà anh luôn thích.

Những ví dụ này bao gồm ý chính, nhưng chúng không nói lên toàn bộ câu chuyện. Trong những ví dụ đó chỉ có một cách để nói đến người đó. Hoặc bằng cách sử dụng tên của họ hoặc bằng cách sử dụng một thuật ngữ chung chung hơn như "anh ấy" và "anh ấy".

Trong trường hợp mã, chúng tôi có 3 cách để giúp thêm rõ ràng. Loại, tên biến và gán. Lấy dòng mã này làm ví dụ:

Person p = GetPerson();

Câu hỏi bây giờ là có đủ thông tin trong dòng mã đó để giúp bạn tìm hiểu chuyện gì đang xảy ra không?

Còn dòng mã sau đây thì sao? Bạn vẫn sẽ biết những gì pcó nghĩa là trong trường hợp này:

var p = GetPerson();

Làm thế nào bây giờ:

var p = Get();

Hay bây giờ:

var person = Get();

Hoặc cái này:

var t = GetPerson();

Hoặc này:

var u = Person.Get();

Việc từ khóa có varhoạt động trong một kịch bản nhất định hay không phụ thuộc rất nhiều vào ngữ cảnh của mã, như cách các biến, lớp và phương thức được đặt tên. Nó cũng phụ thuộc vào độ phức tạp của mã và phần còn lại của mã xung quanh nó.

Cá nhân tôi thích sử dụng vartừ khóa toàn diện hơn đối với tôi hầu hết thời gian. Nhưng tôi cũng có xu hướng đặt tên biến của mình theo loại để tôi không thực sự mất bất kỳ thông tin nào.

Điều đó nói rằng đôi khi tùy thuộc vào bối cảnh tôi đưa ra ngoại lệ, đó là bản chất của bất cứ điều gì phức tạp và phần mềm không là gì nếu không nói là phức tạp.

Tôi không thích điều này là tốt.

Tôi không muốn điều này biến thành một cuộc tranh luận về việc sử dụng var, nó có công dụng của nó nhưng không nên được sử dụng ở mọi nơi.

Điều quan trọng cần nhớ là ReSharper được cấu hình theo bất kỳ tiêu chuẩn mã hóa nào bạn muốn.

Chỉnh sửa: ReSharper và var

Tôi thấy nhiều câu trả lời đúng, nhưng thiếu câu trả lời đầy đủ.

Đúng là ReSharper lạm vardụng theo mặc định. Tôi nghĩ rằng hầu hết mọi người sẽ đồng ý với điều đó. Nó cũng dễ đọc hơn khi varđược sử dụng và loại rõ ràng như khi bạn sử dụng một newcâu lệnh. Tôi thấy một bài viết cho thấy cách cập nhật mức độ nghiêm trọng kiểm tra để chỉ hiển thị gợi ý cho việc sử dụng var.

Tôi đã cố gắng bình luận trên các bài đăng khác trước để thêm vị trí đặt những bài này nhưng không có tiếng tăm. Rõ ràng, tôi cũng không có tiếng để đăng ảnh chụp màn hình của tôi về các cài đặt.

Tôi sẽ giải thích làm thế nào để đạt được điều đó.

Trong Visual Studio> Menu chính> Chia sẻ lại> Tùy chọn> Chỉnh sửa mã> C #> Kiểu mã> Sử dụng Var trong khai báo

• Đối với loại tích hợp Sử dụng loại rõ ràng
• Đối với các loại đơn giản Sử dụng 'var' khi hiển nhiên
• Ở nơi khác sử dụng'var '

Tài liệu trợ giúp của ReSharper: Kiểu cú pháp mã: Nhập ngầm định / gõ ngầm ('var' Keyword) - Định cấu hình tùy chọn sử dụng từ khóa 'var'

Quy tắc của tôi là thế này:

• Bạn đang tuyên bố một loại nguyên thủy (tức là byte, char, string, int[], double?, decimal, vv)? -> Sử dụng loại:

  string myStr = "foo";
  int[] myIntArray = [123, 456, 789];
  double? myDouble = 123.3;

• Bạn đang tuyên bố một loại phức tạp (ví dụ List<T>, Dictionary<T, T>, MyObj)? -> Sử dụng var:

  var myList = List<string>();
  var myDictionary = Dictionary<string, string>();
  var myObjInstance = new MyObj();

Tôi chỉ muốn chỉ ra rằng việc sử dụng "var" được khuyến nghị trong các Quy ước mã hóa C #

khi loại biến rõ ràng từ phía bên phải của bài tập hoặc khi loại chính xác không quan trọng

Vì vậy, đó có thể là lý do tại sao mẹo được bật theo mặc định trong ReSharper. Họ cũng cung cấp một số trường hợp không cải thiện khả năng đọc ngay bên dưới trong cùng một tài liệu.

ReSharper khuyến nghị varvì nó có xu hướng giải phóng việc tạo đối tượng.

So sánh hai ví dụ sau:

StringBuilder bld = new StringBuilder();

var bld = new StringBuilder();

Nó chỉ là một tốc ký được cho là dễ đọc hơn.

Tôi nghĩ sẽ ổn khi bạn tạo đối tượng mới một cách rõ ràng bằng "mới". Trong ví dụ của bạn, tuy nhiên, có thể không rõ ràng nếu các lớp không được đặt tên đúng.

BTW, ReSharper rút ra sự khác biệt giữa 'bạn có thể muốn áp dụng đề xuất này cho mã của mình' và 'mã của bạn bị hỏng, muốn tôi sửa nó?'. Các vartừ khóa là trong hạng mục đề nghị, cùng với những câu như "nghịch nếu để giảm làm tổ"; bạn không cần phải theo nó

Bạn có thể định cấu hình mức độ khó chịu của từng cảnh báo thông qua hộp thoại Tùy chọn hoặc trực tiếp thông qua menu bật lên cho cảnh báo đó. Bạn có thể hạ cấp những thứ như varđề xuất để chúng ít nổi bật hơn hoặc bạn có thể nâng cấp những thứ như cảnh báo 'sử dụng phương thức mở rộng' để nó hiển thị như một lỗi thực tế.

Các vartính năng của Net 3.0 chỉ là suy luận kiểu , đó là loại an toàn và thường làm cho mã của bạn dễ đọc hơn. Nhưng bạn không cần phải và có thể tắt đề xuất đó trong việc chia sẻ lại nếu bạn muốn.

Var thật tuyệt vời! Tôi đã bắt gặp nhiều nhà phát triển có ấn tượng varbị ràng buộc với một loại động, không phải vậy. Nó vẫn được gõ tĩnh, nó chỉ do trình biên dịch quyết định.

Dưới đây là một số tích cực tuyệt vời của việc sử dụng var

Việc gõ var ít hơn và dễ đọc hơn, ví dụ như

Dictionary<int,IList<string>> postcodes = new Dictionary<int,IList<string>>()Yuk.

var postcodes = new Dictionary<int,IList<string>>()\ o / \ o /

Tên biến mô tả khác - một tên khó hiểu nhưng tôi nghĩ điều quan trọng là phải để bản chất lỏng của sự vartỏa sáng ở đây. Như varlà một chút mơ hồ, nó thực sự khuyến khích một tên biến giải thích hơn là để cho loại nói cho chính nó.

Thay đổi mã ít hơn - nếu kiểu trả về của một cuộc gọi phương thức thay đổi. Bạn chỉ phải thay đổi cuộc gọi phương thức, không phải mọi nơi nó được sử dụng.

Các loại ẩn danh - các loại ẩn danh là một khái niệm thực sự mạnh mẽ, đặc biệt là trong các lĩnh vực như tài nguyên một phần của WebApi . Không có var, chúng không thể được sử dụng.

Tuy nhiên, đôi khi rất hữu ích khi khai báo các loại và tôi thấy điều này hữu ích nhất trong các nguyên thủy hoặc cấu trúc. Chẳng hạn, cá nhân tôi thấy cú pháp này rất hữu ích:

for(var i = 0; i < 10; i++) 
{

}

đấu với

for(int i = 0; i < 10; i++) 
{

}

Tất cả tùy thuộc vào sở thích cá nhân nhưng sử dụng varthực sự sẽ tăng tốc độ phát triển của bạn và mở khóa cả một thế giới tốt đẹp loại ẩn danh.

Không có sự khác biệt kỹ thuật, nếu bạn sử dụng var, loại được ngụ ý bởi trình biên dịch. Nếu bạn có một mã như thế này:

var x = 1;

x được ngụ ý là một int và không có giá trị nào khác có thể được gán cho nó.

Từ khóa var rất hữu ích nếu bạn thay đổi loại biến; sau đó bạn chỉ phải thực hiện một thay đổi thay vì hai:

var x = 1; --> var x = "hello";
int x = 1; --> string x = "hello";

Các vartừ khóa được giới thiệu trong C # 3.0 - nó cho phép chúng ta quên đi quy định cụ thể loại của chúng tôi một cách rõ ràng.

Không có sự khác biệt thực sự cho dù bạn sử dụng

MyObject foo = DB.MyObjects.SingleOrDefault(w => w.Id == 1);

hoặc là

var foo = DB.MyObjects.SingleOrDefault(w => w.Id == 1);

ngoại trừ khả năng đọc thuần túy và ít cơ hội cho lỗi.

Nó có vẻ như một ví dụ sáo rỗng, nhưng nói những điều sau đây có thể giúp bạn hiểu:

var myInt = 23;

trả về một intloại, trong khi

var myInt = "23";

trả về một stringloại.

Tham chiếu MSDN

Việc chỉ định một loại đối tượng rõ ràng là bằng cách nào đó dư thừa. Ngay cả khi được dịch bằng tiếng Anh, nó nghe có vẻ dư thừa: "đặt một đối tượng loại X vào một biến loại X" so với "Đặt một đối tượng loại X vào một biến".

Tuy nhiên, sử dụng 'var' có những hạn chế của nó . Nó ngăn chặn việc sử dụng đa hình dưới đây là vẻ đẹp thuần túy :

Giả sử một con chó kéo dài động vật; Cat mở rộng phân cấp lớp Động vật:

Animal x = new Dog();
DoStuffWithDog(x as Dog);

x = new Cat();
DoStuffWithCat(x as Cat);

void DoStuffWithDog(Dog d){}
void DoStuffWithCat(Cat c){}

Cùng một mã, với x được khai báo với 'var' sẽ không biên dịch .

var x = new Dog(); // from this point, x is a Dog
DoStuffWithDog(x as Dog);

x = new Cat(); // cannot assign a Cat instance to a Dog
DoStuffWithCat(x as Cat);

void DoStuffWithDog(Dog d){}
void DoStuffWithCat(Cat c){}

Dù sao, trở lại câu hỏi ban đầu, tôi không sử dụng Resharper, nhưng tôi cho rằng điều đó đủ thông minh để phát hiện khi nào không sử dụng 'var'. :-)

Theo quan điểm của tôi, var chỉ nên được sử dụng khi nó rõ ràng ngay lập tức loại là gì khi xác định giá trị của biến.

Thí dụ:

var s = "string value";

Rõ ràng đó slà một string.

Tôi tin rằng nó cũng thích hợp khi tên loại biến rất phức tạp.

Thí dụ:

Dictionary<SomeCustomKeyType, Dictionary<AnotherCustomKeyType, List<int>>> dict = new Dictionary<SomeCustomKeyType, Dictionary<AnotherCustomKeyType, List<int>>>();

// This is a little easier to read than the above statement
var dict = new Dictionary<SomeCustomKeyType, Dictionary<AnotherCustomKeyType, List<int>>>();

Ngoài những tình huống này, tôi không thấy GAIN nào được thực hiện bằng cách sử dụng var , nhưng tôi có thể nghĩ về một số tình huống có thể gây bất lợi:

Ví dụ, một loại dùng một lần có giá trị biến bên phải không hiển thị rõ ràng loại đó. Việc vứt bỏ IDis Dùng có thể dễ dàng bị lãng quên

Thí dụ:

// returns some file writer
var wr = GetWriter();

wr.add("stuff");
wr.add("more stuff");

//...
//...

// Now `wr` needs to be disposed, 
// but there is no clear indication of the type of `wr`,
// so it will be easily overlooked by code writer and code reviewer.

'var' thêm một loại phần tử "động" vào mã của bạn (mặc dù mã tất nhiên vẫn được nhập đúng). Tôi khuyên bạn không nên sử dụng nó trong trường hợp loại không rõ ràng. Xem xét ví dụ này:

var bar = GetTheObjectFromDatabase();
bar.DoSomething();

ClassA {
  void DoSomething() {
  //does something
  }
}

ClassB {
  void DoSomething() {
  //does something entirely different
  }
}

Nếu loại trả về của GetTheObjectFromDatabase () được thay đổi từ Loại A thành B, chúng tôi sẽ không nhận thấy, vì cả hai Lớp đều triển khai DoS Something (). Mã, tuy nhiên, bây giờ thực sự có thể làm một cái gì đó hoàn toàn khác.

Điều này có thể tinh tế như cả việc viết các nội dung khác nhau vào nhật ký, vì vậy bạn có thể không nhận thấy đơn vị quá muộn.

Việc sử dụng var sau đây sẽ luôn ổn:

var abc = new Something();

Đối với những người không thích sử dụng "var" liên tục, bạn cũng có thể ngăn ReSharper mặc định thành var khi thực hiện "giới thiệu biến". Đây là điều khiến tôi thất vọng trong một thời gian dài, nó luôn được mặc định là var và tôi đã thay đổi nó mỗi lần.

Các cài đặt này nằm trong Chỉnh sửa mã> C #> Kiểu mã

Không có sự khác biệt về kỹ thuật (như eWolf đã chỉ ra). Bạn có thể sử dụng cái này hoặc cái kia, mã CLR được tạo sẽ trông giống nhau.

Theo tôi lợi ích chính là điều này có xu hướng buộc bạn phải sử dụng cách đặt tên biến tốt hơn. Trong ví dụ của bạn, 'foo' là một lựa chọn khá kém cho một tên biến.

Theo JetBrains (tác giả của ReSharper), họ khuyến khích sử dụng var theo mặc định.

Từ trang web của họ :

Việc sử dụng các biến cục bộ được gõ ngầm (còn được gọi là vartừ khóa) được giới thiệu trong C # 3.0 đã trở nên khá phổ biến vì nó cải thiện khả năng đọc trong nhiều tình huống. Theo mặc định, ReSharper cũng khuyến khích sử dụng vartừ khóa, nhưng các tùy chọn sử dụng của nó có thể cấu hình linh hoạt - ví dụ: bạn có thể chọn sử dụng các loại rõ ràng trong các trường hợp cụ thể hoặc ở mọi nơi và ReSharper sẽ giúp bạn thực thi các tùy chọn của mình.