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.

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

typoの修正とdoneの翻訳変更 #337

Merged
merged 3 commits into from
Aug 12, 2022
Merged

typoの修正とdoneの翻訳変更 #337

merged 3 commits into from
Aug 12, 2022

Conversation

t-ae
Copy link
Collaborator

@t-ae t-ae commented Aug 9, 2022

No description provided.

@github-actions github-actions bot requested a review from stzn August 9, 2022 02:48
@@ -495,7 +495,7 @@ print(manualDrawing.draw())
// ***Hello RAVI PATEL!**
```

このコードは機能しますが、少し扱いにづらく感じます。`AllCaps` の後の深くネストされた括弧は読みにくく、`name` が `nil` の場合に「World」を使用するフォールバックロジックは、`??` を使用してインラインで実行する必要があります。これは、より複雑なものに対しては困難です。描画の一部を構成するために `switch` または `for` ループを含める必要があっても、方法はありません。リザルトビルダを使用すると、このようなコードを書き換えて、通常の Swift のコードのように見せることができます。
このコードは機能しますが、少し扱いづらく感じます。`AllCaps` の後の深くネストされた括弧は読みにくく、`name` が `nil` の場合に「World」を使用するフォールバックロジックは、`??` を使用してインラインで記述する必要があります。これは、より複雑なものに対しては困難です。描画の一部を構成するために `switch` または `for` ループを含める必要があっても、方法はありません。リザルトビルダを使用すると、このようなコードを書き換えて、通常の Swift のコードのように見せることができます。
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

原文The fallback logic to use “World” when name is nil has to be done inline using the ?? operator,

インラインで実行する だと意味が通らないので 記述する に変更しました。

@@ -1,6 +1,6 @@
# 高度な演算子\(Advanced Operators\)

最終更新日: 2022/7/15
最終更新日: 2022/7/15
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

日付更新していただけると嬉しいです!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typoの修正くらいなら更新しなくてもいいかなと思ってましたが、この文書は一応翻訳も変えてるので更新しますね。
basic-operators.mdのほうはtypo修正だけなので更新しないでいいでしょうか。

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございますmm

basic-operators.mdのほうはtypo修正だけなので更新しないでいいでしょうか。

はい、内容の変更がないのでいらないと思います。

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

更新しました。

soaringkonoha
soaringkonoha previously approved these changes Aug 10, 2022
Copy link
Owner

@stzn stzn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!

@stzn stzn merged commit c03909c into stzn:master Aug 12, 2022
@t-ae t-ae deleted the fix-typos branch August 12, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants