C# ADB & Fastboot GUI — Basit ve Kullanıcı Dostu ADB Kullanım Arayüzü

3613 dedi:

Super kullanisli olmus. Bir suru batch dosyasi ile ugrasmaktan kurtaracak bir yazilim.

Oneri olarak sunu sunayim, mantikli gelirse eklenir. Scrcpy butonu ekleyip ekran aktarimida olabilir. Boot etmis telefonda bazen telefon ekranindan bir seyler girmek ya da ekrani gormek gerekiyor. Bu sayede telefonu elimize almamiza hic gerek kalmaz.

Genişletmek için tıkla...

pottie4r dedi:

Merhaba; ADB nedir bir fikrim yok, Android cihaz modlamayı da bilmem. Fakat tepkilerce gördüğüm kadarıyla bir işi kolaylaştırmışsınız öncelikle sizi tebrik ediyorum.

Programınız çalışıyor fakat pek iyi kodlandığını söyleyemem. Kendinizi çok fazla tekrar etmek zorunda kalmışsınız. Örneğin her metodunuzda neredeyse şu blok var:

C#:

string selectedDevice = null;

if (ADBFirstDevice.IsChecked == true)
    selectedDevice = ADBFirstDevice.Content.ToString();
else if (ADBSecondDevice.IsChecked == true)
    selectedDevice = ADBSecondDevice.Content.ToString();
else if (ADBThirdDevice.IsChecked == true)
    selectedDevice = ADBThirdDevice.Content.ToString();

Neden mesela bu kodu bir fonksiyona çevirip selectedDevice'ı return etmiyorsunuz? Böylece her seferinde tekrar yazmanız gerekmez.
Ayrıca ileride dördüncü bir cihaz gerektiğinde de bütün kodları baştan aşağı değiştirmemiş olursunuz.

Ayrıca hangi Thread'i kim başlatıyor, ne oluyor çok karışık. Düşük özellikli sistemlerde sorunlar yaşatabilir.

Genişletmek için tıkla...

pottie4r dedi:

Ayrıca hangi Thread'i kim başlatıyor, ne oluyor çok karışık. Düşük özellikli sistemlerde sorunlar yaşatabilir.

Genişletmek için tıkla...

pottie4r dedi:

C#:

private void DoSth1(){
    string selectedDevice = "";
    if(cb1.checked) selectedDevice = "Device 1";
    if(cb2.checked) selectedDevice = "Device 1";
    if(cb3.checked) selectedDevice = "Device 1";
   
    //... Diğer kodlar DoSth1
}

private void DoSth2(){
    string selectedDevice = "";
    if(cb1.checked) selectedDevice = "Device 1";
    if(cb2.checked) selectedDevice = "Device 1";
    if(cb3.checked) selectedDevice = "Device 1";
   
    //... Diğer kodlar DoSth2
}

gibi her metodun içinde bunu yapmak yerine şöyle yapın mesela;

C#:

private string GetSelectedDevice(){
    string selectedDevice = "";
    if(cb1.checked) selectedDevice = "Device 1";
    if(cb2.checked) selectedDevice = "Device 1";
    if(cb3.checked) selectedDevice = "Device 1";
}

private void DoSth1(){
    string device = GetSelectedDevice();
   
    //... Diğer kodlar DoSth1
}

private void DoSth2(){
    string device = GetSelectedDevice();
   
    //... Diğer kodlar DoSth2
}

Bu sayede selectedDevice'ı ayarlama işi tek bir merkezden yönetilir ve güncellemelerde gidip her methoddaki kodu değiştirmek yerine sadece bir methoddan değiştirirsiniz.

Genişletmek için tıkla...

pottie4r dedi:

Bu dediğinizi anlamadım. Şu an kodunuzda her ADB işlemi için yeni bir thread yaratılıp işlem yaptırılıyor. Şu durumda aynı anda 2 ADB komutu eşzamanlı olarak çalıştırılabilir ancak bu sorun yaratır mı mesela?

Zannımca siz program işlem esnasında tıkandığı ve pencere yanıt veremediği için Thread'lere sürekli başvurdunuz. Fakat bunun yerine işlemleri yine sırayla yapmayı garantileyecek şekilde asenkron programlama kullanabilirsiniz. Böylece Thread'leriniz kaybolmaz, fırlatılan hatalar yakalanabilir ve Thread yönetimi kolay olur.

Genişletmek için tıkla...

Thread th = new Thread(() =>
                        {
                            string output = Commander($"fastboot -s {selectedDevice} reboot recovery");
                        });

                        th.IsBackground = true;
                        th.Start();

                        if (selectedDevice == "Device Not Found")
                            MessageBox.Show("CONNECT OR SELECT A DEVICE!");
                        else
                            MessageBox.Show($"{selectedDevice} Rebooting recovery using fastboot.");