Yarn: ResourceReloadListener # reload -> createReloadTask أو شيء من هذا القبيل

تم إنشاؤها على ٢٢ أكتوبر ٢٠١٩ · 3تعليقات · مصدر: FabricMC/yarn

ليس من المفترض أن يقوم ResourceReloadListener#reload بأي إعادة تحميل من تلقاء نفسه. بدلاً من ذلك ، يتمثل هدفه في إرجاع CompletableFuture والذي ، عند تقييمه ، من المفترض أن يرسل مهام إعادة التحميل الفعلية إلى Executor s.

إذا حاولت خداع النظام عن طريق وضع الكود مباشرةً في reload وإعادة بعض الدمية CompletableFuture التي لا تفعل شيئًا ، تتوقف اللعبة عند إعادة التحميل ، لأن المستقبل الذي تم إرجاعه بمقدار Synchronizer#whenPrepared لن يحصل أبدًا مسمى.

يبدو أن تسميته شيئًا مثل createReloadTask طريقة لتثبيط هذا المسار وتوضيح أنه لا يجب إعادة التحميل مباشرة بهذه الطريقة.

discussion enhancement refactor

مصدر

quat1024

👍1

التعليق الأكثر فائدة

imo ، يجب إعادة تسمية ResourceReloadListener إلى ResourceReloader لأنهم هم الأشخاص الذين يقومون بالفعل بإعادة التحميل (قارن بـ ResourceReloadHandler ). اسم reload جيد ، أو يمكننا التفكير في نظام تسمية ثابت لجميع الأسماء ذات الصلة بـ CompletableFuture .

liach في ٢٢ أكتوبر ٢٠١٩

👍2 ❤1

ال 3 كومينتر

liach في ٢٢ أكتوبر ٢٠١٩

👍2 ❤1

بعد نظرة ، لدينا بالفعل أداة إعادة تحميل الموارد التي تراقب إعادة التحميل.

الآن ، أنا مرتبك قليلاً بشأن ما يجب تسميته ، أو ما إذا كان يجب إعادة تسميته أم لا.

liach في ١٠ مارس ٢٠٢١

حسنًا ، ربما ResourceReloader -> SimpleResourceReload ، ResourceReloadListener -> ResourceReloader . بالإضافة إلى ذلك ، قم بتسمية ResourceReloadMonitor إلى ResourceReload وقم بتسمية عمليات التنفيذ وفقًا لذلك. يمثل ResourceReload عملية إعادة تحميل واحدة ويتم التخلص منها على الفور بعد ذلك ، لذلك يبدو هذا الاسم جيدًا بالفعل.