Определение глобальной переменной Сброс

Question

У меня есть класс, предназначенный для получения данных о температуре от API на определенную дату и добавления его в словарь. URL-адрес API хранится в глобальной переменной: baseURL. Он определяется в начале как пустая строка, но позже изменяется. Мой класс ниже:

import UIKit 
import Foundation 
typealias ServiceResponse = (JSON, NSError?) -> Void 

class WeatherManager: NSObject { 

var baseURL: String = "" 
var data: String = "" 
static let sharedInstance = WeatherManager() 


func getRandomUser(onCompletion: (JSON) -> Void) { 
    println("Starting getRandomUser") 
    let route = self.baseURL 
    println(self.baseURL) 
    makeHTTPGetRequest(route, onCompletion: { json, err in 
     onCompletion(json as JSON) 
    }) 
} 

func makeHTTPGetRequest(path: String, onCompletion: ServiceResponse) { 
    let request = NSMutableURLRequest(URL: NSURL(string: path)!) 

    let session = NSURLSession.sharedSession() 

    let task = session.dataTaskWithRequest(request, completionHandler: {data, response, error -> Void in 
     let json:JSON = JSON(data: data) 
     onCompletion(json, error) 
     if error != nil { 
      println("No Error") 
     } else { 
      println("Error") 
     } 
    }) 
    task.resume() 
} 

func addData() { 
    WeatherManager.sharedInstance.getRandomUser { json in 
     var jsonData = json["response"]["version"] 
     self.data = "\(jsonData)" 
     dispatch_async(dispatch_get_main_queue(),{ 
      let alert = UIAlertView() 
      alert.title = "Weather Data Update" 
      if self.data != "null" { 
       println("Value:\(self.data)") 
       alert.message = "The weather data was updated successfully." 
       alert.addButtonWithTitle("OK") 
       alert.show() 
      } else { 
       println("Error Reading Data") 
       println(self.data) 
       alert.message = "HealthTrendFinder encountered an error while updating data." 
       alert.addButtonWithTitle("OK") 
       alert.show() 
      } 
     }) 
    } 
} 

func updateWeatherHistory() { 
    println(self.baseURL) 
    let calendar: NSCalendar = NSCalendar(calendarIdentifier: NSCalendarIdentifierGregorian)! 
    println("Weather Updating...") 
    // This sets the start date to midnight of the current date if no start date has been set. 
    if StorageManager.getValue(StorageManager.StorageKeys.WeatherStartDate) == nil { 
     let startDate: NSDate = calendar.startOfDayForDate(NSDate()) 
     StorageManager.setValue(startDate, forKey: StorageManager.StorageKeys.WeatherStartDate) 

    } 

    // This adds a data array if it hasn't been created yet. 
    if StorageManager.getValue(StorageManager.StorageKeys.WeatherData) == nil { 
     StorageManager.setValue([:], forKey: StorageManager.StorageKeys.WeatherData) 

    } 

    var weatherData: [NSDate: NSObject] = StorageManager.getValue(StorageManager.StorageKeys.WeatherData)! as! [NSDate : NSObject] 
    let startMidnight: NSDate = StorageManager.getValue(StorageManager.StorageKeys.WeatherStartDate) as! NSDate 
    let currentMidnight: NSDate = calendar.startOfDayForDate(NSDate()) 
    let daysFromStartDate: Int = calendar.components(NSCalendarUnit.CalendarUnitDay, fromDate: startMidnight, toDate: currentMidnight, options: nil).day 
    println("Starting Loop") 
    for i: Int in 0..<daysFromStartDate { 

     let dateToBeExamined: NSDate = calendar.dateByAddingUnit(NSCalendarUnit.CalendarUnitDay, value: i, toDate: startMidnight, options: nil)! 
     if weatherData[dateToBeExamined] == nil { 
      let calendarUnits: NSCalendarUnit = .CalendarUnitDay | .CalendarUnitMonth | .CalendarUnitYear 
      let components = NSCalendar.currentCalendar().components(calendarUnits, fromDate: dateToBeExamined) 
      var month: String 
      var day: String 
      if components.month < 10 { 
       month = "0\(components.month)" 
      } else { 
       month = "\(components.month)" 
      } 
      if components.day < 10 { 
       day = "0\(components.day)" 
      } else { 
       day = "\(components.day)" 
      } 
      var dateString = "\(components.year)\(month)\(day)" 
      self.baseURL = "http://api.wunderground.com/api/91e65f0fbb35f122/history_\(dateString)/q/OR/Portland.json" 
      println(self.baseURL) 
      var get:() = WeatherManager.sharedInstance.addData() 
      println(get) 
      weatherData[dateToBeExamined] = self.data 




      // There is no data for the NSDate dateForInspection. You need to pull data and add it to the dictionary. 
     } else { 
      // Data exists for the specified date, so you don't need to do anything. 
     } 
    } 
    println("Loop has finished or been skipped") 

} 

} 

Проблема заключается в том, BaseUrl возвращается в пустую строку, когда getRandomUser выполняется после BaseUrl устанавливается в URL. Почему это происходит, и как я могу это исправить?

Answer 1

Ваш код является излишне сложным, что затрудняет диагностику проблемы без дополнительной информации. Но вот предложение:

Попробуйте сделать это невозможно создать экземпляр более одного экземпляра вашего WeatherManager одноточечного:

class WeatherManager { 

    private static let _sharedInstance = WeatherManager() 

    private init() { super.init() } 

    static func sharedInstance() -> WeatherManager { 
     return _sharedInstance 
    } 
} 

Когда вы работаете с внешним WeatherManager, при обращении к ним по телефону:

let wm = WeatherManager.sharedInstane() 

Затем, когда вы работаете внутри WeatherManager, убедитесь, что все ваши ссылки на self - то есть, self.baseURL = ... или self.updateWeatherHistory(), вместо WeatherManager.sharedInstance.baseURL = ... и т. Д.

Хотя ваш код сложный, я думаю, что происходит, вы фактически имеете два экземпляра WeatherManager в игре. Вы устанавливаете значение baseURL на одном, но не на другом. Если вы хотите, чтобы он был синглом, вам нужно сделать невозможным создание более одного.