आप इस कोड में कुछ सुधार कर सकते हैं।
- हमेशा
use strict
औरuse warnings
आपके कोड में। और उनके द्वारा प्रकट की जाने वाली समस्याओं को ठीक करें (ज्यादातरmy
. के साथ चर घोषित करना इसके लुक से)। - आपके पास
use CGI /:standard/
का उपयोग है जो आपके नाम स्थान में CGI के कार्यों का एक भार आयात करता है, लेकिन तब आप केवल उन कार्यों को अपने CGI ऑब्जेक्ट पर विधियों के रूप में कहते हैं ($q
में) ) ताकि आयात अनावश्यक हो। - आप अप्रत्यक्ष वस्तु संकेतन का उपयोग करते हैं (
new CGI
) अपना CGI ऑब्जेक्ट बनाने के लिए। यह 99% समय काम करेगा लेकिन दुर्लभ मामलों में जब यह टूट जाता है तो आप समस्या को ट्रैक करने में दिन बर्बाद कर देंगे।CGI->new
का उपयोग करना कहीं बेहतर है इसके बजाय। - HTML निर्माण CGI.pm में कार्य करता है (आप
start_html()
. का उपयोग करते हैं ) किया गया है ">कुछ समय के लिए पदावनत . कृपया उनका उपयोग न करें। - आपके पर्ल कोड में कच्चा HTML होना एक रखरखाव समस्या होने की संभावना है (जैसा कि आपको लगता है कि आपने अपने HTML में त्रुटियों को देखते हुए खोज लिया है)। मैं दृढ़ता से अनुशंसा करता हूं कि आप अपने HTML को आपके पर्ल कोड से बाहर और टेम्प्लेट में ले जाएं। CGI::Alternatives ऐसा करने के तरीकों के बारे में कुछ सुझाव हैं।