Почему приложение вылетает с ошибкой?

Question

[Cpper]

При вызове функции macsave приложение вылетает с ошибкой. Помогите пожалуйста. Не могу понять почему.
P.s. я начинающий

package com.example.steshenko.blueterm;

import android.bluetooth.BluetoothAdapter;
import android.bluetooth.BluetoothDevice;
import android.content.Intent;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.EditText;

import java.io.BufferedReader;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.io.OutputStream;
import java.io.OutputStreamWriter;
import java.sql.Struct;
import java.util.Set;

public class MainActivity extends AppCompatActivity {

    private static final int REQUEST_ENABLE_BT = 1;
    public String macadress = "macadress.txt";
    public EditText name;
    public String mac;
    public String named;
    private String fileName = "macadress.txt";

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        BluetoothAdapter bluetooth = BluetoothAdapter.getDefaultAdapter();
        if (bluetooth == null) {
            finish();
        }
        if (!bluetooth.isEnabled()) {
            Intent enableBtIntent = new Intent(BluetoothAdapter.ACTION_REQUEST_ENABLE);
            startActivityForResult(enableBtIntent, REQUEST_ENABLE_BT);
        }
        InputStream inputStream = null;
        try {
            inputStream = openFileInput(fileName);
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        }
        if (inputStream != null) {
            InputStreamReader isr = new InputStreamReader(inputStream);
            BufferedReader reader = new BufferedReader(isr);
            String line;
            StringBuilder builder = new StringBuilder();

            try {
                while ((line = reader.readLine()) != null) {
                    builder.append(line + "\n");
                }
            } catch (IOException e) {
                e.printStackTrace();
            }

            try {
                inputStream.close();
            } catch (IOException e) {
                e.printStackTrace();
            }
            name.setText(builder.toString());
        }
    }


    public void macsave (View view) {
        BluetoothAdapter bluetooth = BluetoothAdapter.getDefaultAdapter();
        name = (EditText) findViewById(R.id.name);
        named = name.getText().toString();
        Set<BluetoothDevice> pairedDevices = bluetooth.getBondedDevices();

        if (pairedDevices.size() > 0) {
            // There are paired devices. Get the name and address of each paired device.
            for (BluetoothDevice device : pairedDevices) {
                String deviceName = device.getName();
                String deviceHardwareAddress = device.getAddress(); // MAC address
                if(deviceName == named){
                    mac = deviceHardwareAddress;
                }
            }
        }
        OutputStream outputStream = null;
        try {
            outputStream = openFileOutput(fileName, 0);
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        }
        OutputStreamWriter osw = new OutputStreamWriter(outputStream);
        try {
            osw.write(mac);
        } catch (IOException e) {
            e.printStackTrace();
        }
        try {
            osw.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
        name.setText(mac.toString());
    }

}

Comments

[artshelom]

А че за ошибка??

[stoptalking]

причин может быть масса, стектрейс в студию

[Cpper]

artshelom, пишет в приложении произошла ошибка и вылетает

[Cpper]

stoptalking, прошу прощения, а где его взять? Пытаюсь найти

[Cpper]

Может быть ошибка из-за одновременного блютуз подключения и потока для работы с файлом?

[Денис Загаевский]

Я ставлю на то, что там NPE, но без лога фиг знает.

[Cpper]

К сожалению я не могу запустить AVD и подключить по отладке настоящий девайс тоже.
И в logcat соответственно пусто ;(
Не могли бы Вы пожалуйста запустить у себя, очень хочется разобраться где ошибка
Весь код прислал

import android.bluetooth.BluetoothAdapter;
import android.bluetooth.BluetoothDevice;
import android.content.Intent;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.EditText;

import java.io.BufferedReader;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.io.OutputStream;
import java.io.OutputStreamWriter;
import java.sql.Struct;
import java.util.Set;

public class MainActivity extends AppCompatActivity {

    private static final int REQUEST_ENABLE_BT = 1;
    public String macadress = "macadress.txt";
    public EditText name;
    public String mac;
    public String named;
    private String fileName = "macadress.txt";

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        name = (EditText) findViewById(R.id.name);
        BluetoothAdapter bluetooth = BluetoothAdapter.getDefaultAdapter();
        if (bluetooth == null) {
            finish();
        }
        if (!bluetooth.isEnabled()) {
            Intent enableBtIntent = new Intent(BluetoothAdapter.ACTION_REQUEST_ENABLE);
            startActivityForResult(enableBtIntent, REQUEST_ENABLE_BT);
        }
        InputStream inputStream = null;
        try {
            inputStream = openFileInput(fileName);
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        }
        if (inputStream != null) {
            InputStreamReader isr = new InputStreamReader(inputStream);
            BufferedReader reader = new BufferedReader(isr);
            String line;
            StringBuilder builder = new StringBuilder();

            try {
                while ((line = reader.readLine()) != null) {
                    builder.append(line + "\n");
                }
            } catch (IOException e) {
                e.printStackTrace();
            }

            try {
                inputStream.close();
            } catch (IOException e) {
                e.printStackTrace();
            }
            name.setText(builder.toString());
        }
    }


    public void macsave (View view) {
        BluetoothAdapter bluetooth = BluetoothAdapter.getDefaultAdapter();
        named = name.getText().toString();
        Set<BluetoothDevice> pairedDevices = bluetooth.getBondedDevices();

        if (pairedDevices.size() > 0) {
            // There are paired devices. Get the name and address of each paired device.
            for (BluetoothDevice device : pairedDevices) {
                String deviceName = device.getName();
                String deviceHardwareAddress = device.getAddress(); // MAC address
                if(deviceName == named){
                    mac = deviceHardwareAddress;
                }
            }
        }
        OutputStream outputStream = null;
        try {
            outputStream = openFileOutput(fileName, 0);
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        }
        OutputStreamWriter osw = new OutputStreamWriter(outputStream);
        try {
            osw.write(mac);
        } catch (IOException e) {
            e.printStackTrace();
        }
        try {
            osw.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
        name.setText(mac.toString());
    }

}

Layout

<EditText
        android:id="@+id/editText"
        android:layout_width="0dp"
        android:layout_height="59dp"
        android:layout_marginBottom="16dp"
        android:layout_marginEnd="8dp"
        android:layout_marginStart="8dp"
        android:layout_marginTop="8dp"
        android:ems="10"
        android:hint="Введите целое число от 0 до 255"
        android:inputType="number"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintVertical_bias="0.215" />

    <Button
        android:id="@+id/button"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginBottom="8dp"
        android:layout_marginEnd="8dp"
        android:layout_marginStart="8dp"
        android:layout_marginTop="8dp"
        android:background="@android:color/background_light"
        android:text="Отправить"
        android:textColor="@color/colorPrimaryDark"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="0.028"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintVertical_bias="0.389" />

    <EditText
        android:id="@+id/name"
        android:layout_width="0dp"
        android:layout_height="48dp"
        android:layout_marginBottom="8dp"
        android:layout_marginEnd="8dp"
        android:layout_marginStart="8dp"
        android:layout_marginTop="8dp"
        android:ems="10"
        android:hint="Введите имя блютуз устройства"
        android:inputType="textPersonName"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="0.102"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintVertical_bias="0.017" />

    <Button
        android:id="@+id/button2"
        android:layout_width="238dp"
        android:layout_height="45dp"
        android:layout_marginBottom="8dp"
        android:layout_marginEnd="8dp"
        android:layout_marginStart="8dp"
        android:layout_marginTop="8dp"
        android:background="@android:color/background_light"
        android:onClick="macsave"
        android:text="Сохранить устройство"
        android:textColor="@color/colorPrimaryDark"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="0.938"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintVertical_bias="0.386" />

Manifest

<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.example.steshenko.blueterm">

    <uses-permission android:name="android.permission.BLUETOOTH" />
    <uses-permission android:name="android.permission.BLUETOOTH_ADMIN" />
    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
    <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />

    <application
        android:allowBackup="true"
        android:icon="@mipmap/ic_launcher"
        android:label="@string/app_name"
        android:roundIcon="@mipmap/ic_launcher_round"
        android:supportsRtl="true"
        android:theme="@style/AppTheme">
        <activity android:name=".MainActivity">
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />

                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
        </activity>
    </application>

</manifest>

Answer 1

[Alex Marken]

Скинь лог (Logcat) ошибки.

UPD:
Cpper, посмотрел Ваш код. К сожалению, у меня на Genymotion нет Bluetooth, чтобы проверить.
Меня насторожили эти строчки в методе macsave:

OutputStream outputStream = null;
        try {
            outputStream = openFileOutput(fileName, 0);
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        }
        OutputStreamWriter osw = new OutputStreamWriter(outputStream);

1:
Для того, чтобы открыть файл в режиме Write, требуется соответствующее разрешение в Manifest. Это касается выполнения outputStream = openFileOutput(fileName, 0).
Далее...Раз у Вас нет разрешения на выполнение записи данных, outputStream останется NULL. В этом случае, при создании потока записи данных "new OutputStreamWriter(outputStream)", приложение выдаст ошибку (outputStream является NULL).

2:
Убедитесь, что открываемый файл существует ("macadress.txt"). В противном случае, его необходимо создать (требуются права на WRITE).

3:
Ещё один момент, неверно указан путь до файла. Если файл лежит в папке приложения, путь указывается следующий: "//data//data//НАЗВАНИЕ ПАКЕТА//macadress.txt".

Comments

[Cpper]

Написал в комментариях под вопросом, лог получить не могу :(

[Alex Marken]

Cpper, прощу прощения, не увидел. Дополни код в комменте под вопросом: нужен полный manifest, gradle

[Cpper]

Alex Marken, полный манифест написал. Что gradle? Вывод gradle консоли добавить?

[Alex Marken]

Cpper, посмотрел Ваш код. К сожалению, у меня на Genymotion нет Bluetooth, чтобы проверить.
Меня насторожили эти строчки в методе macsave:

OutputStream outputStream = null;
        try {
            outputStream = openFileOutput(fileName, 0);
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        }
        OutputStreamWriter osw = new OutputStreamWriter(outputStream);

1:
Для того, чтобы открыть файл в режиме Write, требуется соответствующее разрешение в Manifest. Это касается выполнения outputStream = openFileOutput(fileName, 0).
Далее...Раз у Вас нет разрешения на выполнение записи данных, outputStream останется NULL. В этом случае, при создании потока записи данных "new OutputStreamWriter(outputStream)", приложение выдаст ошибку (outputStream является NULL).

2:
Убедитесь, что открываемый файл существует ("macadress.txt"). В противном случае, его необходимо создать (требуются права на WRITE).

3:
Ещё один момент, неверно указан путь до файла. Если файл лежит в папке приложения, путь указывается следующий: "//data//data//НАЗВАНИЕ ПАКЕТА//macadress.txt".

[Cpper]

Alex Marken, права в манифесте вроде ненужны, я читал в интернете. И я писал типо блокнот и прав в манифесте не давал и все работало. Вроде нужны только если писать на внешнюю память.
Спасибо, все замечания учту

[Cpper]

Alex Marken, при вызове macsave как раз все вылетает

[Alex Marken]

Cpper, для доступа в режиме WRITE права нужны (как раз в методе macsave). В OnCreate у тебя чтение (Input) файла, не путай эти режимы. Вот и ошибки :)

[Cpper]

Спасибо:)
Попробую переделать. Говорю же я ещё начинающий :)

[Cpper]

Alex Marken, вообщем ошибка не с блютуз связана, это точно? :)

[Alex Marken]

Cpper, точно утверждать не могу, как написал уже выше - у меня нет в эмуляторе Bluetooth. Первым делом устрани те ошибки, которые указал ранее (вполне вероятно, только они и приводили к вылету), и проверь на физическом устройстве. Если будет ошибка, скинь лог :)

[Cpper]

Alex Marken, можно вопрос не по теме? :)
Я создал свой блютуз адаптер, напрмер myBluetoothAdapter
Что мне сделать, чтобы я его мог использовать и в следующем активити? :)
Насколько я понимаю заново объявлять это неправильно, я закрывать его в прошлом не подходит :)

[Alex Marken]

Cpper, объяви переменную myBluetoothAdapter через static. Таким образом, ты сможешь получить доступ к ней с других классов, в том числе и активности.

Пример:

class MyClass {
    public static Object obj;
}

class Test {
    public Test() {
        Object j = MyClass.obj;
    }
}

[Cpper]

Alex Marken, благодарю :)

[Cpper]

Alex Marken И снова я. Ищу в интернете примеры работы с блютуз, но находятся только с передачей данных. А меня интересует прием? :) Может знаете где найти понятный пример?

[Alex Marken]

Cpper, запрос в гугле "Receiving data bluetooth Android".
Вот ссылка на пример с GitHub

[Cpper]

Alex Marken, а я в гугле ... read писал и муть какую-то выдавало :)
Спасибо, посмотрю, жаль что не на русском и коменты не везде

Answer 2

[stoptalking]

непонятно что именно вы делаете для того, чтобы получить возврат. Если просто запустить и нажать "сохранить устройство", то это Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'int java.lang.String.length()' on a null object reference в строке osw.write(mac);

Comments

[stoptalking]

Если еще точнее возврат кидается в коде ио райтера:

public void write(String str) throws IOException {
        write(str, 0, str.length());
}

вызов str.length() на нулл-str приведет к нпе

[stoptalking]

кидайте новый код контроллера своего (после того как вы выключили ио) и сценарий при котором крашится

Answer 3

[Cpper]

Всем большое спасибо! Ошибка решилась, все имена и мак адреса приложение показывает.