Question

Je suis en train de créer un tableau qui ne marche pas ajouter des éléments si la valeur / taille est supérieure à 20. cette solution ajoute seulement 1 point, puis arrête même si la valeur est inférieure à 20. comment puis-je le modifier pour qu'il accepte uniquement les valeurs jusqu'à 20 max.

package business;
import java.io.Serializable;
import java.util.ArrayList;

public class Cart implements Serializable
{
private ArrayList<LineItem> items;

    public Cart()
{
    items = new ArrayList<LineItem>();
}

    public ArrayList<LineItem> getItems()
{
    return items;
}

    public int getCount()
{
    return items.size();
}

    public void addItem(LineItem item)
{
    String code = item.getProduct().getCode();
    int quantity = item.getQuantity();
    double credit = item.getProduct().getCHours();
    String credit2 = Double.toString(item.getProduct().getCHours());
    int isize = items.size();


    for (int i = 0; i < items.size(); i++)
    {
        if(isize <= 20)
        {
        LineItem lineItem = items.get(i);
            lineItem.setQuantityCredit(credit);
            return;
       }
    }
    items.add(item);
    }

    public void addItemCredit(LineItem item)
{
        double credit = item.getProduct().getCHours();
        String credit2 = Double.toString(item.getProduct().getCHours());
        String code = item.getProduct().getCode();

    for (int i = 0; i < 20; i++)
    {
        LineItem lineItem2 = items.get(i);
            lineItem2.setQuantityCredit(credit);
            return;

        }
    items.add(item);

    }

public void removeItem(LineItem item)
{
        String code = item.getProduct().getCode();
        for (int i = 0; i < items.size(); i++)
        {
            LineItem lineItem = items.get(i);
            if (lineItem.getProduct().getCode().equals(code))
            {
                items.remove(i);
                return;
            }
        }
    }
}

Je pense que wat u dit est juste si j'ai essayé ce

if(isize <= 20) 
{ 
LineItem lineItem = items.get(i); lineItem.setQuantityCredit(credit); 
} 
return; 
} 
items.add(item); 
} 


and 


if(isize <= 20) 
{ 
LineItem lineItem = items.get(i); lineItem.setQuantityCredit(credit); 
} 
} 
items.add(item); 
return; 
} 

mais ni travaillé. si whr reáliser déclaration de retour d?

Était-ce utile?

La solution

Vous êtes de retour de la méthode de addItem avant d'ajouter l'élément à la liste.

if(isize <= 20)
{
     LineItem lineItem = items.get(i);
     lineItem.setQuantityCredit(credit);
     return; // This is the problem
}

Autres conseils

Je veux juste préciser votre code avant de tenter de répondre.

// get the total number of items in the cart
int isize = items.size();

// loop through the items
   // if there are more than 20 items then set the credit quantity and
     // return without adding
   // otherwise do nothing 
for (int i = 0; i < items.size(); i++)
{
    if(isize <= 20)
    {
    LineItem lineItem = items.get(i);
        lineItem.setQuantityCredit(credit);
        return;
   }
}
// if there are more than 20 items add another
items.add(item);

La raison pour laquelle il ajoute qu'un seul article ne se présente comme suit: La première fois que cette méthode est appelée « isize » est égal à zéro et donc la boucle est ignorée. La deuxième fois que cette méthode est appelée « isize » est égal à zéro et donc la boucle va une fois et retourne avant d'ajouter.

Je ne suis pas certain de ce que vous essayez de faire à l'intérieur de la boucle car il n'a vraiment pas de sens. Si vous voulez juste le tableau d'éléments à se limiter à 20 ou moins juste ajouter

if (items.size() < 20) {
  items.add(item);
}

à votre code. S'il y a une certaine logique plus complexe que vous essayez ensuite d'expliquer vos objectifs.

Licencié sous: CC-BY-SA avec attribution
Non affilié à StackOverflow
scroll top