sarahmadalena/inspire

Daily Challenge: programa de imagens de inspiração e referência diária

Sobre mim

• 🤔   Desenvolvendo soluções e buscando a melhor forma para fazer isso.
• 🎓   Graduada em Letras pela UECE.
• 🎓   Pós-Graduando em Design Gráfico pelo Instituto INFNET.
• 💼   Alumini da Apple Developer Academy IFCE.

Languages and Tools:

Social Medias:

inspire's Issues

Separação de código

Olá Sarah, tudo bem? Eu achei muito legal tua ideia e a implementação tá muito boa também. Queria te dar uma sugestão que acredito que possa facilitar no futuro pra realizar manutenção ou adicionar funcionalidades.

No arquivo api.swift você utiliza a estrutura da seguinte forma:

import Foundation

struct Response: Decodable {
    var page: Int
    var per_page: Int
    var photos: [Photo]
}

struct Photo: Decodable {
    let src: SRC
}

struct SRC: Decodable {
    let original: String
    let medium: String
    let small: String
}


//Função que vai consumir a API e retornar (caso não nulo)
func api (theme: String) {

Seria interessante separar esses modelos em arquivos diferentes para que, no momento em que alguma funcionalidade nova fosse adicionada ou outro modelo tivesse de ser criado, não interferisse na legibilidade do código que lida com a api.

Bug na busca

Oie Sarah, tudo joia? ✨

Adorei a ideia que você propôs! Eu costumava desenhar e quando estava com um bloqueio criativo eu recorria a um site que fazia exatamente isso, e ajudava demais por ser bastante desafiador.

A observação que eu queria fazer é ocorre fatal error ao digitar um tema com mais de uma palavra 😕

Se você quiser podemos trabalhar juntas para resolver esse problema ☺️ estou à disposição

Melhorar variável

Inspire/Sources/Inspire/String.swift

Line 9 in 37b7927

let menssage = """

Sarah, achei incrível a documentação do Readme e o projeto muito legal, única coisa que observei, é que você poderia modificar o nome das suas variáveis de "mensssage", pois pode se tornar um vício para você e acabar te prejudicando futuramente, sugiro que veja sobre CamelCase e sempre escreva variáveis que explique melhor onde você a utiliza, por exemplo "messageStart"

Package

Parabéns pelo código Sarah! Eu gostei demais da ideia do seu programa me retornar uma imagem aleatória, pois incentiva bastante minha criatividade, quando trabalho com a área de prototipação e Design.

Inspire/Sources/Inspire/api.swift

Lines 68 to 78 in 37b7927

	@available(macOS 10.13, *)
	func openG(url : String) {
	let ls = Process()
	ls.executableURL = URL(fileURLWithPath: "/usr/bin/open")
	ls.arguments = [url]
	do{
	try ls.run()
	} catch {
	print(error.localizedDescription)
	}
	}

Como melhoraria, eu gostaria de falar que nesse trecho, é possível evitar o @avaliable se você precisar usar para outras funcionalidades também, ao acrescentar na descrição do seu Package. Exemplo:

let package = Package(
    name: "Inspire",
    platforms: [.macOS(.v10_13)],
    dependencies: [],
    targets: [
        .executableTarget(
            name: "Inspire",
            dependencies: []),
        .testTarget(
            name: "InspireTests",
            dependencies: ["Inspire"]),
    ]
)

Saber mais sobre API

Oi Sarah! Eu achei seu código bem limpo e organizado. ✨ Vi que você usou API no seu projeto e eu gostaria de saber mais sobre o assunto quando você estiver disponível. 🌼

Sugestão para evitar crash de readline().

Inspire/Sources/Inspire/main.swift

Lines 6 to 20 in 979dbda

	themeInput = readLine()!
	// var page = readLine()
	print(menssage2)
	//Deixa o programa inteiro em um loop com uma condicional de saída
	while true {
	if let themeType = Int(themeInput) {
	print("\n Por favor, digite um tema")
	themeInput = readLine()!
	} else {
	let theme = themeInput
	api(theme: theme)
	}
	}

Quando no primeiro readLine() a gente só da enter, o programa não avança.

Eu reescrevi desse jeito pra tentar resolver e funciona pra esse caso: <3

Comentário geral e sugestão de melhoria da jornada do usuário. 🥰

Olá, Sarah! Gostei de mais do seu programa ✨, acho que dá, inclusive, margem para solução de outros problemas, como o uso da imagem para descanso de telas de notebook, desktop e celular 😊. Como eu não tenho experiência com código, mas pelo que vi achei muito organizado.
Como sugestão de melhoria na experiencia do usuário, acho que no final, quando já foi escolhido um tema, poderia ter além da opção de encerrar o programa, a alternativa de pesquisar novamente outro tema. Se não me engano, isso pode ser feito, por exemplo, utilizando a função "while".
No mais, parabéns, ficou muito lindo. 🤩

Só elogios

Adorei! O tema é muito interessante e foi bem desenvolvido.
Não achei algum problema de codificação e acho que você usou muito bem a linguagem.
E parabéns pela organização, código bem limpo, intuitivo e fácil de ler.