canogg/hibo-app

hibo-app's Issues

ScrollView

Oi Carol! Percebi que ao rotacionar o app para orientação Landscape algumas informações são escondidas.

Pensei em 2 possível soluções

1 - Caso não queira a orientação Landscape no app, desativar nas configurações gerais do App

2 - Adicionar as views em ScrollViews. Uma vez que a view esteja dentro de uma ScrollView, evita que elas possam se perder ao rotacionar

ChoicesFrenchView var declarations

Hibo-App/View/ChoichesFrenchView.swift

Lines 82 to 95 in 1cc8f25

	let buttonMatch: UIButton = {
	let button = UIButton(type: .system)
	button.translatesAutoresizingMaskIntoConstraints = false
	button.titleLabel?.font = UIFont.boldSystemFont(ofSize: 24)
	button.setTitleColor(.white, for: .normal)
	button.setTitle("Match!", for: .normal)
	button.backgroundColor = UIColor(red: 190/255.0, green: 30/255.0, blue: 45/255.0, alpha: 1.0)
	button.clipsToBounds = true
	button.layer.cornerRadius = 8
	button.titleLabel?.textAlignment = .center
	button.addTarget(self, action: #selector(nextpagematch), for: .touchUpInside)
	button.addTarget(self, action: #selector(buttonAction), for: .touchUpInside)
	return button
	}()

Oi Carol, tudo bem? Vi que nesse componente, nas linhas 92 e 93 você usa o .addTarget para adicionar um destino ao seu botão quando for pressionado. No Xcode essas linhas estão com um warning que diz Use 'ChoichesFrenchView.self' to silence this warning, mas ao dar um Fix o código quebra. Você pode resolver isso alterando buttonMatch para o tipo lazy var ao invés de let.

Redundant String Enum Value

Hibo-App/Service/UserDefault.swift

Lines 10 to 14 in c4bcd1b

	extension UserDefaults {
	enum UserDefaultsKeys: String {
	case nameUser = "nameUser"
	}

Nesse enum há uma redundância de valores. Você pode corrigir alterando a linha 13 dessa forma:

enum UserDefaultsKeys: String {
case nameUser
}

Rename archive

Hibo-App/Service/UserDefault.swift

Lines 10 to 22 in c4bcd1b

	extension UserDefaults {
	enum UserDefaultsKeys: String {
	case nameUser = "nameUser"
	}
	func setPersonName(value: String){
	set(value, forKey: UserDefaultsKeys.nameUser.rawValue)
	}
	func getPersonName() -> String? {
	return string(forKey: UserDefaultsKeys.nameUser.rawValue)
	}
	}

E aí Carol! Tudo bem? Vi que na sua pasta Service, seu arquivo se chama UserDefault e a extension que ele contém se chama UserDefaults. É uma boa prática igualar o nome das classes, extensions...estruturas principais daquele arquivo com o mesmo nome que foi dado à ele, pra evitar conflitos ou problemas futuros no código. Pelo que vi no restante, você já faz uso dessa boa prática, mas deixo aqui essa issue aberta para caso você não tenha percebido esse detalhe!

reusable code in ChoicesFrenchView

Hibo-App/View/ChoichesFrenchView.swift

Line 10 in c4bcd1b

class ChoichesFrenchView: UIView {

Oi Carol, tudo bem? Vi que em seu projeto, existem dois arquivos extremamente parecidos, o ChoicesFrenchView e o ChoicesEnglishView, você pode unir ambos e reaproveitar código utilizando um conceito chamado Generics. Se fizer sentido pra você quiser conversar mais sobre o assunto, pode me chamar!