nの階乗

Question

問題：整数値を引数とし、その値の階乗を表示する関数を作成せよ。ただし、0以下の値を入力した場合 は1と表示する。
コードfunc twentyfive(o:Int){
if o > 0{
for p in 1...o{
var q:Int = 1
print(q)
q = p * q
}} else {
print("0")
}
}
twentyfive(5)
結果：1が5回表示される
エラー：無し

Answer 1

これ1からnまでの和と同じ間違いしてるよ。
forループがどういうもので、どのようにプログラムが実行されるのか理解してたらこのようなコードは書かないし、間違えて書いたとしてもすぐに気がつくはず。for文やif文など言語の文法をきちっと理解してから、それでもなぜ意図した通りに動かないのかわからない時に、ここに投稿した方がいいと思います。

Comments

[TheLastSoldier]

ゴメンなさい。適当に答えを求めるような質問をしてしまい、その質問にわざわざ答えていただきありがとうございます。次からはちゃんとした質問をします。

Answer 2

if o > 0{
  for p in 1...o{
    var q:Int = 1
    print(q)

ループの先頭で毎回qを初期化してるんでは、print(q)は1しか出力してくれませんよ。

Comments

[TheLastSoldier]

じゃあどうすればいいんですか？

[tkturbo]

ループが開始される直前で「var q:Int = 1」すればいいでしょ？