কোড পর্যালোচনায় ইতিবাচক মন্তব্য করা কি উপযুক্ত, বা এটি কেবল গঠনমূলক সমালোচনার জন্য?


36

আমি ইদানীং প্রচুর কোড পর্যালোচনা করছি এবং কোড পর্যালোচনায় ইতিবাচক এবং / অথবা মজার মন্তব্য দেওয়ার ক্ষেত্রে আমি ইতিবাচক এবং নেতিবাচক প্রভাব এবং পেশাদারিত্ব সম্পর্কে নিশ্চিত নই।

আমরা আমার টিমে গিথুবকে আমাদের কোড পর্যালোচনা প্ল্যাটফর্ম হিসাবে ব্যবহার করি, সুতরাং মন্তব্যগুলি যে কেউ দেখতে পাবে। আমি সাধারণত এই প্ল্যাটফর্মটি ব্যবহার করার চেষ্টা করি যাতে শুরু থেকে শেষ পর্যন্ত সম্পূর্ণ প্রক্রিয়াটি দৃশ্যমান এবং .তিহাসিক হয়।


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

22
ইতিবাচক মন্তব্য সহ আপনি যা দেখতে চান তা চাঙ্গা করা গুরুত্বপূর্ণ is

2
এটি যথাযথ কিনা তা আমি জানি না তবে কখনও কখনও আমি পর্যালোচনাগুলিতে কোডিং হাইকাস পোস্ট করি: জুনিয়র প্রোগ্রামার - আপনার নিরঙ্কুশিত কোডটি - ব্যথার স্প্লিন্টারের মতো।
ইয়ানিস

আমি আসলে জুনিয়র প্রোগ্রামার সিনিয়র কোডটি পর্যালোচনা করছি - আমাদের এখানে বেশ কঠোর প্রক্রিয়া রয়েছে, সমস্ত কোড অবশ্যই পর্যালোচনা করা উচিত :)
কোডম্যান

উত্তর:


53

ইতিবাচক পাশাপাশি নেতিবাচক বিষয়গুলিও হাইলাইট করা গুরুত্বপূর্ণ। আমি জানি যদি আমি একটি নির্দিষ্ট নরকীয় সাবসিস্টেমের রিফ্যাক্টরটি পরিষ্কার-পরিচ্ছন্ন কিছুতে পর্যালোচনা করতাম তবে আমি সম্ভবত প্রোগ্রামারকে তার প্রচেষ্টার জন্য একটি পিৎজা কিনতে পারতাম।

আপনি যদি প্রশিক্ষণ হিসাবে পর্যালোচনাগুলি ব্যবহার করেন তবে এটি দ্বিগুণ গুরুত্বপূর্ণ - একটি ভাল কোডের হাইলাইট করা জুনিয়র প্রোগ্রামাররাও সেই কোডটি পর্যালোচনা করতে সহায়ক হবে। কোনও নির্দিষ্ট পদ্ধতির বা কৌশলটি কেন অন্যটির চেয়ে ভাল সে সম্পর্কে তাদের প্রশ্ন করার সুযোগ থাকবে।


আপনি "কেন এটি গুরুত্বপূর্ণ" হিসাবে একটি নোট যুক্ত করতে পারেন? আপনি এটি করার সময় আমি স্বীকৃত হিসাবে চিহ্নিত করব :)
কোডম্যান

3
এটি লোকেদের জানতে দেয় যে তারা কী করছে (ভাল কোড লেখার জন্য) প্রশংসা করা হয়েছে এবং বৃদ্ধির দিকে মনোনিবেশ রাখে, যা কোড রিভিউগুলির বিষয়ে।
জোনাথন ধনী

1
আপনি কি আপনার উত্তরে এটি যুক্ত করতে পারেন?
কোডম্যান

2
আপনার এই উত্তরটির জন্য ভাল উত্তর ব্যাজ প্রাপ্য 1+। গঠনমূলক সমালোচনাকে যথেষ্ট পরিমাণে মূল্য দেয় না এমন জায়গাগুলি আমাকে সর্বদা অবাক করে দেয়। যখন তারা একটি ভাল কাজ করছেন তখন তারা একটি ভাল কাজ করছেন তাদের বলা অবাক করা কার্যকরভাবে অনুপ্রেরণাকারী হতে পারে।
বেনিয়ামিন গ্রুইনবাউম

@ বেঞ্জামিন: অনেকগুলি সংস্থাগুলি কোড পর্যালোচনাকে 'প্রত্যেকের কোডের ত্রুটিগুলি খুঁজে বের করতে' রাবার স্ট্যাম্পিং প্রক্রিয়া হিসাবে বিবেচনা করে এবং 'একটি উচ্চতর পারফর্মিং দল তৈরি করতে দেয়' হিসাবে নয় treat আমার অভিজ্ঞতায়, কোড রিভিউগুলি বাধা নিরসনের দ্রুত এবং সর্বোত্তম উপায় যা সফ্টওয়্যার দলগুলি সত্যই সম্পাদন করে। আমাদের স্বরলিপিটির একটি মান আছে যেখানে আমরা মন্তব্যগুলি -2 (অবশ্যই স্থির করে নেওয়া উচিত) + 2 (দুর্দান্ত কাজ) সহ প্রিফিক্স করি।
mattnz

8

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

ইতিবাচক: অবশ্যই। সংজ্ঞা অনুসারে পর্যালোচনাতে ইতিবাচক এবং নেতিবাচক / গঠনমূলক উভয়ই অন্তর্ভুক্ত।

ইতিবাচক প্রতিক্রিয়া প্রত্যেককে সহায়তা করে:

থাম্বস আপ প্রাপ্ত ব্যক্তির জন্য, আপনি তাদের আত্মবিশ্বাসকে শক্তিশালী করেন এবং আপনার ইতিবাচক প্রতিক্রিয়ার মাধ্যমে তাদের আরও কিছু করার জন্য অনুপ্রাণিত করেন।

বাকি, অন্যদের উল্লেখ করেছি, তারা শিখতে হবে কি করতে হবে তা , সেইসাথে কি না । তারাও এক্সেল করতে উত্সাহিত হবে, তাই তারাও একদিন স্পটলাইটে থাকতে পারে।

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


3

আমি মন্তব্যগুলি পরিষ্কার এবং বিন্দুতে বলতে চাই, ঠিক সংস্কৃতির কারণে।

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


1
ভাল যুক্তি. মন্তব্য করার ক্ষেত্রে "মজার" হওয়া খুব সহজেই পিছিয়ে যেতে পারে, বিশেষত বহু-ভাষী, বহু-সাংস্কৃতিক পরিবেশে।
ডেভিড নাভারে

2

একটি কোড পর্যালোচনায় মন্তব্য করা পরিচালনা করা

মন্তব্যগুলি পরিচালনার একটি সরঞ্জাম হিসাবে বিবেচনা করুন

একটি কোড পর্যালোচনাতে মন্তব্যগুলি সন্নিবেশ করানো পরিচালনার একটি ফর্ম। সেই হিসাবে এটি পরিচালনা করার সরঞ্জাম হিসাবে যোগাযোগ করা উচিত।

মন্তব্য করার সময় পরিচালিত অনুশীলনগুলি ব্যবহার করুন

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

পরিবেশ

সংস্কৃতি

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

শৈলী

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

নেতৃত্ব

প্রাথমিক ধারনা

মন্তব্য করার সময় বিবেচনার জন্য নেতৃত্বের তিনটি মূল স্তম্ভ রয়েছে। যথা এগুলি হ'ল দৃষ্টি, যোগাযোগ এবং বিচার।

Vision

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

Communication

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

Judgement

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

নির্মাতা

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

নিয়ামক

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

রাজনৈতিক

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

ধারণাসঙ্গত

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

আন্তঃব্যক্তিগত

পরিচালনা করার ক্ষেত্রে আন্তঃব্যক্তিক দক্ষতা খুব গুরুত্বপূর্ণ। এটিও একটি বিশাল বিষয়। আন্তঃব্যক্তিক দক্ষতার সাথে বিবেচনা করার জন্য কয়েকটি গুরুত্বপূর্ণ বিষয় হ'ল গাইডিং, গঠনমূলক সমালোচনা এবং "হার্পুনিং"।

Mentoring

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

Constructive Criticism

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

"Harpooning"

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


নির্বাহী সারসংক্ষেপ

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


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

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

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

0

কোড পর্যালোচনাগুলি কোডের গুণমান উন্নত করার জন্য একটি সরঞ্জাম, অংশে ত্রুটিগুলি উদ্ঘাটন করে। আরও গুরুত্বপূর্ণ, আপনি ভাল কোডিং অনুশীলন স্থাপন করতে চান।

এই দৃষ্টিকোণ থেকে, ভাল করা জিনিস সম্পর্কে মন্তব্য করা গুরুত্বপূর্ণ। প্রশিক্ষণের প্রসঙ্গে উন্নতির বিষয়েও মন্তব্য করা উচিত। যদি আপনার সংস্কৃতি ক্রমাগত উন্নতি হয় তবে আপনার সর্বদা উন্নতিতে মন্তব্য করা উচিত।

ভুল, বাগ এবং খারাপ কোডিং হবে। এগুলিকে একটি ব্যাক্তিগতভাবে বিন্দুতে দেখান এবং প্রত্যাশার সাথে তাদের আচরণ করুন।

আচরণগত পরিবর্তনের দৃষ্টিকোণ থেকে, শাস্তির চেয়ে পরিবর্তন আনতে পুরষ্কার অনেক ভাল। আমি পুরস্কার হিসাবে ভাল কাজ লক্ষ্য করা বিবেচনা করব।

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