ការវិភាគនៃសំណើបញ្ចូលគ្នានៅក្នុង GitLab ដោយប្រើ PVS-Studio សម្រាប់ C#

ស្រឡាញ់ GitLab និងស្អប់កំហុស? ចង់កែលម្អគុណភាពនៃកូដប្រភពរបស់អ្នកទេ? បន្ទាប់មកអ្នកបានមកដល់កន្លែងដែលត្រឹមត្រូវ។ ថ្ងៃនេះយើងនឹងប្រាប់អ្នកពីរបៀបកំណត់រចនាសម្ព័ន្ធឧបករណ៍វិភាគ PVS-Studio C# ដើម្បីពិនិត្យមើលសំណើបញ្ចូលគ្នា។ មានអារម្មណ៍ unicorn និងរីករាយក្នុងការអានដល់អ្នករាល់គ្នា។

ភី។ អេស។ អេស។ ស្ទូឌីយោ — គឺជាឧបករណ៍មួយសម្រាប់កំណត់អត្តសញ្ញាណកំហុស និងភាពងាយរងគ្រោះដែលអាចកើតមាននៅក្នុងកូដប្រភពនៃកម្មវិធីដែលសរសេរជា C, C++, C# និង Java។ វាដំណើរការលើប្រព័ន្ធ 64 ប៊ីត។ Windows, Linux и macOSវាអាចវិភាគកូដដែលកំណត់គោលដៅលើវេទិកា ARM 32-bit, 64-bit និង embedded។

ដោយវិធីនេះ យើងបានចេញផ្សាយ PVS-Studio 7.08 ដែលក្នុងនោះយើងបានធ្វើរឿងជាច្រើន។ គួរឱ្យចាប់អារម្មណ៍។ ឧទាហរណ៍ៈ

• អ្នកវិភាគ C# នៅក្រោម Linux и macOS;
• កម្មវិធីជំនួយសម្រាប់ Rider;
• របៀបពិនិត្យបញ្ជីឯកសារថ្មី។

របៀបពិនិត្យបញ្ជីឯកសារ

ពីមុន ដើម្បីពិនិត្យមើលឯកសារជាក់លាក់ ចាំបាច់ត្រូវឆ្លងកាត់ .xml ដែលមានបញ្ជីឯកសារទៅអ្នកវិភាគ។ ប៉ុន្តែដោយសារវាមិនងាយស្រួលខ្លាំង យើងបានបន្ថែមសមត្ថភាពក្នុងការផ្ទេរ .txt ដែលធ្វើឱ្យជីវិតសាមញ្ញបំផុត។

ដើម្បីពិនិត្យមើលឯកសារជាក់លាក់ អ្នកត្រូវតែបញ្ជាក់ទង់ --sourceFiles (-f) និងផ្ទេរ .txt ជាមួយនឹងបញ្ជីឯកសារ។ វាមើលទៅដូចនេះ៖

pvs-studio-dotnet -t path/to/solution.sln -f fileList.txt -o project.json

ប្រសិនបើអ្នកចាប់អារម្មណ៍ក្នុងការដំឡើង commit checking ឬទាញសំណើ អ្នកក៏អាចធ្វើវាបានដោយប្រើរបៀបនេះផងដែរ។ ភាពខុសគ្នានឹងស្ថិតនៅក្នុងការទទួលបានបញ្ជីឯកសារដើម្បីវិភាគ ហើយនឹងអាស្រ័យលើប្រព័ន្ធអ្វីដែលអ្នកកំពុងប្រើ។

គោលការណ៍នៃការត្រួតពិនិត្យការស្នើសុំបញ្ចូលគ្នា

ខ្លឹមសារសំខាន់នៃការត្រួតពិនិត្យគឺដើម្បីធានាថាបញ្ហាដែលបានរកឃើញដោយអ្នកវិភាគកំឡុងពេលបញ្ចូលគ្នាមិនធ្លាក់ចូលទៅក្នុង មេ សាខា។ យើងក៏មិនចង់វិភាគគម្រោងទាំងមូលរាល់ពេលដែរ។ លើសពីនេះ ពេលបញ្ចូលសាខា យើងមានបញ្ជីឯកសារដែលបានផ្លាស់ប្តូរ។ ដូច្នេះ ខ្ញុំស្នើឱ្យបន្ថែមការត្រួតពិនិត្យសំណើបញ្ចូលគ្នា។

នេះ​ជា​អ្វី​ដែល​សំណើ​បញ្ចូល​គ្នា​មើល​ទៅ​ដូច​ជា​មុន​ពេល​អនុវត្ត​កម្មវិធី​វិភាគ​ឋិតិវន្ត៖

នោះគឺកំហុសទាំងអស់ដែលមាននៅក្នុងសាខា ការផ្លាស់ប្តូរនឹងផ្លាស់ទីទៅសាខាមេ។ ដោយ​សារ​យើង​មិន​ចង់​បាន​វា យើង​បន្ថែម​ការ​វិភាគ ហើយ​ឥឡូវ​នេះ ដ្យាក្រាម​មើល​ទៅ​ដូច​នេះ៖

ការវិភាគ ការផ្លាស់ប្តូរ 2 ហើយប្រសិនបើមិនមានកំហុសទេ យើងទទួលយកសំណើបញ្ចូលគ្នា បើមិនដូច្នេះទេ យើងបដិសេធវា។

ដោយវិធីនេះ ប្រសិនបើអ្នកចាប់អារម្មណ៍ក្នុងការវិភាគការសន្យា និងទាញសំណើសម្រាប់ C/C++ នោះអ្នកអាចអានអំពីវា នៅទីនេះ.

GitLab

GitLab គឺជាឧបករណ៍ជីវិត DevOps ដែលមានមូលដ្ឋានលើបណ្តាញបើកចំហ ដែលផ្តល់ប្រព័ន្ធគ្រប់គ្រងឃ្លាំងកូដសម្រាប់ Git ជាមួយនឹងវិគីផ្ទាល់ខ្លួន ប្រព័ន្ធតាមដានបញ្ហា បំពង់បង្ហូរ CI/CD និងលក្ខណៈពិសេសផ្សេងទៀត។

មុនពេលអ្នកចាប់ផ្តើមវិភាគសំណើបញ្ចូលគ្នា អ្នកត្រូវចុះឈ្មោះ និងបង្ហោះគម្រោងរបស់អ្នក។ ប្រសិនបើអ្នកមិនដឹងពីរបៀបធ្វើវាទេនោះខ្ញុំណែនាំ អត្ថបទ មិត្តរួមការងាររបស់ខ្ញុំ។

ការកត់សម្គាល់. វិធីសាស្រ្តនៃការរៀបចំបរិស្ថានដែលបានពិពណ៌នាខាងក្រោមគឺជាវិធីមួយដែលអាចធ្វើទៅបាន។ គោលដៅគឺដើម្បីបង្ហាញពីជំហានសម្រាប់ការរៀបចំបរិយាកាសចាំបាច់សម្រាប់ការវិភាគ និងការបើកដំណើរការឧបករណ៍វិភាគ។ ប្រហែលជាក្នុងករណីរបស់អ្នក វាជាការប្រសើរបំផុតក្នុងការបែងចែកដំណាក់កាលនៃការរៀបចំបរិស្ថាន (បន្ថែមឃ្លាំង ដំឡើងឧបករណ៍វិភាគ) និងការវិភាគ៖ ឧទាហរណ៍ ការរៀបចំរូបភាព Docker ជាមួយនឹងបរិស្ថានចាំបាច់ និងការប្រើប្រាស់ពួកវា ឬវិធីសាស្រ្តផ្សេងទៀត។

ដើម្បីយល់កាន់តែច្បាស់អំពីអ្វីដែលនឹងកើតឡើងឥឡូវនេះ ខ្ញុំស្នើឱ្យមើលដ្យាក្រាមខាងក្រោម៖

ឧបករណ៍វិភាគតម្រូវឱ្យមាន .NET Core SDK 3 ដូច្នេះមុនពេលដំឡើងឧបករណ៍វិភាគ អ្នកត្រូវបន្ថែមឃ្លាំង Microsoft ដែលភាពអាស្រ័យរបស់ឧបករណ៍វិភាគនឹងត្រូវបានដំឡើង។ ការបន្ថែមឃ្លាំង Microsoft សម្រាប់ការចែកចាយផ្សេងៗ។ Linux បានពិពណ៌នានៅក្នុងឯកសារដែលត្រូវគ្នា។.

ដើម្បីដំឡើង PVS-Studio តាមរយៈកម្មវិធីគ្រប់គ្រងកញ្ចប់ អ្នកក៏នឹងត្រូវបន្ថែមឃ្លាំង PVS-Studio ផងដែរ។ ការបន្ថែមឃ្លាំងសម្រាប់ការចែកចាយផ្សេងៗគ្នាត្រូវបានពិពណ៌នាលម្អិតបន្ថែមទៀតនៅក្នុង ផ្នែកពាក់ព័ន្ធនៃឯកសារ.

ឧបករណ៍វិភាគទាមទារសោអាជ្ញាប័ណ្ណដើម្បីដំណើរការ។ អ្នកអាចទទួលបានអាជ្ញាប័ណ្ណសាកល្បងនៅ ទំព័រទាញយកកម្មវិធីវិភាគ.

ការកត់សម្គាល់. សូមចំណាំថារបៀបប្រតិបត្តិការដែលបានពិពណ៌នា (ការវិភាគសំណើបញ្ចូលគ្នា) ទាមទារអាជ្ញាប័ណ្ណសហគ្រាស។ ដូច្នេះហើយ ប្រសិនបើអ្នកចង់សាកល្បងរបៀបប្រតិបត្តិការនេះ កុំភ្លេចបង្ហាញនៅក្នុងវាល "សារ" ថាអ្នកត្រូវការអាជ្ញាប័ណ្ណសហគ្រាស។

ប្រសិនបើសំណើបញ្ចូលគ្នាកើតឡើង នោះយើងគ្រាន់តែត្រូវការវិភាគបញ្ជីឯកសារដែលបានផ្លាស់ប្តូរ បើមិនដូច្នេះទេយើងវិភាគឯកសារទាំងអស់។ បន្ទាប់ពីការវិភាគ យើងត្រូវបំប្លែងកំណត់ហេតុទៅជាទម្រង់ដែលយើងត្រូវការ។

ឥឡូវ​នេះ ដោយ​មាន​ក្បួន​ដោះស្រាយ​ការងារ​នៅ​ចំពោះ​មុខ​អ្នក អ្នក​អាច​បន្ត​ទៅ​សរសេរ​ស្គ្រីប​បាន។ ដើម្បីធ្វើដូចនេះអ្នកត្រូវផ្លាស់ប្តូរឯកសារ .gitlab-ci.yml ឬប្រសិនបើវាមិនមានទេ បង្កើតវា។ ដើម្បីបង្កើតវាអ្នកត្រូវចុចលើឈ្មោះគម្រោងរបស់អ្នក -> ដំឡើង CI/CD.

ឥឡូវនេះយើងត្រៀមខ្លួនរួចរាល់ហើយដើម្បីសរសេរស្គ្រីប។ ដំបូងយើងសរសេរកូដដែលនឹងដំឡើងឧបករណ៍វិភាគ ហើយបញ្ចូលអាជ្ញាប័ណ្ណ៖

before_script:
  - apt-get update && apt-get -y install wget gnupg 

  - apt-get -y install git
  - wget https://packages.microsoft.com/config/debian/10/
packages-microsoft-prod.deb -O packages-microsoft-prod.deb
  - dpkg -i packages-microsoft-prod.deb
  - apt-get update
  - apt-get install apt-transport-https
  - apt-get update
  
  - wget -q -O - https://files.viva64.com/etc/pubkey.txt | apt-key add -
  - wget -O /etc/apt/sources.list.d/viva64.list
https://files.viva64.com/etc/viva64.list
  - apt-get update
  - apt-get -y install pvs-studio-dotnet

  - pvs-studio-analyzer credentials $PVS_NAME $PVS_KEY
  - dotnet restore "$CI_PROJECT_DIR"/Test/Test.sln

ដោយសារការដំឡើង និងការធ្វើឱ្យសកម្មត្រូវតែកើតឡើងមុនពេលស្គ្រីបផ្សេងទៀតទាំងអស់ យើងប្រើស្លាកពិសេស before_script. ខ្ញុំសូមពន្យល់ពីបំណែកនេះបន្តិច។

កំពុងរៀបចំដំឡើងឧបករណ៍វិភាគ៖

  - wget https://packages.microsoft.com/config/debian/10/
packages-microsoft-prod.deb -O packages-microsoft-prod.deb
  - dpkg -i packages-microsoft-prod.deb
  - apt-get update
  - apt-get install apt-transport-https
  - apt-get update

ការបន្ថែមឃ្លាំង PVS-Studio និងឧបករណ៍វិភាគ៖

  - wget -q -O - https://files.viva64.com/etc/pubkey.txt | apt-key add -
  - wget -O /etc/apt/sources.list.d/viva64.list
https://files.viva64.com/etc/viva64.list
  - apt-get update
  - apt-get -y install pvs-studio-dotnet

ការធ្វើឱ្យសកម្មអាជ្ញាប័ណ្ណ៖

  - pvs-studio-analyzer credentials $PVS_NAME $PVS_KEY

$PVS_NAME - ឈ្មោះ​អ្នកប្រើប្រាស់។

$PVS_KEY - លេខ​កូ​ខ​ទំនិញ។

ការងើបឡើងវិញនូវភាពអាស្រ័យរបស់គម្រោងដែលជាកន្លែងដែល $CI_PROJECT_DIR - ផ្លូវពេញទៅកាន់ថតគម្រោង៖

  - dotnet restore "$CI_PROJECT_DIR"/Path/To/Solution.sln

សម្រាប់ការវិភាគត្រឹមត្រូវ គម្រោងត្រូវតែត្រូវបានសាងសង់ដោយជោគជ័យ ហើយភាពអាស្រ័យរបស់វាត្រូវតែត្រូវបានស្ដារឡើងវិញ (ឧទាហរណ៍ កញ្ចប់ NuGet ចាំបាច់ត្រូវតែទាញយក)។

អ្នកអាចកំណត់អថេរបរិស្ថានដែលមានព័ត៌មានអាជ្ញាប័ណ្ណដោយចុច ការកំណត់, និងបន្ទាប់ពី - នៅលើ CI/CD.

នៅក្នុងបង្អួចដែលបើកសូមស្វែងរកធាតុ អថេរចុចលើប៊ូតុងនៅខាងស្តាំ ពង្រីក និងបន្ថែមអថេរ។ លទ្ធផលគួរតែមើលទៅដូចនេះ៖

ឥឡូវនេះអ្នកអាចបន្តទៅការវិភាគ។ ជាដំបូង ចូរយើងបន្ថែមស្គ្រីបសម្រាប់ការវិភាគពេញលេញ។ ដល់ទង់ជាតិ -t យើងឆ្លងកាត់ផ្លូវទៅកាន់ដំណោះស្រាយទៅទង់ -o សរសេរផ្លូវទៅកាន់ឯកសារដែលលទ្ធផលនៃការវិភាគនឹងត្រូវបានសរសេរ។ យើងក៏ចាប់អារម្មណ៍នឹងលេខកូដត្រឡប់មកវិញដែរ។ ក្នុងករណីនេះ យើងចាប់អារម្មណ៍លើការបញ្ឈប់ប្រតិបត្តិការ នៅពេលដែលលេខកូដត្រឡប់មកវិញមានព័ត៌មានដែលការព្រមានត្រូវបានចេញកំឡុងពេលវិភាគ។ នេះជាអ្វីដែលបំណែកនេះមើលទៅ៖

job:
  script:
  - exit_code=0
  - pvs-studio-dotnet -t "$CI_PROJECT_DIR"/Test/Test.sln -o 
PVS-Studio.json || exit_code=$?
  - exit_code=$((($exit_code & 8)/8))
  - if [[ $exit_code == 1 ]]; then exit 1; else exit 0; fi

កូដ​ត្រឡប់​ដំណើរការ​លើ​គោលការណ៍ bitmask។ ឧទាហរណ៍ ប្រសិនបើ​ការវិភាគ​បាន​បង្ហាញ​ពី​ការព្រមាន កូដ​ត្រឡប់​នឹង​មាន 8។ ប្រសិនបើ​អាជ្ញាប័ណ្ណ​ផុតកំណត់​ក្នុង​រយៈពេល​មួយខែ កូដ​ត្រឡប់​នឹង​មាន 4។ ប្រសិនបើ​ការវិភាគ​រកឃើញ​កំហុស ហើយ​អាជ្ញាប័ណ្ណ​ផុតកំណត់​ក្នុង​រយៈពេល​មួយខែ កូដ​ត្រឡប់​នឹង​មាន​តម្លៃ​ទាំងពីរ៖ បូក​លេខ​ជាមួយគ្នា​ដើម្បី​ទទួលបាន​កូដ​ត្រឡប់​ចុងក្រោយ—8 + 4 = 12។ ដូច្នេះ ដោយ​ការត្រួតពិនិត្យ​ប៊ីត​ដែលត្រូវគ្នា អ្នកអាច​ទទួលបាន​ព័ត៌មាន​អំពី​ស្ថានភាព​ផ្សេងៗ​ក្នុងអំឡុងពេល​វិភាគ។ កូដ​ត្រឡប់​ត្រូវបាន​ពិពណ៌នា​លម្អិត​បន្ថែម​នៅក្នុង​ផ្នែក "កូដ​ត្រឡប់ pvs-studio-dotnet"។Linux / macOS) "ឯកសារ"កំពុងពិនិត្យមើលគម្រោង Visual Studio / MSBuild / .NET Core ពីបន្ទាត់ពាក្យបញ្ជាដោយប្រើ PVS-Studio".

ក្នុងករណីនេះយើងចាប់អារម្មណ៍លើលេខកូដត្រឡប់មកវិញទាំងអស់ដែលលេខ 8 លេចឡើង។

  - exit_code=$((($exit_code & 8)/8))

យើងនឹងទទួលបាន 1 នៅពេលដែលលេខកូដត្រឡប់មានប៊ីតនៃលេខដែលយើងចាប់អារម្មណ៍ បើមិនដូច្នេះទេ យើងនឹងទទួលបាន 0 ។

វាដល់ពេលហើយដើម្បីបន្ថែមការវិភាគសំណើបញ្ចូលគ្នា។ មុននឹងយើងធ្វើនេះ យើងរៀបចំកន្លែងសម្រាប់ស្គ្រីប។ យើង​ត្រូវ​ការ​វា​ដើម្បី​ប្រតិបត្តិ​តែ​នៅ​ពេល​ដែល​សំណើ​បញ្ចូល​គ្នា​កើត​ឡើង។ វាមើលទៅដូចនេះ៖

merge:
  script:
  only:
  - merge_requests

ចូរបន្តទៅស្គ្រីបខ្លួនឯង។ ខ្ញុំត្រូវប្រឈមមុខនឹងការពិតដែលថាម៉ាស៊ីននិម្មិតមិនដឹងអ្វីទាំងអស់។ ប្រភពដើម / មេ. ដូច្នេះសូមជួយនាងបន្តិច៖

  - git fetch origin

ឥឡូវនេះយើងទទួលបានភាពខុសគ្នារវាងសាខានិងរក្សាទុកលទ្ធផលនៅក្នុង txt ឯកសារ៖

  - git diff --name-only origin/master $CI_COMMIT_SHA > pvs-fl.txt

កន្លែងណា $CI_COMMIT_SHA - សញ្ញានៃការប្តេជ្ញាចិត្តចុងក្រោយ។

បន្ទាប់យើងចាប់ផ្តើមវិភាគបញ្ជីឯកសារដោយប្រើទង់ -f. យើងផ្ទេរឯកសារ .txt ដែលបានទទួលពីមុនទៅវា។ ជាការប្រសើរណាស់, ដោយការប្រៀបធៀបជាមួយនឹងការវិភាគពេញលេញ, យើងពិនិត្យមើលលេខកូដត្រឡប់មកវិញ:

  - exit_code=0
  - pvs-studio-dotnet -t "$CI_PROJECT_DIR"/Test/Test.sln -f 
pvs-fl.txt -o PVS-Studio.json || exit_code=$?
  - exit_code=$((($exit_code & 8)/8))
  - if [[ $exit_code == 1 ]]; then exit 1; else exit 0; fi

ស្គ្រីបពេញលេញសម្រាប់ពិនិត្យមើលសំណើបញ្ចូលគ្នានឹងមើលទៅដូចនេះ៖

merge:
  script:
  - git fetch origin
  - git diff --name-only origin/master $CI_COMMIT_SHA > pvs-fl.txt
  - exit_code=0
  - pvs-studio-dotnet -t "$CI_PROJECT_DIR"/Test/Test.sln -f 
pvs-fl.txt -o PVS-Studio.json || exit_code=$?
  - exit_code=$((($exit_code & 8)/8))
  - if [[ $exit_code == 1 ]]; then exit 1; else exit 0; fi
  only:
  - merge_requests

អ្វីដែលនៅសេសសល់គឺត្រូវបន្ថែមការបម្លែងកំណត់ហេតុ បន្ទាប់ពីស្គ្រីបទាំងអស់ត្រូវបានដំណើរការ។ យើងប្រើស្លាក after_script និងឧបករណ៍ប្រើប្រាស់ កម្មវិធីបម្លែង plog:

after_script:
  - plog-converter -t html -o eLog ./PVS-Studio.json

ឧបករណ៍ប្រើប្រាស់ កម្មវិធីបម្លែង plog គឺជាគម្រោងប្រភពបើកចំហដែលត្រូវបានប្រើដើម្បីបំប្លែងរបាយការណ៍កំហុសញែកទៅជាទម្រង់ផ្សេងៗដូចជា HTML ជាដើម។ ការពិពណ៌នាលម្អិតបន្ថែមទៀតអំពីឧបករណ៍ប្រើប្រាស់ត្រូវបានផ្តល់ឱ្យនៅក្នុងផ្នែករង "ឧបករណ៍បំប្លែង Plog" ផ្នែកពាក់ព័ន្ធនៃឯកសារ.

ដោយវិធីនេះ ប្រសិនបើអ្នកចង់ធ្វើការយ៉ាងងាយស្រួលជាមួយរបាយការណ៍ .json ក្នុងស្រុកពី IDE នោះខ្ញុំស្នើឱ្យយើងណែនាំ កម្មវិធីជំនួយ សម្រាប់ IDE Rider ។ ការប្រើប្រាស់របស់វាត្រូវបានពិពណ៌នាលម្អិតបន្ថែមទៀតនៅក្នុង ឯកសារពាក់ព័ន្ធ.

ដើម្បីភាពងាយស្រួលគឺនៅទីនេះ .gitlab-ci.yml ពេញ៖

image: debian

before_script:
  - apt-get update && apt-get -y install wget gnupg 

  - apt-get -y install git
  - wget https://packages.microsoft.com/config/debian/10/
packages-microsoft-prod.deb -O packages-microsoft-prod.deb
  - dpkg -i packages-microsoft-prod.deb
  - apt-get update
  - apt-get install apt-transport-https
  - apt-get update
  
  - wget -q -O - https://files.viva64.com/etc/pubkey.txt | apt-key add -
  - wget -O /etc/apt/sources.list.d/viva64.list
https://files.viva64.com/etc/viva64.list
  - apt-get update
  - apt-get -y install pvs-studio-dotnet

  - pvs-studio-analyzer credentials $PVS_NAME $PVS_KEY
  - dotnet restore "$CI_PROJECT_DIR"/Test/Test.sln

merge:
  script:
  - git fetch origin
  - git diff --name-only origin/master $CI_COMMIT_SHA > pvs-fl.txt
  - exit_code=0
  - pvs-studio-dotnet -t "$CI_PROJECT_DIR"/Test/Test.sln -f 
pvs-fl.txt -o PVS-Studio.json || exit_code=$?
  - exit_code=$((($exit_code & 8)/8))
  - if [[ $exit_code == 1 ]]; then exit 1; else exit 0; fi
  only:
  - merge_requests

job:
  script:
  - exit_code=0
  - pvs-studio-dotnet -t "$CI_PROJECT_DIR"/Test/Test.sln -o 
PVS-Studio.json || exit_code=$?
  - exit_code=$((($exit_code & 8)/8))
  - if [[ $exit_code == 1 ]]; then exit 1; else exit 0; fi
  
after_script:
  - plog-converter -t html -o eLog ./PVS-Studio.json

នៅពេលដែលអ្នកបានបន្ថែមអ្វីគ្រប់យ៉ាងទៅក្នុងឯកសារសូមចុចលើ ធ្វើការផ្លាស់ប្តូរ. ដើម្បី​មើល​ថា​អ្វី​គ្រប់​យ៉ាង​គឺ​ត្រូវ​ទៅ​កាន់​ ស៊ីអាយ / ស៊ីឌី -> បំពង់ -> ការរត់. បង្អួចម៉ាស៊ីននិម្មិតនឹងបើក ដែលនៅចុងបញ្ចប់គួរតែមានដូចខាងក្រោម៖

ស ការងារបានជោគជ័យ - ជោគជ័យអ្វីគ្រប់យ៉ាងគឺល្អ។ ឥឡូវនេះអ្នកអាចសាកល្បងអ្វីដែលអ្នកបានធ្វើ។

ឧទាហរណ៍នៃការងារ

សម្រាប់ឧទាហរណ៍នៃការងារ ចូរយើងបង្កើតគម្រោងសាមញ្ញមួយ (ក្នុង មេ) ដែលនឹងមានឯកសារជាច្រើន។ បន្ទាប់ពីនោះ នៅក្នុងសាខាមួយទៀត យើងនឹងផ្លាស់ប្តូរឯកសារតែមួយ ហើយព្យាយាមធ្វើការស្នើសុំបញ្ចូលគ្នា។

ចូរយើងពិចារណាករណីពីរ៖ នៅពេលដែលឯកសារដែលបានកែប្រែមានកំហុស និងនៅពេលដែលវាមិនមាន។ ទីមួយឧទាហរណ៍ជាមួយកំហុសមួយ។

ចូរនិយាយថាមានឯកសារមួយនៅក្នុងសាខាមេ កម្មវិធី.csដែលមិនមានកំហុស ប៉ុន្តែនៅក្នុងសាខាមួយផ្សេងទៀត អ្នកអភិវឌ្ឍន៍បានបន្ថែមកូដដែលមានកំហុស ហើយចង់ធ្វើការស្នើសុំបញ្ចូលគ្នា។ តើ​កំហុស​អ្វី​ដែល​គាត់​ធ្វើ​មិន​សំខាន់​នោះ​ទេ រឿង​សំខាន់​គឺ​វា​មាន។ ឧទាហរណ៍ប្រតិបត្តិករភ្លេច បោះ (បាទ! ខុស):

void MyAwesomeMethod(String name)
{
  if (name == null)
    new ArgumentNullException(....);
  // do something
  ....
}

សូមក្រឡេកមើលលទ្ធផលនៃការវិភាគឧទាហរណ៍ជាមួយកំហុស។ ដើម្បីប្រាកដថាមានតែឯកសារមួយប៉ុណ្ណោះដែលត្រូវបានញែក ខ្ញុំបន្ថែមទង់ -r ទៅកាន់បន្ទាត់ចាប់ផ្តើម pvs-studio-dotnet៖

យើងឃើញថាអ្នកវិភាគបានរកឃើញកំហុសមួយ និងមិនអនុញ្ញាតឱ្យមានការបញ្ចូលគ្នានៃសាខា។

តោះពិនិត្យមើលឧទាហរណ៍ដោយគ្មានកំហុស។ ការកែកូដ៖

void MyAwesomeMethod(String name)
{
  if (name == null)
    throw new ArgumentNullException(....);
  // do something
  ....
}

បញ្ចូលលទ្ធផលការវិភាគសំណើ៖

ដូចដែលយើងឃើញហើយ គ្មានកំហុសត្រូវបានរកឃើញទេ ហើយការប្រតិបត្តិភារកិច្ចបានជោគជ័យ ដែលជាអ្វីដែលយើងចង់ពិនិត្យ។

សេចក្តីសន្និដ្ឋាន

ការដក​កូដ​អាក្រក់​ចេញ​មុន​ពេល​រួម​បញ្ចូល​មែក​គឺ​ងាយ​ស្រួល​និង​រីករាយ។ ដូច្នេះប្រសិនបើអ្នកកំពុងប្រើ CI/CD សូមសាកល្បងបង្កប់ឧបករណ៍វិភាគឋិតិវន្តដើម្បីពិនិត្យ។ លើសពីនេះទៀតនេះត្រូវបានធ្វើយ៉ាងសាមញ្ញ។

សូមអរគុណចំពោះការយកចិត្តទុកដាក់របស់អ្នក។

ប្រសិនបើអ្នកចង់ចែករំលែកអត្ថបទនេះជាមួយទស្សនិកជនដែលនិយាយភាសាអង់គ្លេស សូមប្រើតំណបកប្រែ៖ Nikolay Mironov ។ ការវិភាគនៃសំណើបញ្ចូលគ្នានៅក្នុង GitLab ដោយប្រើ PVS-Studio សម្រាប់ C#.

ប្រភព: www.habr.com