cari
Semakan kod pertamaSep 14, 2024 am 06:28 AM

Semalam saya telah menyemak kod saya oleh orang lain buat kali pertama. Sebagai salah satu tugasan kami untuk kursus pembangunan sumber terbuka yang saya ambil, kami terpaksa menyemak kod masing-masing. Untuk latihan ini saya berpasangan dengan Vinh, rakan saya yang juga seorang pengaturcara yang cukup baik. Kami ditugaskan untuk menguji dan memfailkan isu pada kerja masing-masing untuk alat baris arahan yang setiap satu telah kami usahakan dalam kursus.

First code review

Vinh Nhan

/vinhyan

Pendekatan Asynchronous vs Synchronous

Kami melakukan semakan kod kami secara serentak melalui teks dan tidak segerak atas isu GitHub. Saya mendapati pendekatan segerak membawa kepada hasil yang lebih pantas kerana saya boleh merujuk pengarang kod tentang sebab mereka mengambil pendekatan tertentu semasa menulis kod mereka, dan segera mendapat respons. Walau bagaimanapun, pendekatan tak segerak menghilangkan keperluan untuk mencari masa tetap dalam jadual kedua-dua orang untuk menyelesaikan kerja.

Menguji program Vinh

Vinh mencipta alat baris perintah yang dipanggil tanpa halangan yang menggunakan AI untuk menterjemah frasa teks ke dalam bahasa lain, yang saya fikir adalah idea yang bagus. Apabila saya mula menguji program Vinh, ia masih dalam pembangunan awal, jadi ia belum mempunyai README lagi (ia ada sekarang, pergi semaknya!).

First code review vinhyan / tanpa halangan

Apa itu Tanpa Penghalang

Barrierless ialah alat baris arahan yang direka untuk memecahkan halangan bahasa dengan menyediakan terjemahan lancar dari satu bahasa ke bahasa lain. Dikuasakan oleh GROQCloud, alat ini membolehkan pengguna menterjemah teks dengan cepat ke dalam bahasa sasaran yang diingini, menjadikan komunikasi merentas bahasa berbeza menjadi mudah.

Ciri

  • Auto mengesan bahasa.
  • Sokongan Pelbagai Bahasa: Terjemah teks antara pelbagai bahasa.
  • Penyepaduan GROQCloud: Menggunakan API terjemahan berprestasi tinggi GROQCloud.
  • Mudah Digunakan: Antara muka baris perintah yang ringkas untuk terjemahan pantas.
  • Boleh disesuaikan: Boleh dilanjutkan dengan mudah untuk ciri bahasa tambahan atau sokongan API.

Cara menggunakan

Pemasangan

  1. Klon repositori dan navigasi ke direktori projek:
git clone git@github.com:vinhyan/barrierless.git
  1. Navigasi ke direktori projek:
cd barrierless
  1. Pasang kebergantungan yang diperlukan:
npm install
  1. Buat fail .env untuk menyimpan Kunci API Groq
    Nota: rujuk .env.contoh untuk arahan tentang cara mendapatkan dan menyimpan Kunci API Groq

  2. Tinggalkan langkah ini jika npm install -g digunakan dalam langkah 3…

Lihat di GitHub

A feature I really liked is the colorful output text which makes the user experience a little bit more pleasant - something I neglected in my own program in trying to model it after CLI tools like git.

I read the package.json file to find out how the program should be run, and when it immediately crashed I realized I forgot to add the API key as an environment variable. After adding my API key, the program ran without errors, although I did find an interesting quirk - the program defaults the output language to English, so if you didn't specify one, and the input was in English, it seemed to choose a language to translate to on its own - either randomly, or based on context from the input.

First code review

First code review

I opened a few other issues, mostly to do with improving code quality:

  • A missing try/catch block around an async function call

Uncaught exception in index.js #7

First code review
uday-rana posted on

index.js contains the following async function calls which are not wrapped in a try/catch block and may lead to an uncaught exception:

export async function main(text, targetLang) {
  const chatCompletion = await getGroqChatCompletion(text, targetLang);
  console.log(chatCompletion.choices[0]?.message?.content || '');
}

...

program
  ...
  .action(async (text, options) => {
    console.log(chalk.blue(`Translating  <span class="pl-s1"><span class="pl-kos">${text}</span>...`</span>));

    await main(text, options.language);
  });
Enter fullscreen mode Exit fullscreen mode
View on GitHub
  • Some suggestions to make code easier to understand

Could simplify code #8

First code review
uday-rana posted on

Some changes may be made to to the project make it easier to understand and work on:

  • [x] Move Groq configuration above program initialization with commander
  • [x] main() seems unnecessary since it contains two lines of code and there are more lines of code involved in creating and invoking the function than if it was omitted
  • [ ] prompt.js seems unnecessary since it just contains a single function which places arguments into a template literal and returns them
  • [x] Exporting main() and getGroqChatCompletion() seems unnecessary
View on GitHub
  • Adding a comment to explain the use of both import and require statements

Add comments explaining mixed import/require #9

First code review
uday-rana posted on

Projek ini menggunakan import ES6 dan keperluan CommonJS kerana modul kapur yang memerlukan penggunaan import dan menggunakan import untuk package.json yang membawa kepada ralat. Adalah berguna untuk menambah ulasan yang menerangkan perkara ini.

Lihat di GitHub

giliran saya

Seterusnya giliran saya untuk disemak. Saya tidak pasti jenis isu yang akan timbul, tetapi Vinh akhirnya menemui banyak isu yang saya tidak ambil perhatian:

  • Menambah pautan npm sebagai pilihan lain kerana tidak perlu memberi awalan alat dengan nod dalam arahan dalam README

README.md tidak termasuk arahan untuk menjalankan `pautan npm` #2

First code review
vinhyan disiarkan pada

Fail README.md tiada arahan untuk menjalankan pautan npm, yang diperlukan untuk pembangunan tempatan dan ujian alat CLI

Lihat di GitHub
  • Penugasan arahan yang tidak perlu menggunakan commander.js

`program.command("run")` tidak diperlukan kerana CLI tidak mempunyai subperintah #3

First code review
vinhyan disiarkan pada
Lihat di GitHub
  • Taip nama berubah

Nama boleh ubah kesilapan menaip #4

First code review
vinhyan disiarkan pada

baris index.js 31: kesilapan menaip dalam nama berubah: reponseStream

Lihat di GitHub

Kesimpulan

Saya fikir saya telah melakukan kerja yang cukup baik tetapi menunjukkan bahawa sentiasa ada pepijat yang mungkin terlepas atau ciri yang boleh diperbaiki. Ia sangat bagus apabila sepasang mata segar meneliti kod yang saya tulis. Buat masa ini, saya membetulkan kesilapan menaip dan mengemas kini README, tetapi isu lain memerlukan ujian dan saya merancang untuk mendapatkannya sebelum saya mengeluarkan versi 0.1.

Atas ialah kandungan terperinci Semakan kod pertama. Untuk maklumat lanjut, sila ikut artikel berkaitan lain di laman web China PHP!

Kenyataan
Kandungan artikel ini disumbangkan secara sukarela oleh netizen, dan hak cipta adalah milik pengarang asal. Laman web ini tidak memikul tanggungjawab undang-undang yang sepadan. Jika anda menemui sebarang kandungan yang disyaki plagiarisme atau pelanggaran, sila hubungi admin@php.cn
Ganti aksara rentetan dalam javascriptGanti aksara rentetan dalam javascriptMar 11, 2025 am 12:07 AM

Penjelasan terperinci mengenai kaedah penggantian rentetan javascript dan Soalan Lazim Artikel ini akan meneroka dua cara untuk menggantikan watak rentetan dalam JavaScript: Kod JavaScript dalaman dan HTML dalaman untuk laman web. Ganti rentetan di dalam kod JavaScript Cara yang paling langsung ialah menggunakan kaedah pengganti (): str = str.replace ("cari", "ganti"); Kaedah ini hanya menggantikan perlawanan pertama. Untuk menggantikan semua perlawanan, gunakan ungkapan biasa dan tambahkan bendera global g: str = str.replace (/fi

Tutorial Persediaan API Carian Google CustomTutorial Persediaan API Carian Google CustomMar 04, 2025 am 01:06 AM

Tutorial ini menunjukkan kepada anda bagaimana untuk mengintegrasikan API carian Google tersuai ke dalam blog atau laman web anda, menawarkan pengalaman carian yang lebih halus daripada fungsi carian tema WordPress standard. Ia menghairankan mudah! Anda akan dapat menyekat carian ke y

Contoh warna json failContoh warna json failMar 03, 2025 am 12:35 AM

Siri artikel ini ditulis semula pada pertengahan 2017 dengan maklumat terkini dan contoh segar. Dalam contoh JSON ini, kita akan melihat bagaimana kita dapat menyimpan nilai mudah dalam fail menggunakan format JSON. Menggunakan notasi pasangan nilai utama, kami boleh menyimpan apa-apa jenis

10 JQuery Syntax Highlighters10 JQuery Syntax HighlightersMar 02, 2025 am 12:32 AM

Tingkatkan Penyampaian Kod Anda: 10 Penyeret Sintaks untuk Pemaju Coretan kod perkongsian di laman web atau blog anda adalah amalan biasa bagi pemaju. Memilih penyapu sintaks yang betul dapat meningkatkan daya tarikan dan daya tarikan visual dengan ketara. T

Bina Aplikasi Web Ajax anda sendiriBina Aplikasi Web Ajax anda sendiriMar 09, 2025 am 12:11 AM

Jadi di sini anda, bersedia untuk mempelajari semua perkara ini yang dipanggil Ajax. Tetapi, apa sebenarnya? Istilah Ajax merujuk kepada kumpulan teknologi longgar yang digunakan untuk membuat kandungan web yang dinamik dan interaktif. Istilah Ajax, yang asalnya dicipta oleh Jesse J

8 plugin susun atur halaman jquery yang menakjubkan8 plugin susun atur halaman jquery yang menakjubkanMar 06, 2025 am 12:48 AM

Leverage JQuery untuk Layouts Laman Web yang mudah: 8 Plugin Essential JQuery memudahkan susun atur laman web dengan ketara. Artikel ini menyoroti lapan plugin jQuery yang kuat yang menyelaraskan proses, terutamanya berguna untuk penciptaan laman web manual

10 JavaScript & JQuery MVC Tutorial10 JavaScript & JQuery MVC TutorialMar 02, 2025 am 01:16 AM

Artikel ini membentangkan pemilihan lebih daripada 10 tutorial mengenai rangka kerja javascript dan jquery model-view-controller (MVC), sesuai untuk meningkatkan kemahiran pembangunan web anda pada tahun baru. Tutorial ini merangkumi pelbagai topik, dari Foundatio

Apa itu ' ini ' Dalam JavaScript?Apa itu ' ini ' Dalam JavaScript?Mar 04, 2025 am 01:15 AM

Mata teras Ini dalam JavaScript biasanya merujuk kepada objek yang "memiliki" kaedah, tetapi ia bergantung kepada bagaimana fungsi dipanggil. Apabila tidak ada objek semasa, ini merujuk kepada objek global. Dalam penyemak imbas web, ia diwakili oleh tetingkap. Apabila memanggil fungsi, ini mengekalkan objek global; tetapi apabila memanggil pembina objek atau mana -mana kaedahnya, ini merujuk kepada contoh objek. Anda boleh mengubah konteks ini menggunakan kaedah seperti panggilan (), memohon (), dan mengikat (). Kaedah ini memanggil fungsi menggunakan nilai dan parameter yang diberikan. JavaScript adalah bahasa pengaturcaraan yang sangat baik. Beberapa tahun yang lalu, ayat ini

See all articles

Alat AI Hot

Undresser.AI Undress

Undresser.AI Undress

Apl berkuasa AI untuk mencipta foto bogel yang realistik

AI Clothes Remover

AI Clothes Remover

Alat AI dalam talian untuk mengeluarkan pakaian daripada foto.

Undress AI Tool

Undress AI Tool

Gambar buka pakaian secara percuma

Clothoff.io

Clothoff.io

Penyingkiran pakaian AI

AI Hentai Generator

AI Hentai Generator

Menjana ai hentai secara percuma.

Alat panas

Dreamweaver CS6

Dreamweaver CS6

Alat pembangunan web visual

SecLists

SecLists

SecLists ialah rakan penguji keselamatan muktamad. Ia ialah koleksi pelbagai jenis senarai yang kerap digunakan semasa penilaian keselamatan, semuanya di satu tempat. SecLists membantu menjadikan ujian keselamatan lebih cekap dan produktif dengan menyediakan semua senarai yang mungkin diperlukan oleh penguji keselamatan dengan mudah. Jenis senarai termasuk nama pengguna, kata laluan, URL, muatan kabur, corak data sensitif, cangkerang web dan banyak lagi. Penguji hanya boleh menarik repositori ini ke mesin ujian baharu dan dia akan mempunyai akses kepada setiap jenis senarai yang dia perlukan.

Pelayar Peperiksaan Selamat

Pelayar Peperiksaan Selamat

Pelayar Peperiksaan Selamat ialah persekitaran pelayar selamat untuk mengambil peperiksaan dalam talian dengan selamat. Perisian ini menukar mana-mana komputer menjadi stesen kerja yang selamat. Ia mengawal akses kepada mana-mana utiliti dan menghalang pelajar daripada menggunakan sumber yang tidak dibenarkan.

EditPlus versi Cina retak

EditPlus versi Cina retak

Saiz kecil, penyerlahan sintaks, tidak menyokong fungsi gesaan kod

mPDF

mPDF

mPDF ialah perpustakaan PHP yang boleh menjana fail PDF daripada HTML yang dikodkan UTF-8. Pengarang asal, Ian Back, menulis mPDF untuk mengeluarkan fail PDF "dengan cepat" dari tapak webnya dan mengendalikan bahasa yang berbeza. Ia lebih perlahan dan menghasilkan fail yang lebih besar apabila menggunakan fon Unicode daripada skrip asal seperti HTML2FPDF, tetapi menyokong gaya CSS dsb. dan mempunyai banyak peningkatan. Menyokong hampir semua bahasa, termasuk RTL (Arab dan Ibrani) dan CJK (Cina, Jepun dan Korea). Menyokong elemen peringkat blok bersarang (seperti P, DIV),