আমার সহকর্মীদের উত্স নিয়ন্ত্রণ ব্যবস্থা থেকে একে অপরের কোড পর্যালোচনা করা উচিত?


9

সুতরাং এটি আমার গল্প: আমার সহকর্মীদের একজন পুনর্বিবেচনা সিস্টেমে হোস্ট করা সমস্ত কোড পর্যালোচনা করতে ব্যবহার করে। আমি তার অংশগুলির পরিবর্তনগুলির পর্যাপ্ত পর্যালোচনা সম্পর্কে বলছি না। তিনি কোড ফাইল ফাইল করতে দেখেন, লাইনে টু লাইনে দেখেন। প্রতিটি নতুন ফাইল এবং প্রতিটি পরিবর্তিত। আমি গুপ্তচরবৃত্তি করা ঠিক মত অনুভব!

আমার ধারণা হ'ল কোডটি যদি ইতিমধ্যে সিস্টেম নিয়ন্ত্রণ করতে হোস্ট করা থাকে তবে আপনার এটিকে অন্তত কার্যকর হিসাবে বিশ্বাস করা উচিত। আমার প্রশ্নটি হ'ল, সম্ভবত আমি খুব বেড়ান এবং একে অপরের কোড পর্যালোচনা করার অনুশীলন ভাল?

পিএস: আমরা কেবল তিনটি বিকাশকারীদের দল, এবং আমি আশঙ্কা করছি যে আমাদের মধ্যে আরও কিছু উপস্থিত থাকলে সহকর্মীর কাছে আমরা যে কোডটি লিখব তার সমস্ত কোড পর্যালোচনা করার সময় পাবে না।

উত্তর:


19

আমি বলব হ্যাঁ!

এর দুটি দ্রুত কারণ:

1) কোডটি যদি উত্পাদনে থাকে তবে আপনি এটি সঠিক বলে ধরে নিতে পারবেন না। সিস্টেমের অন্য কোথাও যে কোনও পরিবর্তন ত্রুটিগুলি প্রবর্তন করতে পারে। আমি মনে করি যে নিয়মিতভাবে কোডটি পরীক্ষা করা খুব জরুরি। এইভাবে, কোডটি ঝরঝরে এবং "আরও" সঠিক রেখে (নিয়মিত সম্ভবত আপত্তিকর ভাল) রেফ্যাক্টরিং করা হয়।

2) কোড পড়তে সক্ষম হওয়া একটি খুব গুরুত্বপূর্ণ দক্ষতা যদি আপনি প্রোগ্রামার হতে চলেছেন। এবং এটি একটি দক্ষতা, আপনার কিছু কাজ করা উচিত। যে কোনও প্রোগ্রামার বিদ্যমান কোড বেসে কাজ শুরু করার জন্য, যদি তিনি অন্য লোকের কোড পড়তে অভ্যস্ত না হন, সেখানে যা চলছে তার সাথে আধুনিকীকরণের চেষ্টা করার একটি খাড়া লার্নিং কার্ভ রয়েছে।

আমি মনে করি না যে আপনি গুপ্তচরবৃত্তি করা উচিত। কেউ আপনাকে যে কোনও সমালোচনা দেয় তা গ্রহণ করুন (যদি তা অবশ্যই কার্যকর হয়)। এবং নির্দ্বিধায় অন্য ব্যক্তিকে ভাল সমালোচনা দিতে পারেন। এটি আমরা শিখার উপায়। একবার আমরা শেখা বন্ধ করে (বা থামাতে চাই), তারপরে বড় সমস্যা রয়েছে problems


12

যদি বলা হয় সহকর্মী ভাল এবং গঠনমূলক প্রতিক্রিয়া সরবরাহ করে, এটি একটি দুর্দান্ত জিনিস এবং আপনার এটির খুব প্রশংসা করা উচিত।

এই হবে ত্রুটিগুলি আপনি যখন এটি লেখার ভাবিনি ধরতে। এই হবে আপনি ভাল কোড লেখা স্থাপিত কারণ আপনি জানেন যে অন্যদের তা দেখতে হবে।


4

পুরো টিম যদি একজন ব্যক্তির পরিবর্তে কোড পর্যালোচনা করে তবে এটি স্বাস্থ্যকর হবে। আদর্শভাবে প্রত্যেকেই কাউকে সম্পূর্ণ করার পরে তাদের কোড পর্যালোচনা করার জন্য আমন্ত্রণ জানাত। এটিকে অনানুষ্ঠানিক রাখতে (পরিচালকদের দূরে রাখতে) এবং পর্যালোচককে তার অনুসন্ধানের মাধ্যমে আপনাকে কথা বলতে সহায়তা করা সহায়ক। আদর্শভাবে পর্যালোচক কেবল প্রতিক্রিয়া দেয় এবং কোড পরিবর্তন করে না, অবশ্যই আপনি এটিতে কিছুটা জুড়ি দিতে পারেন।

ধীরে ধীরে সাদা স্থান এবং কোড স্টাইল সম্পর্কে পর্যালোচনা আলোচনা এড়ানোর জন্য কোডিং মানকগুলিতে এটি সত্যই সহায়তা করে। বিল্ড মেশিনে কিছু স্ট্যাটিক কোড বিশ্লেষণ থাকা কিছু আলোচনা বাইরে রাখার ক্ষেত্রেও সহায়ক হতে পারে।

সময়ের দিক সম্পর্কে, তত্ত্বটি হল এটি আপনার সময় সাশ্রয় করবে। পরবর্তী ত্রুটিগুলি তারা আরও ব্যয়বহুল হিসাবে পাওয়া যায়, দ্রুত নীতি ব্যর্থ করে। পিয়ার কোড পর্যালোচনা বেশ কিছুটা সমস্যা ধরতে পারে।


1
+1 সম্মত। সমস্ত পর্যালোচনা করা একজন ব্যক্তি দলে অস্বস্তি বাড়ে। এটি মারাত্মকভাবে ভুল হতে পারে এবং আমার কোডটি আপনার কোডের চেয়ে ভাল বলে ব্যবহার করা যেতে পারে । এটি একটি দলের কাজ করা উচিত।
অড্রিয়াস

@ অ্যান্ড্রিয়াস: দু: খিত, আমি কী বোঝাতে চেয়েছি তা বুঝতে পেরেছি
kizzx2


3

আমি আমাদের সংস্করণ নিয়ন্ত্রণ সিস্টেমটি একইভাবে দেখি। আমাদের কোডবেস প্রতিটি লাইন দেখতে খুব বড়, তবে আমি বেশিরভাগ পরিবর্তনের জন্য একটি উচ্চ স্তরের অনুভূতি পাওয়ার চেষ্টা করি। আমি চেকিনগুলির জন্যও লক্ষ্য রাখি যার সাইড এফেক্ট থাকতে পারে এবং লাইন দ্বারা সেই লাইনগুলি পর্যালোচনা করে। আমি এই কাজটি করতে ন্যূনতম সময়ের জন্য, বেতনটি বিশাল। (আরও মনে রাখবেন: এই অভ্যাসটি সহ আমি আমাদের দলের একমাত্র বিকাশকারী নই))

এই ধরণের পর্যালোচনাগুলি সাপ্তাহিক ভিত্তিতে বাগগুলি ধরা বা আলোচনার আহ্বান জানায়। যা কিউএ করার সময় সময় সাশ্রয় করে। আলোচনার সেরা অনুশীলন থেকে শুরু করে অ্যালগরিদম ডিজাইন এবং আরও অনেক কিছু। এই ফ্রন্টের মূলটি হ'ল প্রত্যেকে এটিকে গঠনমূলক হিসাবে দেখে।

ব্যক্তিগতভাবে, এটি আমাকে নিয়মিতভাবে স্পর্শ না করে কোডবেসের অন্যান্য অংশে কী চলছে তার একটি বৃহত্তর উপলব্ধি দেয়। যখন অন্যের সাহায্যের প্রয়োজন হয়, আমি আরও দ্রুত ঝাঁপিয়ে পড়তে সক্ষম হয়েছি। এছাড়াও, যখন নতুন ধারণা উপস্থিত হয় আমি তাড়াতাড়ি তাদের সুবিধা নিতে পারি।


1

আপনি কি এটা গুপ্তচরবৃত্তি হিসাবে মনে হচ্ছে (!)? তবে আপনার সহকর্মী দৃষ্টিকোণ থেকে আমি বলব তিনি তার ক্যারিয়ার বিকাশের জন্য সঠিক কাজ করছেন। অন্যদের কোড পড়ুন এবং কীভাবে তারা যুক্তিটি ডিজাইন এবং প্রয়োগ করেন, এটি আপনাকে প্রচুর পরিমাণে অর্জন করবে!

আইএমএইচও যদি কোনও একটি আপনার কোডটিতে কিছু ভুল দেখায়, আপনাকে অবশ্যই এটি গ্রহণ করতে হবে এবং একটি ভাল কোড কীভাবে লিখবেন সে সম্পর্কে তাদের কাছ থেকে শিখতে হবে


1

6--7 মাসের সময় আমি একই কাজ করছিলাম। গুপ্তচরবৃত্তি করতে নয়, মান নিয়ন্ত্রণ করতে। সক্রিয়ভাবে বিকশিত অ্যাপ্লিকেশনটির জন্য কোডের প্রতিটি লাইন, কেন্দ্রীয় সংগ্রহশালার প্রতিশ্রুতিবদ্ধ, 2 টি প্রধান ভাষা, কয়েকটি আরও কয়েকটি ভাষা, 4 প্ল্যাটফর্মের জন্য বিশাল মেকফিলগুলি।

এটা খুব খারাপ অনুশীলন । কোনও দিন আমি জানতে পেরেছি যে দৃ the়তার কারণে আমি সমস্ত কিছু ক্যাপচার করতে পারি না। এর বিরুদ্ধে আর একটি যুক্তি হ'ল সাবজেক্টিভিটি - প্রত্যেকে ভুল হতে পারে।

এটি উন্নত হয় যখন বিকাশকারীরা একে অপরের কোডগুলি পর্যালোচনা করে এবং চূড়ান্ত সিদ্ধান্ত গ্রহণ এবং দিকনির্দেশ নির্ধারণ করার জন্য অভিজ্ঞ কেউ আছেন।


1

একটি দলের মধ্যে কোড পর্যালোচনাগুলি ( ফিশিয়ে , ক্রুশিবল বা অন্যান্য সরঞ্জাম ব্যবহার করে) অত্যন্ত গুরুত্বপূর্ণ এবং দরকারী। প্রথমবারের মতো সিস্টেমে যে কোডটি আসে সেটিকে প্রথমে ভালভাবে বিবেচনা করা হয় এবং একাধিক ব্যক্তির মস্তিষ্কের মধ্য দিয়ে যায় তা নিশ্চিত করার জন্য কেবল সরাসরি জুটি প্রোগ্রামিংই ভাল।


0

আমার দলে এটি একবার হয়েছিল। দুর্ভাগ্যক্রমে এটি একটি দোষের খেলায় পরিণত হয়েছিল। লোকেরা নিয়মিতভাবে অন্যদের কোডটি চেক-ইন করার জন্য অপেক্ষা করত এবং সর্বদা এতে কোনও ভুল খুঁজে পাওয়ার চেষ্টা করত এবং দোষের খেলাটি সর্বদা খেলত।

আমি আশা করি আপনি আরও পরিপক্ক শ্রোতা আছে।


কোডারটি তাকে / তার নিজের কাউকে তাদের কোড পর্যালোচনা করার জন্য আমন্ত্রণ জানানো ভাল, চেক-ইন করার আগেই সম্ভব। এটি আপনার বর্ণনা করা উন্মত্ততা রোধ করতে পারে।
জোপ্প

@ টুঙ্গা: মজার বিষয়টি হ'ল কেবল পর্যালোচনা কোডটি পরীক্ষা করা হয়েছিল তবে তবুও তারা সকলেই তার শ্রেষ্ঠত্ব প্রমাণের জন্য এতটাই আগ্রহী ছিল যে তারা কোডার এবং পর্যালোচককে নিষেধাজ্ঞার বিষয়ে কিছু মনে করবে না। আমি এটি দেখতে পেয়েছি খুব মজাদার :-)
গীক

0

এটি শিল্পে বেশ স্ট্যান্ডার্ড অনুশীলন। আমি যে সংস্থাগুলিতে কাজ করেছি তাদের কাছে অত্যন্ত কঠোর কোড পর্যালোচনা নির্দেশিকা রয়েছে। কোড পর্যালোচনা না করা হলে একটি আপনাকে প্রতিশ্রুতিবদ্ধ হতে দেয় না।

অপরাধ নেবেন না, বা দেখে দেখে বোধ করবেন না। এটিকে একটি সুরক্ষা জাল এবং একটি শেখার অভিজ্ঞতা হিসাবে ভাবেন।


0

আগের চাকরিতে, প্রবীণ বিকাশকারী সমস্ত চেক-ইন দেখে এবং পর্যালোচনা করেছিলেন এবং আমি প্রায়শই চমৎকার প্রতিক্রিয়া পাই যা আমাকে আরও উন্নত বিকাশকারী হিসাবে সহায়তা করে।

আমার বর্তমান চাকরিতে, আমি অনেকগুলি চেক-ইন দেখেছি এবং তিন দিন আগে আমি একটি বাগ পেয়েছি এবং বিকাশকারীকে অবহিত করেছি।

এই অনুশীলনটি পুরোপুরি বাগগুলি ধরে ফেলবে এবং আপনার পুরো দলটিকে আরও ভাল করে তুলবে, যদি আপনি এটি আলিঙ্গন করেন।

আমাদের সাইট ব্যবহার করে, আপনি স্বীকার করেছেন যে আপনি আমাদের কুকি নীতি এবং গোপনীয়তা নীতিটি পড়েছেন এবং বুঝতে পেরেছেন ।
Licensed under cc by-sa 3.0 with attribution required.