2016-12-24 35 views
2

我有以下課程來返回NOAA氣象觀測站的列表。我正在使用它來學習如何處理XML。但是,我在func returnWxStation() - >(wxObservationStations)處得到了「使用未聲明類型'wxObservationStations'」作爲錯誤。我正在使用SWXMLHash反序列化XML,但我不認爲這是我的問題(儘管我只是在學習,所以可能會這樣)。在Swift中使用未聲明的類型3

class WxObservationStations { 

let wxObserStationsURL = URL(string: "http://w1.weather.gov/xml/current_obs/index.xml") 

struct wxStation: XMLIndexerDeserializable { 
    let stationName: String 
    let stationState: String 
    let latitude: Double 
    let longitude: Double 

    static func deserialize(_ node: XMLIndexer) throws -> wxStation { 
     return try wxStation(
      stationName: node["station_name"].value(), 
      stationState: node["state"].value(), 
      latitude: node["latitude"].value(), 
      longitude: node["longitude"].value() 
     ) 
    } 
} 

public var wxObservationStations: [wxStation] = [] 


private func getStationNamesAndLocations(url: URL, completion:@escaping (XMLIndexer) ->()) { 

    Alamofire.request(url).responseJSON { response in 
     //    print(response) // To check XML data in debug window. 
     let wxStationList = SWXMLHash.parse(response.data!) 
     print(wxStationList) 
     completion(wxStationList) 

    } 
} 

//The error is here: 
func returnWxStation() -> (wxObservationStations) { 
    getStationNamesAndLocations(url: wxObserStationsURL!, completion: { serverResponse in 
     do { 
      self.wxObservationStations = try serverResponse["wx_station_index"]["station"].value() 

     } catch { 

     } 
    }) 
     return self.wxObservationStations 
} 
} 

有什麼想法?該變量在類中聲明,我想用它來將數據發送回請求對象。提前致謝。

回答

1

wxObservationStations是不是一個類型的,所以它沒有任何意義說

func returnWxStation() -> (wxObservationStations) { ... } 

你返回self.wxObservationStations,這是[wxStation]類型。因此,該方法的聲明應該是

func returnWxStation() -> [wxStation] { ... } 

順便說一句,如果你堅持使用可可的命名約定你的生活會更容易,即類型應該大寫字母開頭。因此,而不是wxStation類型,我建議WxStation


你下面的方法不能達到你想要的東西:

func returnWxStation() -> [wxStation] { 
    getStationNamesAndLocations(url: wxObserStationsURL!, completion: { serverResponse in 
     do { 
      self.wxObservationStations = try serverResponse["wx_station_index"]["station"].value() 
     } catch { 

     } 
    }) 

    return self.wxObservationStations 
} 

getStationNamesAndLocations異步運行的方法和你的self.wxObservationStations不會受returnWxStation實際返回的時間進行填充。

getStationNamesAndLocations方法的全部用途是爲完成處理程序提供一個很好的異步方法。我會完全從你的代碼中刪除returnWxStation。或做類似:

func returnWxStation(completionHandler: ([wxStation]?) -> Void) { 
    getStationNamesAndLocations(url: wxObserStationsURL!) { serverResponse in 
     do { 
      let stations = try serverResponse["wx_station_index"]["station"].value() 
      completionHandler(stations) 
     } catch { 
      completionHandler(nil) 
     } 
    } 
} 

你會使用它像這樣:

returnWxStation() { stations in 
    guard let stations = stations else { 
     // handle error here 
     return 
    } 

    // use `stations` here 
} 

// but not here 
+0

謝謝!我確實努力遵守慣例,但正如我所瞭解的,我不確定事物是否是類型或對象。至於代碼是否正常工作,我懷疑它不會出於這個原因,但我試圖讓它達到它將構建和運行的程度,以便我可以從那裏開始工作,所以您的額外答案是一個很好的幫助。 – Yrb

+0

@Rob,優秀的答案,除非你看起來像編輯錯誤。 returnWxStation函數中的代碼不應該返回'try serverResponse ...'。該行應該將該表達式傳遞給完成處理程序。 –

+0

@DuncanC - 很對。謝謝。固定。 – Rob

相關問題