0

Xcode 8.3.3で以下のソースを変換してbuildしたところエラーが発生しました。

open func drChar(fieldNo : Int32, row : Int32, column : Int32, strBuf : [UInt8]?, useDirectChg : Int32) -> Int32 {
    var retValue : Int32 = 0
    if strBuf == nil {
        retValue = drChar(fieldNo, row, column, nil, useDirectChg)
    } else {
        retValue = drChar(fieldNo, row, column, UnsafeMutablePointer<Int8>(strBuf!), useDirectChg)
    }
    return retValue
}

エラーの場所はUnsafeMutablePointerで、エラーメッセージは

'init' is unavailable: use 'withMemoryRebound(to:capacity:_)' to temporarily view memory as another layout-compatible type.

です。

どのように修正すればよいのでしょうか?

【追加】すみません。ヘッダー部分は以下です。

public func DrChar(_ iFldNo: Int32, _ iRow: Int32, _ iCol: Int32, _ strBuf: UnsafeMutablePointer<Int8>!, _ iDChg: Int32) -> Int32
6
  • あなたのコードはdrCharというメソッドを定義していますが、その中で呼び出されている同名のdrCharは、別シグニチャーで定義された別メソッドのようです。そちらの宣言(関数定義のヘッダー部分)はわかりますでしょうか。正しい変換のためにはそちらの情報が必要となります。
    – OOPer
    2017年8月8日 7:35
  • public func DrChar(_ iFldNo: Int32, _ iRow: Int32, _ iCol: Int32, _ strBuf: UnsafeMutablePointer<Int8>!, _ iDChg: Int32) -> Int32 です。
    – GRRN9
    2017年8月8日 8:14
  • @GRRN9 追加情報は質問文を編集して追記してください。編集ボタンから編集することが出来ます。コメントには直接記載せず追記した旨を知らせる場合にコメントしてください。
    – user10259
    2017年8月8日 8:37
  • @Myaku質問文を編集しました。ご指摘ありがとうございます。
    – GRRN9
    2017年8月8日 9:08
  • コメント・【追加】では大文字スタートのDrCharとなっていますが、元コードは小文字スタートのdrCharで呼び出しているので、とりあえず後者のつもりで回答しておきました。ご確認ください。
    – OOPer
    2017年8月8日 10:41

1 件の回答 1

2

Swift 3では、「Swift2まではポインター型の変換が簡単すぎた」という反省のもと、いくつかのイニシャライザが廃止になっており、単純な置き換え用のメソッドが用意されていないこともあって、Migratorでもうまく変換できないことが多いようです。

なお、UnsafePointer(配列)UnsafeMutablePointer(&配列)と言う書き方は大変危険なもので、特定の条件で動いているように見えても、ちょっとしたことで全く意図しない動作になってしまうことがあります。たとえコンパイルが通る場合でも、ただちに修正していただいた方が良いでしょう。


さて、本題ですが、ご質問に掲載されたコードは、

  • 配列(の先頭要素)へのポインターを取り出す
  • 呼び出し先の引数型に合わせてそのデータ型を変換する

と言う2種類のことをUnsafeMutablePointer<Int8>のイニシャライザで同時に実現していました。Swift 3では、それぞれ別々に行う必要があるので、少々面倒なことになります。

  • 配列(の先頭要素)へのポインターを取り出す

    ⇒配列のwithUnsafeMutableBufferPointer(_:)またはwithUnsafeBufferPointer(_:)を呼び出す

  • 呼び出し先の引数型に合わせてそのポインターのデータ型を変換する

    ⇒ポインターのwithMemoryRebound(to:capacity:_)を呼び出す

元の構造をできるだけ保つように書き直すと、こんな感じになります。

open func drChar(fieldNo: Int32, row: Int32, column: Int32, strBuf: [UInt8]?, useDirectChg: Int32) -> Int32 {
    var retValue: Int32
    if strBuf == nil {
        retValue = DrChar(fieldNo, row, column, nil, useDirectChg)
    } else {
        var strBuffer = strBuf!
        retValue = strBuffer.withUnsafeMutableBufferPointer {bufferPointer in
            bufferPointer.baseAddress!.withMemoryRebound(to: Int8.self, capacity: strBuffer.count) {
                DrChar(fieldNo, row, column, $0, useDirectChg)
            }
        }
    }
    return retValue
}

なお、最終的にUnsafeMutablePointerにしないといけないので、immutableなパラメータのstrBufを直接使うことはせず、mutableとなるようvarにコピーしています。

もうちょっとだけSwiftっぽく書くとこんな感じになるでしょうか。

open func drChar(fieldNo: Int32, row: Int32, column: Int32, strBuf: [UInt8]?, useDirectChg: Int32) -> Int32 {
    var retValue: Int32
    if var strBuffer = strBuf {
        retValue = strBuffer.withUnsafeMutableBufferPointer {bufferPointer in
            bufferPointer.baseAddress!.withMemoryRebound(to: Int8.self, capacity: strBuffer.count) {
                DrChar(fieldNo, row, column, $0, useDirectChg)
            }
        }
    } else {
        retValue = DrChar(fieldNo, row, column, nil, useDirectChg)
    }
    return retValue
}

少々面倒に見えるかもしれませんが、今現在判明しているSwiftやSwift標準ライブラリーの仕様に合わせて危険な使い方は一切しないようになっています。

※「正しくは大文字で呼び出していました」のコメントをいただいたので、「ヘッダー部分は以下です」の内容通り、呼び出される側をDrCharに修正しました。
DrCharが内部で引数のポインターを保存して後々も再利用するような場合には、もっと大規模な修正が必要になります。


C言語で書かれたライブラリーのSwift側のラッパーでしょうか?最初に書いたような変更の他にも、Swift 3ではポインターの取り扱いに大きな変更がありましたので、そのようなライブラリーを多用していると大変かもしれません。ご質問のコードに直接関係ある事柄はこの回答へのコメントとしてお知らせ下さい。別のコードでうまくいかない部分がある場合には別のご質問としてご投稿くださるようお願いします。

(もうじきXcode9の正式版も例年通りの時期に登場しそうです。Xcode9には、Swift2.xからの移行ツールは含まれませんので、要移行のプロジェクトが他にもあるのでしたら、急がれた方が良いでしょう。)

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。