项目中使用 rc-util/es/warning 后，打包后的生产代码为什么会包含警告代码？

Aperta
#301 5 commenti 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
javascript, typescript, vite
Ambito
build-system, frontend

Direzione di ricerca

Inizia dalla riproduzione StackBlitz e dall’entry point rc-util/es/warning, quindi esamina le righe 7356-7467 del dist/assets/index.57005310.js generato. Confronta il bundle di produzione con l’implementazione di warning mostrata nell’issue; il lavoro è completato quando il codice di warning viene rimosso dall’output di produzione senza compromettere il comportamento in sviluppo.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

问题如题所述

复现案例

https://stackblitz.com/edit/vitejs-vite-zdg5hc?file=src%2FApp.tsx
请看案例中 dist/assets/index.57005310.js 文件中 7356-7467 代码行

期待行为：

正常来说生产代码不应该包含警告（console.log/console.warn/console.error）等不影响项目执行的代码

造成问题的原因:
// 源码实现如下(只是列出部分代码)
export function warning(valid, message) {
  if (process.env.NODE_ENV !== 'production' && !valid && console !== undefined) {
    console.error(message)
  }
}

// 相比以上实现，下列实现则会在打包生产代码时移除相关代码
function noop() {}
export let warning = noop

if (process.env.NODE_ENV !== 'production') {
  warning = function () {
    if (!valid && console !== undefined) {
      console.error(message)
    }
  }
}

另外，console !== undefined 可以移除，客户端和服务端不是都有 console 对象么？

Lingua principale
TypeScript
Stelle
670
Fork
205
Merge medio
15g 17h
PR unite (30g)
6

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di react-component/util

Tutte le issue di react-component/util

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.