Django视图无法保存数据至模型问题求助

Hey there! Let's figure out why your Coinbase data isn't sticking to the user's account. Based on what you've shared, here are the most common issues and fixes to try:

Common Issues & Fixes #

1. You're not properly targeting the logged-in user #

Since you're using login_required, request.user gives you the current authenticated user—but you need to explicitly link this user to their UserAddress record. If you skip this association, the system has no idea which account to save the data to.

2. You forgot to call .save() on the model instance #

This is the #1 mistake! Even if you set the field value on your UserAddress object, you have to call .save() to write those changes to the database. No save call = no data stored.

3. You're not handling the UserAddress instance correctly #

Do you want to create a new record if the user doesn't have one, or update an existing entry? Use get_or_create to cover both scenarios, or get (with error handling) if you know the record already exists for the user.

4. JSON-to-string conversion is off #

response.json() returns a Python dictionary, not a string. You need to use json.dumps() to convert that dict into a storable string that your model field can accept.

5. No error handling means you're missing critical clues #

If something breaks (like a failed API call or misnamed model field), you won't know unless you catch exceptions. Adding try/except blocks will help you pinpoint exactly where things go wrong.

Fixed Example Code #

Here's how to adjust your view to cover all these bases:

from django.shortcuts import render
from django.http import HttpResponse
from django.contrib.auth.decorators import login_required
from ico_login.models import UserAddress
import requests
import json

@login_required
def fetch_coinbase_data(request):
    # Only handle POST requests (button clicks should use POST to avoid duplicate submissions)
    if request.method == 'POST':
        try:
            # Fetch data from Coinbase API (update the endpoint to match your needs)
            coinbase_response = requests.get(
                "https://api.coinbase.com/v2/accounts",
                auth=('YOUR_API_KEY', 'YOUR_API_SECRET')  # Tip: Use environment variables for credentials in production!
            )
            # Raise an error if the API call fails (e.g., 404, 500)
            coinbase_response.raise_for_status()
            
            # Convert the JSON response to a string
            coinbase_data_str = json.dumps(coinbase_response.json())
            
            # Get or create the UserAddress record tied to the current user
            user_address, was_created = UserAddress.objects.get_or_create(user=request.user)
            
            # Assign the data and save to the database
            user_address.coinbase_data = coinbase_data_str  # Replace with your actual model field name
            user_address.save()
            
            return HttpResponse("Coinbase data saved successfully!")
        
        # Handle API request errors
        except requests.exceptions.RequestException as e:
            return HttpResponse(f"Failed to fetch Coinbase data: {str(e)}", status=500)
        
        # Handle missing UserAddress records (if you use get() instead of get_or_create)
        except UserAddress.DoesNotExist:
            return HttpResponse("No address record found for this user!", status=404)
        
        # Catch-all for other unexpected errors
        except Exception as e:
            return HttpResponse(f"Error saving data: {str(e)}", status=500)
    
    # Reject non-POST requests (since this is tied to a button click)
    else:
        return HttpResponse("Invalid request method. Use POST.", status=400)

Double-Check Your Model #

Make sure your UserAddress model has a field capable of storing long strings (use TextField—CharField has strict length limits):

from django.contrib.auth.models import User
from django.db import models

class UserAddress(models.Model):
    user = models.OneToOneField(User, on_delete=models.CASCADE)
    coinbase_data = models.TextField()  # This field will store your JSON string
    # Add your other model fields here...

Quick Debug Tips #

• Print request.user in your view to confirm you're targeting the correct logged-in user.
• Print coinbase_data_str to verify it's a valid, properly formatted string.
• Check your database directly (via Django admin or a database tool) to see if the record is being created/updated.

内容的提问来源于stack exchange，提问作者Jiroscopes