পিয়ার / কোড পর্যালোচনা হতাশা


27

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

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

আমি প্রায়শই একই সহকর্মীদের মত সমালোচনা মন্তব্যগুলি দেখতে পাই -

  • পরে একটি জায়গা যোগ করা ভাল হবে <INSERT SOMETHING HERE>
  • পদ্ধতির মধ্যে অযাচিত অতিরিক্ত লাইন
  • ডকব্লকগুলিতে মন্তব্যের শেষে ফুল স্টপ ব্যবহার করা উচিত।

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

  • আপনি এর মাধ্যমে সাইক্লোমেটিক জটিলতা হ্রাস করতে পারেন ...
  • তাড়াতাড়ি প্রস্থান করুন এবং / অন্যথায় এড়ানো
  • আপনার ডিবি ক্যোয়ারীটিকে কোনও ভাণ্ডার থেকে বিমূর্ত করুন
  • এই যুক্তিটি এখানে সত্যই অন্তর্ভুক্ত নয়
  • নিজেকে পুনরাবৃত্তি করবেন না - বিমূর্ত এবং পুনরায় ব্যবহার করুন
  • Xপদ্ধতির যুক্তি হিসাবে পাস হলে কী হবে Y?
  • এর জন্য ইউনিট পরীক্ষাটি কোথায়?

আমি দেখতে পেয়েছি যে এটি সর্বদা একই ধরণের লোক যারা প্রসাধনী ধরণের পর্যালোচনা দেয় এবং একই ধরণের লোকেরা যারা আমার মতে "গুণমান এবং যুক্তি ভিত্তিক" পিয়ার পর্যালোচনা দেয়।

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

যদি আমি সঠিক হয় - আমি কীভাবে সহকর্মীদের কসমেটিক টাচ-আপের পরামর্শ দিয়ে ভারসাম্যগুলিতে কোডের ত্রুটিগুলি খুঁজে পেতে উত্সাহিত করব?

যদি আমি ভুল হয় - দয়া করে আমাকে আলোকিত করুন। আসলে কোনও ভাল কোড পর্যালোচনা গঠনের জন্য কি থাম্বের কোনও নিয়ম রয়েছে? আমি কি কোড পর্যালোচনাগুলি এর বিন্দুটি মিস করেছি?

আমার দৃষ্টিকোণ থেকে - কোড পর্যালোচনা কোডের জন্য ভাগ করা দায়িত্ব সম্পর্কে about যুক্তি, পাঠযোগ্যতা এবং কার্যকারিতা সম্বোধন / পরীক্ষা করা ছাড়া কোডগুলিতে থাম্বস আপ দিতে স্বাচ্ছন্দ্যবোধ করব না। আমি যদি দৃ noticed় কোডের জন্য একীকরণকে আটকাতে বিরক্ত করব না তবে যদি আমি লক্ষ্য করি যে কোনও ডক-ব্লকে কোনও পুরো স্টপ বাদ দিয়েছে।

আমি যখন কোড পর্যালোচনা করি তখন আমি প্রতি 500 লোকের মধ্যে 15-45 মিনিটের মধ্যে ব্যয় করি। এই অগভীর পর্যালোচনাগুলি যদি তারা সম্পাদন করে তবে পর্যালোচনাটির গভীরতা যদি 10 মিনিটেরও বেশি সময় নেয় তবে আমি ভাবতে পারি না। আরও, অগভীর পর্যালোচকটির কাছ থেকে থাম্ব-আপের কত মূল্য? অবশ্যই এর অর্থ হ'ল সমস্ত থাম্বগুলি সমান ওজনের নয় এবং সম্ভবত এটি 2-পাসের পর্যালোচনা প্রক্রিয়া হওয়া দরকার। গভীর পর্যালোচনার জন্য একটি থাম্ব এবং "পলিশিং" এর জন্য দ্বিতীয় থাম্ব?


2
আপনি কি সেই ব্যক্তির সাথে এটি উল্লেখ করার চেষ্টা করেছেন?
ব্রায়ান ওকলে

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

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

2
অহঙ্কারী / অভিজাত পর্যালোচনা সহজ এবং প্রায় কোনও প্রচেষ্টা প্রয়োজন require একটি প্রযুক্তিগত পর্যালোচনা করতে আপনাকে কোডটি পড়তে হবে এবং বুঝতে হবে এবং তারপরে আরও ভাল সমাধানের জন্য চিন্তা করতে হবে ... এর অর্থ হল আপনাকে কাজ করতে হবে। আপনার প্রস্তাবের ফলাফল দেখে আমি অবাক হই না। কিছু অর্জনের জন্য আপনার পরিমাপযোগ্য এবং সঠিকভাবে সংজ্ঞায়িত কার্যগুলির সাথে পর্যালোচনা প্রক্রিয়াটি সংগঠিত করা উচিত।
জোলিনরোজ

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

উত্তর:


22

পর্যালোচনা প্রকার

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

পর্যালোচকদের প্রকার

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

পিয়ার পর্যালোচনা হ'ল সহযোগিতা, কোনও ট্যাগের খেলা নয়

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

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

পরামর্শ

আপনার প্রশ্নের শেষের দিকে আপনি লিখেছেন:

উজ্জ্বল নান্দনিক ত্রুটির সাথে ভারসাম্যহীনভাবে কোডের ত্রুটিগুলি খুঁজে পেতে আমি কীভাবে সহকর্মীদের উত্সাহিত করব?

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

আরও বাস্তববাদী নোটে, আমি ব্যক্তিগতভাবে কোড পর্যালোচনা মন্তব্যগুলিকে দুটি শিবিরে বিভক্ত করি এবং সেগুলি যথাযথভাবে বাক্যগুলি: যে জিনিসগুলি স্থির করতে হবে এবং যে জিনিসগুলি আরও প্রসাধনী। কোনও ফাইলের শেষে যদি অনেক বেশি ফাঁকা লাইন থাকে তবে আমি শক্ত, কার্যকারী কোডটি চেক ইন করা থেকে কখনই বাধা দেব না। তবে আমি এটি উল্লেখ করব, তবে আমি এই জাতীয় কিছু দিয়ে করব "আমাদের নির্দেশিকাগুলি শেষে বলেছে একটি ফাঁকা রেখা আছে, এবং আপনার কাছে 20 আছে It's এটি কোনও শো-স্টপার নয়, তবে আপনি যদি চান্স পান তবে আপনি এটি ঠিক করতে চাইবে "।

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

রিভিউ পরে কি করবেন

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


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

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

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

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

1
বেশিরভাগ লোকেরা যখন ভাষায় ত্রুটিগুলি তাদের দিকে নির্দেশিত হয় তখন তারা প্রশংসা করে যাতে তারা উন্নতি করতে পারে।
gnasher729

7

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

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

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

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

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


আমি আপনার মন্তব্যে আপনার সাথে একমত। এবং আপনি যদি ocean of shitআমার লেখা এটি দেখতে পান - তবে আমি আপনাকে পুনরায় লেখার পরামর্শ দিতে উত্সাহিত করব। তবে আপনি যদি উপেক্ষা করেন shitতবে আমাকে কসমেটিক জিনিসগুলি ঠিক করতে বলেন - যা আমাকে হতাশ করে ts
গ্রেভি

4

ব্যবহারিক উত্তর এখানে আসে:

পরিস্থিতি 1 : আপনি প্রবীণ সদস্য এবং অনুশীলনের সিদ্ধান্ত নিতে পারেন এমন কেউ

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

পরিস্থিতি 2 : আপনি অনুশীলনটি সিদ্ধান্ত নেন এমন কেউ নন বা আপনি দলের তুলনামূলকভাবে জুনিয়র সদস্য

আপনার সেরা বাজি হ'ল আপনি কোড 1 এ পৌঁছানোর ব্যবস্থা না করা পর্যন্ত কোড পর্যালোচনা সম্পর্কিত সমস্যাগুলি সমাধান করা ।


2

"তুচ্ছ" কোড পর্যালোচনা মন্তব্যগুলি রোধ করার সহজ উত্তরটি হ'ল (দক্ষতার জন্য) যে সেগুলি ঠিক করার জন্য পর্যালোচনা করা উচিত। সুতরাং যদি পর্যালোচনা করে দেখা যায় যে আপনি (হরর!) একটি ফুলস্টপ মিস করেছেন বা কোনও মন্তব্য ভুল করেছেন, তবে তাদের উচিত এটি ঠিক করে ঠিক করা।

আমার অভিজ্ঞতায় এর অর্থ এই:

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

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


0

পর্যালোচনা প্রক্রিয়া পর্যালোচনা

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

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

এর পরে, সংগঠন বা দলটি গুড কোড as হিসাবে কী অনুধাবন করে এবং কী এন্টি-প্যাটার্নগুলি এড়ানো উচিত সেগুলি (looseিলে )ালাভাবে) একটি মান এবং মানদণ্ডের একটি সেট আনুষ্ঠানিকভাবে গ্রহণ করা ভাল ধারণা হতে পারে। এটা sth স্থাপন সম্পর্কে নয়। কংক্রিট, কিন্তু সম্পর্কে একটি সাধারণ ঐক্যমত্য পেয়ে সম্পর্কে প্রথম থেকেই কোড মানের


0

যে কেউ এর দু'পাশে ছিলেন (অন্যের লিখিত কোড পর্যালোচনা করার পাশাপাশি আমি পর্যালোচনা করে লিখেছি কোড থাকা), আমি বলব যে আমার কাছে তিনটি বিভাগ রয়েছে যা কোনও প্রতিক্রিয়াতে আসে। ঠিক আছে, চার, "সব ঠিক আছে" এর আনন্দদায়ক ঘটনাও রয়েছে।

"চমৎকার লাগবে, তবে এটি আপনাকে অবরুদ্ধ করবে না" (যদি সমস্ত প্রতিক্রিয়া এই বিভাগে আসে তবে আমি "এক্সএক্সএক্সএক্সএক্সএক্সএক্সএলএক্সএলএক্সএতে মার্জ হয়ে যাবে") দিয়ে মার্জ করার অনুরোধটি প্রেরণ করতে পারি, যদি না আপনি আমাকে বলেন যে আপনি সেগুলি ঠিক করতে চান " , বা "নিশ্চিতভাবে, পরীক্ষা করতে এগিয়ে যান, সিস্টেমটি যেই ব্লকটি ফেলবে তা অক্ষম করা হয়েছে"):

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

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

  • মাইনর কুইবলস ("আপনি একটি বুলিয়ান সাথে তুলনা করছেন true, এটি কিছুটা ভৌতিক ...", ...)
  • গৌণ স্টাইলের লঙ্ঘন, ("স্টাইল গাইড এক্স বলেছেন, আপনি যা করেছেন তার বাইরে এটি; আপনি কোন পথে যেতে চান তার উপর নির্ভর করে আমি ওয়াই বা জেড করব")
  • কয়েকটি অনুপস্থিত পরীক্ষার কেস, এটি লেখা শক্ত হবে না

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

  • এটি "না, সত্যিই এটির লেখার আরও অনেক ভাল উপায় আছে", "আপনি" যা আশা করেন তা গণনা করছেন না, কারণ আপনার ইউনিট পরীক্ষায় এই প্রান্তের মামলাগুলি মিস হয় "এবং অন্যান্য সমস্ত গুরুতর সমস্যা রয়েছে like

0

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

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

যদি আপনি পর্যালোচনা প্রক্রিয়াটিকে আপনার সহকর্মীদের নাম লেখার বা তাদের জন্য কাজ তৈরি করার সুযোগ হিসাবে দেখেন তবে আপনি এটি ভুল করছেন।

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