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

第二章を加筆しました #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

AnaTofuZ
Copy link
Member

今までの入学式の資産を使いつつ加筆しました。
またshebangについてコラムを書いてみました

my $mul = $foo * $bar; # 4 * 2 = 8
my $div = $foo / $bar; # 4 / 2 = 2
my $mod = $foo % $bar; # 4 % 2 = 0
my $exp = $foo ** $bar; # 4の2乗は 16
Copy link
Member

Choose a reason for hiding this comment

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

サッとマージするとも限らないので, この辺りの誤字修正系は別のPull Requestで切り出してもらえると良さそうでした.

Copy link
Member Author

Choose a reason for hiding this comment

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

なるほどです


### コラム 2017年までのお約束

昨年までのPerl入学式ではお約束として**3点セット**を受講生の皆さんにお伝えてしていました。
Copy link
Member

Choose a reason for hiding this comment

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

あくまで2018年の資料を作っているので, 去年までの内容に触れる必要はないと思いました. どういう意図がありますか?

Copy link
Member Author

Choose a reason for hiding this comment

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

Wanbok以外の環境で資料として使用する場合がある事を想定しています。
教科書的に触れておく(講義では触れないで利用する)と良いかなと考えています

Copy link
Member

Choose a reason for hiding this comment

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

それであれば, 2017年までのお約束 というタイトルは不適切かなと思いました.

Wanbok以外の環境で資料として使用する場合

ということを想定したタイトルになっていた方がわかりやすそうです.

$fuga -= 1; # $fuga = $fuga - 1;と同じ
$fuga *= 2; # $fuga = $fuga * 2;と同じ
$fuga /= $hoge; # $fuga = $fuga / $hoge;と同じ
```
Copy link
Member

Choose a reason for hiding this comment

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

省略記法はそれこそ省略して良さそうと思っています. $fuga = $fuga + 1 などで代替できるので.

Copy link
Member Author

Choose a reason for hiding this comment

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

確かに入門では突飛感があるので無くしてもいいかなとは思っていたのですが、教科書的に使うなら一応書いておいても良いかななどと思っています。(入門向けのProgateというサイト代入をした後のフェーズで触れられています)

Copy link
Member

Choose a reason for hiding this comment

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

Progateが触れているのでPerl入学式でも触れる, だとちょっと説得力が弱い気がします.
例えばカリキュラムを終了した後のappendix的な資料で触れるなどの手もありそうです.

Copy link
Member Author

Choose a reason for hiding this comment

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

なるほどです。一旦削除してみます

@@ -58,10 +78,17 @@ print $foo; # => 4
変数に格納された数値を, 1ずつ増やしたい/減らしたいときは, インクリメント/デクリメントという演算子を利用することができます.
(1)のように, 変数の後ろにインクリメント演算子(`++`)を書くと, その変数に格納された数値が1増えます.
同様に, (2)のように, 変数の後ろにデクリメント演算子(`--`)を書くと, その変数に格納された数値が1減ります.
また、1ずつ増やす事をインクリメント,減らす事をデクリメントと呼びます.
Copy link
Member

Choose a reason for hiding this comment

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

ここで唐突に説明すると違和感がありました. そもそもインクリメント/デクリメントについては, 上の 変数に格納された数値を, 1ずつ増やしたい/減らしたいときは, インクリメント/デクリメントという演算子を利用することができます. で説明できていそうです.

変数に格納された数値を, 1ずつ増やしたい(インクリメント)/減らしたい(デクリメント)ときは, それぞれインクリメント演算子++とデクリメント演算子--という演算子を利用することができます.

あたりがよさそう?

Copy link
Member Author

Choose a reason for hiding this comment

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

すっきりして良さそうですね!ありがとうございます!

@papix
Copy link
Member

papix commented Mar 19, 2018

あと, 細かい/かつコーディングルール規定していなかった方も悪いのだけれど, 句読点は,.に統一したい... (完全に個人の趣味)

@AnaTofuZ
Copy link
Member Author

レビューありがとうございます! 句読点修正しておきます

Copy link
Member

@papix papix left a comment

Choose a reason for hiding this comment

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

幾つかコメントしています. ご確認下さい!

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.

2 participants