Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve messages in specified to prefer "dearu" or "desumasu" #50

Merged
merged 1 commit into from
Oct 4, 2024
Merged

improve messages in specified to prefer "dearu" or "desumasu" #50

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 19 additions & 9 deletions src/MixedChecker.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -128,22 +128,32 @@ export default class MixedChecker {
*/
outputMessage(token) {
const overType = this.getOverType();
let topMessage = "";
if (overType === "である") {
// である優先 => 最後の"ですます"を表示
return `"である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "${token.value}"
Total:
である : ${this.dearuCount}
ですます: ${this.desumasuCount}
`;
if (this.options.preferDearu) {
// である優先が指定されている場合
topMessage = `"である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります`;
} else {
topMessage = `"である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります`;
}
} else if (overType === "ですます") {
// ですます優先 => 最後の"である"を表示
return `"である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "${token.value}"
if (this.options.preferDesumasu) {
// ですます優先が指定されている場合
topMessage = `"ですます"調 でなければなりません
=> "ですます"調 であるべき箇所に、次の "である"調 の箇所があります`;
} else {
topMessage = `"である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります`;
}
}
return `${topMessage}: "${token.value}"
Total:
である : ${this.dearuCount}
ですます: ${this.desumasuCount}
`;
}
}
}
72 changes: 36 additions & 36 deletions test/no-mix-dearu-desumasu-test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -301,8 +301,8 @@ Total:
errors: [
// 優先指定した、"である" に統一するのを優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 3
Expand All @@ -311,8 +311,8 @@ Total:
column: 8
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 3
Expand All @@ -321,8 +321,8 @@ Total:
column: 8
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 3
Expand All @@ -345,8 +345,8 @@ Total:
errors: [
// 優先指定した、"である" に統一するのを優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 3
Expand All @@ -355,8 +355,8 @@ Total:
column: 8
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ますね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ますね。"
Total:
である : 2
ですます: 3
Expand All @@ -365,8 +365,8 @@ Total:
column: 11
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ますね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ますね。"
Total:
である : 2
ですます: 3
Expand Down Expand Up @@ -427,8 +427,8 @@ Total:
},
errors: [
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 1
Expand All @@ -447,8 +447,8 @@ Total:
},
errors: [
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ますね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ますね。"
Total:
である : 2
ですます: 1
Expand All @@ -470,8 +470,8 @@ CはDです。
},
errors: [
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
message: `本文: "ですます"調 でなければなりません
=> "ですます"調 であるべき箇所に、次の "である"調 の箇所があります: "である。"
Total:
である : 1
ですます: 1
Expand All @@ -493,8 +493,8 @@ CはDになります。
},
errors: [
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
message: `本文: "ですます"調 でなければなりません
=> "ですます"調 であるべき箇所に、次の "である"調 の箇所があります: "である。"
Total:
である : 1
ですます: 1
Expand All @@ -515,8 +515,8 @@ Total:
errors: [
// preferInで"ですます"が設定されているため、"である"に統一された文章でも、"ですます"を優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
message: `本文: "ですます"調 でなければなりません
=> "ですます"調 であるべき箇所に、次の "である"調 の箇所があります: "である。"
Total:
である : 2
ですます: 0
Expand All @@ -525,8 +525,8 @@ Total:
column: 8
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
message: `本文: "ですます"調 でなければなりません
=> "ですます"調 であるべき箇所に、次の "である"調 の箇所があります: "である。"
Total:
である : 2
ですます: 0
Expand All @@ -545,8 +545,8 @@ Total:
errors: [
// preferInで"である"が設定されているため、"ですます"に統一された文章でも、"である"を優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 0
ですます: 2
Expand All @@ -555,8 +555,8 @@ Total:
column: 8
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
message: `本文: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 0
ですます: 2
Expand All @@ -576,8 +576,8 @@ Total:
errors: [
// preferInで"ですます"が設定されているため、"である"に統一された文章でも、"ですます"を優先する
{
message: `見出し: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である"
message: `見出し: "ですます"調 でなければなりません
=> "ですます"調 であるべき箇所に、次の "である"調 の箇所があります: "である"
Total:
である : 1
ですます: 0
Expand All @@ -597,8 +597,8 @@ Total:
errors: [
// preferInで"である"が設定されているため、"ですます"に統一された文章でも、"である"を優先する
{
message: `見出し: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ますね"
message: `見出し: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ますね"
Total:
である : 0
ですます: 1
Expand All @@ -618,8 +618,8 @@ Total:
errors: [
// preferInで"ですます"が設定されているため、"である"に統一された文章でも、"ですます"を優先する
{
message: `箇条書き: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である"
message: `箇条書き: "ですます"調 でなければなりません
=> "ですます"調 であるべき箇所に、次の "である"調 の箇所があります: "である"
Total:
である : 1
ですます: 0
Expand All @@ -639,8 +639,8 @@ Total:
errors: [
// preferInで"である"が設定されているため、"ですます"に統一された文章でも、"である"を優先する
{
message: `箇条書き: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ますね"
message: `箇条書き: "である"調 でなければなりません
=> "である"調 であるべき箇所に、次の "ですます"調 の箇所があります: "ますね"
Total:
である : 0
ですます: 1
Expand Down
Loading