Magento 1 केcoding standard लिए कुछ ऐसा है , जिसे कहा जाता है कि कई चीजों से बचना चाहिए या उन तरीकों का इस्तेमाल नहीं किया जाना चाहिए।ECG
आइए एक सूची बनाने की कोशिश करें जो आपके कोड में "त्रुटियों" या "चेतावनी" को फेंकने वाले सभी मामलों को कवर करती है।
नीचे आपको सभी संभावित चेतावनियों के साथ एक सूची मिलेगी। मैं नियमित अंतराल पर इस पोस्ट को अपडेट करूंगा और दिए गए और उत्कीर्ण उत्तरों के लिंक दूंगा।
नोट: कृपया डुप्लिकेट उत्तरों से बचने की कोशिश करें;)
डाउनलोड करें: https://github.com/magento-ecg/coding-standard
ईसीजी सूँघता है
कक्षाएं
मायिकल 4 कक्षाएं अप्रचलित हैं।
- संसाधन मॉडल को पुनर्लेखन करते समय हटाए गए "मैसूरल 4" कक्षाओं को कैसे संभालना है
- मैसेंजर से रिसोर्स फ़ोल्डर को रिसोर्स में बदलते समय Magento 1.9 मॉड्यूल अजीब तरह से टूट रहा है
Magento में प्रत्यक्ष वस्तु तात्कालिकता (वर्ग% s) हतोत्साहित किया जाता है।
protected $disallowedClassPrefixes = array(
'Mage_',
'Enterprise_',
);
...
पीएचपी
गोटो का उपयोग हतोत्साहित किया जाता है।
...
Namespace for "'। $ अपवाद .ClassName।" "वर्ग निर्दिष्ट नहीं है।
...
निजी वर्ग के सदस्य का पता चला।
...
निजी वर्ग के सदस्य का पता चला।
...
प्रदर्शन
Magento डेटा संग्रह की अनावश्यक लोडिंग। इसके बजाय getSize () विधि का उपयोग करें।
fetchAll () बड़े डेटा सेट के लिए स्मृति अक्षम हो सकती है।
...
getFirstItem () संग्रह भार के परिणाम को एक आइटम तक सीमित नहीं करता है।
- GetFirstItem - https://magento.stackexchange.com/a/179309/46249
लूप में पाया गया आकार आकार गणना फ़ंक्शन% s है
मॉडल एलएसडी विधि% s लूप में पाई गई
डेटा लोड% s विधि लूप में पाई गई
protected $countFunctions = array(
'sizeof',
'count'
);
protected $modelLsdMethods = array(
'load',
'save',
'delete'
);
सुरक्षा
% S वर्ग में% s () ACL विधि गुम है।
const PARENT_CLASS_NAME = 'Mage_Adminhtml_Controller_Action';
const REQUIRED_ACL_METHOD_NAME = '_isAllowed';
...
public $forbiddenFunctions = array(
'^is_dir' => null,
'^is_file$' => null,
'^pathinfo$' => null,
);
...
public $forbiddenFunctions = array(
'^assert$' => null,
'^bind_textdomain_codeset$' => null,
'^bindtextdomain$' => null,
'^bz.*$' => null,
'^call_user_func$' => null,
'^call_user_func_array$' => null,
'^chdir$' => null,
'^chgrp$' => null,
'^chmod$' => null,
'^chown$' => null,
'^chroot$' => null,
'^com_load_typelib$' => null,
'^copy$' => null,
'^create_function$' => null,
'^curl_.*$' => null,
'^cyrus_connect$' => null,
'^dba_.*$' => null,
'^dbase_.*$' => null,
'^dbx_.*$' => null,
'^dcgettext$' => null,
'^dcngettext$' => null,
'^dgettext$' => null,
'^dio_.*$' => null,
'^dirname$' => null,
'^dngettext$' => null,
'^domxml_.*$' => null,
'^exec$' => null,
'^fbsql_.*$' => null,
'^fdf_add_doc_javascript$' => null,
'^fdf_open$' => null,
'^fopen$' => null,
'^fsockopen$' => null,
'^ftp_.*$' => null,
'^fwrite$' => null,
'^gettext$' => null,
'^gz.*$' => null,
'^header$' => null,
'^highlight_file$' => null,
'^ibase_.*$' => null,
'^id3_set_tag$' => null,
'^ifx_.*$' => null,
'^image.*$' => null,
'^imap_.*$' => null,
'^ingres_.*$' => null,
'^ircg_.*$' => null,
'^ldap_.*$' => null,
'^link$' => null,
'^mail$' => null,
'^mb_send_mail$' => null,
'^mkdir$' => null,
'^move_uploaded_file$' => null,
'^msession_.*$' => null,
'^msg_send$' => null,
'^msql$' => null,
'^msql_.*$' => null,
'^mssql_.*$' => null,
'^mysql_.*$' => null,
'^odbc_.*$' => null,
'^opendir$' => null,
'^openlog$' => null,
'^ora_.*$' => null,
'^ovrimos_.*$' => null,
'^parse_ini_file$' => null,
'^parse_str$' => null,
'^parse_url$' => null,
'^parsekit_compile_string$' => null,
'^passthru$' => null,
'^pcntl_.*$' => null,
'^posix_.*$' => null,
'^pfpro_.*$' => null,
'^pfsockopen$' => null,
'^pg_.*$' => null,
'^php_check_syntax$' => null,
'^popen$' => null,
'^print_r$' => null,
'^printf$' => null,
'^proc_open$' => null,
'^putenv$' => null,
'^readfile$' => null,
'^readgzfile$' => null,
'^readline$' => null,
'^readlink$' => null,
'^register_shutdown_function$' => null,
'^register_tick_function$' => null,
'^rename$' => null,
'^rmdir$' => null,
'^scandir$' => null,
'^session_.*$' => null,
'^set_include_path$' => null,
'^set_ini$' => null,
'^set_time_limit$' => null,
'^setcookie$' => null,
'^setlocale$' => null,
'^setrawcookie$' => null,
'^shell_exec$' => null,
'^sleep$' => null,
'^socket_.*$' => null,
'^stream_.*$' => null,
'^sybase_.*$' => null,
'^symlink$' => null,
'^syslog$' => null,
'^system$' => null,
'^touch$' => null,
'^trigger_error$' => null,
'^unlink$' => null,
'^vprintf$' => null,
'^mysqli.*$' => null,
'^oci_connect$' => null,
'^oci_pconnect$' => null,
'^quotemeta$' => null,
'^sqlite_popen$' => null,
'^time_nanosleep$' => null,
'^base64_decode$' => null,
'^base_convert$' => null,
'^basename$' => null,
'^chr$' => null,
'^convert_cyr_string$' => null,
'^dba_nextkey$' => null,
'^dns_get_record$' => null,
'^extract$' => null,
'^fdf_.*$' => null,
'^fget.*$' => null,
'^fread$' => null,
'^fflush$' => null,
'^get_browser$' => null,
'^get_headers$' => null,
'^get_meta_tags$' => null,
'^getallheaders$' => null,
'^getenv$' => null,
'^getopt$' => null,
'^headers_list$' => null,
'^hebrev$' => null,
'^hebrevc$' => null,
'^highlight_string$' => null,
'^html_entity_decode$' => null,
'^ibase_blob_import$' => null,
'^id3_get_tag$' => null,
'^import_request_variables$' => null,
'^ircg_nickname_unescape$' => null,
'^ldap_get_values$' => null,
'^mb_decode_mimeheader$' => null,
'^mb_parse_str$' => null,
'^mcrypt_decrypt$' => null,
'^mdecrypt_generic$' => null,
'^msg_receive$' => null,
'^ngettext$' => null,
'^ob_get_contents$' => null,
'^ob_get_flush$' => null,
'^rawurldecode$' => null,
'^shm_get_var$' => null,
'^stripcslashes$' => null,
'^stripslashes$' => null,
'^token_get_all$' => null,
'^unpack$' => null,
'^convert_uudecode$' => null,
'^iconv_mime_decode$' => null,
'^iconv_mime_decode_headers$' => null,
'^iconv_mime_encode$' => null,
'^iconv_set_encoding$' => null,
'^php_strip_whitespace$' => null,
'^addcslashes$' => null,
'^addslashes$' => null,
'^escapeshellarg$' => null,
'^escapeshellcmd$' => null,
'^gettype$' => null,
'^var_dump$' => null,
'^tempnam$' => null,
'^realpath$' => null,
'^linkinfo$' => null,
'^lstat$' => null,
'^stat$' => null,
'^lchgrp$' => null,
'^lchown$' => null,
'^show_source$' => null,
'^is_executable$' => null,
'^is_link$' => null,
'^is_readable$' => null,
'^is_writable$' => null,
'^is_writeable$' => null,
'^is_uploaded_file$' => null,
'^glob$' => null,
'^ssh2_.*$' => null,
'^delete$' => null,
'^file.*$' => null,
);
- ...
- curl_: - https://magento.stackexchange.com/a/178642/46249
- file_exists - https://magento.stackexchange.com/a/178641/46249
- ...
"% s" कथन का पता चला। फ़ाइल हेरफेर को हतोत्साहित किया जाता है।
... कथन एक फ़ंक्शन नहीं है, कोई कोष्ठक की आवश्यकता नहीं है।
... यूरेल पास करना मना है।
... कॉन्टेनेट करना मना है।
... अंदर के चर असुरक्षित हैं।
public $urlPattern = '#(https?|ftp)://.*#i';...
बैक कोट स्ट्रिंग निरंतर का गलत उपयोग। वापस उद्धरण हमेशा स्ट्रिंग्स के अंदर होना चाहिए।
% S भाषा निर्माण का उपयोग हतोत्साहित किया जाता है।
return array(
T_EXIT,
T_ECHO,
T_PRINT,
T_BACKTICK
);
...
% S सुपरग्लोबल का प्रत्यक्ष उपयोग पाया गया।
public $superGlobalErrors = array(
'$GLOBALS',
'$_GET',
'$_POST',
'$_SESSION',
'$_REQUEST',
'$_ENV'
);
public $superGlobalWarning = array(
'$_FILES',
'$_COOKIE',
'$_SERVER',
);
Sql
संभावित कच्चे SQL कथन% s का पता चला
public $statements = array(
'SELECT',
'UPDATE',
'INSERT',
'CREATE',
'DELETE',
'ALTER',
'DROP'
);
public $queryFunctions = array(
'query',
'raw_query'
);
...
संभावित धीमा SQL कथन% s का पता चला
संभावित धीमी SQL विधि% s का पता चला
public $adapterMethods = array(
'group',
'having',
'distinct',
'addLikeEscape',
'escapeLikeValue',
'union',
'orHaving',
);
public $rawStatements = array(
'GROUP BY',
'HAVING',
'DISTINCT',
'LIKE',
'UNION',
);...
स्ट्रिंग्स
% S में संभावित निष्पादन योग्य नियमित अभिव्यक्ति। सुनिश्चित करें कि पैटर्न में "e" संशोधक नहीं है
public $functions = array(
'preg_replace',
);...
पता लगाया गया दो तारों को जोड़ने के लिए + ऑपरेटर का उपयोग
...
% S फ़ंक्शन के रिटर्न मान के परीक्षण के लिए आइडेंटिकल ऑपरेटर === का उपयोग नहीं किया जाता है
public $functions = array(
'strpos',
'stripos',
);- http://php.net/manual/en/function.strpos.php#refsect1-function.strpos-examples
- http://php.net/manual/en/function.stripos.php#refsect1-function.stripos-examples
सर्वश्रेष्ठ अभ्यास के लिए संबंधित प्रश्न और उत्तर