DEVTOOLS-185 Remplacer le json en dur des images resgen #11

2024-04-19T12:52:50+02:00

q.bandera commented

2024-04-19 12:52:50 +02:00

No description provided.

q.bandera added 1 commit 2024-04-19 12:52:50 +02:00

DEVTOOLS-185 Remplacer le json en dur des images resgen

gitea-openium/resgen.swift/pipeline/head There was a failure building this commit

Details

gitea-openium/resgen.swift/pipeline/pr-master There was a failure building this commit

Details

c8cfe82109

q.bandera requested review from t.schmitt 2024-04-19 12:53:01 +02:00

t.schmitt requested changes

2024-04-19 14:42:59 +02:00

Dismissed

Sources/ResgenSwift/Images/Model/ImageContent.swift Outdated

						
				@@ -0,0 +12,4 @@

				    let info: Info

				    static func == (lhs: ImageContent, rhs: ImageContent) -> Bool {

				        let lhsImagesCount = lhs.images.count

t.schmitt commented

2024-04-19 14:41:26 +02:00

Tu peux optimiser les performances:

guard lhs.images.count == rhs.images.count else { return false }
let lhsImages = ...
....
return lhsImages == rhsImages

Pas besoin de check les infos.
Si tu as pas le meme count, tu ne vas pas parcourir ton tableau images c'est toujours ca de gagné.

Tu peux optimiser les performances: ``` guard lhs.images.count == rhs.images.count else { return false } let lhsImages = ... .... return lhsImages == rhsImages ``` Pas besoin de check les infos. Si tu as pas le meme count, tu ne vas pas parcourir ton tableau `images` c'est toujours ca de gagné.

q.bandera marked this conversation as resolved

Sources/ResgenSwift/Images/Model/ImageContent.swift Outdated

						
				@@ -0,0 +30,4 @@

				    let filename: String

				}

				struct Info: Codable, Equatable {

t.schmitt commented

2024-04-19 14:38:12 +02:00

Aprs réflexion, on va changer le nom des 3 structs:

ImageContent -> AssetContent
Image -> AssetImageDescription
Info -> AssetInfo
Ces noms sont plus représentatifs de la data qu'ils contiennent.

Aprs réflexion, on va changer le nom des 3 structs: - `ImageContent` -> `AssetContent` - `Image` -> `AssetImageDescription` - `Info` -> `AssetInfo` Ces noms sont plus représentatifs de la data qu'ils contiennent.

q.bandera marked this conversation as resolved

Sources/ResgenSwift/Images/Model/ParsedImage.swift Outdated

						
				@@ -71,0 +46,4 @@

				        let encoder = JSONEncoder()

				        encoder.outputFormatting = .prettyPrinted

				        guard let data = try? encoder.encode(imageContent) else { return nil }

t.schmitt commented

2024-04-19 14:42:43 +02:00

Si la méthode appelante ne le fait pas déjà, dans le else regarde comment j'ai fais pour générer un warning pour Xcode. Avec le warning, le dev saura qu'il y a eu un soucis