无法理解程序背后的逻辑

Question

试图制定一个基本的C程序来查找向量，我以为我在某处，但我已经着手停止，而不一定是在错误方面，而是在它背后的逻辑。这里是我的代码：

#include<stdio.h> 
#include <math.h> 

int norm_vec(int *x, int n) { 

int i; 
float modul; 

for(i=0;i<n;i++;) 
    { 
     modul=++ x[i]*x[i]; 
    } 
     modul = sqrt(modul); 

     for(i=0;i<n;i++;) 
      { 
       x[i] = x[i]/modul 
      } 
} 

Answer 1

让我先分清你的代码，这样它更具可读性并纠正一些错误。

#include <stdio.h> 
#include <math.h> 

int norm_vec(int * x, int n) 
{ 
    int i; 
    // initialize it at 0 for good practice 
    // to my knowledge if you don't initialize a float, it will be 0, but let's stay safe 
    float modul = 0; 

    for (i = 0; i < n; i++) { 
     modul += x[i]*x[i]; 
    } 

    modul = sqrt(modul); 

    for (i = 0; i < n; i++) { 
     x[i] = x[i]/modul; 
    } 
} 

对我而言，你的代码似乎在数学上是正确的。你首先计算向量的范数（你称之为modul），然后你将向量的每个分量除以范数，这就是归一化的结果。

但是，你的函数应该返回一个int，但它什么都不返回。你应该决定如何处理它。它应该返回规范还是没有规定？

Answer 2

将问题分解成更小的部分，您将有更容易的时间。归一化矢量需要将矢量的每个分量除以矢量的幅度。所以你需要一种计算量级的方法。这是一件很常见的事情，所以它保证它自己的功能。

您也可能想要一种打印矢量的方式，以便您可以看到您的函数按照您的预期工作。我为Vector写了一个打印功能的例子。

#include <stdio.h> 
#include <math.h> 
#include <stdlib.h> 
#include <string.h> 

typedef struct Vector { 
    int *components; 
    int arity; 
} Vector; 

double squaredMagnitude(Vector); 
double magnitude(Vector); 
void normalize(Vector); 
void printVector(Vector); 

double squaredMagnitude(Vector v) { 
    double sum = 0; 
    for (int i = 0; i < v.arity; i++) { 
     int component = v.components[i]; 
     sum += component * component; 
    } 
    return sum; 
} 

double magnitude(Vector v) { 
    return sqrt(squaredMagnitude(v)); 
} 

void normalize(Vector v) { 
    double mag = magnitude(v); 
    for (int i = 0; i < v.arity; i++) v.components[i] /= mag; 
} 

void printVector(Vector v) { 
    printf("{"); 
    for (int i = 0; i < v.arity - 1; i++) printf("%i, ", v.components[i]); 
    if (v.arity != 0) printf("%i", v.components[v.arity - 1]); 
    printf("}\n"); 
} 

int main() { 
    int components[] = {0, 5, 0}; 
    int componentCount = sizeof(components)/sizeof(*components); 

    Vector v; 
    v.components = malloc(componentCount); 
    memcpy(v.components, components, sizeof(components)); 
    v.arity = componentCount; 

    normalize(v); 

    printVector(v); 
}