java-Methoden, die Multiplikation von Elementen in einem array mit Hilfe einer Schleife

Definieren Sie eine Java-Methode namens weightedSum (), der zwei integer-arrays als Argumente. Die
die Methode verwendet eine Schleife multipliziert entsprechende Elemente der beiden arrays zusammen (d.h., es vermehrt sich das erste argument von jedem array zusammen, gefolgt von multiplizieren, das zweite element jedes Arrays zusammen, und so weiter) und gibt die Summe dieser Produkte (die ebenfalls eine ganze Zahl). Sie können davon ausgehen, dass beide arrays gleich lang sind.

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;

Bin ich Probleme beim schreiben diese Methode für meine Aufgabe . Ich unter stehen, es akzeptiert zu arrays, aber ich habe Probleme beim schreiben der Schleife selbst, so dass Sie vermehrt sich jedes einzelne element des Arrays mit seinem Gegenstück im Gegenteil-array pos [1] * pos [1], und fügen Sie dann die beiden Werte togather mit pos [2] + pos[2] und um die Summe für alle Werte

  • Tipp: du hast eine variable namens sum, aber Sie nutzen Sie nicht...
  • testen Sie Ihre Methode, verwenden Sie einen ähnlichen drucken von Nachrichten wie hier beschrieben - docs.oracle.com/javase/tutorial/java/nutsandbolts/arrays.html - um zu verstehen, was passiert, in jeder iteration. Beginnen Sie mit 2 arrays mit 3 Elementen definiert, die in jedem und fügen Sie einige drucken in Ihrem code, um es zu verstehen.
InformationsquelleAutor n94pro | 2014-05-07
  1. 1

    unten sind die erforderlichen änderungen in Ihrem code. Im Grunde haben Sie nie aktualisiert die variable Summe, nachdem Sie berechnen das Produkt der entsprechenden Elemente in den beiden arrays. Auch Sie könnte verwenden möchte, long-Datentyp zum speichern des Ergebnisses der Summe & das Produkt als int-Wert könnte überlaufen, wenn die Elemente im array sind ausreichend groß.

    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;
}
    • Diese noch nicht kompiliert. sum ist nicht definitiv zugewiesen. (Ich würde auch den Bereich ändern, der value, aber das ist eine etwas andere Sache.)
    • initialisieren Sie die Summe mit null wie int Summe = 0
    • "gibt die Summe dieser Produkte (die ebenfalls eine ganze Zahl)" Dies entspricht nicht mehr den spec.
    • Ich änderte den Vertrag über die Zwecke anzugeben, für die OP, dass wenn int zurückgegeben wird, gibt es eine Möglichkeit, falsche Ergebnisse wegen überlauf. Die OP ist frei zu entscheiden, ob er/Sie will halten Sie den Rückgabetyp als int/long
    InformationsquelleAutor Kakarot
  2. 1

    Versuchen Sie dies:

    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

    Meisten der code korrekt aussieht, aber die zweite Zeile in der Schleife nicht verändert werden, der Wert der variable, die variable Summe aktualisiert werden soll, mit dem Wert.
    so etwas wie Summe += Wert.
    dann wieder Summe.

    InformationsquelleAutor Narble
  4. -2

    Änderung:

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

    Auch initialisieren value zu 0 bevor Sie es verwenden.
    Entfernen Sie auch die sum variable.

    • Das wird nicht kompiliert, und es wird sicherlich nicht die Summe.
    • Was für einen Kommentar code ist sinnlos, wenn es nicht benötigt wird, dann sollte es gelöscht werden
    • die Summe ist nie benutzt, im original-code, ist es immer updates Wert
    • Downvoters zu schnell hier 🙂 Meine Antwort ist in Ordnung.
    • Ihre erste Antwort war nicht einfach nur gut. Es ist nicht die downvoters, die waren zu schnell: war es Ihre Antwort, die falsch war. Ich habe entfernt die downvote jetzt, aber man kann nicht wirklich beschweren sich über die immer downvotes für eine falsche Antwort. (Es ist noch nicht ein guter Antwort IMO, dass es nicht wirklich alles erklären...)
    • Ich will mich nicht beschweren, ich finde es lustig 😉
    • Mein Kommentar hat nicht erwähnt sum variable; es bezog sich auf die Summe der Werte, was in der Frage gefragt wurde. Peter, die erste Antwort kam nicht zurück die Summe, die so falsch war.
    • Ja, Jon, der sehr erste war falsch. Ich Streite nicht darüber. Noch, downvoters sind zu schnell... klicken Sie auf klicken Sie auf klicken 😉
    • Schauen Sie auf Ihre Bemerkung, die kritisierte, dass das Verhalten der Menschen downvoting Ihre Antwort, die war natürlich zunächst falsch. Sieht aus wie Sie finden es lustig, oder sieht es wie eine Beschwerde?
    • Der smiley gibt, hier die Antwort 🙂 xkcd.com/438
    • Wenn eine Antwort falsch ist, es ist wenig hilfreich, so verdient ein downvote. Sind Sie darauf hindeutet, dass vor downvoting, sollte man hängen herum für eine Weile, nur für den Fall, die Antwort ist behoben? Schlechte Idee IMO - ich würde sagen, es ist bis zu den Nutzer zu machen, ob die Antwort korrekt ist, bevor Sie buchen. In diesem Fall Sie waren zu schnell, nicht die downvoters. (Und Nein, das ist nicht zu sagen, meine Antworten sind immer richtig, mit zu beginnen - aber wenn jemand downvotes eine falsche Antwort von mir, ich würde das nie sagen, Sie sind "zu schnell". Sie Verhalten sich ganz entsprechend.)

    InformationsquelleAutor peter.petrov
Schreibe einen Kommentar