méthodes de java en multipliant les éléments dans un tableau à l'aide d'une boucle

Définir une méthode en Java nommé weightedSum() qui prend deux entiers tableaux comme arguments. L'
la méthode utilise une boucle pour multiplier les éléments correspondants des deux tableaux (c'est à dire, il multiplie le premier argument de chaque tableau d'ensemble, suivie en multipliant le deuxième élément de chaque tableau, et ainsi de suite), et renvoie la somme de ces produits (qui est aussi un entier). On peut supposer que les deux tableaux sont de longueur égale.

public int weightedSum(int [] a ,int [] b)


    {
        int value;
        int sum ;

        for (int i = 0 ; i < a.length ; i++)
        {
            value = a[i] * b [i];
            value  = value +value ;

        }

        return value;

J'ai du mal à écrire cette méthode pour mon affectation . J'en vertu de la stand de il accepte de tableaux, mais j'ai du mal à écrire la boucle elle-même de sorte qu'il multiplie chaque élément du tableau avec son homologue de l'inverse de la matrice de sorte pos [1] * pos [1], puis ajoutez les deux valeurs togather avec pos [2] + pos[2] et pour obtenir la somme totale de toutes les valeurs

  • Astuce: vous avez une variable appelée sum, mais vous ne l'utilisez pas...
  • pour tester votre méthode, similaire à celui de l'impression de messages comme décrit ici - docs.oracle.com/javase/tutorial/java/nutsandbolts/arrays.html - pour comprendre ce qui se passe dans chaque itération. Commencez avec 2 tableaux à 3 éléments définis dans chaque et ajouter un peu de l'impression dans votre code pour le comprendre.
InformationsquelleAutor n94pro | 2014-05-07
  1. 1

    ci-dessous sont les changements nécessaires dans votre code. Fondamentalement, vous n'avez jamais mis à jour la variable somme après permet de calculer le produit des éléments correspondants dans les deux tableaux. Aussi, vous pourriez voulez utiliser pour le type long pour stocker le résultat de la somme & le produit que l'int valeur peut déborder si les éléments du tableau sont suffisamment importantes.

    public long weightedSum(int [] a ,int [] b)
{
    long value = 0;
    long sum = 0 ;

    for (int i = 0 ; i < a.length ; i++)
    {
        value = a[i] * b [i];
        sum = sum +value ;

    }

    return sum;
}
    • Ce ne parvenez toujours pas à compiler. sum n'est certainement pas attribué. (Je voudrais aussi changer le champ d'application de value, mais ça c'est une autre affaire.)
    • initialisation de la somme à zéro, comme int somme = 0
    • "les retours de la somme de ces produits (qui est aussi un entier)" Ce n'est plus conforme à la spéc.
    • J'ai changé le contrat sur le but d'indiquer à l'OP que si l'int est retourné il y a une possibilité de résultat incorrect en raison de dépassement de capacité. L'OP est libre de décider si elle veut garder le retour de type int/long
    InformationsquelleAutor Kakarot
  2. 1

    Essayez ceci:

    public long dotProduct(int [] a, int [] b) {
    if (a == null) throw new IllegalArgumentException("a array cannot be null");
    if (b == null) throw new IllegalArgumentException("b array cannot be null");
    if (a.length != b.length) throw new IllegalArgumentException("arrays must have equal lengths");
    long sum = 0L;
    for (int i = 0; i < a.length; i++) {
        sum += a[i]*b[i];
    }
    return sum;
}
    InformationsquelleAutor duffymo
  3. 0

    La plupart du code est correct, cependant la deuxième ligne dans la boucle ne devrait pas changer la valeur de la variable, la variable de la somme doit être mis à jour avec la valeur.
    quelque chose comme somme += valeur.
    puis retour somme.

    InformationsquelleAutor Narble
  4. -2

    Changer à cela:

    value += a[i] * b [i];
//value = value + value;

    Aussi initialiser value à 0 avant de l'utiliser.
    Retirez également le sum variable.

    • Qui ne compile pas, et il ne sera certainement pas de retour de la somme.
    • Ce qui suggère commenter du code est inutile, s'il n'est pas nécessaire, alors il devrait être supprimé
    • la somme n'est jamais utilisé, dans le code d'origine, c'est toujours les mises à jour de la valeur
    • Downvoters sont trop rapide ici 🙂 Ma réponse est tout simplement parfait.
    • Votre première réponse était pas très bien. Ce n'est pas le downvoters qui ont été trop rapide: il a été votre réponse, ce qui était erroné. J'ai enlevé le downvote maintenant, mais vous ne pouvez pas vraiment se plaindre arriver downvotes pour une réponse incorrecte. (Il n'est toujours pas un bon réponse de l'OMI, en ce qu'il n'a pas vraiment d'expliquer quoi que ce soit...)
    • Je ne me plains pas, je trouve ça drôle 😉
    • Mon commentaire ne mentionne pas la sum variable; il faisait référence à la somme des valeurs, qui est le sens de la question posée. Pierre première réponse n'a pas renvoyé de la somme, donc, était incorrecte.
    • Oui, Jon, le très initiale était erronée. Je ne suis pas d'argumenter à ce sujet. Encore, downvoters sont trop rapides... cliquez sur cliquez sur cliquez sur 😉
    • Regardez votre commentaire, qui a critiqué le comportement de ceux qui downvoting votre réponse, qui était certainement initialement incorrect. Le fait ressembler à vous de trouver ça drôle, ou faut-il regarder comme une plainte?
    • Le smiley donne la réponse ici 🙂 xkcd.com/438
    • Si la réponse est incorrecte, il est inutile, à ce titre, mérite un downvote. Suggérez-vous qu'avant de downvoting, on devrait traîner pendant un moment juste au cas où la réponse est fixe? Mauvaise idée de l'OMI - je dirais que c'est à la répondeur pour vous assurer que la réponse est correcte avant de poster. Dans ce cas, vous ont été trop rapide, pas la downvoters. (Et non, ce n'est pas dire que mes réponses sont toujours correct de commencer avec soit - mais si quelqu'un downvotes une réponse incorrecte de la mienne, j'aurais jamais dire qu'ils sont "trop rapide". Ils se comporter entièrement de façon appropriée.)

    InformationsquelleAutor peter.petrov