if文

Question

###実現したいこと
重複なく値を出力させたい。

###該当のソースコード

//: Playground - noun: a place where people can play

import UIKit

var RandomNumber = arc4random\(\) % 4

RandomNumber \+= 1

//空の配列を用意
var party = \[String\]\(\)

switch \(RandomNumber\) {

case 1:

print\("a"\)
//配列に値を追加
party\.append\("やった！"\)
break


case 2:
print\("b"\)
party\.append\("できた！"\)
break
    

case 3:
print\("c"\)

party\.append\("すごい！"\)

break

case 4:
print\("d"\)


party\.append\("完成！"\)
break
    
    
default:
    
    break

}

###やったこと
まず空の配列を用意し、caseで選ばれたものを空の配列の中に入れました。

###できないこと
配列の中に同じ値が入っていたらfor文で初めから読み込み配列の中に値が入ってなかったらその値を出力させるプログラムの書き方がわかりません。（重複させない）
アドバイスなどありましたらお願いします

Comments

[fuzzball]

同じ質問をしないで下さい。

Answer 1

配列の中に同じ値が入っていたらfor文で初めから読み込み配列の中に値が入ってなかったらその値を出力させるプログラムの書き方がわかりません。（重複させない）

というのを解釈し直し、配列[1,2,3,4]をランダムにソート（もちろん重複しません）してやって、それぞれの値に該当するstringを取得するようにしてみてはいかがでしょう？

//: Playground - noun: a place where people can play

import Cocoa

// http://qiita.com/codelynx/items/4c5c6c2faf8315c208fe
extension Array {
    mutating func shuffle() -> Array {
        for i in 0 ..< (count - 1) {
            let j = Int(arc4random_uniform(UInt32(count - i))) + i
            if i != j {
                swap(&self[i], &self[j])
            }
        }
        return self
    }
}

// party に入れる代わりに関数で返すように
func getMessage(_ RandomNumber: Int) -> String {
    switch (RandomNumber) {
    case 1:
        return "やった！"
        
    case 2:
        return "できた！"
        
    case 3:
        return "すごい！"
        
    case 4:
        return "完成！"
        
    default:
        return ""
    }
}

var numbers: [Int] = [1, 2, 3, 4]


// mapを使うとfor文を使って逐次 [string] に append() するよりスッキリ書けます
var party: [String] = numbers.shuffle().map{ getMessage($0) }

// playground で見やすくするため
party

Comments

[退会済みユーザー]

func getMessage(_ RandomNumber: Int) -> String {

の部分はどのような意味になりますか？

[TakeoAsai]

関数です。
何度も使うものは関数として分けてやるとわかりやすくなります。

Answer 2

申し訳ございません。
こちらでいかがでしょうか？
func removeOverlap(arr: Array<String>) -> Array<String>から始まる部分は独立したメソッドしてソースコード内にコピペしてみてください。

//空の配列を用意
var arr = String

    while arr.count < 4 {
        var RandomNumber = arc4random() % 4
        
        RandomNumber += 1
        
        switch (RandomNumber) {
            
        case 1:
            
            
            //配列に値を追加
            arr.append("やった！")
            break
            
            
        case 2:
            
            arr.append("できた！")
            break
            
            
        case 3:
            
            
            arr.append("すごい！")
            
            break
            
        case 4:
            
            
            
            arr.append("完成！")
            break
            
            
        default:
            
            break
            
        }
        
        arr = removeOverlap(arr: arr)
    }


    func removeOverlap(arr: Array<String>) -> Array<String> {
        let arrCnt = arr.count
        let order = NSOrderedSet(array: arr)
        let reArr = order.array as! [String]
        
        if arrCnt == reArr.count {
            print(reArr[arrCnt - 1])
        }
        return reArr
    }

Comments

[退会済みユーザー]

大丈夫でした！！！
ありがとうございます！！

Answer 3

以下の感じでいかがでしょうか？

//空の配列を用意
var arr = String

while arr.count < 4 {
var RandomNumber = arc4random() % 4

RandomNumber += 1

switch (RandomNumber) {

case 1:

print("a")
//配列に値を追加
arr.append("やった！")
break

case 2:
print("b")
arr.append("できた！")
break

case 3:
print("c")

arr.append("すごい！")

break

case 4:
print("d")

arr.append("完成！")
break

default:

break

}

arr = removeOverlap(arr: arr)
}

func removeOverlap(arr: Array<String>) -> Array<String> {
let arrCnt = arr.count
let order = NSOrderedSet(array: arr)
let reArr = order.array as! [String]

if arrCnt == reArr.count {
    print(reArr[arrCnt - 1]) 
}
return reArr

}

Comments

[退会済みユーザー]

ありがとうございます
試して見たところ、
コードの
arr = removeOverlap(arr: arr)
で: Use of unresolved identifier 'removeOverlap'というエラーが発生しました。
未解決？だというふうなエラーだと。
しかし、プログラムはきちんと動いて結果は
a
やった！
c
すごい！
a
b
できた！
d
完成！

でした。途中abとなっています