Swift for文の中で条件を満たしたらfor文の手前にもう一度戻りたい

Question

お世話になっております。
Swiftの基礎の質問になると思います。よろしくお願い致します。

画像を使ったアプリを作っており
FileManagerで画像を保存していて
数字を使ってファイル名を決めています。

ファイル名が重複することがないように
ファイル名の配列の中に無いファイル名を作りたいです。

//以下コード
fileNumberはファイル名に使う数字
imageStringArrayは画像ファイル名の二次元配列

fileNumber += 1
let imageString = "image(fileNumber)"
for i in 0..<imageStringArray.count{
if imageStringArray[i].contains(imageString){
   fileNumber += 1   
   }
}

上記のような感じのやり方でやろうと思っています。
for文で配列の中身を一つずつ確認し、配列の中に同じ要素がある場合はfileNumberを+=1しています。

配列に重複しているものが見つかりfileNumberを+=1したらfor文の前に戻ってまたfor文を最初からやり直したいのですがどう書けばいいのでしょうか？

またファイル名の扱い方としてあまり良くない点などがあればアドバイスもお願いしたいです。

ご回答よろしくお願い致します。

Answer 1

もう解決した質問ですが、気になってしまったのですみません。
安直に「flatにしてやればいいのでは？ 」と書きましたが、さすがに乱暴すぎたので少し考えなおしました。
Arrayに対して２次元配列の場合その中身に対象のインスタンスが存在するかを確認するメソッドを追加してみました。

extension Array {
  
  func deepContains<E: Equatable>(_ element: E) -> Bool
    where Element == [E] {
    
      contains { $0.contains(element) }
  }
}

let imageStringArray = [
    ["image1", "image5", "image3"],
    ["image2", "image6"],
    ["image4"],
]

let fileName = (1...)
    .lazy
    .map { "image($0)" }
    .first { !imageStringArray.deepContains($0) }!  // nilになることはない。 Int.maxを超えるのでクラッシュするから。

条件によりますが少しパフォーマンスが上がっているはずです。

---

ややこしく考えずにimageStringArrayをflatにしてやればいいのでは？

let imageNames = imageStringArray.flatMap { $0 }
let fileName = (1...)
    .lazy
    .map { "image($0)" }
    .first { !imageNames.contains($0) }

Comments

[kizahashi]

できました！
いやー、全てを一つの配列にするという発想はありませんでした...
.flatMap { $0 }色々なところに活用できそうですね。
ご回答ありがとうございました!

[退会済みユーザー]

(1...).lazy.map { } 初めて知りました。
ありがとうございます。

Answer 2

playground調子が悪かったので…

---

備忘目的追記

import UIKit

class ViewController: UIViewController {

    let fileNames = ["03", "fileName_094", "fileName_056", "BBB", "45", "fileName_001", "fileName_002", "fileName_005"]
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        print(fileNames.newFileName)　// => "fileName_004"
    }
}

extension String {
    // 同一接頭句でないと連番の意味が薄れるが念の為。
    var numberStr: String {
        
        guard let regex = try? NSRegularExpression(pattern: "^*[\d]+$", options: []) else {
            return ""
        }
        
        let results = regex.matches(in: self, options: [], range: NSMakeRange(0, self.count))
        
        var str: String = ""
        
        for i in 0 ..< results.count{
            for j in 0 ..< results[i].numberOfRanges{
                let range = results[i].range(at: j)
                str.append((self as NSString).substring(with: range))
            }
        }
        return str
    }
}
    
extension Array where Element == String {
    
    var newFileName: String {
        
        let prefixStr: String = "fileName_"
        
        let maxDigit = self.map { $0.numberStr.count }.max() ?? 1
        
        let suffixNumbers: [Int] = self.map { Int($0.numberStr) ?? 0 }.sorted(by: <)
        
        let newSuffixNumber: String
            = String(format: "%0(maxDigit)d", ((1...).lazy.map { $0 }.first { !suffixNumbers.contains($0) }) ?? 0 )
        
        return prefixStr + newSuffixNumber
        
    }
}

---

import UIKit

class ViewController: UIViewController {

    let fileNames = ["AA003y", "094", "556", "BBB", "DE45", "BB001", "CCDFR002", "005", "kaegei990785304743", "08307320683"]
    
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        print("num: (fileNames.fileNumber)") => num: Optional("3")

        
    }

}

extension String {
    
    var number: Int? {
        // 正規表現わかんない
        guard let regex = try? NSRegularExpression(pattern: "^*[\d]+$", options: []) else {
            return nil
        }
        
        let results = regex.matches(in: self, options: [], range: NSMakeRange(0, self.count))
        
        var str: String? = ""
        
        for i in 0 ..< results.count{
            for j in 0 ..< results[i].numberOfRanges{
                let range = results[i].range(at: j)
                str?.append((self as NSString).substring(with: range))
            }
        }
        
        if let str = str {
            return Int(str) ?? 0
        } else {
            return nil
        }
    }
}
    
extension Array where Element == String {
    
    var fileNumber: String? {
        
        // 配列が空だったら？ -> 1を返す？
　　　　　// ファイル名に数値がなかったら -> 1を返す?
        // 桁数調整(入ってくる連番の桁数とのかねあいも考慮)
        // 最終的にはStringを返す？

        let numArray: [Int] = self.map { ($0.number ?? 0) }.sorted(by: <)
         
        guard let max = numArray.max() else { return nil }
        
        for i in 0 ... max {
            
            if !(numArray.contains(i)) { return String(i) }
            
        }
        
        return nil
    }
}

時間があったら完成させようと思う

Comments

[kizahashi]

playground調子悪くなる事多いですよね...
考えていただきありがとうございます！

Answer 3

目的の「for文の前に戻ってまたfor文を最初からやり直したい」については hoshi-takanori さんの回答が最適に感じます。

こちらは補足としての回答なのですけれど、文字列を操作して番号を取得する操作は複雑になりがちなので、もしかすると先にファイル名からファイル番号の一覧を作ってしまって、そこから次の番号を得る方法を考えると見通しが利きやすくなるかもしれないです。

/// ファイル名からファイル番号を取得します。
/// - Parameter filename: ファイル名です。
/// - Returns: 該当するファイル番号です。接頭辞が一致しない場合は `nil` を、一致する場合で番号がない場合は `0` になります。
func fileNumber<T:StringProtocol>(of filename: T) -> Int? {

    let prefix = "iamge"
    
    guard filename.hasPrefix(filename) else {
        
        return nil
    }
    
    // 補足：番号なしのファイル名を許容しない場合は `?? 0` を削除して差し支えないはずです。
    return Int(filename.suffix(filename.count - prefix.count), radix: 10) ?? 0
}

// ファイル番号のリストを生成します。
let fileNumbers = imageStringArray.flatMap {$0}.compactMap(fileNumber(of:))

たとえば fileNumber(of:) という、与えられたファイル名からファイル番号を取得する関数を作っておいて、それを imageStringArray に compactMap メソッドで実行してあげれば、ファイル番号の配列を作ることができます。

ここから先は、好きなように、たとえば欠番を作っても良いなら、ループを使わずに max メソッドを使う方法でシンプルに組み立てられます。欠番を埋めていく場合は関数を用意する分だけ複雑になりそうですけれど、番号の並び替えなどもしやすくなっていて、考えをまとめやすくなる、融通が効きやすくなるのがこの方法の場合のメリットのように感じます。

// ファイル番号の最大値が得られたら、その次の番号を次のファイル番号とします。
if let maxFileNumber = fileNumbers.max() {
    
    let nextFileNumber = maxFileNumber + 1
    
    
}

Comments

[kizahashi]

まだまだ勉強不足でプロトコル？ジェネリクス？などわからないので使えるように勉強してみたいと思います。
ファイル名なのでよく考えたら欠番ありでも大丈夫ですね！
考えていただきありがとうございました。

Answer 4

こんな感じでしょうか。「元に戻ってやり直す」というのは基本的にループになります。そして、外側のループを break や continue するにはラベルを使います。
参考: Swiftは多重ループなどを一気にbreakできる - Qiita

func nextFilename(_ imageStringArray: [[String]]) -> String {
    var fileNumber = 1
    outerLoop: while true {
        let imageString = "image(fileNumber)"
        for subArray in imageStringArray {
            if subArray.contains(imageString) {
                fileNumber += 1   
                continue outerLoop
            }
        }
        return imageString
    }
}

let imageStringArray = [
    ["image1", "image5", "image3"],
    ["image2", "image6"],
    ["image4"],
]

print(nextFilename(imageStringArray))

Comments

[kizahashi]

できました。すごいですね！
回答ありがとうございました〜！

Answer 5

containsはコレクションに適用できますので、必ず1番から始まっているという保証があれば、こういう風に書き換えることができると思います。

var fileNumber = 0

repeat {
    fileNumber += 1
} while imageStringArray.contains("image(fileNumber)")

// つづきの番号
print(fileNumber)

ただし、1番から始まらない、途中に空き要素があるなどの要因が考えられるのであれば別の方法を検討する必要がありそうです。

Comments

[kizahashi]

試してみましたが難しそうです。空の配列も含まれている時もありimageStringArrayに入っている順番がバラバラなので。
ご回答ありがとうございます。