Changeset 10 in code for trunk/engines/google.go

Timestamp:

Sep 30, 2022, 8:16:23 PM (3 years ago)

Author:

fattalion

Message:

Make some identifiers more concise

See ​https://go.dev/doc/effective_go#package-names, specifically:

The importer of a package will use the name to refer to its contents,
so exported names in the package can use that fact to avoid
repetition.

For example, engines.GoogleTranslateEngine needlessly repeats
"engine," so just get rid of that duplication by renaming it to
engines.GoogleTranslate.

Renaming engines.TranslationEngine to engines.Engine may be
debatable, so if somebody disagrees, feel free to leave a comment
stating your disagreement and with an explanation of why you disagree.

File:

• trunk/engines/google.go (modified) (2 diffs)

trunk/engines/google.go

@@ -9 +9 @@
 )
 
-type GoogleTranslateEngine struct{}
+type GoogleTranslate struct{}
 
-func (_ *GoogleTranslateEngine) InternalName() string { return "google" }
+func (_ *GoogleTranslate) InternalName() string { return "google" }
 
-func (_ *GoogleTranslateEngine) DisplayName() string { return "Google" }
+func (_ *GoogleTranslate) DisplayName() string { return "Google" }
 
-func (_ *GoogleTranslateEngine) getLangs(type_ string) ([]Language, error) {
+func (_ *GoogleTranslate) getLangs(type_ string) ([]Language, error) {
         var langsType string
         switch type_ {
@@ -84 +84 @@
 }
 
-func (e *GoogleTranslateEngine) SourceLanguages() ([]Language, error) {
+func (e *GoogleTranslate) SourceLanguages() ([]Language, error) {
         return e.getLangs("source")
 }
 
-func (e *GoogleTranslateEngine) TargetLanguages() ([]Language, error) {
+func (e *GoogleTranslate) TargetLanguages() ([]Language, error) {
         return e.getLangs("target")
 }
 
-func (_ *GoogleTranslateEngine) SupportsAutodetect() bool { return true }
+func (_ *GoogleTranslate) SupportsAutodetect() bool { return true }
 
-func (_ *GoogleTranslateEngine) DetectLanguage(text string) (Language, error) { return Language{}, nil }
+func (_ *GoogleTranslate) DetectLanguage(text string) (Language, error) { return Language{}, nil }
 
-func (_ *GoogleTranslateEngine) Translate(text string, from Language, to Language) (TranslationResult, error) {
+func (_ *GoogleTranslate) Translate(text string, from Language, to Language) (TranslationResult, error) {
         requestURL, err := url.Parse("https://translate.google.com/m")