Js-beautify: Spasi sebelum argumen fungsi (spasi-setelah-fungsi) (spasi-setelah-nama-fungsi)

Dibuat pada 27 Jan 2015  ·  16Komentar  ·  Sumber: beautify-web/js-beautify

Harus ada bendera untuk memaksa spasi sebelum argumen fungsi seperti ini:

function foo (bar, baz) {
    // …
}

mungkin: {"space_before_arguments": true}

good first issue help wanted enhancement

Komentar yang paling membantu

+1
Saya sangat berharap ini mendapat perhatian lebih karena 'gaya standar javascript' semakin populer.

Semua 16 komentar

Ada --space-after-anon-function . Mungkin sebut ini --space-after-function .

: +1:

Ya silahkan.

begitu?

Ini peningkatan. Saya sedang memperbaiki masalah dengan prioritas yang lebih tinggi. Permintaan tarik diterima.

Adakah yang bisa menunjukkan kepada saya cara meretas ini ke /js/lib/beautify.js sendiri? Sejalan 681 saya menemukan pernyataan menangani anon_functions. Saya tidak bisa melakukan apa pun dengan mengutak-atik berbagai jenis pernyataan, misalnya: if (current_token.text === '(' && last_last_word === 'function') ...

@bitwiseman Saya pikir menyebutnya --space-after-named-function akan menjadi yang paling jelas, ya? Seperti @atrioom , saya tertarik untuk mengerjakan ini. Di mana saya memulai?

@atrioom Aku tidak benar-benar tahu apa yang saya lihat, tapi mungkin di sini 's tempat yang baik untuk melihat?

@jsejcksn - Nama terdengar bagus.
Dan Anda berada di area yang benar, tetapi Anda ingin memisahkan else if dengan last_type === 'TK_WORD' .

Harap pertimbangkan juga fungsi asinkron

const a = async () {...}
async function b () {...}

--space-after-named-function adalah nama yang bagus

+1
Saya sangat berharap ini mendapat perhatian lebih karena 'gaya standar javascript' semakin populer.

@zekida Dan seharusnya, itu "standar"! 😄

CATATAN: Ketika diimplementasikan ini tidak perlu merusak "dynamic import() " - # 1393

Tidak terdengar kejam, tetapi mengingat ini adalah spesifikasi standar js, agak mengejutkan itu belum menjadi prioritas yang lebih tinggi. Saya membayangkan sebagian besar dari kita lint dengan standar hari ini, sih, sebagian besar framework Vue memilikinya secara default. Ini semacam membuat semuanya menjadi sangat berlebihan jika Anda harus kembali dan merapikan ruang.

@bayu_joo
Keberadaan proyek ini (dan daftar permintaan yang terus bertambah untuk opsi pemformatan yang berbeda) adalah satu lagi bukti nyata bahwa perang pemformatan masih hidup dan baik di dunia JavaScript. "Standar" Anda adalah "tidak pernah melakukan itu" dari pengembang lain.

Anda benar sekali: akan sangat bagus jika ada lebih banyak orang yang bersedia meluangkan waktu untuk mengimplementasikan fitur untuk pustaka yang banyak digunakan ini. Kalau saja ada lebih banyak kontributor ...

Apakah halaman ini membantu?
0 / 5 - 0 peringkat