2017-09-16 129 views
0

用户必须通过点击地图来添加标记。我的目标是将名称,类别,纬度和经度发送到SQL数据库。
我跟着这个问题:How can you pass multiple primitive parameters to AsyncTask?,但我在这一行出现错误:AsyncTask执行错误

backgroundTask.execute(method,Name,Category,Latitude,Longitude); 

看到代码:

AddShopActivity.java

public class AddShopActivity extends MainScreen implements OnMapReadyCallback, GoogleApiClient.ConnectionCallbacks, GoogleApiClient.OnConnectionFailedListener, com.google.android.gms.location.LocationListener { 

Spinner spinner; 
ArrayAdapter<CharSequence> adapter; 


GoogleMap mGoogleMap; 
GoogleApiClient mGoogleApiClient; 

String Name, Category; 
Double Latitude, Longitude, Latitude_pass, Longitude_pass; 

EditText shop_name = (EditText)findViewById(R.id.Shop_Name); 
Spinner shop_category = (Spinner)findViewById(R.id.spinner); 

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_add_shop); 
    initMap(); 
    spinner = (Spinner) findViewById(R.id.spinner); 
    adapter = ArrayAdapter.createFromResource(this, R.array.eidoskatastimatos, android.R.layout.simple_spinner_item); 
    adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item); 
    spinner.setAdapter(adapter); 
} 

private void initMap() { 
    MapFragment mapFragment = (MapFragment) getFragmentManager().findFragmentById(R.id.mapFragment); 
    mapFragment.getMapAsync(this); 
} 

@Override 
public void onMapReady(GoogleMap googleMap) { 
    mGoogleMap = googleMap; 
    mGoogleMap.setMapType(GoogleMap.MAP_TYPE_NORMAL); 
    mGoogleMap.getUiSettings().setZoomControlsEnabled(true); 
    mGoogleApiClient = new GoogleApiClient.Builder(this) 
      .addApi(LocationServices.API) 
      .addConnectionCallbacks(this) 
      .addOnConnectionFailedListener(this) 
      .build(); 
    mGoogleApiClient.connect(); 
} 

LocationRequest mLocationsRequest; 

@Override 
public void onConnected(Bundle bundle) { 
    mLocationsRequest = LocationRequest.create(); 
    mLocationsRequest.setPriority(LocationRequest.PRIORITY_HIGH_ACCURACY); 
    mLocationsRequest.setInterval(5000); 
     if (ActivityCompat.checkSelfPermission(this, android.Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED && ActivityCompat.checkSelfPermission(this, android.Manifest.permission.ACCESS_COARSE_LOCATION) != PackageManager.PERMISSION_GRANTED) { 
      // TODO: Consider calling 
      // ActivityCompat#requestPermissions 
      // here to request the missing permissions, and then overriding 
      // public void onRequestPermissionsResult(int requestCode, String[] permissions, 
      //           int[] grantResults) 
      // to handle the case where the user grants the permission. See the documentation 
      // for ActivityCompat#requestPermissions for more details. 
      return; 
     } 

    LocationServices.FusedLocationApi.requestLocationUpdates(mGoogleApiClient, mLocationsRequest, this); 


    mGoogleMap.setOnMapClickListener(new GoogleMap.OnMapClickListener() { 
     @Override 
     public void onMapClick(LatLng latLng) { 

      MarkerOptions marker = new MarkerOptions() 
             .position(new LatLng(latLng.latitude, latLng.longitude)) 
             .draggable(true) 
             .title(shop_name.getText().toString()) 
             .snippet(shop_category.getSelectedItem().toString()); 


      CameraUpdate update = CameraUpdateFactory.newLatLngZoom(latLng, 16); 
      mGoogleMap.animateCamera(update); 

      mGoogleMap.clear(); 
      mGoogleMap.addMarker(marker); 

      Latitude_pass = latLng.latitude; 
      Longitude_pass = latLng.longitude; 
     } 
    }); 

} 

public void shopReg(View view) 
{ 
    Name = shop_name.getText().toString(); 
    Category = shop_category.getSelectedItem().toString(); 
    Latitude = Latitude_pass; 
    Longitude = Longitude_pass; 
    String method = "save"; 
    BackgroundTask backgroundTask = new BackgroundTask(this); 
    backgroundTask.execute(method,Name,Category,Latitude,Longitude); //here is the error. It asks me to create the method 'execute' but the guy in a tutorial I am watching doen't do this// 
    finish(); 
} 

@Override 
public void onConnectionSuspended(int i) { 

} 

@Override 
public void onConnectionFailed(ConnectionResult connectionResult) { 

} 

@Override 
public void onLocationChanged(Location location) { 
    if (location == null){ 
     Toast.makeText(this, "Can't get current location", Toast.LENGTH_LONG).show(); 
    } else { 
     LatLng ll = new LatLng(location.getLatitude(), location.getLongitude()); 
     CameraUpdate update = CameraUpdateFactory.newLatLngZoom(ll, 16); 
     mGoogleMap.animateCamera(update); 
    } 

} 
} 

BackgroundTask.java

public class BackgroundTask extends AsyncTask<String,Void,String> { 

String Name, Category; 
Double Latitude, Longitude; 

BackgroundTask(String method, String Name, String Category, Double Latitude, Double Longitude) { 
    this.Name = Name; 
    this.Category = Category; 
    this.Latitude = Latitude; 
    this.Longitude = Longitude; 
} //Trying to pass 2 types of variablew, string and double, because lat and long are double// 

Context ctx; 

BackgroundTask(Context ctx){ 
    this.ctx = ctx; 
} 

@Override 
protected void onPreExecute() { 
    super.onPreExecute(); 
} 

@Override 
protected String doInBackground(String... params) { 
    String reg_url = "http://10.0.2.2/shop/register.php"; 
    String method = params[0]; 
    if(method.equals("save")) 
    { 
     String Name = params[1]; 
     String Category = params[2]; 
     String Latitude = params[3]; 
     String Longitude = params[4]; 
     try { 
      URL url = new URL(reg_url); 
      HttpURLConnection httpURLConnection = (HttpURLConnection) url.openConnection(); 
      httpURLConnection.setRequestMethod("POST"); 
      httpURLConnection.setDoOutput(true); 
      OutputStream OS = httpURLConnection.getOutputStream(); 
      BufferedWriter bufferedWriter = new BufferedWriter(new OutputStreamWriter(OS, "UTF-8")); 
    String data = URLEncoder.encode("Name", "UTF-8") +"="+URLEncoder.encode(Name,"UTF-8")+"&"+ 
      URLEncoder.encode("Category", "UTF-8") +"="+URLEncoder.encode(Category,"UTF-8")+"&"+ 
      URLEncoder.encode("Latitude", "UTF-8") +"="+URLEncoder.encode(Latitude,"UTF-8")+"&"+ 
      URLEncoder.encode("Longitude", "UTF-8") +"="+URLEncoder.encode(Longitude,"UTF-8"); 
      bufferedWriter.write(data); 
      bufferedWriter.flush(); 
      bufferedWriter.close(); 
      OS.close(); 
      InputStream IS = httpURLConnection.getInputStream(); 
      IS.close(); 
      return "Το κατάστημα προστέθηκε!"; 
     } catch (MalformedURLException e) { 
      e.printStackTrace(); 
     } catch (IOException e) { 
      e.printStackTrace(); 
     } 

    } 

    return null; 
} 

@Override 
protected void onProgressUpdate(Void... values) { 
    super.onProgressUpdate(values); 
} 

@Override 
protected void onPostExecute(String result) { 
    Toast.makeText(ctx,result,Toast.LENGTH_LONG).show(); 
} 
} 
+1

请添加错误 – cutiko

回答

3

查看链接的问题agai ñ。你错过了“Params”类。

execute的参数传递给doInBackground,您的预期值仅为String[]。然而,这并不完全错误所在

您的其他构造函数没有被用来获取2个字符串和2张双人床

你只使用的上下文一个

new BackgroundTask(this); 

教程我看着

我可以说是再次尝试观看,或找到另一个

我敢打赌,它仅调用execute("save");并具有这样的事情

// add the context to this constructor also 
new BackgroundTask(this, Name, Category, Latitude, Longitude); 

或者,看的AsyncTask,你需要使用Double.parseDouble,你必须通过串

String Name = params[1]; 
    String Category = params[2]; 
    Double Latitude = Double.parseDouble(params[3]); 
    Double Longitude = Double.parseDouble(params[4]); 
+0

在链接中看到第3个答案(与63的投票)。我试图申请的那一个。同时检查_edited code_,我做了一些改变,如你所说。这次没有出现红色下划线的错误,应用程序正在运行,但是当我点击调用'shopReg'的按钮**时,它会崩溃。 –

+0

我无法修复代码中的所有运行时异常。阅读日志并了解如何修复它们。只有当你真的被卡住了,但实际上试图了解错误日志告诉你什么 –

+0

而且我认为这个答案可以修复你的初始帖子。编辑你的问题会使你在这里收到的答案无效。建议采取的行动是接受一个答案,然后使用logcat消息将更正后的代码重新发布为[mcve] –

0

AsyncTask<String...的参数传递到execute方法都是字符串类型。你正在传球,字符串和双。

作为一般性建议,按照骆驼案例惯例编写变量。使用私人accesor。

你的AsyncTask应该


public class BackgroundTask extends AsyncTask { 

private String name, category; 
private double latitude, longitude; 

BackgroundTask(double latitude, double longitude) { 
    this.latitude = latitude; 
    this.longitude = longitude; 
} 
} 

现在你的构造函数初始化双打和你现在做它从PARAMS得到的字符串。

相关问题