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

Issue 21 エラーメッセージの改善 #25

Merged
merged 4 commits into from
Apr 24, 2020
Merged

Issue 21 エラーメッセージの改善 #25

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3cfb480
chore(docker): Add temporary Node.JS env for dev
KEINOS Apr 22, 2020
b000bb2
chore(outputMessage): イマイチ意図がわかりにくいエラーメッセージを改善
KEINOS Apr 22, 2020
a463ff6
test(outputMessage): エラー時のメッセージの変更にともなうテストを修正
KEINOS Apr 22, 2020
470dca0
delete: temp Dockerfile
KEINOS Apr 22, 2020
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
4 changes: 2 additions & 2 deletions src/MixedChecker.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -125,15 +125,15 @@ export default class MixedChecker {
if (overType === "である") {
// である優先 => 最後の"ですます"を表示
return `"である"調 と "ですます"調 が混在
=> "${token.value}" がですます調
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "${token.value}"
Total:
である : ${this.dearuCount}
ですます: ${this.desumasuCount}
`;
} else if (overType === "ですます") {
// ですます優先 => 最後の"である"を表示
return `"である"調 と "ですます"調 が混在
=> "${token.value}" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "${token.value}"
Total:
である : ${this.dearuCount}
ですます: ${this.desumasuCount}
Expand Down
48 changes: 24 additions & 24 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 @@ -113,7 +113,7 @@ CはDになります。`,
// 同数である場合は、"ですます"に統一するのを優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 1
ですます: 1
Expand All @@ -131,7 +131,7 @@ Total:
// 同数である場合は、"ですます"に統一するのを優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 1
ですます: 1
Expand All @@ -153,7 +153,7 @@ Total:
// 同数である場合は、"ですます"に統一するのを優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 2
ですます: 3
Expand All @@ -163,7 +163,7 @@ Total:
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 2
ですます: 3
Expand All @@ -184,7 +184,7 @@ Total:
// 同数である場合は、"ですます"に統一するのを優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 2
ですます: 3
Expand All @@ -194,7 +194,7 @@ Total:
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 2
ですます: 3
Expand All @@ -215,7 +215,7 @@ Total:
// 同数である場合は、"ですます"に統一するのを優先する
{
message: `見出し: "である"調 と "ですます"調 が混在
=> "である" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である"
Total:
である : 1
ですます: 1
Expand All @@ -235,7 +235,7 @@ Total:
// 同数である場合は、"ですます"に統一するのを優先する
{
message: `見出し: "である"調 と "ですます"調 が混在
=> "である" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である"
Total:
である : 1
ですます: 1
Expand All @@ -255,7 +255,7 @@ Total:
// 同数である場合は、"ですます"に統一するのを優先する
{
message: `箇条書き: "である"調 と "ですます"調 が混在
=> "である" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である"
Total:
である : 1
ですます: 1
Expand All @@ -274,7 +274,7 @@ Total:
// 同数である場合は、"ですます"に統一するのを優先する
{
message: `箇条書き: "である"調 と "ですます"調 が混在
=> "である" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である"
Total:
である : 1
ですます: 1
Expand All @@ -297,10 +297,10 @@ Total:
preferInBody: "である"
},
errors: [
// 優先指定した、"ですます" に統一するのを優先する
// 優先指定した、"である" に統一するのを優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ですね。" がですます調
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 3
Expand All @@ -310,7 +310,7 @@ Total:
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ですね。" がですます調
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 3
Expand All @@ -320,7 +320,7 @@ Total:
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ですね。" がですます調
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 3
Expand All @@ -341,10 +341,10 @@ Total:
preferInBody: "である"
},
errors: [
// 優先指定した、"ですます" に統一するのを優先する
// 優先指定した、"である" に統一するのを優先する
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ですね。" がですます調
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 3
Expand All @@ -354,7 +354,7 @@ Total:
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ますね。" がですます調
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ますね。"
Total:
である : 2
ですます: 3
Expand All @@ -364,7 +364,7 @@ Total:
},
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ますね。" がですます調
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ますね。"
Total:
である : 2
ですます: 3
Expand All @@ -384,7 +384,7 @@ Total:
[
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 1
ですます: 1
Expand All @@ -404,7 +404,7 @@ Total:
[
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 1
ですます: 1
Expand All @@ -426,7 +426,7 @@ Total:
errors: [
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ですね。" がですます調
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ですね。"
Total:
である : 2
ですます: 1
Expand All @@ -446,7 +446,7 @@ Total:
errors: [
{
message: `本文: "である"調 と "ですます"調 が混在
=> "ますね。" がですます調
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "ますね。"
Total:
である : 2
ですます: 1
Expand All @@ -469,7 +469,7 @@ CはDです。
errors: [
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 1
ですます: 1
Expand All @@ -492,7 +492,7 @@ CはDになります。
errors: [
{
message: `本文: "である"調 と "ですます"調 が混在
=> "である。" がである調
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である。"
Total:
である : 1
ですます: 1
Expand Down