C-toxcore: .h फ़ाइलों में OS विशिष्ट शीर्षलेख शामिल न करें

को निर्मित 11 सित॰ 2016 · 5टिप्पणियाँ · स्रोत: TokTok/c-toxcore

वर्तमान में, network.h में बहुत सारे प्लेटफ़ॉर्म विशिष्ट शीर्षलेख शामिल हैं (जैसे u.h , windows.h , और unistd.h )। हमें इन्हें एक ही फाइल (या कुछ फाइलों) तक सीमित रखना चाहिए।

मेरा सुझाव है कि हम नेटवर्किंग के लिए ओएस सुविधाओं पर एक पतली अमूर्तता बनाएं, और दूसरा मोनोटोनिक घड़ी के लिए। तब हमारे पास प्रति प्लेटफ़ॉर्म प्रकार एक फ़ाइल हो सकती थी (जैसे ${module}_win32.c , _unix.c , _osx.c , _posix.c , ...) और सशर्त रूप से उन्हें प्रीप्रोसेसर के अनुसार संकलित करें प्रतीक, जैसे हम पहले से ही करते हैं। नकारात्मक पक्ष यह है कि यह थोड़ा अधिक घोषणा ओवरहेड है, क्योंकि वर्तमान में वे #ifdef s फ़ंक्शन के अंदर हैं जिन्हें प्रत्येक प्लेटफ़ॉर्म के लिए फिर से परिभाषित करने की आवश्यकता होगी। लाभ यह है कि कोड में ही कोई या कुछ #ifdef s नहीं हैं, और प्लेटफ़ॉर्म विशिष्ट कोड 1100 लाइन फ़ाइल मिक्सिंग प्लेटफ़ॉर्म एब्स्ट्रैक्शन के बजाय (जैसा कि अभी है) के बजाय छोटी फ़ाइलों के एक सेट में है। आवेदन कोड।

P1 cleanup good first issue

स्रोत

iphydf

👍3

सबसे उपयोगी टिप्पणी

हम एक "सिस्टम" इंटरफ़ेस कैसे बनाते हैं जिसे "विंडोज सिस्टम" और "पॉज़िक्स सिस्टम" और "टेस्ट सिस्टम" द्वारा कार्यान्वित किया जा सकता है? परीक्षण प्रणाली का उपयोग यूनिट परीक्षणों में किया जा सकता है और वास्तव में सिस्टम कॉल किए बिना सिस्टम का अनुकरण करता है।

iphydf 18 दिस॰ 2016

👍2

सभी 5 टिप्पणियाँ

बढ़िया है

nurupo 12 सित॰ 2016

iphydf 18 दिस॰ 2016

👍2

मैं कार्यान्वयन सुझाव से असहमत हूं। मैं एक ही फ़ाइल में सभी कार्यों को देखना चाहता हूं {network,time,...}.c , जिसमें #ifdef s का उपयोग उन कार्यों को लपेटने के लिए किया जाता है जिन्हें प्रत्येक प्लेटफॉर्म के लिए अलग-अलग लिखने की आवश्यकता होती है।

अलग-अलग फाइलों पर इस पद्धति का लाभ यह है कि हम सभी प्लेटफ़ॉर्म विशिष्ट कोड को एक स्थिर फ़ंक्शन के रूप में बाध्य कर सकते हैं, और टॉक्सकोर के उच्च स्तर के लिए एक एकल एपीआई को उजागर कर सकते हैं। किसी भी प्लेटफ़ॉर्म कार्यान्वयन विवरण को एक फ़ाइल से आसानी से देखा जा सकता है। प्लेटफ़ॉर्म की ख़ासियत के लिए कई फ़ाइलों के बजाय, एक फ़ाइल के भीतर अमूर्तता और हैंडलिंग की आवश्यकता होगी। यह cmakelists.txt फ़ाइल को बनाए रखना भी आसान बना देगा (यह पहले से ही हमसे दूर होने लगा है।) और बाद में समीक्षा करें, हमारी वर्तमान समीक्षा प्रणाली के साथ।

GrayHatter 30 दिस॰ 2016

👍1

पूरी तरह से तय नहीं

Diadlo 27 जन॰ 2017

@ ग्रेहैटर सहमत हुए। आइए ओएस विनिर्देशों को लागू करने वाली एक ही फाइल है।

मैं अभी के लिए @Diadlo को अनअसाइन कर रहा हूं। मुझे लगता है कि नए योगदानकर्ताओं के लिए काम करने के लिए यह एक बड़ी बग है, क्योंकि यह बहुत से कोडबेस और नेटवर्क कोड को एक्सपोजर देता है। यह कुछ हद तक दिलचस्प रिफैक्टरिंग है जिसका कार्यक्षमता पर कोई प्रभाव नहीं पड़ता है, इसलिए परीक्षण करना आसान है।

iphydf 16 जन॰ 2018

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

C-toxcore: .h फ़ाइलों में OS विशिष्ट शीर्षलेख शामिल न करें

सबसे उपयोगी टिप्पणी

सभी 5 टिप्पणियाँ

संबंधित मुद्दों