Yarn: MinecraftClient.openScreen -> setScreen

نعم ، هذا منطقي تمامًا بالنسبة لي. لا يوجد سوى واحد منهم.

نعم ، يتم تبديل الشاشة أيضًا ، لذا فإن setScreen أفضل بالفعل

لست معجبًا بهذا التغيير ، حقًا. يجعل setScreen الأمر يبدو وكأنه أداة ضبط بسيطة ، ولكن إذا كان هناك إجماع على ذلك ... حسنًا

أو present / display / show ؟

إذا ذهبنا للحصول على اقتراحات الاتصال ، في هذه المرحلة ، يبدو أننا نغير الاسم من أجل تغيير الأسماء

لول وإلا فإنك بحاجة إلى اشتقاق واحد يرضي Shartte

setScreen تجعلها تبدو وكأنها أداة ضبط بسيطة

الهدف الكامل من أداة الضبط وليس مجرد حقل هو تشغيل منطق إضافي عند تعيين الحقل (سواء كان هذا المنطق الإضافي موجودًا بالفعل ، أو تريد أن يكون لديك إمكانية إضافته في المستقبل دون كسر الكود الذي يحدد الحقل ).

liach ناه ، لا داعي لإرضاء لي. لا أعتقد أن هذا _بشكل خاص_ مهم ، وأعتقد أن الاسم الحالي جيد ، لكن إعادة تسميته setScreen ليس نهاية العالم :-D

اقتراح آخر: switchScreen

لا يزال switchScreen(null) منطقيًا تمامًا ، وليس لديه مشكلة في الظهور كجهاز ضبط أو عملية عكسية.

إذا ذهبنا للحصول على اقتراحات الاتصال ، في هذه المرحلة ، يبدو أننا نغير الاسم من أجل تغيير الأسماء

اعتقدت أن هذه كانت المشكلة هنا؟ أجد أن openScreen محيرًا للغاية ، أقضي خمس دقائق جيدة في البحث عن closeScreen أو lastScreen أو popScreen . من المتوقع أن يكون للكائنات التي تحتوي على open close على الأقل.

present المحتمل أن يكون لدى display / show نفس المشكلة ، على الرغم من أن انعكاساتها ليست عالمية تقريبًا ( destroy ، conceal ، hide ).

setScreen على ما يرام ، على الرغم من ذلك ، نظرًا لأن المستوطنين يمكن أن يكون لديهم رمز أكثر من مجرد field = param . يستدعي الغزل بالفعل العديد من المستوطنين الأطول setX على الرغم من أنهم يقومون بأكثر من مجرد تعيين قيمة الحقل.