Assignment - Fix Calculator and make NetworkModel by Dunkkkk · Pull Request #12 · LandvibeDev/2022-iOS

Dunkkkk · 2022-09-02T13:24:38Z

네트워크모듈과 파라미터에 대한 설명은 저번 시간에 알려주신 주석의 방법으로 적어놓았습니다.

Parameters:
method: HttpMethod라는 열거형이거, get, put, post, delete를 지원합니다.
url: url은 string 형태로 파라미터를 제거한 형태의 url을 입력하면 됩니다.
entity: 원하는 reposne의 모델 그 중 naver의 경우 items의 타입을 기입하면 해당되는 모델로 클로저의 파라미터를 통해 방출합니다.
paramaters: 원하는 파라미터를 넣으면 됩니다. 이때. value는 반드시 string protocol을 따라야합니다.
header: 토큰이나 필수 요소를 파라미터와 같은 형태로 넣으면 됩니다.
completion: 제네릭의 넣은 모델을 바탕으로 결과값을 보내줍니다.

모듈을 바탕으로 뷰에서 이미지만 볼 수 있도록 간단한 테스트 뷰를 만들었습니다.

I make Network Module It's looks like Alamofire description write on the code and fix Calculator

gen-com · 2022-09-03T10:44:58Z

week4/Calculator/Calculator/CalculatorApp.swift

    var body: some Scene {
        WindowGroup {
-          ContentView(calculatorManager: caculatorManager)
+            ContentView().environmentObject(caculatorManager)


caculatorManager -> ca"l"culatorManager

gen-com · 2022-09-03T10:46:11Z

week4/Calculator/Calculator/Model/CalculatorModel.swift

-      print("error")
+    var showingText: String
+    private(set) var previousNumber: Decimal? {
+


이거 왜 공백을 다 주도록 선택했나요 ? 오히려 보기 힘든 거 같은데

gen-com · 2022-09-03T10:49:16Z

week4/Calculator/Calculator/View/ContentView.swift

@@ -8,27 +8,40 @@
 import SwiftUI

 struct ContentView: View {


gen-com · 2022-09-03T10:54:19Z

week4/Calculator/Calculator/CalculatorApp.swift

@@ -14,7 +14,7 @@ struct CalculatorApp: App {



계산기는 공백이나, 오타 등이 수정이 안돼서 천천히 읽어보면서 확인해 주세요.
리뷰는 보류합니다.

함수의 시작부에 공백은 하지말고, 구조체나 클래스, 확장 때에만 쓸지 말지 정해주세요.

다시 수정하겠습니다

gen-com · 2022-09-03T10:57:26Z

week5/NetworkModule/NetworkModule/Network/NetworkModule.swift

+
+struct NetworkModule {
+
+    static let shared = NetworkModule()


shared를 만든 이유는 무엇인가요 ?

이걸 부르는 모든 곳에서 같은 행동을 할 것이고, 따라서 부를때마다 인스턴스를 만드는 것보다 정적으로 만드는 것이 좋다고 생각했습니다.
또 url session에서 shared 를 사용해서 하기때문에 굳이 인스턴스를 각자 만들어도 session에서 staic하게 동작하기 때문에 static으로 하지 않는 것은 손해라고 생각했습니다

싱글턴 패턴의 효과를 보려는 건가요 ?
근데 NetworkModule은 인스턴스를 만들어 낼 수 있지 않나요 ?
싱글턴 패턴의 단점은 무엇일까요 ?

gen-com · 2022-09-03T10:57:51Z

week5/NetworkModule/NetworkModule/Network/NetworkModule.swift

+    ///   - paramaters: 원하는 파라미터를 넣으면 됩니다. 이때. value는 반드시 string protocol을 따라야합니다.
+    ///   - header: 토큰이나 필수 요소를 파라미터와 같은 형태로 넣으면 됩니다.
+    ///   - completion: 제네릭의 넣은 모델을 바탕으로 결과값을 보내줍니다.
+    func fetchData<EntityType: Decodable>(_ url: String, entity: EntityType.Type, httpMethod: HttpMethod, paramaters: [String: String]? = nil, header: [String: String]? = nil,completion: @escaping (NetworkResult<EntityType>?) -> Void) {


매개변수가 많습니다. 줄일 수 있지 않을까요 ?
사용자 정의 타입, struct를 활용하는 건 어떤가요 ?

요청이 실패한 것도 알려야 하지 않을까요 ?

수정하겠습니다

질문이 있습니다. 만약에 struct로 parameter들을 한 번 정리해서 하는 것도 해당 구조체를 초기화 할 때, 똑같은 과정이 필요한고, 그걸 사용하는 함수내에서도 struct.url 이렇게 써야 할 거 같아서 안했는데 struct로 파라미터를 하는게 더 좋을까요?
요청 실패는 처리하겠습니다

어떤 요청에 대한 설정을 하는 것을 메소드에 호출에서 맡기는 것보다,
요청을 설정하는 부분을 외부로 빼고 메소드는 요청 작업만 신경쓰도록 하는 것은 어떤가요 ?

gen-com · 2022-09-03T11:01:30Z

week5/NetworkModule/NetworkModule/Network/NetworkModule.swift

+            guard let reponseData = try? decoder.decode(EntityType.self, from: data)
+            else { return }
+
+            switch httpURLResponse.statusCode {


NaverOpenAPI의 Status Code를 제대로 읽어봤나요 ?

gen-com · 2022-09-03T11:15:00Z

week5/NetworkModule/NetworkModule/View/ContentView.swift

+struct ContentView: View {
+
+    @ObservedObject var viewModel: Finder
+    @State var movieTitle: String


@State 는 해당 뷰의 내부에서만 사용되므로 private으로 막아줘야 합니다.

leejun6694 · 2022-09-03T15:42:43Z

week5/NetworkModule/NetworkModule/Network/NetworkModule.swift

+        paramaters?.forEach { key, value in
+            queryItems.append(URLQueryItem(name: key, value: value))
+        }
+        urlComponent?.queryItems = queryItems


http method 가 POST 인 경우에도 위 동작이 필요한가요~?

leejun6694 · 2022-09-03T15:43:25Z

week5/NetworkModule/NetworkModule/Network/NetworkModule.swift

+        }
+        urlComponent?.queryItems = queryItems
+
+        guard let url = urlComponent?.url else { return }


열심히 만들고 있는 모듈이니 에러 처리 등을 좀 더 신경써볼까요?

leejun6694 · 2022-09-03T15:46:26Z

week5/NetworkModule/NetworkModule/Network/NetworkModule.swift

+
+            completion(networkResult)
+        }).resume()
+    }


fetchData method 가 담고 있는 역할과 책임이 너무 많은 것 같아요
만들고 처리하는 부분을 조금씩 나눠보면 어떨까요?

rose6649 · 2022-09-05T05:48:33Z

week5/NetworkModule/NetworkModule/Network/NaverOpenAPI.swift

+    static let baseURL = "https://openapi.naver.com/v1"
+    static let searchURL = baseURL + "/search"
+    static let movieJsonURL = searchURL + "/movie.json"
+    static let ClientID = "SjK7VjRrkujvwETKupas"


rose6649 · 2022-09-05T05:51:57Z

week5/NetworkModule/NetworkModule/Network/NetworkModule.swift

+    }
+}
+
+protocol RequestEnum {


protocol을 쓴 이유는 무엇일까요?

rose6649 · 2022-09-05T05:52:36Z

week5/NetworkModule/NetworkModule/ViewModel/Finder.swift

+class Finder: ObservableObject {
+
+    @Published var movieList: [Movie]?
+    @Published var testResult: UIImage = UIImage(named: "tempImage")!


rose6649 · 2022-09-05T05:53:05Z

week5/NetworkModule/NetworkModule/ViewModel/Finder.swift

+
+            case .success(let data):
+                DispatchQueue.main.async { [weak self] in
+                    print(data)


rose6649 · 2022-09-05T05:53:48Z

week5/NetworkModule/NetworkModule/ViewModel/Finder.swift

+                    print(data)
+                    guard let self = self else { return }
+                    self.movieList = data.items
+                    guard let data = try? Data(contentsOf: URL(string: data.items.first?.image ?? "")!),


가급적 강제 언래핑을 지양하도록 해요~

make Network Module and fix Calculator

07dc4b1

I make Network Module It's looks like Alamofire description write on the code and fix Calculator

gen-com requested review from gen-com, leejun6694 and rose6649 September 2, 2022 15:32

gen-com assigned Dunkkkk Sep 2, 2022

gen-com reviewed Sep 3, 2022

View reviewed changes

week4/Calculator/Calculator/View/ContentView.swift

@@ -8,27 +8,40 @@

import SwiftUI

struct ContentView: View {

Copy link

Contributor

gen-com Sep 3, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네이밍

gen-com reviewed Sep 3, 2022

View reviewed changes

leejun6694 reviewed Sep 3, 2022

View reviewed changes

rose6649 reviewed Sep 5, 2022

View reviewed changes

week5/NetworkModule/NetworkModule/Network/NetworkModule.swift

}

}

protocol RequestEnum {

Copy link

rose6649 Sep 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

protocol을 쓴 이유는 무엇일까요?

rose6649 reviewed Sep 5, 2022

View reviewed changes

make networkModuleMoyatype

e71f0ec

Conversation

Dunkkkk commented Sep 2, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

gen-com Sep 3, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

gen-com Sep 3, 2022 •

edited

Loading