Extract duplicated string literals into constants in main code (java:S1192)

未關閉 適合新手
#738 0 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視

維護者通常 1 天內回覆

還沒有人認領這個 Issue。

評估

難度
2/5
預估耗時
1-3 小時
新手友好度
70/100
Issue 類型
重構
描述清晰度
基本清楚
活躍度
活躍
技術堆疊
java
領域
tooling

研究方向

閱讀 CONTRIBUTING.md 並檢視連結的 SonarCloud 發現,從 ExplorePage 和 Compute 中列出的重複項目開始。選擇一個模組,在註解中說明你的選擇,並將合適的重複名稱或鍵提取為常數;如果常數會降低可讀性,就不要改動 HTML 片段。完成的標準是:為該模組提交一個 PR,行為保持不變,並說明任何略過的發現。

由索引模型根據 Issue 內容生成。

描述

good first issue

SonarCloud flags duplicated string literals: inspector 7 (ExplorePage), writer 5, reader 2 (Compute:55, :56). List: java:S1192 open issues.

Out of scope: fbs-gen and proto-gen (32 findings). Those are code-generator templates where a constant would hurt readability.

How: pick one module, say so in a comment here, and send one PR. Use judgment: a constant for a name or key repeated across methods is good; for fragments like "</td>" in an HTML builder it may not be, so leave those and explain on the Sonar finding. Behavior must not change. See CONTRIBUTING.md.

主要語言
Java
星號
3
分支
2
平均合併
9 小時 19 分鐘
30 天內合併 PR
154

環境準備

  • 沒有 Dockerfile 或 Docker Compose 檔案
  • 沒有 Pull Request 範本
  • 閱讀貢獻指南

從這裡開始

  1. 先讀完整個 Issue,再讀專案的貢獻指南。
  2. 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
  3. Fork 儲存庫,在一個分支上完成修改。
  4. 送出 Pull Request,並在描述裡引用這個 Issue 編號。

dfa1/vortex-java 的其他 Issue

查看 dfa1/vortex-java 的全部 Issue

相似的 Issue

更多 Java Issue

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。