ฉันสงสัยเกี่ยวกับการสร้างตัวแปรใหม่หรือวิธีการเรียกสองสามครั้ง อะไรจะดีไปกว่าประสิทธิภาพโดยรวมและการทำความสะอาด GC ลองดูสิ:
public static string GetValue(RegistryKey key, string value)
{
if (key.GetValue(value) == null)
return null;
string newValue = key.GetValue(value).ToString();
if (String.IsNullOrWhiteSpace(newValue))
return null;
return newValue.ToLower();
}
ฉันจะทำให้รหัสนี้ชัดเจนได้อย่างไร
โดยทั่วไป เมื่อคุณจำเป็นต้องใช้ผลลัพธ์ของการเรียกใช้เมธอดหลายครั้ง คุณควรกำหนดให้กับตัวแปรท้องถิ่นก่อน ตัวแปรโลคัลต้องการให้สแต็กเฟรมของเมธอดของคุณมีขนาดใหญ่ขึ้นสองสามไบต์ (8 ไบต์สำหรับตัวแปรอ็อบเจ็กต์บน 64 บิต) และจะถูกล้างโดยอัตโนมัติเมื่อเมธอดส่งคืน ซึ่งไม่มีผลกระทบต่อ GC
ในทางกลับกัน การเรียกใช้เมธอดซ้ำจะต้องใช้ตรรกะทั้งหมดเพื่อดำเนินการอีกครั้ง ซึ่งทำให้เกิดการจัดสรรเฟรมสแต็กที่ต้องการ และอาจสร้างอินสแตนซ์ของอ็อบเจ็กต์ได้ ในกรณีของคุณ RegistryKey.GetValue ทำให้สถานการณ์แย่ลงไปอีก เนื่องจากจำเป็นต้องเข้าถึงรีจิสทรีของ Windows ซึ่งทำให้ช้ากว่าการเข้าถึงตัวแปรในเครื่องหลายลำดับ นอกจากนี้ คุณอาจเผชิญกับสภาวะการแข่งขันที่การโทรทั้งสองส่งคืนค่าต่างกัน
public static string GetValue(RegistryKey key, string name)
{
object value = key.GetValue(name);
if (value == null)
return null;
string valueStr = value.ToString()
if (String.IsNullOrWhiteSpace(valueStr))
return null;
return valueStr.ToLower();
}
โปรดทราบว่าปัญหานี้จะได้รับการแก้ไขเป็นส่วนใหญ่ใน C# 6 ซึ่งคุณสามารถใช้ตัวดำเนินการแบบมีเงื่อนไขว่างได้:
public static string GetValue(RegistryKey key, string name)
{
string value = key.GetValue(name)?.ToString();
if (String.IsNullOrWhiteSpace(value))
return null;
return value.ToLower();
}
a?.Foo() จะประเมินเป็น null ทันทีหาก a คือ null มันจะเรียกใช้ (และส่งคืนผลลัพธ์ของ) a.Foo() ถ้า a ไม่ใช่ null
- person Douglas; 24.04.2015
สร้างตัวแปรท้องถิ่นและเรียกใช้เมธอดหนึ่งครั้ง มีค่าใช้จ่าย (เล็กน้อยที่เป็นที่ยอมรับ) สำหรับการเรียกเมธอดในเครื่อง หากคุณใช้วิธีระยะไกล การเรียกความแตกต่างจะเด่นชัดกว่ามาก
โดยใช้ ? โอเปอเรเตอร์ทำให้อ่านง่ายขึ้นและประสิทธิภาพดีขึ้นอย่างที่คุณเห็น
public static string GetValue(RegistryKey key, string value)
{
string valueStr=(string)key.GetValue(value);
return string.IsNullOrWhiteSpace(valueStr)?null:valueStr.ToLower();
}
RegistryKey.GetValueไม่แคชผลลัพธ์ การเรียกนี้จะต้องมีการเข้าถึงรีจิสทรี ซึ่งมี ลำดับความสำคัญ ช้ากว่าการเข้าถึงตัวแปรในเครื่องหลายระดับ - person Douglas schedule 24.04.2015