Swift/tvOS: Complex Arrays with multiple strings?

Question

had to edit this because I wrote it in a confused way.

I'm looking to create an Array, where each entry can be used to replace a function's parameters like this:

  func playMe(inputfile: String, inputtype: String)

with

  playMe(videoArray[0])

and that would then replace

  inputfile: String, inputtype: String

with

  inputfile: "file", inputtype: "mp4"

How would I go about creating it? I've tried

   var videoArray = [""Nameofvideo", inputtype: "mp4"", ""Nameofvideo2", inputtype: "mp4""]

but it simply does not want to work. What am I missing? Am I trying to do this in a too complex fashion? I was hoping to emerge with

   playMe(videoArray[1])

and then follow it with audioArray[1] for the followup mp3. Hmm?

Here is my playMe code:

    override func viewDidLoad() {
        super.viewDidLoad()
        NSNotificationCenter.defaultCenter().addObserver(self, selector: "itemDidFinishPlaying:", name: AVPlayerItemDidPlayToEndTimeNotification, object: nil)
    }

    func playMe(inputfile: String, inputtype: String) {
        let path = NSBundle.mainBundle().pathForResource(inputfile, ofType:inputtype)!
        let videoURL = NSURL(fileURLWithPath: path)
        let playerItem = AVPlayerItem(URL: videoURL)
        let player = AVPlayer(playerItem: playerItem)
        playerLayer = AVPlayerLayer(player: player)
        playerLayer.frame = self.view.bounds
        self.view.layer.addSublayer(playerLayer)
        player.play()
    }

    func itemDidFinishPlaying(notification: NSNotification) {
        playerLayer.removeFromSuperlayer()
    }

also considering the limited number of types you might want to create an enumeration with the player accepted types — Leo Dabus
– Leo Dabus, Commented Jan 1, 2016 at 20:27
@LeoDabus Added playMe code and the required notifiers to return back to menu after playback stops. — esaruoho
– esaruoho, Commented Jan 1, 2016 at 20:38
you are declaring your AVPlayer inside viewDidLoad method so it will go out of existence almost immediately. You have to declare the player out of that method, inside your view controller class — Leo Dabus
– Leo Dabus, Commented Jan 1, 2016 at 22:01
try to keep your code as readable as possible. Close your brackets the next line as the Swift convention — Leo Dabus
– Leo Dabus, Commented Jan 1, 2016 at 22:37

Devran Cosmo Uenal · Accepted Answer · 2016-01-02 00:02:59Z

4

You can do this:

var videoArray2 = [
    ["name": "Nameofvideo", "inputType": "mp4"],
    ["name": "Nameofvideo1", "inputType": "mp4"],
    ["name": "Nameofvideo2", "inputType": "mp4"]
]

or better:

struct Video {
    let name: String
    let inputType: String
}

var videoArray = [
    Video(name: "Nameofvideo", inputType: "mp4"),
    Video(name: "Nameofvideo2", inputType: "mp4"),
    Video(name: "Nameofvideo3", inputType: "mp4"),
    Video(name: "Nameofvideo4", inputType: "mp4")
]

if you can, consider using an enum type for inputType. like this:

enum InputType: String {
    case MP4 = "mp4"
    case MP3 = "mp3"
    case MKV = "mkv"
}

struct Video {
    let name: String
    let inputType: InputType
}

var videoArray = [
    Video(name: "Nameofvideo",  inputType: .MP4),
    Video(name: "Nameofvideo2", inputType: .MP4),
    Video(name: "Nameofvideo3", inputType: .MP4),
    Video(name: "Nameofvideo4", inputType: .MKV)
]

to call playMe:inputtype: write this:

let video = videoArray[1]
playMe(video.name, inputtype: video.inputType.rawValue) {

or update your playMe method to:

func playMe(video: Video) {
    let path = NSBundle.mainBundle().pathForResource(video.name, ofType: video.inputType.rawValue)!
    ...
}

and call it like this:

self.playMe(videoArray[1])

edited Jan 2, 2016 at 0:02

answered Jan 1, 2016 at 20:13

Devran Cosmo Uenal

6,2052 gold badges28 silver badges29 bronze badges

Sign up to request clarification or add additional context in comments.

9 Comments

Luca Angeletti Over a year ago

Very good. But VideoObject should be renamed as Video since a Struct is not an object ;) Anyway you have my upvote.

Luca Angeletti Over a year ago

@DevranCosmoUenal: Another refinement: the properties in Video should be declared as let. Infact, IMHO, they are not supposed to change.

Leo Dabus Over a year ago

@appzYourLife I almost mentioned that

Leo Dabus Over a year ago

@DevranCosmoUenal make your enum InputType: String and give them a string value "mp4"

Devran Cosmo Uenal Over a year ago

@appzYourLife: but this works: dropbox.com/s/z6tanip3umklbxv/… ? appzYourLife: but this works: dropbox.com/s/z6tanip3umklbxv/… ? LeoDabus: updated, thanks :)

|

Community · Accepted Answer · 2017-05-23 12:15:20Z

2

To add Devran's answer, here is what you can do

enum MediaType: String {

    case mp4
    case mp3

    var description: String {
        return rawValue
    }
}

struct Media {

    var title: String
    var type: MediaType
}

let medias = [
    Media(title: "Game Of Thrones", type: .mp4),
    Media(title: "Harry Potter", type: .mp4),
    Media(title: "Coldplay Clocks", type: .mp3),
    Media(title: "In the End", type: .mp3),
]


func playMedia(media: Media) {

    if let path = NSBundle.mainBundle().pathForResource(media.title, ofType: media.type.description) {
        // play media at path
    }
}

edited May 23, 2017 at 12:15

CommunityBot

11 silver badge

answered Jan 1, 2016 at 20:45

BLC

2,31025 silver badges29 bronze badges

12 Comments

Devran Cosmo Uenal Over a year ago

But Media and MediaType are way better names :)

BLC Over a year ago

Thanks @DevranCosmoUenal

Leo Dabus Over a year ago

I would just change var mediaType: MediaType to var type: MediaType

Leo Dabus Over a year ago

besides naming you should use URLForResource(withExtension:) to get the fileURL instead of the filePath

BLC Over a year ago

Thanks @LeoDabus for suggestions. I have adhered some of the suggestion and removed the playMedia code as this might not be relevant for the quesion.

|

Alain T. · Accepted Answer · 2016-01-13 01:30:13Z

2

You can use an array of tuples to feed parameters as long as you match your function's call signature exactly:

let arrayOfMedia = 
[ 
 ( "Game Of Thrones", inputtype: ".mp4"),
 ( "Harry Potter",    inputtype: ".mp4"),
 ( "Coldplay Clocks", inputtype: ".mp3"),
 ( "In the End",      inputtype: ".mp3")     
]

playMe(arrayOfMedia[0])

if you want to be explicit for the first parameter name in your array of tuples, you need to change your function's signature to :

func playMe(inputfile inputfile: String, inputtype: String)

answered Jan 13, 2016 at 1:30

Alain T.

42.2k4 gold badges36 silver badges57 bronze badges

Comments

esaruoho · Accepted Answer · 2016-01-02 08:17:50Z

1

You can use an array of dictionaries:

 var videoArray = [["videoName":"Nameofvideo", "inputtype": "mp4"], ["videoName":"Nameofvideo2", "inputtype": "mp4"]]

eventually, instead of repeating "videoName" and "inputtype", you could define two constants like:

static  let VideoName = "videoName"
static  let InputType = "inputtype"

and use them to define and access values contained within dictionaries.

edited Jan 2, 2016 at 8:17

esaruoho

9662 gold badges9 silver badges27 bronze badges

answered Jan 1, 2016 at 20:12

chiarotto.alessandro

1,5991 gold badge14 silver badges33 bronze badges

Collectives™ on Stack Overflow

Swift/tvOS: Complex Arrays with multiple strings?

4 Answers 4

9 Comments

12 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

9 Comments

12 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related