Как построить логику приложения?

Question

[DS_JS]

Доброго времени суток, я только начал изучать програмирование и у меня есть небольшая задачка
имеем поле для ввода текста вводим слово и нажимаем на кнопку и получаем цифру
есть алфавит в котором к каждому символу присвоена своя цифра и поле для ввода текста editText полученное слово нужно разбить на отдельные символы и посчитать сумму всех цифр принадлежащих етим символам
подскажите как правильно построить логику
создавать каждый символ как отдельную переменную и присваивать нужную цифру или лучше загнать алфавит в массив А и создать отдельный массив В с цифрами и через поиск по индексу присваивать нужную цифру?
Хотелось бы подчеркнуть что так как я новичек и хочу всему научиться прошу более детально пояснить почему и как лучше

Comments

[Алексей Черемисин]

Или создать hashmap с ключем буквы и значением цифры.
Но и два масива тоже годятся. С ними может быть и комфортнее.

Answer 1

[Орхан Гасанлы]

Можно по-разному реализовать вашу задачу. Если никаких ограничений нет, то можете посмотреть в сторону:
1) использовать 2 массива. Первый char[], а второй int[]. Далее как вы и упомянули по индексу достаете нужное значение.
2) Либо использовать Map. Например, ее реализацию HashMap.
3) Либо например, в пакете apache commons lang3 есть объект Pair. И можно использовать что-то типа List<Pair<Character, Integer>>.

В любом случае все будет сводится к следующему:
- получаете слово (строку) разбиваете его на characters.
- объявляете переменную int, в которой будут суммироваться значения букв
- далее в зависимости от выбранного массива или коллекции итерируете по нему и при совпадении буквы получаете ее числовое значение и прибавляете к вышеобъявленной переменной.

Примерно как-то так

Comments

[DS_JS]

Благодарю за ответ !)
Чесно признаюсь я самоучка и пытаюсь читать разные матереалы и уроки но идет с трудом
вот что у меня получилось
ну вернее не получилось ..

package com.example.num;

import androidx.appcompat.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.EditText;
import android.widget.TextView;
import java.util.Objects;


public class MainActivity extends AppCompatActivity {

    @Override      // Новая Activity
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
    }


    String[] ukr  = {"А", "Б", "В", "Г", "Ґ", "Д", "Е", "Є", "Ж", "З", "И", "І", "Ї", "Й", "К", "Л", "М", "Н", "О", "П", "Р", "С", "Т", "У", "Ф", "Х", "Ц", "Ч", "Ш", "Щ", "Ь", "Ю", "Я"};

    int[] ukrn  = {1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6};

    TextView textView = (TextView) findViewById(R.id.textView);
    EditText editText = (EditText) findViewById(R.id.editText);

    int simwol = -1;
    int indexA = -1;
    int nomer = -1;
    // ========================  Преобразуем строку str в массив символов  =========================
    public class Example {
        public void main(String args[]) {
         String tekst = Integer.parseInt(editText.getText().toString());
            char[] strTXT = tekst.toCharArray();

            // ==================  Перебор по массивам и поис нужного значения  ====================
            for (int s = 0; s < strTXT.length; s++) {
                simwol = strTXT[s];
                 for (int a = 0; a < ukr.length; a++) {
                     if (Objects.equals(strTXT[s], ukr[a])) {
                         indexA = ukr[a];
                         nomer += ukrn.length[indexA];
                    }
                } 
                 break;
            }
        }
    }
  
    // ==============================  Обработка нажатия кнопки  ===================================

    public void sendMessage(View view ) {
        textView.setText(nomer); //
    }
}

То есть насколько я понял через вложенный цикл
сначала открываем первый цикл и берем с массива1 первый символ допустим "К"
Открываем след цикл и находим "К" в массиве2 с алфавитом но тут нужно уже взять только индекс данного символа после чего точно такой же индекс просто достаем с массива3 с кодом символа и добавляем в переменную которую потом нам нужно вывести в качестве ответа

[Орхан Гасанлы]

DS_JS, Я бы рекомендовал вам прежде, чем изучать андроид разработку, серьезно изучить основы java или kotlin. Не владея простыми инструментами самого языка будет довольно сложно изучать андроид. Ведь, дальше вы натолкнетесь на различные паттерны проектирования, которые применяются в андроид и не только. Например, Adapter, Builder. Возможно, столкнетесь с технологией ORM и др.
В первую очередь изучите язык и тогда проще будет осознанно писать код.

Также проанализировав ваш код я могу отметить следующие нюансы, на которые стоит обратить внимание:

1)

String[] ukr  = {"А", "Б", "В", "Г", "Ґ", "Д", "Е", "Є", "Ж", "З", "И", "І", "Ї", "Й", "К", "Л", "М", "Н", "О", "П", "Р", "С", "Т", "У", "Ф", "Х", "Ц", "Ч", "Ш", "Щ", "Ь", "Ю", "Я"};

В массиве у вас только заглавные буквы и когда внутри цикла вы будете их сравнивать, то не факт что будут совпадения. Причина - слово же не написано целиком в uppercase (например, "АЛФАВИТ"), а скорее всего "Aлфавит". Т.е. вам нужно привести слово к lowercase или наоборот к uppercase.

2) По факту вам скорее нужен не строковый массив, а char[]
Иначе вам сперва нужно будет разделить слово на массив символов, затем проитерировать по строковому массиву ukr и внутри преобразовывать каждую строку в символ и сравнивать с каждой буквой слова. Это лишнее телодвижение.

3) Рекомедую прочитать про java naming convention. В частности, camelCase
char[] strTXT
более корректно будет назвать strTxt, а еще лучше давать осознанные названия переменным и названиями методов, классов и т.д. Это касается всех переменных, аргументов и т.д.

4) Также непонятно зачем вы вкладываете публичный класс в другой публичный класс. Рекомендую прочитать про классы, про ООП, про отношения классов, внутренние классы, анонимные классы и т.д.

public class MainActivity extends AppCompatActivity {
...
public class Example {
....

5)

String tekst = Integer.parseInt(editText.getText().toString());

Давайте проговорим словами вот, эту строчку кода:
"Спарсить текст полученный из ввода и перевести его в строку, затем перевести его в числовое значение и присвоить это значение к строке".
Явно стоит подумать над этим кусочком кода.

6) Непонятен смысл break для выхода из цикла в данном сниппете

for (int s = 0; s < strTXT.length; s++) {
                simwol = strTXT[s];
                 for (int a = 0; a < ukr.length; a++) {
                     if (Objects.equals(strTXT[s], ukr[a])) {
                         indexA = ukr[a];
                         nomer += ukrn.length[indexA];
                    }
                } 
                 break;
            }

7) непонятен смысл инициализации значений этим переменным

int simwol = -1;
    int indexA = -1;
    int nomer = -1;

8) В этой строчке кода вы сравниваете слона с арбузом и соответствено, не будет совпадения. Т.е. вы сравниваете строку с символом.
(Objects.equals(strTXT[s], ukr[a]))

9) Метод main в данном классе явно лишний
public void main(String args[]) {

В общем, мой совет - изучите сперва основы java и ООП, а затем уже начинайте изучать android разработку. По себе знаю, к каким последствиям это может привести потом...

Чесно признаюсь я самоучка и пытаюсь читать разные матереалы и уроки но идет с трудом
вот что у меня получилось

Я тоже начинал изучение Java сам. Рекомендую воспользоваться академической литературой. Например, Г. Шилдт - Полное руководство Java. Б. Эккель - Философия Java

[DS_JS]

Доброго времени суток
Получилось немного поправить код но теперь ругаеться на последнюю строку вывода результата в textView код ошибки: Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.TextView.setText(int)' on a null object reference
at com.example.num.MainActivity.sendMessage(MainActivity.java:73)

Добавлю весь код

package com.example.num;

import androidx.appcompat.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.EditText;
import android.widget.TextView;
import java.util.Arrays;
import android.widget.Button;

public class MainActivity extends AppCompatActivity {

    TextView textView;
    EditText editText;

    @Override      // Новая Activity
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
    }


    public int sum = 0;
    public int number = 0;

    public void main(String[] args) {

        textView = (TextView) findViewById(R.id.textView);
        editText = (EditText) findViewById(R.id.editText);

        char[] nameArr = editText.getText().toString().toUpperCase().toCharArray();
        
       // char[] nameArr;
        Character[] ukr = {'А', 'Б', 'В', 'Г', 'Ґ', 'Д', 'Е', 'Є', 'Ж', 'З', 'И', 'І', 'Ї', 'Й', 'К', 'Л', 'М', 'Н', 'О', 'П', 'Р', 'С', 'Т', 'У', 'Ф', 'Х', 'Ц', 'Ч', 'Ш', 'Щ', 'Ь', 'Ю', 'Я'};
       // name = name.toUpperCase();
        //nameArr = name.toCharArray();

        // Character[] name = {'Д', 'М', 'И', 'Т', 'Р', 'О'};

        int[] ukrn = {1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6};



        for (int s = 0; s < nameArr.length; s++)  {
            int nameFind = nameArr[s];

            for (int a = 0; a < ukr.length; a++) {

                if (nameFind == ukr[a]) {
                    int index = Arrays.asList(ukr).indexOf(ukr[a]);
                    number += ukrn[index];
                    //System.out.println(number);
                    //break;
                }
            }

        }

        int num = number;
        sum = 0;
        while (num > 0) {
            sum = sum + num % 10;
            num = num / 10;
        }


    }
    public void sendMessage(View view ) {
        String str = Integer.toString(sum);
      // String result = Integer.toString(sum);
        textView.setText(str);
    }
}

[Орхан Гасанлы]

Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.TextView.setText(int)' on a null object reference

Вы пытаетесь засеттить null и получаете NPE. Попробуйте подебажить ваш код

[DS_JS]

когда запускаю отдельно вот эту часть

package com.example.start;

import java.util.Arrays;


public class Main {

    public static void main(String[] args) {
        int number = 0;
        String name = "Вода";
        char[] nameArr;
        Character[] ukr = {'А', 'Б', 'В', 'Г', 'Ґ', 'Д', 'Е', 'Є', 'Ж', 'З', 'И', 'І', 'Ї', 'Й', 'К', 'Л', 'М', 'Н', 'О', 'П', 'Р', 'С', 'Т', 'У', 'Ф', 'Х', 'Ц', 'Ч', 'Ш', 'Щ', 'Ь', 'Ю', 'Я'};
        name = name.toUpperCase();
        nameArr = name.toCharArray();



        int[] ukrn = {1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6};

        for (int s = 0; s < nameArr.length; s++)  {
            int nameFind = nameArr[s];

            for (int a = 0; a < ukr.length; a++) {

                if (nameFind == ukr[a]) {
                    int index = Arrays.asList(ukr).indexOf(ukr[a]);
                    number += ukrn[index];
                    //System.out.println(number);
                    //break;
                }
            }
        }

        int num = number;
        int sum = 0;
        while (num > 0) {
            sum = sum + num % 10;
            num = num / 10;
        }
        System.out.println(sum);
    }
}

В консоли Android Studio все работает
Когда подставляю в MainActivity кричит что что-то не так

[Орхан Гасанлы]

DS_JS, ну так потому, что в Activity main метода и не должно быть. Вам нужно код внутри метода main вынести в отдельный метод и в Activity вызывать его.

Когда подставляю в MainActivity кричит что что-то не так

В разработке есть более объективные причины, чем "что-то не так". Есть некий баг, который вы словили и его нужно решить. Дебаг кода в этом случае поможет

Answer 2

[Руслан .]

Можно и одним массивом обойтись, если положить по индексу равном коду символа его цифру.
Например если ваш алфавит начинается с латинской А, то можно посчитать её за 0 и
заполнить массив так
int [] code;
code[0] =1; // символ A
code[1] =2; // символ B
...
А искать нужный код так:
code[(char) x - (char) 'A')]
x - это переменная в которую последовательно считывается символы из введенной строки.